[FFmpeg-devel] [PATCH] lavfi: add ff_all_channel_layouts internal symbol

[Michael Niedermayer]

On Thu, Aug 25, 2011 at 01:38:52AM +0200, Stefano Sabatini wrote:
> On date Wednesday 2011-08-24 05:48:55 +0200, Michael Niedermayer encoded:
> > On Tue, Aug 23, 2011 at 03:31:35PM +0200, Stefano Sabatini wrote:
> [...]
> > > +const int64_t avfilter_all_channel_layouts_list[] = {
> > > +    AV_CH_LAYOUT_MONO,
> > > +    AV_CH_LAYOUT_STEREO,
> > > +    AV_CH_LAYOUT_4POINT0,
> > > +    AV_CH_LAYOUT_QUAD,
> > > +    AV_CH_LAYOUT_5POINT0,
> > > +    AV_CH_LAYOUT_5POINT0_BACK,
> > > +    AV_CH_LAYOUT_5POINT1,
> > > +    AV_CH_LAYOUT_5POINT1_BACK,
> > > +    AV_CH_LAYOUT_5POINT1|AV_CH_LAYOUT_STEREO_DOWNMIX,
> > > +    AV_CH_LAYOUT_7POINT1,
> > > +    AV_CH_LAYOUT_7POINT1_WIDE,
> > > +    AV_CH_LAYOUT_7POINT1|AV_CH_LAYOUT_STEREO_DOWNMIX,
> > > +    -1
> > > +};
> > 
> > There are more possible layouts than can easily be listed i think
> 
> Yes, this should be meant as the "default channel layouts which are
> automatically selected when choosing 'all channels' in audio filters".
> 
> So maybe it should be better named
> avfilter_default_all_channel_layouts_list.
> 
> If you don't like it please suggest alternatives.

assume the front and rear can be 0,1,2,3 speakers, side 0 or 2
and wide 0 or 2 and there can be a LFE channel and there can be stereo
downmix channels
you get 256 combinations
IMHO they all should be in the list at least

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored awnser#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- 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/20110826/1157c0cb/attachment.asc>