From: Pekka Pessi <[email protected]>

Do not add warning flag if corresponding or opposite option is already
in CFLAGS.
---
 acinclude.m4 |   22 +++++++++++++++-------
 1 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/acinclude.m4 b/acinclude.m4
index 84086fd..fcfb132 100644
--- a/acinclude.m4
+++ b/acinclude.m4
@@ -15,13 +15,21 @@ AC_DEFUN([COMPILER_FLAGS], [
                CFLAGS="-Wall -O2 -D_FORTIFY_SOURCE=2"
        fi
        if (test "$USE_MAINTAINER_MODE" = "yes"); then
-               CFLAGS+=" -Werror -Wextra"
-               CFLAGS+=" -Wno-unused-parameter"
-               CFLAGS+=" -Wno-missing-field-initializers"
-               CFLAGS+=" -Wdeclaration-after-statement"
-               CFLAGS+=" -Wmissing-declarations"
-               CFLAGS+=" -Wredundant-decls"
-               CFLAGS+=" -Wcast-align"
+               for flag in \
+                       -Werror -Wextra \
+                       -Wno-unused-parameter \
+                       -Wno-missing-field-initializers \
+                       -Wdeclaration-after-statement \
+                       -Wmissing-declarations \
+                       -Wredundant-decls \
+                       -Wcast-align
+               do
+                       neg=`echo $flag | sed 's/^-Wno-/-W/;t;s/^-W/-Wno-/'`
+                       case " $CFLAGS " in
+                       *" $flag "* | *" $neg "* ) ;;
+                       * ) CFLAGS="$CFLAGS $flag" ;;
+                       esac
+               done
        fi
 ])
 
-- 
1.7.0.4

_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to