[MPlayer-dev-eng] [PATCH] Automatic downmix

Nicolas George nicolas.george at normalesup.org
Wed Sep 8 15:23:25 CEST 2010


Le septidi 27 thermidor, an CCXVIII, ubitux a écrit :
> +static void downmix_if_needed()

I think MPlayer coding style requires "(void)".

> +        af_add(mpctx->mixer.afilter, "pan=2:"
> +                                     "0.4:0:"      // Front left
> +                                     "0:0.4:"      // Front right
> +                                     "0.15:0:"     // Back left
> +                                     "0:0.15:"     // Back right
> +                                     "0.25:0.25:"  // Center
> +                                     "0.1:0.1:"    // LFE
> +                                     "0.1:0:"      // Auxiliary left
> +                                     "0:0.1"       // Auxiliary right
> +        );

The indentation is unusual, but it looks nice. I believe it's ok.

>      mpctx->mixer.volstep = volstep;
> +
> +    downmix_if_needed();

Is the extra empty line really necessary?


Le duodi 22 fructidor, an CCXVIII, Diego Biurrun a écrit :
> Two committers have been discussing this patch with you.
> Guys, get a move on and either commit or reject the patch please.

I believe one of them is me, although I was not committer at the time I
actually discussed the patch, I believe.

There is still a problem with this patch: the audio output gets initialized
for the original number of channels, not the reduced stereo. Example:

$ ./mplayer -channels 2 -v /tmp/6_Channel_ID.ac3 |& grep 'alsa.*device'
alsa-init: using device default
$ ./mplayer -channels 2 -v /tmp/6_Channel_ID.wav |& grep 'alsa.*device'
alsa-init: device set to surround51
alsa-init: using device surround51

(First call: AC3, built-in downmix, second call: WAVE, new generic downmix.)

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100908/b0e35192/attachment.pgp>


More information about the MPlayer-dev-eng mailing list