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

[Justin Ruggles]

Jason Tackaberry wrote:

> Hi,
> 
> The attached patch implements sane channel layout behaviour for 7 and 8
> channel Vorbis files.
> 
> The Vorbis specification [1] says that the layout for greater than six
> channels is left up to the application.  I think the most sensible and
> consistent layout for 7 and 8 channels is their 5 and 6 channel
> counterparts with back left/right channels appended.

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.

-Justin