[MPlayer-dev-eng] [PATCH] fixing srate, channels etc.

[Nico Sabbi]

Reimar Döffinger wrote:

>Hi,
>this is a patch I could need some help testing, since my last attempt
>actually broke things even more...
>First, it introduces a new option decchannels, since using the channels
>for specifying both number of in- and output channels creates problems
>with -af hrtf, -af pan etc.
>With -channels 0 you can now e.g. let MPlayer automatically select the
>number of channels, as the default is still 2 (also for decchannels).
>It also removes some code duplication by doing preinit by calling init.
>I guess the man page part could really be improved...
>
>Greetings,
>Reimar Döffinger
>  
>
>------------------------------------------------------------------------
>
>  
>
Hi,
I didn't test the patch, but the introduction of one more *channel 
parameter seems
overly confusing to me.
At the moment -channels identifies the number of channels output by the 
decoder,
not the number of audio out channels; the problem is that it's not (or 
wasn't)
well explained in the man page.
When you make this point clear the current situation is not so bad.

I'm not sure that with your patch applied things will improve.

    Nico