[MPlayer-dev-eng] [PATCH] Automatic downmix
Clément Bœsch
ubitux at gmail.com
Sun Sep 26 10:29:15 CEST 2010
On Sun, Sep 26, 2010 at 10:21:28AM +0200, Nicolas George wrote:
> Le quartidi 4 vendémiaire, an CCXIX, Clément Bœsch a écrit :
> > Well, if I understand it well, I don't have to change something; new
> > version attached.
>
> The code seems quite simple, and all tests I could think of behaved
> satisfactorily. Just a small point:
>
> > + return af_append(s, s->first, af_pan_str);
> > + return NULL;
>
> > + af_downmix(s);
>
> Is there a reason to return a value and then just ignore it instead of
> returning nothing at all?
>
It's "historical"; there was a time where I needed to check the return
value in order to reinit or not the chain list. As I said in a previous
mail, I can remove it but I'm not sure it's harmless to keep it (if it
needs to be checked later).
--
Clément B.
More information about the MPlayer-dev-eng
mailing list