yggdrasil/binutils-gdb
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove strneq macro and use startswith.

Browse Source 
bfd/ChangeLog:

	* ecoff.c (strneq): Remove strneq and use startswith.
	(_bfd_ecoff_slurp_armap): Likewise.

binutils/ChangeLog:

	* elfcomm.h (strneq): Remove strneq and use startswith.
	* readelf.c (ia64_process_unwind): Likewise.
	(process_note): Likewise.

gas/ChangeLog:

	* config/obj-coff.c (strneq): Remove strneq and use startswith.
	(weak_is_altname): Likewise.
	(obj_coff_section): Likewise.
	* config/tc-cr16.c (process_label_constant): Likewise.
	* config/tc-crx.c (strneq): Likewise.

include/ChangeLog:

	* opcode/cr16.h (strneq): Remove strneq and use startswith.

ld/ChangeLog:

	* ldbuildid.c (strneq): Remove strneq and use startswith.
	(validate_build_id_style): Likewise.
	(compute_build_id_size): Likewise.

opcodes/ChangeLog:

	* arm-dis.c (strneq): Remove strneq and use startswith.
	* cr16-dis.c (print_insn_cr16): Likewise.
	* score-dis.c (streq): Likewise.
	(strneq): Likewise.
	* score7-dis.c (strneq): Likewise.
This commit is contained in:
Martin Liska 2021-03-22 14:56:16 +01:00
parent d34049e8bb
commit e9b095a538
18 changed files with 65 additions and 51 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
bfd/ChangeLog
View File

@ -1,3 +1,8 @@
2021-04-01 Martin Liska <mliska@suse.cz>
* ecoff.c (strneq): Remove strneq and use startswith.
(_bfd_ecoff_slurp_armap): Likewise.
2021-04-01 Martin Liska <mliska@suse.cz>
* elf-bfd.h (bfd_section_is_ctf): Use startswith function.

5
bfd/ecoff.c
View File

