[FFmpeg-devel] [PATCH] 8 channel layout fix for Vorbis decoder

Jason Tackaberry tack
Wed Nov 4 02:33:11 CET 2009


On Tue, 2009-11-03 at 20:10 -0500, Justin Ruggles wrote:
> How about using AVCodecContext.request_channel_layout?  I don't agree
> with unconditionally assuming channel layouts that are not in the
> specification.  But I think adding support for reordering common layouts
> when requested would be fine.

In practice, ffmpeg is currently already assuming a channel layout
that's not in the specification anyway.

I think the patch moves the default behaviour for 8 channel content much
closer to the realm of objectively sane.  I can't fathom anyone would
expect or desire the current behaviour when passing it 8 channel Vorbis
content.

Cheers,
Jason.




More information about the ffmpeg-devel mailing list