Fix warnings for clang
authorDaniel Johnson <daniel@daniel-johnson.org>
Mon, 22 Mar 2010 00:46:33 +0000 (20:46 -0400)
committerDaniel Stenberg <daniel@haxx.se>
Mon, 22 Mar 2010 18:44:45 +0000 (19:44 +0100)
ares/m4/cares-compilers.m4
m4/curl-compilers.m4

index ae32988..3a57389 100644 (file)
@@ -835,11 +835,11 @@ AC_DEFUN([CARES_SET_COMPILER_WARNING_OPTS], [
         #
         if test "$want_warnings" = "yes"; then
           dnl All versions of clang support the same warnings as at least
-          dnl gcc 4.2.1.
+          dnl gcc 4.2.1 except -Wunused.
           tmp_CFLAGS="$tmp_CFLAGS -pedantic"
           tmp_CFLAGS="$tmp_CFLAGS -Wall -Wextra"
           tmp_CFLAGS="$tmp_CFLAGS -Wpointer-arith -Wwrite-strings"
-          tmp_CFLAGS="$tmp_CFLAGS -Wunused -Wshadow"
+          tmp_CFLAGS="$tmp_CFLAGS -Wshadow"
           tmp_CFLAGS="$tmp_CFLAGS -Winline -Wnested-externs"
           tmp_CFLAGS="$tmp_CFLAGS -Wmissing-declarations"
           tmp_CFLAGS="$tmp_CFLAGS -Wmissing-prototypes"
@@ -853,6 +853,11 @@ AC_DEFUN([CARES_SET_COMPILER_WARNING_OPTS], [
           tmp_CFLAGS="$tmp_CFLAGS -Wcast-align"
           tmp_CFLAGS="$tmp_CFLAGS -Wno-system-headers"
           tmp_CFLAGS="$tmp_CFLAGS -Wshorten-64-to-32"
+          #
+          dnl Only clang 1.1 or later
+          if test "$compiler_num" -ge "101"; then
+            tmp_CFLAGS="$tmp_CFLAGS -Wunused"
+          fi
         fi
         ;;
         #
index 9577bf7..3f434d4 100644 (file)
@@ -840,11 +840,11 @@ AC_DEFUN([CURL_SET_COMPILER_WARNING_OPTS], [
         #
         if test "$want_warnings" = "yes"; then
           dnl All versions of clang support the same warnings as at least
-          dnl gcc 4.2.1.
+          dnl gcc 4.2.1 except -Wunused.
           tmp_CFLAGS="$tmp_CFLAGS -pedantic"
           tmp_CFLAGS="$tmp_CFLAGS -Wall -Wextra"
           tmp_CFLAGS="$tmp_CFLAGS -Wpointer-arith -Wwrite-strings"
-          tmp_CFLAGS="$tmp_CFLAGS -Wunused -Wshadow"
+          tmp_CFLAGS="$tmp_CFLAGS -Wshadow"
           tmp_CFLAGS="$tmp_CFLAGS -Winline -Wnested-externs"
           tmp_CFLAGS="$tmp_CFLAGS -Wmissing-declarations"
           tmp_CFLAGS="$tmp_CFLAGS -Wmissing-prototypes"
@@ -858,6 +858,11 @@ AC_DEFUN([CURL_SET_COMPILER_WARNING_OPTS], [
           tmp_CFLAGS="$tmp_CFLAGS -Wcast-align"
           tmp_CFLAGS="$tmp_CFLAGS -Wno-system-headers"
           tmp_CFLAGS="$tmp_CFLAGS -Wshorten-64-to-32"
+          #
+          dnl Only clang 1.1 or later
+          if test "$compiler_num" -ge "101"; then
+            tmp_CFLAGS="$tmp_CFLAGS -Wunused"
+          fi
         fi
         ;;
         #