gcc41/gcc44: Remove a modification to exclude -Wpointer-sign from -Wall.
authorSascha Wildner <saw@online.de>
Sat, 14 Jul 2012 16:19:13 +0000 (18:19 +0200)
committerSascha Wildner <saw@online.de>
Sat, 14 Jul 2012 16:19:13 +0000 (18:19 +0200)
We had this as a local change in our compilers but the problem with this
approach is that it needs to be added to each future compiler we support.

This isn't so much a problem with the compilers we have in base but with
those from pkgsrc that we support via CCVER and compilers.conf(5).

Instead, add -Wno-pointer-sign to our Makefiles whenever -Wall comes into
play.

contrib/gcc-4.1/gcc/c-opts.c
contrib/gcc-4.4/gcc/c-opts.c
share/mk/bsd.sys.mk
sys/conf/bsd.kern.mk

index 0474e6c..551893f 100644 (file)
@@ -417,6 +417,9 @@ c_common_handle_option (size_t scode, const char *arg, int value)
       cpp_opts->warn_comments = value;
       cpp_opts->warn_num_sign_change = value;
       cpp_opts->warn_multichar = value;        /* Was C++ only.  */
+
+      if (warn_pointer_sign == -1)
+       warn_pointer_sign = 1;
       break;
 
     case OPT_Wcomment:
index 8af6486..28bdc31 100644 (file)
@@ -422,11 +422,8 @@ c_common_handle_option (size_t scode, const char *arg, int value)
       cpp_opts->warn_comments = value;
       cpp_opts->warn_num_sign_change = value;
 
-#if 0
-      /* DragonFly: disabled in -Wall */
       if (warn_pointer_sign == -1)
        warn_pointer_sign = value;
-#endif
       break;
 
     case OPT_Wbuiltin_macro_redefined:
index 9c76250..3cd9bdb 100644 (file)
@@ -29,7 +29,7 @@ CWARNFLAGS    +=      -Werror
 .   endif
 .  endif
 .  if ${WARNS} >= 2
-CWARNFLAGS     +=      -Wall
+CWARNFLAGS     +=      -Wall -Wno-pointer-sign
 .  endif
 .  if ${WARNS} >= 3
 CWARNFLAGS     +=      -W -Wno-unused-parameter -Wstrict-prototypes\
index 5493b8e..d13a63e 100644 (file)
@@ -9,7 +9,7 @@
 
 CWARNFLAGS?=   -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes \
                -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual \
-               -Wold-style-definition -std=c99
+               -Wold-style-definition -Wno-pointer-sign -std=c99
 
 .if ${CCVER} != "gcc41"
 CWARNFLAGS+=   -Wold-style-declaration