From: Frederic Marchal <fmarchal@users.sourceforge.net>
Date: Thu, 21 Nov 2013 08:31:53 +0000 (+0100)
Subject: Clang compatibility fix
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=dfbbe8fa1e04912858a1ccffd2d2855fa72fa71a;p=thirdparty%2Fsarg.git

Clang compatibility fix

Option to not fail on integers sign comparison must come after the extra
warnings have been enabled or it is ignored.
---

diff --git a/configure.in b/configure.in
index 59f7721..7c09b8e 100644
--- a/configure.in
+++ b/configure.in
@@ -27,7 +27,7 @@ dnl C99 support is required to define LLONG_MAX (at least on CentOS 5.7)
 AC_PROG_CC_C99
 
 # Report more warnings to improve code quality.
-CFLAGS="${CFLAGS} -Wall -Wno-sign-compare"
+CFLAGS="${CFLAGS} -Wall"
 
 dnl Check for supported compiler options
 
@@ -40,6 +40,9 @@ if test "$have_extra_warnings" = "no" ; then
     CFLAGS="${saved_CFLAGS}"
 fi
 
+# Don't compare signs as it is a mess
+CFLAGS="${CFLAGS} -Wno-sign-compare"
+
 AC_MSG_CHECKING([for implicit-function-declaration error flag in $CC])
 saved_CFLAGS="${CFLAGS}"
 CFLAGS="${CFLAGS} -Werror=implicit-function-declaration"