[FFmpeg-devel] [PATCH 05/17] lavfi: rename all_channel_layouts.h to all_channel_layouts.inc.

[Stefano Sabatini]

On date Thursday 2012-05-10 00:42:39 +0200, Nicolas George encoded:
> Le duodi 22 floréal, an CCXX, Stefano Sabatini a écrit :
> > No opinion. Ideally we should get rid of the file once we remove the
> > <=8 channels limitation.
> 
> Also, there is a deep flaw with that implementation:
> avfilter_make_all_channel_layouts does not really report all channel
> layouts, that would be a 2^63 elements list. It only returns the list of
> known channel layouts. And at least any of the 2^24 combinations of defined
> channels (or possibly sum(binom(24,i),i,1,8)) is potentially useful.
> 
> I have not yet come with a satisfactory solution to that problem. The best I
> have come with yet is to abuse format_count = UINT_MAX to mean "all
> formats", but that may cause some trouble with code not ready for that.

My plan was to add a new field to mean "all available values for this
type of format". This could be used for audio/video formats,
samplerates, channel layouts (and avoid some code duplication).
-- 
FFmpeg = Fiendish and Frenzy Monstrous Pitiful Extravagant Guide