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

Nicolas George nicolas.george at normalesup.org
Thu May 10 00:42:39 CEST 2012


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.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120510/d3758744/attachment.asc>


More information about the ffmpeg-devel mailing list