2005-10-07 Robert Love <rml@novell.com>

* configure.in: Change our compile flags for the betterment of mankind.
          Remove "-Wno-strict-aliasing" because we should not be doing bad
          aliasing or type punning that throws off optimizations (this is esp.
          important in gcc 4.1).  Add "-Wstrict-prototypes" because we comply
          anyhow and missing a prototype is very bad on 64-bit platforms as
          types default to int but sizeof(int) != sizeof(long), add
          "-Wmissing-prototypes" and "-Wmissing-declarations" to warn if we
          define an exported function but fail to put it in a header.


git-svn-id: http://svn-archive.gnome.org/svn/NetworkManager/trunk@1002 4912f4e0-d625-0410-9fb7-b9a5a253dbdc
This commit is contained in:
Robert Love 2005-10-07 17:57:10 +00:00 committed by Robert Love
parent 0f08c505a2
commit 7864ca262d
2 changed files with 14 additions and 1 deletions

View file

@ -1,3 +1,14 @@
2005-10-07 Robert Love <rml@novell.com>
* configure.in: Change our compile flags for the betterment of mankind.
Remove "-Wno-strict-aliasing" because we should not be doing bad
aliasing or type punning that throws off optimizations (this is esp.
important in gcc 4.1). Add "-Wstrict-prototypes" because we comply
anyhow and missing a prototype is very bad on 64-bit platforms as
types default to int but sizeof(int) != sizeof(long), add
"-Wmissing-prototypes" and "-Wmissing-declarations" to warn if we
define an exported function but fail to put it in a header.
2005-10-07 Robert Love <rml@novell.com>
* src/NetworkManagerWireless.c: remove stale, unused function, who goes

View file

@ -244,7 +244,9 @@ if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then
AC_MSG_RESULT(yes)
CFLAGS="-Wall -Werror -std=gnu89 $CFLAGS"
for option in -Wno-unused -Wno-strict-aliasing -Wno-sign-compare -Wdeclaration-after-statement -Wno-pointer-sign ; do
for option in -Wno-unused -Wno-sign-compare -Wno-pointer-sign \
-Wdeclaration-after-statement -Wstrict-prototypes \
-Wmissing-prototypes -Wmissing-declarations; do
SAVE_CFLAGS="$CFLAGS"
CFLAGS="$CFLAGS $option"
AC_MSG_CHECKING([whether gcc understands $option])