diff options
| author | Dan Williams <dcbw@redhat.com> | 2012-08-10 09:06:05 -0500 |
|---|---|---|
| committer | Dan Williams <dcbw@redhat.com> | 2012-08-22 20:26:51 -0500 |
| commit | 8d0abc1bdd28232ffe46045a88d15834830ec87e (patch) | |
| tree | bc384e578e11d3f3f6657187d4130daa82dba329 /m4 | |
| parent | bc1d88da9ee4d18b1a8872db2e2b546929e279d7 (diff) | |
build: enable more compiler warnings: --enable-more-warnings=yes/no/error
Diffstat (limited to 'm4')
| -rw-r--r-- | m4/compiler-warnings.m4 | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/m4/compiler-warnings.m4 b/m4/compiler-warnings.m4 index 511b898..539eae0 100644 --- a/m4/compiler-warnings.m4 +++ b/m4/compiler-warnings.m4 @@ -1,16 +1,20 @@ AC_DEFUN([LIBQMI_COMPILER_WARNINGS], [AC_ARG_ENABLE(more-warnings, - AS_HELP_STRING([--enable-more-warnings], [Maximum compiler warnings]), - set_more_warnings="$enableval",set_more_warnings=yes) -AC_MSG_CHECKING(for more warnings, including -Werror) + AS_HELP_STRING([--enable-more-warnings], [Possible values: no/yes/error]), + set_more_warnings="$enableval",set_more_warnings=error) +AC_MSG_CHECKING(for more warnings) if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then AC_MSG_RESULT(yes) - CFLAGS="-Wall -Werror -std=gnu89 $CFLAGS" + CFLAGS="-Wall -std=gnu89 $CFLAGS" for option in -Wmissing-declarations -Wmissing-prototypes \ -Wdeclaration-after-statement -Wstrict-prototypes \ -fno-strict-aliasing -Wno-deprecated-declarations \ - -Wint-to-pointer-cast; do + -Wint-to-pointer-cast -Wfloat-equal -Wno-unused-parameter \ + -Wno-sign-compare -Wunused-but-set-variable \ + -Wundef -Wimplicit-function-declaration \ + -Wpointer-arith -Winit-self -Wshadow \ + -Wmissing-include-dirs -Waggregate-return; do SAVE_CFLAGS="$CFLAGS" CFLAGS="$CFLAGS $option" AC_MSG_CHECKING([whether gcc understands $option]) @@ -25,6 +29,9 @@ if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then unset SAVE_CFLAGS done unset option + if test "x$set_more_warnings" = xerror; then + CFLAGS="$CFLAGS -Werror" + fi else AC_MSG_RESULT(no) fi |