[PATCH] Add option to control compiler warnings for gcc

[Anton Lundin]

On 08 January, 2015 - Anton Lundin wrote:

> On 08 January, 2015 - Jef Driesen wrote:
> 
> > On 2015-01-06 23:28, Anton Lundin wrote:
> > >This adds a configure option, --enable-gcc-compile-warnings which
> > >controls which CFLAGS to pass to gcc. It defaults to -Wall but can with
> > >the configure argument be turned to different levels.
> > >
> > >Signed-off-by: Anton Lundin <glance at acc.umu.se>
> > >---
> > >
> > >I think i kinda over-did-it, but hey.
> > >
> > >I had something quite close laying around from another project so i picked
> > >the whole thingie.
> > 
> > You don't really need a new option to use specific CFLAGS (or others). Just
> > pass them to the configure script (or make) directly:
> > 
> > ./configure CFLAGS='...'
> > 
> > or
> > 
> > make CFLAGS='...'
> > 
> > I always build with -Wall -Wextra -pedantic using the first variant (which
> > means I only have to set the right options once).
> > 
> 
> Yes, i know, but its usually good practise to make sure a -Wall is
> there by default. Thats why i wrote this one.
> 

I still would like to see such flags added by default. Yes, you can
enable them yourself, but as noted earlier, the conception of this patch
was due to that it would ease development to have them enabled by
default.

Do you disagree?


//Anton


-- 
Anton Lundin	+46702-161604