Don't declare xmalloc or xrealloc in bucomm.h
authorAlan Modra <amodra@gmail.com>
Tue, 1 Aug 2023 22:23:26 +0000 (07:53 +0930)
committerAlan Modra <amodra@gmail.com>
Tue, 1 Aug 2023 22:23:26 +0000 (07:53 +0930)
It's better to include the proper header, which has declarations with
various attributes.  Commit 096aefc040 in 1994 introduced this wart.

* bucomm.h (xmalloc, xrealloc): Delete declaration.
* od-macho.c: Include libiberty.h.
* od-xcoff.c: Include libiberty.h.

binutils/bucomm.h
binutils/od-macho.c
binutils/od-xcoff.c

index 0c11c0b868d87d876a5a776863bb78b8176c7fd6..b70879078ee0e53c25f3e5b15da80d08c7d1a71c 100644 (file)
@@ -76,12 +76,6 @@ extern void set_times (const char *, const struct stat *);
 extern int smart_rename (const char *, const char *, int,
                         struct stat *, bool);
 
-
-/* In libiberty.  */
-void *xmalloc (size_t);
-
-void *xrealloc (void *, size_t);
-
 #if __GNUC__ >= 7
 #define _mul_overflow(a, b, res) __builtin_mul_overflow (a, b, res)
 #else
index 17253d4f70bb99fe2e54223cdf045ca41600b3f3..c2bd7d38a89c0153cc1fd6ca1a460e23eac166a9 100644 (file)
@@ -23,6 +23,7 @@
 #include <stddef.h>
 #include <time.h>
 #include "safe-ctype.h"
+#include "libiberty.h"
 #include "bfd.h"
 #include "objdump.h"
 #include "bucomm.h"
index 92d17e99fbf1fa83b23900a3beeeedd5a846d31b..e6b2f08c24505d3c04b2b4d4f4562d7c129a0805 100644 (file)
@@ -23,6 +23,7 @@
 #include <stddef.h>
 #include <time.h>
 #include "safe-ctype.h"
+#include "libiberty.h"
 #include "bfd.h"
 #include "objdump.h"
 #include "bucomm.h"