changeset 16582:60f9ac8421ef

Add a couple more CFLAGS to highlight some more warnings and change how we specify FORTIFY_SOURCE, I think this is more correct
author Stu Tomlinson <stu@nosnilmot.com>
date Sat, 28 Apr 2007 12:58:16 +0000
parents 467698ab6bf0
children ce049678a67b
files configure.ac
diffstat 1 files changed, 3 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/configure.ac	Sat Apr 28 10:07:22 2007 +0000
+++ b/configure.ac	Sat Apr 28 12:58:16 2007 +0000
@@ -850,10 +850,12 @@
 			"-Waggregate-return" \
 			"-Wcast-align" \
 			"-Wdeclaration-after-statement" \
+			"-Wendif-labels" \
 			"-Werror-implicit-function-declaration" \
 			"-Wextra -Wno-sign-compare -Wno-unused-parameter" \
 			"-Winit-self" \
 			"-Wmissing-declarations" \
+			"-Wmissing-noreturn" \
 			"-Wmissing-prototypes" \
 			"-Wnested-externs" \
 			"-Wpointer-arith" \
@@ -890,7 +892,7 @@
 			}
 		], [
 			AC_MSG_RESULT(yes)
-			DEBUG_CFLAGS="$DEBUG_CFLAGS -D_FORTIFY_SOURCE=2"
+			DEBUG_CFLAGS="$DEBUG_CFLAGS -Wp,-D_FORTIFY_SOURCE=2"
 		], [
 			AC_MSG_RESULT(no)
 		])