[sane-devel] [AWARE] -Wall warning count down by 50%
paddy-hack at member.fsf.org
Mon Dec 28 11:44:46 UTC 2015
A month ago I asked about what warning options to keep besides -Wall
and -Wextra (formerly known as -W). I didn't get any feedback so I just
went ahead and pushed a pile of commits yesterday.
The first changes compiler warning policy:
-Wall always enabled when using gcc
-Wextra enabled by default during the development cycle, disabled
by default for releases. The --enable-warnings option can
be used the change the default behaviour.
-pedantic same as -Wextra
No other warnings than the above are enabled. Speak up if you want to
see other warnings added.
# BTW, I am not sure I want to keep -pedantic. Suggestions welcome.
The following forty some commits fix half of the -Wall warnings I see on
two Debian GNU/Linux 8.2 build setups. I've split the fixes so that
you should be able to find those that affect the parts of the code you
care about fairly easily from the one-line commit summaries.
I think I can half the count of the remaining -Wall warnings this year.
The rest may need some discussion with backend maintainers. Feel free
to start fixing -Wextra warnings! I'll still have to get the autotool
reform into shape too ;-)
Hope this helps,
Olaf Meeuwissen, LPIC-2 FSF Associate Member since 2004-01-27
Support Free Software Support the Free Software Foundation
GnuPG key: F84A2DD9/B3C0 2F47 EA19 64F4 9F13 F43E B8A4 A88A F84A 2DD9
More information about the sane-devel