Give up on -Wsuggest-attribute=const

The attribute doesn't help performance significantly, and the
warning seems to be more trouble than it's worth.  See the thread at:
http://lists.gnu.org/archive/html/emacs-devel/2015-01/msg00361.html
* configure.ac (WERROR_CFLAGS): Don't use -Wsuggest-attribute=const.
* lib-src/make-docfile.c (write_globals):
Remove special hack for Fnext_read_file_uses_dialog_p.
* src/decompress.c (Fzlib_available_p):
* src/gnutls.c (Fgnutls_available_p):
* src/gtkutil.h (xg_uses_old_file_dialog):
* src/xdisp.c (Ftool_bar_height):
* src/xmenu.c (popup_activated):
No longer const, since it's not const on at lest some
configurations, and we shouldn't lie to the compiler.

diff --git a/ChangeLog b/ChangeLog
index cca9100ddf7c28fb70d97983d39680eca0853ec5..309b04f26abefa6ef1c39084a16aeb458775bd4a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2015-01-16  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Give up on -Wsuggest-attribute=const
+       The attribute doesn't help performance significantly, and the
+       warning seems to be more trouble than it's worth.  See the thread at:
+       http://lists.gnu.org/archive/html/emacs-devel/2015-01/msg00361.html
+       * configure.ac (WERROR_CFLAGS): Don't use -Wsuggest-attribute=const.
+
 2015-01-11  Paul Eggert  <eggert@cs.ucla.edu>
 
        Default to 'configure --enable-silent-rules'

diff --git a/configure.ac b/configure.ac
index 4cad214e92800fe30994bc98dfcb682e6d0dd266..9db4bdecfcca6f4ba2d57774a80188d4adbfd308 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -892,6 +892,10 @@ else
   # Emacs's use of alloca inhibits protecting the stack.
   nw="$nw -Wstack-protector"
 
+  # Emacs's use of partly-const functions such as Fgnutls_available_p
+  # make this option problematic.
+  nw="$nw -Wsuggest-attribute=const"
+
   # Emacs's use of partly-pure functions such as CHECK_TYPE make this
   # option problematic.
   nw="$nw -Wsuggest-attribute=pure"
@@ -1974,7 +1978,6 @@ fi
 if test "$window_system" = none && test "$gl_gcc_warnings" = yes; then
    # Too many warnings for now.
    nw=
-   nw="$nw -Wsuggest-attribute=const"
    nw="$nw -Wsuggest-attribute=noreturn"
    gl_MANYWARN_COMPLEMENT([WARN_CFLAGS], [$WARN_CFLAGS], [$nw])
 

diff --git a/lib-src/ChangeLog b/lib-src/ChangeLog
index e9205fdd12a465c2909d3d8475db5245bf869bf5..7cbf3274946129683404c56e45896f5286c47dd7 100644 (file)
--- a/lib-src/ChangeLog
+++ b/lib-src/ChangeLog
@@ -1,3 +1,9 @@
+2015-01-16  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Give up on -Wsuggest-attribute=const
+       * make-docfile.c (write_globals):
+       Remove special hack for Fnext_read_file_uses_dialog_p
+
 2015-01-13  Paul Eggert  <eggert@cs.ucla.edu>
 
        Don't say Fnext_read_file_uses_dialog_p is const

diff --git a/lib-src/make-docfile.c b/lib-src/make-docfile.c
index 741fa4bfa425561b6d1a92c690f741c5d919576a..79d421a0a8ec09542b1634aeab3d3d3cfefe0c67 100644 (file)
--- a/lib-src/make-docfile.c
+++ b/lib-src/make-docfile.c
@@ -729,18 +729,6 @@ write_globals (void)
 
          if (globals[i].flags & DEFUN_const)
            fputs (" ATTRIBUTE_CONST", stdout);
-         else if (strcmp (globals[i].name, "Fnext_read_file_uses_dialog_p")
-                  == 0)
-           {
-             /* It would be nice to have a cleaner way to deal with this
-                special hack.  */
-             fputs (("\n"
-                     "#if ! (defined USE_GTK || defined USE_MOTIF \\\n"
-                     "       || defined HAVE_NS || defined HAVE_NTGUI)\n"
-                     "\tATTRIBUTE_CONST\n"
-                     "#endif\n"),
-                    stdout);
-           }
 
          puts (";");
        }

