[MPlayer-cvslog] r23937 - trunk/libmpcodecs/ad_ffmpeg.c

[Reimar Döffinger]

Hello,
On Thu, Aug 02, 2007 at 12:34:52AM +0300, Uoti Urpala wrote:
> On Mon, 2007-07-30 at 17:19 +0200, reimar wrote:
> > Indicate desired number of output channels to ffmpeg audio decoder.
> 
> > +   // Set desired number of channels
> > +   lavc_context->channels = audio_output_channels;
> 
> This breaks at least ffvorbis which doesn't reset the number of channels
> during each decode call (only sets it during init). I'm not sure what
> the API is supposed to be - is this a bug in ffvorbis?

I fear there is no real "API" there yet, but it would be enough if it
set the number of channels after the first decode...
But after rereading the code, I think that this code should probably be
before the avcodec_open.

Greetings,
Reimar Döffinger