[Ffmpeg-devel] [PATCH] ac3 decoder

[Benjamin Larsson]

Justin Ruggles wrote:

>Benjamin Larsson wrote:
>  
>
>>Here is the patch with the ac3 decoder from SoC. Included is the patch
>>to enable the sse imdct from Loren Merrit.
>>
>>MvH
>>Benjamin Larsson
>>    
>>
>
>I have a question about the behavior of the ac3 decoder.  With the lgpl
>decoder patch, it defaults to downmixing stereo to mono if '-ac 2' is
>not specified, but does not downmix 5.1 by default.  This has to do with
>the ac3 parser, but I can't tell right off if that was the intended
>behavior.  If not, it's an easy fix.
>  
>
Well I think it should downmix (if needed) to stereo by default.

>I'm checking to see if this was the behavior of the liba52 decoder as
>well, but I have to do a full source recompile, so my machine may take a
>little while.  Either way, I think it's not good behavior to downmix
>stereo to mono by default.  Or if so, at least make it consistant and
>downmix all channel configurations to mono unless otherwise specified.
>
>  
>
Agree, I think this is a leftover from the development process.

>Thanks,
>-Justin
>  
>
MvH
Benjamin Larsson

-- 
new tiny signature