From eed1fa76e902396a6a3497ad877af6ce1ceca06a Mon Sep 17 00:00:00 2001 From: Robert Love Date: Fri, 7 Oct 2005 19:11:55 +0000 Subject: [PATCH] Add back -Wno-strict-aliasing because we type pun a /lot/. Note that you are not supposed to mix pointers to functions and objects!!! git-svn-id: http://svn-archive.gnome.org/svn/NetworkManager/trunk@1003 4912f4e0-d625-0410-9fb7-b9a5a253dbdc --- ChangeLog | 12 +++++------- configure.in | 3 ++- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/ChangeLog b/ChangeLog index ebf223ea5..d4667c08a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,13 +1,11 @@ 2005-10-07 Robert Love * 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. + 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 diff --git a/configure.in b/configure.in index cc9b5e185..1bc4a416b 100644 --- a/configure.in +++ b/configure.in @@ -246,7 +246,8 @@ if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then for option in -Wno-unused -Wno-sign-compare -Wno-pointer-sign \ -Wdeclaration-after-statement -Wstrict-prototypes \ - -Wmissing-prototypes -Wmissing-declarations; do + -Wmissing-prototypes -Wmissing-declarations \ + -Wno-strict-aliasing; do SAVE_CFLAGS="$CFLAGS" CFLAGS="$CFLAGS $option" AC_MSG_CHECKING([whether gcc understands $option])