Re: -Wextra

[Rodney Dawes <dobey novell com>]

I was about to just remove this completely anyway, since it breaks even
on gcc 3.3. There's no reason we should be requiring 3.4 to build. It
doesn't make sense to, anyway. So, I'm all for removing it. I think it
was committed accidentally by Kjarten anyway, as I didn't see a 
ChangeLog entry for it.

-- dobey


On Tue, 2005-11-22 at 17:09 +0000, Jens Granseuer wrote:
> Hi,
> 
> Rodrigo told me to post this here, so...
> 
> Current CVS uses the -Wextra compiler flag to enable some extra warnings for gcc.
> AFAICS -Wextra has only been added (or rather renamed) in gcc 3.4.0, and it
> certainly isn't supported by gcc 2.95 which I am using.
> 
> Therefore I filed bug 321779 [1] where I also posted a patch to simply remove
> -Wextra completely. However, if the flag is dear enough to your hearts that
> you can't stand dropping it, I'm also willing to try to come up with a more
> sophisticated patch that uses -Wextra only where appropriate and -W otherwise.
> 
> Opinions? Please discuss away.
> 
> Jens
> 
> [1] http://bugzilla.gnome.org/show_bug.cgi?id=321779