[FFmpeg-devel] [PATCH]Make CH_LAYOUT_5POINTx_BACK the normal case

Carl Eugen Hoyos cehoyos at ag.or.at
Thu Aug 25 20:01:36 CEST 2011


Stefano Sabatini <stefano.sabatini-lala <at> poste.it> writes:

> > > > AV_CH_LAYOUT_5POINTx_BACK is what every other application expects for
> > > > 5.0/5.1 when reading a wav file which is where FFmpeg's channel layout
> > > > comes from, so this should be the "normal" case, and
> > > > AV_CH_LAYOUT_5POINTx the exception.
> > > 
> > > How is this related to the channel_layout_map array?
> > 
> > How is it not related to it?
> 
> I see channel_layout_map is used for mapping strings <-> channel
> layouts, so the only difference I'd expect by this change is that now
> the mapping is changed. In other words the string is only used for
> describing the channel layout to *users*, so it is an UI change.
> 
> Or no?

I thought when you wrote "This is required for distinguishing between the two
channel layouts starting from the channel layout description." you meant that
this will be used in the future by users to request channel layouts that the
input should be converted into.

If I misunderstood, I agree the patch is not that important.

http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/133747

Carl Eugen



More information about the ffmpeg-devel mailing list