[MPlayer-dev-eng] squelch some warnings (patches incl.)

Aurelien Jacobs aurel at gnuage.org
Tue Feb 26 20:49:15 CET 2008


Uoti Urpala wrote:

> On Mon, 2008-02-25 at 03:20 +0100, Michael Niedermayer wrote:
> > Also, if you think if(<this will be 0>) something_nonexistant is
> > ugly look at some commits from aurel, like allcodecs.c. And fix
> > this, which frankly would be identical to reverting aurels work and
> > iam pretty sure he will like this as much as i do like these
> > changes to snow.c.
> 
> Also the allcodecs.c stuff is all inside macros. Those macros could be
> written in a standard-compliant way that doesn't rely on the compiler
> ignoring illegal code inside "if (0)", and the per-codec macro
> invocations would not need to change.

If you really think this can be done, please, please provide a patch !
Until then, please stop the FUD.

Aurel



More information about the MPlayer-dev-eng mailing list