diff --git a/src/ChangeLog b/src/ChangeLog
index 40d8b267d8454ff1cad9d233b65c751081e65c1c..ae634f318f039b8cc9c5eb5d2de5fc8c287d83a9 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,14 @@
+2015-01-16  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Give up on -Wsuggest-attribute=const
+       * decompress.c (Fzlib_available_p):
+       * gnutls.c (Fgnutls_available_p):
+       * gtkutil.h (xg_uses_old_file_dialog):
+       * xdisp.c (Ftool_bar_height):
+       * xmenu.c (popup_activated):
+       No longer const, since it's not const on at lest some
+       configurations, and we shouldn't lie to the compiler.
+
 2015-01-15  Eli Zaretskii  <eliz@gnu.org>
 
        * fileio.c: Include binary-io.h.

diff --git a/src/decompress.c b/src/decompress.c
index b78dacee207251b3221bb8d7dacbdd772e395921..b14f0a2cd79f75f7c976d6ddc6de6586ae4bc548 100644 (file)
--- a/src/decompress.c
+++ b/src/decompress.c
@@ -88,8 +88,7 @@ unwind_decompress (void *ddata)
 }
 
 DEFUN ("zlib-available-p", Fzlib_available_p, Szlib_available_p, 0, 0, 0,
-       doc: /* Return t if zlib decompression is available in this instance of Emacs.  */
-       attributes: const)
+       doc: /* Return t if zlib decompression is available in this instance of Emacs.  */)
      (void)
 {
 #ifdef WINDOWSNT

diff --git a/src/gnutls.c b/src/gnutls.c
index 909542f42124ce72fb23e862e5ea27cd375c3b56..35f0eb48bc110665fc3e3a4ac2d4b4573c6915fe 100644 (file)
--- a/src/gnutls.c
+++ b/src/gnutls.c
@@ -1619,8 +1619,7 @@ This function may also return `gnutls-e-again', or
 #endif /* HAVE_GNUTLS */
 
 DEFUN ("gnutls-available-p", Fgnutls_available_p, Sgnutls_available_p, 0, 0, 0,
-       doc: /* Return t if GnuTLS is available in this instance of Emacs.  */
-       attributes: const)
+       doc: /* Return t if GnuTLS is available in this instance of Emacs.  */)
      (void)
 {
 #ifdef HAVE_GNUTLS

diff --git a/src/gtkutil.h b/src/gtkutil.h
index 7d712c92e0dad9ef3bbe37cd8e131c6786942e69..0ac49ca7db58bf9e46d0de082fd9fabae2823041 100644 (file)
--- a/src/gtkutil.h
+++ b/src/gtkutil.h
@@ -78,7 +78,7 @@ typedef struct xg_menu_item_cb_data_
 
 } xg_menu_item_cb_data;
 
-extern bool xg_uses_old_file_dialog (void) ATTRIBUTE_CONST;
+extern bool xg_uses_old_file_dialog (void);
 
 extern char *xg_get_file_name (struct frame *f,
                                char *prompt,

diff --git a/src/xdisp.c b/src/xdisp.c
index 041a022a370c41eaa8d7e46dc318b0623629242e..f006f8e0b94fe32b74c134f0385abc7a657cbd5d 100644 (file)
--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -12276,8 +12276,7 @@ DEFUN ("tool-bar-height", Ftool_bar_height, Stool_bar_height,
        0, 2, 0,
        doc: /* Return the number of lines occupied by the tool bar of FRAME.
 If FRAME is nil or omitted, use the selected frame.  Optional argument
-PIXELWISE non-nil means return the height of the tool bar in pixels.  */
-       attributes: const)
+PIXELWISE non-nil means return the height of the tool bar in pixels.  */)
   (Lisp_Object frame, Lisp_Object pixelwise)
 {
   int height = 0;

diff --git a/src/xmenu.c b/src/xmenu.c
index 9063a8a2a52dc7399479ffe631af0d2c4a0db500..fdf1f6f4d84e0cfe5c3a187206d7324c2cb19703 100644 (file)
--- a/src/xmenu.c
+++ b/src/xmenu.c
@@ -2288,7 +2288,7 @@ x_menu_show (struct frame *f, int x, int y, int menuflags,
 /* Detect if a dialog or menu has been posted.  MSDOS has its own
    implementation on msdos.c.  */
 
-int ATTRIBUTE_CONST
+int
 popup_activated (void)
 {
   return popup_activated_flag;