aboutsummaryrefslogtreecommitdiff
path: root/bfd
diff options
context:
space:
mode:
authorAlan Modra <amodra@gmail.com>2021-03-31 10:02:08 +1030
committerAlan Modra <amodra@gmail.com>2021-03-31 10:47:59 +1030
commit57ae980e3290c0c1a9fb4a93144cc5b24457f05a (patch)
treeb639bd1f64e84addc898ae6a29c97ffbe7c757ad /bfd
parentedf71419e06105a73c59a95d3e74557a6c6f313b (diff)
downloadfsf-binutils-gdb-57ae980e3290c0c1a9fb4a93144cc5b24457f05a.zip
fsf-binutils-gdb-57ae980e3290c0c1a9fb4a93144cc5b24457f05a.tar.gz
fsf-binutils-gdb-57ae980e3290c0c1a9fb4a93144cc5b24457f05a.tar.bz2
Include string.h in bfd.h and delete LITMEMCPY, LITSTRCPY
This fixes the issue that startswith depends on strncpy being declared, and not all projects using bfd.h include string.h before bfd.h. I've also deleted some macros that don't find much use anywhere. bfd/ * bfd-in.h: Include string.h. (LITMEMCPY, LITSTRCPY): Delete. * bfd-in2.h: Regenerate. binutils/ * prdbg.c (pr_function_type): Replace LITSTTCPY with strcpy.
Diffstat (limited to 'bfd')
-rw-r--r--bfd/ChangeLog6
-rw-r--r--bfd/bfd-in.h8
-rw-r--r--bfd/bfd-in2.h8
3 files changed, 8 insertions, 14 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index a94223d..5936279 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,9 @@
+2021-03-31 Alan Modra <amodra@gmail.com>
+
+ * bfd-in.h: Include string.h.
+ (LITMEMCPY, LITSTRCPY): Delete.
+ * bfd-in2.h: Regenerate.
+
2021-03-30 Alan Modra <amodra@gmail.com>
* elflink.c (elf_link_add_object_symbols): Don't set h->indx
diff --git a/bfd/bfd-in.h b/bfd/bfd-in.h
index fda9fe0..f90f718 100644
--- a/bfd/bfd-in.h
+++ b/bfd/bfd-in.h
@@ -37,6 +37,7 @@ extern "C" {
#include "bfd_stdint.h"
#include "diagnostics.h"
#include <stdarg.h>
+#include <string.h>
#include <sys/stat.h>
#if defined (__STDC__) || defined (ALMOST_STDC) || defined (HAVE_STRINGIZE)
@@ -57,13 +58,6 @@ extern "C" {
is error prone, so using this macro is safer. */
#define STRING_COMMA_LEN(STR) (STR), (sizeof (STR) - 1)
- /* strcpy() can have a similar problem, but since we know we are
- copying a constant string, we can use memcpy which will be faster
- since there is no need to check for a NUL byte inside STR. We
- can also save time if we do not need to copy the terminating NUL. */
-#define LITMEMCPY(DEST,STR2) memcpy ((DEST), (STR2), sizeof (STR2) - 1)
-#define LITSTRCPY(DEST,STR2) memcpy ((DEST), (STR2), sizeof (STR2))
-
#define BFD_SUPPORTS_PLUGINS @supports_plugins@
/* The word size used by BFD on the host. This may be 64 with a 32
diff --git a/bfd/bfd-in2.h b/bfd/bfd-in2.h
index 8874da8..426ab86d 100644
--- a/bfd/bfd-in2.h
+++ b/bfd/bfd-in2.h
@@ -44,6 +44,7 @@ extern "C" {
#include "bfd_stdint.h"
#include "diagnostics.h"
#include <stdarg.h>
+#include <string.h>
#include <sys/stat.h>
#if defined (__STDC__) || defined (ALMOST_STDC) || defined (HAVE_STRINGIZE)
@@ -64,13 +65,6 @@ extern "C" {
is error prone, so using this macro is safer. */
#define STRING_COMMA_LEN(STR) (STR), (sizeof (STR) - 1)
- /* strcpy() can have a similar problem, but since we know we are
- copying a constant string, we can use memcpy which will be faster
- since there is no need to check for a NUL byte inside STR. We
- can also save time if we do not need to copy the terminating NUL. */
-#define LITMEMCPY(DEST,STR2) memcpy ((DEST), (STR2), sizeof (STR2) - 1)
-#define LITSTRCPY(DEST,STR2) memcpy ((DEST), (STR2), sizeof (STR2))
-
#define BFD_SUPPORTS_PLUGINS @supports_plugins@
/* The word size used by BFD on the host. This may be 64 with a 32