From 87da1eab9b570eb2aec5cd017d90ac147677ff53 Mon Sep 17 00:00:00 2001 From: sergiodj Date: Wed, 9 Oct 2013 05:23:23 +0000 Subject: bfd/ChangeLog 2013-10-09 Sergio Durigan Junior PR binutils/15993 * elf32-m32c.c (dump_symtab): Add missing "break;" statement on each "case". Reindent "switch" statements. --- bfd/ChangeLog | 6 +++++ bfd/elf32-m32c.c | 80 ++++++++++++++++++++++++++++++++++++++++++++------------ 2 files changed, 69 insertions(+), 17 deletions(-) (limited to 'bfd') diff --git a/bfd/ChangeLog b/bfd/ChangeLog index e8534ddaee9..a0110199fea 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,9 @@ +2013-10-09 Sergio Durigan Junior + + PR binutils/15993 + * elf32-m32c.c (dump_symtab): Add missing "break;" statement on each + "case". Reindent "switch" statements. + 2013-10-08 Tom Tromey * bfd-in2.h: Rebuild. diff --git a/bfd/elf32-m32c.c b/bfd/elf32-m32c.c index 5205b7b7ce5..8e77c5ec172 100644 --- a/bfd/elf32-m32c.c +++ b/bfd/elf32-m32c.c @@ -921,32 +921,78 @@ dump_symtab (bfd * abfd, void *internal_syms, void *external_syms) { switch (ELF_ST_TYPE (isym->st_info)) { - case STT_FUNC: st_info_str = "STT_FUNC"; - case STT_SECTION: st_info_str = "STT_SECTION"; - case STT_FILE: st_info_str = "STT_FILE"; - case STT_OBJECT: st_info_str = "STT_OBJECT"; - case STT_TLS: st_info_str = "STT_TLS"; - default: st_info_str = ""; + case STT_FUNC: + st_info_str = "STT_FUNC"; + break; + + case STT_SECTION: + st_info_str = "STT_SECTION"; + break; + + case STT_FILE: + st_info_str = "STT_FILE"; + break; + + case STT_OBJECT: + st_info_str = "STT_OBJECT"; + break; + + case STT_TLS: + st_info_str = "STT_TLS"; + break; + + default: + st_info_str = ""; } + switch (ELF_ST_BIND (isym->st_info)) { - case STB_LOCAL: st_info_stb_str = "STB_LOCAL"; - case STB_GLOBAL: st_info_stb_str = "STB_GLOBAL"; - default: st_info_stb_str = ""; + case STB_LOCAL: + st_info_stb_str = "STB_LOCAL"; + break; + + case STB_GLOBAL: + st_info_stb_str = "STB_GLOBAL"; + break; + + default: + st_info_stb_str = ""; } + switch (ELF_ST_VISIBILITY (isym->st_other)) { - case STV_DEFAULT: st_other_str = "STV_DEFAULT"; - case STV_INTERNAL: st_other_str = "STV_INTERNAL"; - case STV_PROTECTED: st_other_str = "STV_PROTECTED"; - default: st_other_str = ""; + case STV_DEFAULT: + st_other_str = "STV_DEFAULT"; + break; + + case STV_INTERNAL: + st_other_str = "STV_INTERNAL"; + break; + + case STV_PROTECTED: + st_other_str = "STV_PROTECTED"; + break; + + default: + st_other_str = ""; } + switch (isym->st_shndx) { - case SHN_ABS: st_shndx_str = "SHN_ABS"; - case SHN_COMMON: st_shndx_str = "SHN_COMMON"; - case SHN_UNDEF: st_shndx_str = "SHN_UNDEF"; - default: st_shndx_str = ""; + case SHN_ABS: + st_shndx_str = "SHN_ABS"; + break; + + case SHN_COMMON: + st_shndx_str = "SHN_COMMON"; + break; + + case SHN_UNDEF: + st_shndx_str = "SHN_UNDEF"; + break; + + default: + st_shndx_str = ""; } printf ("isym = %p st_value = %lx st_size = %lx st_name = (%lu) %s " -- cgit v1.2.1