diff options
| author | Sergio Durigan Junior <sergiodj@linux.vnet.ibm.com> | 2013-10-09 05:23:23 +0000 |
|---|---|---|
| committer | Sergio Durigan Junior <sergiodj@linux.vnet.ibm.com> | 2013-10-09 05:23:23 +0000 |
| commit | 03fc0bdbb22d4f9b66fd8c43bf0fcf32a7066d75 (patch) | |
| tree | bca8067e327ec7ec073fa270b6c3ebb50a5ff27d /bfd | |
| parent | 3e97608bb7ca2422119703ac1929384f2670a88e (diff) | |
| download | binutils-redhat-03fc0bdbb22d4f9b66fd8c43bf0fcf32a7066d75.tar.gz | |
bfd/ChangeLog
2013-10-09 Sergio Durigan Junior <sergiodj@redhat.com>
PR binutils/15993
* elf32-m32c.c (dump_symtab): Add missing "break;" statement on each
"case". Reindent "switch" statements.
Diffstat (limited to 'bfd')
| -rw-r--r-- | bfd/ChangeLog | 6 | ||||
| -rw-r--r-- | bfd/elf32-m32c.c | 80 |
2 files changed, 69 insertions, 17 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index e8534ddaee..a0110199fe 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,9 @@ +2013-10-09 Sergio Durigan Junior <sergiodj@redhat.com> + + PR binutils/15993 + * elf32-m32c.c (dump_symtab): Add missing "break;" statement on each + "case". Reindent "switch" statements. + 2013-10-08 Tom Tromey <tromey@redhat.com> * bfd-in2.h: Rebuild. diff --git a/bfd/elf32-m32c.c b/bfd/elf32-m32c.c index 5205b7b7ce..8e77c5ec17 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 " |