@ -46,7 +46,6 @@
#include "libiberty.h"
#define streq(a, b) (strcmp ((a), (b)) == 0)
#define strneq(a, b, n) (strncmp ((a), (b), (n)) == 0)
/* This stuff is somewhat copied from coffcode.h. */
@ -2870,14 +2869,14 @@ _bfd_ecoff_slurp_armap (bfd *abfd)
return bfd_slurp_armap (abfd);
/* See if the first element is an armap. */
if (! strneq (nextname, ecoff_backend (abfd)->armap_start, ARMAP_START_LENGTH)
if (strncmp (nextname, ecoff_backend (abfd)->armap_start, ARMAP_START_LENGTH) != 0
|| nextname[ARMAP_HEADER_MARKER_INDEX] != ARMAP_MARKER
|| (nextname[ARMAP_HEADER_ENDIAN_INDEX] != ARMAP_BIG_ENDIAN
&& nextname[ARMAP_HEADER_ENDIAN_INDEX] != ARMAP_LITTLE_ENDIAN)
|| nextname[ARMAP_OBJECT_MARKER_INDEX] != ARMAP_MARKER
|| (nextname[ARMAP_OBJECT_ENDIAN_INDEX] != ARMAP_BIG_ENDIAN
&& nextname[ARMAP_OBJECT_ENDIAN_INDEX] != ARMAP_LITTLE_ENDIAN)
|| ! strneq (nextname + ARMAP_END_INDEX, ARMAP_END, sizeof ARMAP_END - 1))
|| strncmp (nextname + ARMAP_END_INDEX, ARMAP_END, sizeof ARMAP_END - 1) != 0)
{
abfd->has_armap = false;
return true;

6
binutils/ChangeLog
View File

@ -1,3 +1,9 @@
2021-04-01 Martin Liska <mliska@suse.cz>
* elfcomm.h (strneq): Remove strneq and use startswith.
* readelf.c (ia64_process_unwind): Likewise.
(process_note): Likewise.
2021-04-01 Martin Liska <mliska@suse.cz>
* dllwrap.c: Use startswith function.

1
binutils/elfcomm.h
View File

@ -47,7 +47,6 @@ extern void byte_get_64 (const unsigned char *, elf_vma *, elf_vma *);
/* This is just a bit of syntatic sugar. */
#define streq(a,b) (strcmp ((a), (b)) == 0)
#define strneq(a,b,n) (strncmp ((a), (b), (n)) == 0)
/* Structure to hold information about an archive file. */

14
binutils/readelf.c
View File

@ -8235,8 +8235,8 @@ ia64_process_unwind (Filedata * filedata)
}
}
else if (SECTION_NAME_VALID (unwsec)
&& strneq (SECTION_NAME (unwsec),
ELF_STRING_ia64_unwind_once, len))
&& startswith (SECTION_NAME (unwsec),
ELF_STRING_ia64_unwind_once))
{
/* .gnu.linkonce.ia64unw.FOO -> .gnu.linkonce.ia64unwi.FOO. */
len2 = sizeof (ELF_STRING_ia64_unwind_info_once) - 1;
@ -8245,8 +8245,8 @@ ia64_process_unwind (Filedata * filedata)
i < filedata->file_header.e_shnum;
++i, ++sec)
if (SECTION_NAME_VALID (sec)
&& strneq (SECTION_NAME (sec),
ELF_STRING_ia64_unwind_info_once, len2)
&& startswith (SECTION_NAME (sec),
ELF_STRING_ia64_unwind_info_once)
&& streq (SECTION_NAME (sec) + len2, suffix))
break;
}
@ -8258,13 +8258,13 @@ ia64_process_unwind (Filedata * filedata)
len2 = sizeof (ELF_STRING_ia64_unwind_info) - 1;
suffix = "";
if (SECTION_NAME_VALID (unwsec)
&& strneq (SECTION_NAME (unwsec), ELF_STRING_ia64_unwind, len))
&& startswith (SECTION_NAME (unwsec), ELF_STRING_ia64_unwind))
suffix = SECTION_NAME (unwsec) + len;
for (i = 0, sec = filedata->section_headers;
i < filedata->file_header.e_shnum;
++i, ++sec)
if (SECTION_NAME_VALID (sec)
&& strneq (SECTION_NAME (sec), ELF_STRING_ia64_unwind_info, len2)
&& startswith (SECTION_NAME (sec), ELF_STRING_ia64_unwind_info)
&& streq (SECTION_NAME (sec) + len2, suffix))
break;
}
@ -20449,7 +20449,7 @@ process_note (Elf_Internal_Note * pnote,
/* NetBSD-specific core file notes. */
return process_netbsd_elf_note (pnote);
else if (strneq (pnote->namedata, "SPU/", 4))
else if (startswith (pnote->namedata, "SPU/"))
{
/* SPU-specific core file notes. */
nt = pnote->namedata + 4;

8
gas/ChangeLog
View File

@ -1,3 +1,11 @@
2021-04-01 Martin Liska <mliska@suse.cz>
* config/obj-coff.c (strneq): Remove strneq and use startswith.
(weak_is_altname): Likewise.
(obj_coff_section): Likewise.
* config/tc-cr16.c (process_label_constant): Likewise.
* config/tc-crx.c (strneq): Likewise.
2021-04-01 Martin Liska <mliska@suse.cz>
* as.c (select_emulation_mode): Use startswith.

5
gas/config/obj-coff.c
View File

@ -33,7 +33,6 @@
#endif
#define streq(a,b) (strcmp ((a), (b)) == 0)
#define strneq(a,b,n) (strncmp ((a), (b), (n)) == 0)
/* I think this is probably always correct. */
#ifndef KEEP_RELOC_INFO
@ -1058,7 +1057,7 @@ obj_coff_val (int ignore ATTRIBUTE_UNUSED)
static int
weak_is_altname (const char * name)
{
return strneq (name, weak_altprefix, sizeof (weak_altprefix) - 1);
return startswith (name, weak_altprefix);
}
/* Return the name of the alternate symbol
@ -1667,7 +1666,7 @@ obj_coff_section (int ignore ATTRIBUTE_UNUSED)
/* Add SEC_LINK_ONCE and SEC_LINK_DUPLICATES_DISCARD to .gnu.linkonce
sections so adjust_reloc_syms in write.c will correctly handle
relocs which refer to non-local symbols in these sections. */
if (strneq (name, ".gnu.linkonce", sizeof (".gnu.linkonce") - 1))
if (startswith (name, ".gnu.linkonce"))
flags |= SEC_LINK_ONCE | SEC_LINK_DUPLICATES_DISCARD;
#endif

26
gas/config/tc-cr16.c
View File

@ -905,34 +905,34 @@ process_label_constant (char *str, ins * cr16_ins)
cr16_ins->rtype = BFD_RELOC_NONE;
relocatable = 1;
if (strneq (input_line_pointer, "@c", 2))
if (startswith (input_line_pointer, "@c"))
symbol_with_at = 1;
if (strneq (input_line_pointer, "@l", 2)
|| strneq (input_line_pointer, ":l", 2))
if (startswith (input_line_pointer, "@l")
|| startswith (input_line_pointer, ":l"))
symbol_with_l = 1;
if (strneq (input_line_pointer, "@m", 2)
|| strneq (input_line_pointer, ":m", 2))
if (startswith (input_line_pointer, "@m")
|| startswith (input_line_pointer, ":m"))
symbol_with_m = 1;
if (strneq (input_line_pointer, "@s", 2)
|| strneq (input_line_pointer, ":s", 2))
if (startswith (input_line_pointer, "@s")
|| startswith (input_line_pointer, ":s"))
symbol_with_s = 1;
if (strneq (input_line_pointer, "@cGOT", 5)
|| strneq (input_line_pointer, "@cgot", 5))
if (startswith (input_line_pointer, "@cGOT")
|| startswith (input_line_pointer, "@cgot"))
{
if (GOT_symbol == NULL)
GOT_symbol = symbol_find_or_make (GLOBAL_OFFSET_TABLE_NAME);
symbol_with_at_gotc = 1;
}
else if (strneq (input_line_pointer, "@GOT", 4)
|| strneq (input_line_pointer, "@got", 4))
else if (startswith (input_line_pointer, "@GOT")
|| startswith (input_line_pointer, "@got"))
{
if ((strneq (input_line_pointer, "+", 1))
|| (strneq (input_line_pointer, "-", 1)))
if ((startswith (input_line_pointer, "+"))
|| (startswith (input_line_pointer, "-")))
as_warn (_("GOT bad expression with %s."), input_line_pointer);
if (GOT_symbol == NULL)

1
gas/config/tc-crx.c
View File

@ -43,7 +43,6 @@
/* Utility macros for string comparison. */
#define streq(a, b) (strcmp (a, b) == 0)
#define strneq(a, b, c) (strncmp (a, b, c) == 0)
/* Assign a number NUM, shifted by SHIFT bytes, into a location
pointed by index BYTE of array 'output_opcode'. */

4
include/ChangeLog
View File

@ -1,3 +1,7 @@
2021-04-01 Martin Liska <mliska@suse.cz>
* opcode/cr16.h (strneq): Remove strneq and use startswith.
2021-03-31 Alan Modra <amodra@gmail.com>
* bfdlink.h: Replace bfd_boolean with bool throughout.

1
include/opcode/cr16.h
View File

@ -425,7 +425,6 @@ extern const char * cr16_no_op_insn[];
/* Utility macros for string comparison. */
#define streq(a, b) (strcmp (a, b) == 0)
#define strneq(a, b, c) (strncmp (a, b, c) == 0)
/* Long long type handling. */
/* Replace all appearances of 'long long int' with LONGLONG. */

6
ld/ChangeLog
View File

@ -1,3 +1,9 @@
2021-04-01 Martin Liska <mliska@suse.cz>
* ldbuildid.c (strneq): Remove strneq and use startswith.
(validate_build_id_style): Likewise.
(compute_build_id_size): Likewise.
2021-03-31 Alan Modra <amodra@gmail.com>
* sysdep.h (POISON_BFD_BOOLEAN): Define.

7
ld/ldbuildid.c
View File

@ -30,13 +30,12 @@
#endif
#define streq(a,b) strcmp ((a), (b)) == 0
#define strneq(a,b,n) strncmp ((a), (b), (n)) == 0
bool
validate_build_id_style (const char *style)
{
if ((streq (style, "md5")) || (streq (style, "sha1"))
|| (streq (style, "uuid")) || (strneq (style, "0x", 2)))
|| (streq (style, "uuid")) || (startswith (style, "0x")))
return true;
return false;
@ -51,7 +50,7 @@ compute_build_id_size (const char *style)
if (streq (style, "sha1"))
return 160 / 8;
if (strneq (style, "0x", 2))
if (startswith (style, "0x"))
{
bfd_size_type size = 0;
/* ID is in string form (hex). Count the bytes. */
@ -156,7 +155,7 @@ generate_build_id (bfd *abfd,
(size_t) size < sizeof (UUID) ? (size_t) size : sizeof (UUID));
#endif /* __MINGW32__ */
}
else if (strneq (style, "0x", 2))
else if (startswith (style, "0x"))
{
/* ID is in string form (hex). Convert to bits. */
const char *id = style + 2;

8
opcodes/ChangeLog
View File

@ -1,3 +1,11 @@
2021-04-01 Martin Liska <mliska@suse.cz>
* arm-dis.c (strneq): Remove strneq and use startswith.
* cr16-dis.c (print_insn_cr16): Likewise.
* score-dis.c (streq): Likewise.
(strneq): Likewise.
* score7-dis.c (strneq): Likewise.
2021-04-01 Alan Modra <amodra@gmail.com>
PR 27675

5
opcodes/arm-dis.c
View File

@ -39,11 +39,6 @@
#include "elf/arm.h"
#include "mach-o.h"
/* FIXME: Belongs in global header. */
#ifndef strneq
#define strneq(a,b,n) (strncmp ((a), (b), (n)) == 0)
#endif
/* Cached mapping symbol state. */
enum map_type
{

2
opcodes/cr16-dis.c
View File

@ -800,7 +800,7 @@ print_insn_cr16 (bfd_vma memaddr, struct disassemble_info *info)
/* If found, print the instruction's mnemonic and arguments. */
if (is_decoded > 0 && (cr16_words[0] != 0 || cr16_words[1] != 0))
{
if (strneq (instruction->mnemonic, "cinv", 4))
if (startswith (instruction->mnemonic, "cinv"))
info->fprintf_func (info->stream,"%s",
getcinvstring (instruction->mnemonic));
else

8
opcodes/score-dis.c
View File

@ -477,14 +477,6 @@ static struct score_opcode score_opcodes[] =
};
#ifndef streq
#define streq(a,b) (strcmp ((a), (b)) == 0)
#endif
#ifndef strneq
#define strneq(a,b,n) (strncmp ((a), (b), (n)) == 0)
#endif
#ifndef NUM_ELEM
#define NUM_ELEM(a) (sizeof (a) / sizeof (a)[0])
#endif

4
opcodes/score7-dis.c
View File

@ -37,10 +37,6 @@
#define streq(a,b) (strcmp ((a), (b)) == 0)
#endif
#ifndef strneq
#define strneq(a,b,n) (strncmp ((a), (b), (n)) == 0)
#endif
#ifndef NUM_ELEM
#define NUM_ELEM(a) (sizeof (a) / sizeof (a)[0])
#endif