Remove some unnecessary declarations and configury

configure checks for declarations of free, malloc, and realloc; but
the results are only used in a single spot: utils.c.  I think these
checks are long since obsolete, so this patch removes them.

Grepping the entire tree for these HAVE_DECL_ symbols, the only uses
of possible interest to gdb come from bfd/sysdep.h; but this is not
(nor should be) included by gdb.  (And furthermore I think the code
there is probably also obsolete.)

gdb/ChangeLog
2018-07-24  Tom Tromey  <tom@tromey.com>

	* utils.c (malloc, realloc, free): Don't declare.
	* configure, config.in: Rebuild.
	* configure.ac: Don't check for declarations of free, malloc, or
	realloc.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index efa6a5a0a0f94bd2a12610041ee179ca6ec7e46a..11f78c59c586b884d6b498ea1e19b3ba6ff5ba51 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,10 @@
+2018-07-24  Tom Tromey  <tom@tromey.com>
+
+       * utils.c (malloc, realloc, free): Don't declare.
+       * configure, config.in: Rebuild.
+       * configure.ac: Don't check for declarations of free, malloc, or
+       realloc.
+
 2018-07-22  Simon Marchi  <simon.marchi@polymtl.ca>
 
        * aarch64-linux-nat.c

diff --git a/gdb/config.in b/gdb/config.in
index 527290296ecd30cf2fb856d6c3f2d2a8957c3983..01acda124b940b3802b993da8c1675ecfcd98483 100644 (file)
--- a/gdb/config.in
+++ b/gdb/config.in
@@ -102,26 +102,14 @@
 /* Define to 1 if you have the declaration of `ffs', and to 0 if you don't. */
 #undef HAVE_DECL_FFS
 
-/* Define to 1 if you have the declaration of `free', and to 0 if you don't.
-   */
-#undef HAVE_DECL_FREE
-
 /* Define to 1 if you have the declaration of `getthrds', and to 0 if you
    don't. */
 #undef HAVE_DECL_GETTHRDS
 
-/* Define to 1 if you have the declaration of `malloc', and to 0 if you don't.
-   */
-#undef HAVE_DECL_MALLOC
-
 /* Define to 1 if you have the declaration of `ptrace', and to 0 if you don't.
    */
 #undef HAVE_DECL_PTRACE
 
-/* Define to 1 if you have the declaration of `realloc', and to 0 if you
-   don't. */
-#undef HAVE_DECL_REALLOC
-
 /* Define to 1 if you have the declaration of `snprintf', and to 0 if you
    don't. */
 #undef HAVE_DECL_SNPRINTF

diff --git a/gdb/configure b/gdb/configure
index 3ee88e4eb3910c0b572a9c81b81851ff09b9b4e0..9cd0036848cbf58bbb29802098eed4647feb0a0b 100755 (executable)
--- a/gdb/configure
+++ b/gdb/configure
@@ -12394,36 +12394,6 @@ _ACEOF
 
 
 
-ac_fn_c_check_decl "$LINENO" "free" "ac_cv_have_decl_free" "$ac_includes_default"
-if test "x$ac_cv_have_decl_free" = xyes; then :
-  ac_have_decl=1
-else
-  ac_have_decl=0
-fi
-
-cat >>confdefs.h <<_ACEOF
-#define HAVE_DECL_FREE $ac_have_decl
-_ACEOF
-ac_fn_c_check_decl "$LINENO" "malloc" "ac_cv_have_decl_malloc" "$ac_includes_default"
-if test "x$ac_cv_have_decl_malloc" = xyes; then :
-  ac_have_decl=1
-else
-  ac_have_decl=0
-fi
-
-cat >>confdefs.h <<_ACEOF
-#define HAVE_DECL_MALLOC $ac_have_decl
-_ACEOF
-ac_fn_c_check_decl "$LINENO" "realloc" "ac_cv_have_decl_realloc" "$ac_includes_default"
-if test "x$ac_cv_have_decl_realloc" = xyes; then :
-  ac_have_decl=1
-else
-  ac_have_decl=0
-fi
-
-cat >>confdefs.h <<_ACEOF
-#define HAVE_DECL_REALLOC $ac_have_decl
-_ACEOF
 ac_fn_c_check_decl "$LINENO" "snprintf" "ac_cv_have_decl_snprintf" "$ac_includes_default"
 if test "x$ac_cv_have_decl_snprintf" = xyes; then :
   ac_have_decl=1

diff --git a/gdb/configure.ac b/gdb/configure.ac
index dfd38a730f4c4686878b3f3f2287136eeabec470..13bc5f9a8f2285f12ec932719142569c748a55b5 100644 (file)
--- a/gdb/configure.ac
+++ b/gdb/configure.ac
@@ -1317,7 +1317,7 @@ AC_CHECK_HEADERS(term.h, [], [],
 
 libiberty_INIT
 
-AC_CHECK_DECLS([free, malloc, realloc, snprintf])
+AC_CHECK_DECLS([snprintf])
 AM_LC_MESSAGES
 
 # ----------------------- #

diff --git a/gdb/utils.c b/gdb/utils.c
index bfcb45bc950971f431f9cd37b2497731edea1b5e..7a8c80c64edbe3a2936c8c605b5d18bd391452c8 100644 (file)
--- a/gdb/utils.c
+++ b/gdb/utils.c
@@ -72,16 +72,6 @@
 #include <algorithm>
 #include "common/pathstuff.h"
 
-#if !HAVE_DECL_MALLOC
-extern PTR malloc ();          /* ARI: PTR */
-#endif
-#if !HAVE_DECL_REALLOC
-extern PTR realloc ();         /* ARI: PTR */
-#endif
-#if !HAVE_DECL_FREE
-extern void free ();
-#endif
-
 void (*deprecated_error_begin_hook) (void);
 
 /* Prototypes for local functions */