[Ffmpeg-devel] [PATCH] ac3 decoder

[Justin Ruggles]

Justin Ruggles wrote:
> Benjamin Larsson wrote:
> 
>>Agree, I think this is a leftover from the development process.
> 
> 
> I think I've narrowed the problem down to the recent change to
> initializes AVCodecContext with the AVOption defaults.  Prior to that,
> avctx->channels was set to zero (by memset).  Now that it is set to 1,
> the ac3 parser thinks the user manually set it, so it does not override
> it with the value from the ac3 header.
> 
> Given that, I don't know how the parser could determine if the mono
> downmix is a user request or just the default...  If there is no way
> around this, maybe the parser should be changed to always leave the
> number of channels alone and force the user to set it?

This patch makes things work as expected.

-Justin

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ffmpeg-ac3-channels.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20061001/c4c40fa2/attachment.asc>