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@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