From db86b2dce129156cc7d749f88ce1b324db0ce77e Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Thu, 25 Oct 2007 21:01:55 +0000 Subject: [PATCH] bfd/ * bfd-in.h (STRING_COMMA_LEN): Don't handle NULL STR case. * bfd-in2.h: Regenerate. ld/ * pe-dll.c (autofilter_symbollist_generic) (autofilter_symbollist_i386, autofilter_liblist) (autofilter_objlist, autofilter_symbolprefixlist) (autofilter_symbolsuffixlist): Don't use STRING_COMMA_LEN with NULL. --- bfd/ChangeLog | 5 +++++ bfd/bfd-in.h | 5 ++--- bfd/bfd-in2.h | 5 ++--- ld/ChangeLog | 8 ++++++++ ld/pe-dll.c | 12 ++++++------ 5 files changed, 23 insertions(+), 12 deletions(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 7034943990e..d9d53304c8d 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,8 @@ +2007-10-25 Pedro Alves + + * bfd-in.h (STRING_COMMA_LEN): Don't handle NULL STR case. + * bfd-in2.h: Regenerate. + 2007-10-25 Daniel Jacobowitz * elf32-ppc.c (ppc_elf_merge_obj_attributes): Add support for diff --git a/bfd/bfd-in.h b/bfd/bfd-in.h index 07fdf036497..be5c2e59fa7 100644 --- a/bfd/bfd-in.h +++ b/bfd/bfd-in.h @@ -46,9 +46,8 @@ extern "C" { /* This is a utility macro to handle the situation where the code wants to place a constant string into the code, followed by a comma and then the length of the string. Doing this by hand - is error prone, so using this macro is safer. The macro will - also safely handle the case where a NULL is passed as the arg. */ -#define STRING_COMMA_LEN(STR) (STR), ((STR) ? sizeof (STR) - 1 : 0) + is error prone, so using this macro is safer. */ +#define STRING_COMMA_LEN(STR) (STR), (sizeof (STR) - 1) /* Unfortunately it is not possible to use the STRING_COMMA_LEN macro to create the arguments to another macro, since the preprocessor will mis-count the number of arguments to the outer macro (by not diff --git a/bfd/bfd-in2.h b/bfd/bfd-in2.h index 35aad28a5c6..5e548674e1f 100644 --- a/bfd/bfd-in2.h +++ b/bfd/bfd-in2.h @@ -53,9 +53,8 @@ extern "C" { /* This is a utility macro to handle the situation where the code wants to place a constant string into the code, followed by a comma and then the length of the string. Doing this by hand - is error prone, so using this macro is safer. The macro will - also safely handle the case where a NULL is passed as the arg. */ -#define STRING_COMMA_LEN(STR) (STR), ((STR) ? sizeof (STR) - 1 : 0) + is error prone, so using this macro is safer. */ +#define STRING_COMMA_LEN(STR) (STR), (sizeof (STR) - 1) /* Unfortunately it is not possible to use the STRING_COMMA_LEN macro to create the arguments to another macro, since the preprocessor will mis-count the number of arguments to the outer macro (by not diff --git a/ld/ChangeLog b/ld/ChangeLog index 5a4b2e29dcf..104a0ff8a2a 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,11 @@ +2007-10-25 Pedro Alves + + * pe-dll.c (autofilter_symbollist_generic) + (autofilter_symbollist_i386, autofilter_liblist) + (autofilter_objlist, autofilter_symbolprefixlist) + (autofilter_symbolsuffixlist): Don't use STRING_COMMA_LEN with + NULL. + 2007-10-24 H.J. Lu * configure.in: Move AM_PROG_LIBTOOL after AC_GNU_SOURCE. diff --git a/ld/pe-dll.c b/ld/pe-dll.c index cdbfa70f0e3..d7547477008 100644 --- a/ld/pe-dll.c +++ b/ld/pe-dll.c @@ -195,7 +195,7 @@ static const autofilter_entry_type autofilter_symbollist_generic[] = /* Runtime pseudo-reloc. */ { STRING_COMMA_LEN ("_pei386_runtime_relocator") }, { STRING_COMMA_LEN ("do_pseudo_reloc") }, - { STRING_COMMA_LEN (NULL) } + { NULL, 0 } }; static const autofilter_entry_type autofilter_symbollist_i386[] = @@ -231,7 +231,7 @@ static const autofilter_entry_type autofilter_symbollist_i386[] = { STRING_COMMA_LEN ("_impure_ptr") }, { STRING_COMMA_LEN ("_fmode") }, { STRING_COMMA_LEN ("environ") }, - { STRING_COMMA_LEN (NULL) } + { NULL, 0 } }; #define PE_ARCH_i386 1 @@ -321,7 +321,7 @@ static const autofilter_entry_type autofilter_liblist[] = { STRING_COMMA_LEN ("libsupc++") }, { STRING_COMMA_LEN ("libobjc") }, { STRING_COMMA_LEN ("libgcj") }, - { STRING_COMMA_LEN (NULL) } + { NULL, 0 } }; static const autofilter_entry_type autofilter_objlist[] = @@ -336,7 +336,7 @@ static const autofilter_entry_type autofilter_objlist[] = { STRING_COMMA_LEN ("gcrt2.o") }, { STRING_COMMA_LEN ("crtbegin.o") }, { STRING_COMMA_LEN ("crtend.o") }, - { STRING_COMMA_LEN (NULL) } + { NULL, 0 } }; static const autofilter_entry_type autofilter_symbolprefixlist[] = @@ -350,13 +350,13 @@ static const autofilter_entry_type autofilter_symbolprefixlist[] = { STRING_COMMA_LEN ("_nm_") }, /* Don't export symbols specifying internal DLL layout. */ { STRING_COMMA_LEN ("_head_") }, - { STRING_COMMA_LEN (NULL) } + { NULL, 0 } }; static const autofilter_entry_type autofilter_symbolsuffixlist[] = { { STRING_COMMA_LEN ("_iname") }, - { STRING_COMMA_LEN (NULL) } + { NULL, 0 } }; #define U(str) (pe_details->underscored ? "_" str : str) -- 2.30.2