* configure.in (C_WARNINGS_SWITCH): Do not prepend -Wno-pointer-sign.

3 files changed, 10 insertions, 44 deletions

diff --git a/ChangeLog b/ChangeLog
index 07cafd1f33a..b390cfb7266 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2011-02-20  Paul Eggert  <eggert@cs.ucla.edu>
+
+        * configure.in (C_WARNINGS_SWITCH): Do not prepend -Wno-pointer-sign.
+        This undoes the 2006-01-02 change.  The -Wno-pointer-sign option
+        is no longer needed, due to the recent SSDATA and related changes.
+        Perhaps -Wno-pointer-sign should also be removed from
+        nextstep/Cocoa/Emacs.xcodeproj/project.pbxproj but I have no easy
+        way to test this so I left it alone.
+        * configure: Regenerate.
+
 2011-02-20  Christoph Scholtes <cschol2112@gmail.com>
 
         * lib/makefile.w32-in ($(BLD)/md5.$(O)): New recipe, moved from
diff --git a/configure b/configure
index d3607aec316..558ed63da17 100755
--- a/configure
+++ b/configure
@@ -6177,37 +6177,6 @@ else
   test "x$NON_GCC_TEST_OPTIONS" != x && CC="$CC $NON_GCC_TEST_OPTIONS"
 fi
 
-### Use -Wno-pointer-sign if the compiler supports it
-{ $as_echo "$as_me:${as_lineno-$LINENO}: checking whether gcc understands -Wno-pointer-sign" >&5
-$as_echo_n "checking whether gcc understands -Wno-pointer-sign... " >&6; }
-SAVE_CFLAGS="$CFLAGS"
-CFLAGS="$CFLAGS -Wno-pointer-sign"
-cat confdefs.h - <<_ACEOF >conftest.$ac_ext
-/* end confdefs.h.  */
-
-int
-main ()
-{
-
-  ;
-  return 0;
-}
-_ACEOF
-if ac_fn_c_try_compile "$LINENO"; then :
-  has_option=yes
-else
-  has_option=no
-fi
-rm -f core conftest.err conftest.$ac_objext conftest.$ac_ext
-if test $has_option = yes; then
-   C_WARNINGS_SWITCH="-Wno-pointer-sign $C_WARNINGS_SWITCH"
-fi
-{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $has_option" >&5
-$as_echo "$has_option" >&6; }
-CFLAGS="$SAVE_CFLAGS"
-unset has_option
-unset SAVE_CFLAGS
-
 ### Use -Wdeclaration-after-statement if the compiler supports it
 { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether gcc understands -Wdeclaration-after-statement" >&5
 $as_echo_n "checking whether gcc understands -Wdeclaration-after-statement... " >&6; }
diff --git a/configure.in b/configure.in
index ed25373a12d..376c82c4ca4 100644
--- a/configure.in
+++ b/configure.in
@@ -699,19 +699,6 @@ else
   test "x$NON_GCC_TEST_OPTIONS" != x && CC="$CC $NON_GCC_TEST_OPTIONS"
 fi
 
-### Use -Wno-pointer-sign if the compiler supports it
-AC_MSG_CHECKING([whether gcc understands -Wno-pointer-sign])
-SAVE_CFLAGS="$CFLAGS"
-CFLAGS="$CFLAGS -Wno-pointer-sign"
-AC_TRY_COMPILE([], [], has_option=yes, has_option=no,)
-if test $has_option = yes; then
-   C_WARNINGS_SWITCH="-Wno-pointer-sign $C_WARNINGS_SWITCH"
-fi
-AC_MSG_RESULT($has_option)
-CFLAGS="$SAVE_CFLAGS"
-unset has_option
-unset SAVE_CFLAGS
-
 ### Use -Wdeclaration-after-statement if the compiler supports it
 AC_MSG_CHECKING([whether gcc understands -Wdeclaration-after-statement])
 SAVE_CFLAGS="$CFLAGS"