[Emacs-diffs] /srv/bzr/emacs/trunk r103366: * configure.in (C_WARNINGS_SWITCH): Do not prepend -Wno-pointer-sign.

[Paul Eggert]

------------------------------------------------------------
revno: 103366
committer: Paul Eggert <address@hidden>
branch nick: trunk
timestamp: Sun 2011-02-20 11:46:08 -0800
message:
  * configure.in (C_WARNINGS_SWITCH): Do not prepend -Wno-pointer-sign.
modified:
  ChangeLog
  configure
  configure.in

=== modified file 'ChangeLog'
--- a/ChangeLog 2011-02-20 18:50:26 +0000
+++ b/ChangeLog 2011-02-20 19:46:08 +0000
@@ -1,3 +1,13 @@
+2011-02-20  Paul Eggert  <address@hidden>
+
+       * 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 <address@hidden>
 
        * lib/makefile.w32-in ($(BLD)/md5.$(O)): New recipe, moved from

=== modified file 'configure'
--- a/configure 2011-02-19 07:28:29 +0000
+++ b/configure 2011-02-20 19:46:08 +0000
@@ -6177,37 +6177,6 @@
   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; }

=== modified file 'configure.in'
--- a/configure.in      2011-02-18 07:44:39 +0000
+++ b/configure.in      2011-02-20 19:46:08 +0000
@@ -699,19 +699,6 @@
   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"