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

@@ -1,13 +1,11 @@
 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
+	  Add "-Wstrict-prototypes" because we comply anyhow and missing a
-	  aliasing or type punning that throws off optimizations (this is esp.
+	  prototype is very bad on 64-bit platforms as types default to int but
-	  important in gcc 4.1).  Add "-Wstrict-prototypes" because we comply
+	  sizeof(int) != sizeof(long), add "-Wmissing-prototypes" and
-	  anyhow and missing a prototype is very bad on 64-bit platforms as
+	  "-Wmissing-declarations" to warn if we define an exported function
-	  types default to int but sizeof(int) != sizeof(long), add
+	  but fail to put it in a header.
-	  "-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>
 

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])