[MPlayer-dev-eng] [PATCH] yet another batch of compiler warning fixes

Dominik Mierzejewski dominik at rangers.eu.org
Mon Nov 25 02:41:07 CET 2002


On Monday, 25 November 2002, Arpi wrote:
> Hi,
> 
> > > PS. Do you think I could start applying such patches myself directly
> > > into CVS? After posting them here for review, of course. 
> > IMHO yes, but arpi is the one who must decide that
> 
> it's ok.

Thanks.

> but please stop sending (and applying) those 'unused variables' patches.
> they are there mostly for some uncommented/disabled debug code, and if
> you remove them it makes debugging a lot harder.

OK. Will do.

> rather find the right gcc option to hide this kind of warning if you dislike
> it.

I'm on it.

> it's ok to fix other kind of warnings, especially 10l ones (missing return
> value, etc).
> 
> and i really don't like the default -Wall everywhere.
> it's a nightmare to develop code when gcc bombs pages of useless warnings
> and it takes minutes to find the real error or important warning messages
> there.
> 
> if you want to hunt warnings, let's add -Wall, make a patch, but don't force
> everyone to read those warnings!

Very well. Understood.
 
-- 
MPlayer RPMs maintainer: http://www.piorunek.pl/~dominik/linux/pkgs/mplayer/
"The Universe doesn't give you any points for doing things that are easy."
        -- Sheridan to Garibaldi in Babylon 5:"The Geometry of Shadows"



More information about the MPlayer-dev-eng mailing list