-Wimplicit-fallthrough warning fixes

Comment changes.

bfd/
	* coff-h8300.c: Spell fall through comments consistently.
	* coffgen.c: Likewise.
	* elf32-hppa.c: Likewise.
	* elf32-ppc.c: Likewise.
	* elf32-score.c: Likewise.
	* elf32-score7.c: Likewise.
	* elf64-ppc.c: Likewise.
	* elfxx-aarch64.c: Likewise.
	* elfxx-mips.c: Likewise.
	* cpu-ns32k.c: Add missing fall through comments.
	* elf-m10300.c: Likewise.
	* elf32-arm.c: Likewise.
	* elf32-avr.c: Likewise.
	* elf32-bfin.c: Likewise.
	* elf32-frv.c: Likewise.
	* elf32-i386.c: Likewise.
	* elf32-microblaze.c: Likewise.
	* elf32-nds32.c: Likewise.
	* elf32-ppc.c: Likewise.
	* elf32-rl78.c: Likewise.
	* elf32-rx.c: Likewise.
	* elf32-s390.c: Likewise.
	* elf32-sh.c: Likewise.
	* elf32-tic6x.c: Likewise.
	* elf64-ia64-vms.c: Likewise.
	* elf64-ppc.c: Likewise.
	* elf64-s390.c: Likewise.
	* elf64-x86-64.c: Likewise.
	* elflink.c: Likewise.
	* elfnn-aarch64.c: Likewise.
	* elfnn-ia64.c: Likewise.
	* ieee.c: Likewise.
	* oasys.c: Likewise.
	* pdp11.c: Likewise.
	* srec.c: Likewise.
	* versados.c: Likewise.
opcodes/
	* aarch64-opc.c: Spell fall through comments consistently.
	* i386-dis.c: Likewise.
	* aarch64-dis.c: Add missing fall through comments.
	* aarch64-opc.c: Likewise.
	* arc-dis.c: Likewise.
	* arm-dis.c: Likewise.
	* i386-dis.c: Likewise.
	* m68k-dis.c: Likewise.
	* mep-asm.c: Likewise.
	* ns32k-dis.c: Likewise.
	* sh-dis.c: Likewise.
	* tic4x-dis.c: Likewise.
	* tic6x-dis.c: Likewise.
	* vax-dis.c: Likewise.
binutils/
	* dlltool.c: Spell fall through comments consistently.
	* objcopy.c: Likewise.
	* readelf.c: Likewise.
	* dwarf.c: Add missing fall through comments.
	* elfcomm.c: Likewise.
	* sysinfo.y: Likewise.
	* readelf.c: Likewise.  Also remove extraneous comments.
gas/
	* app.c: Add missing fall through comments.
	* dw2gencfi.c: Likewise.
	* expr.c: Likewise.
	* config/tc-alpha.c: Likewise.
	* config/tc-arc.c: Likewise.
	* config/tc-arm.c: Likewise.
	* config/tc-cr16.c: Likewise.
	* config/tc-crx.c: Likewise.
	* config/tc-dlx.c: Likewise.
	* config/tc-h8300.c: Likewise.
	* config/tc-hppa.c: Likewise.
	* config/tc-i370.c: Likewise.
	* config/tc-i386.c: Likewise.
	* config/tc-i960.c: Likewise.
	* config/tc-ia64.c: Likewise.
	* config/tc-m68hc11.c: Likewise.
	* config/tc-m68k.c: Likewise.
	* config/tc-mep.c: Likewise.
	* config/tc-metag.c: Likewise.
	* config/tc-microblaze.c: Likewise.
	* config/tc-mips.c: Likewise.
	* config/tc-ns32k.c: Likewise.
	* config/tc-rx.c: Likewise.
	* config/tc-score.c: Likewise.
	* config/tc-score7.c: Likewise.
	* config/tc-sh.c: Likewise.
	* config/tc-tic4x.c: Likewise.
	* config/tc-vax.c: Likewise.
	* config/tc-xstormy16.c: Likewise.
	* config/tc-z80.c: Likewise.
	* config/tc-z8k.c: Likewise.
	* config/obj-elf.c: Likewise.
	* config/tc-i386.c: Likewise.
	* depend.c: Spell fall through comments consistently.
	* config/tc-arm.c: Likewise.
	* config/tc-d10v.c: Likewise.
	* config/tc-i960.c: Likewise.
	* config/tc-ia64.c: Likewise.
	* config/tc-m68k.c: Likewise.
	* config/tc-mcore.c: Likewise.
	* config/tc-mep.c: Likewise.
	* config/tc-ns32k.c: Likewise.
	* config/tc-visium.c: Likewise.
	* config/tc-xstormy16.c: Likewise.
	* config/tc-z8k.c: Likewise.
gprof/
	* gprof.c: Add missing fall through comments.
ld/
	* lexsup.c: Spell fall through comments consistently and add
	missing fall through comments.
This commit is contained in:
Alan Modra
2016-10-05 18:17:02 +10:30
parent 1e0f0b4d01
commit 1a0670f374
96 changed files with 415 additions and 63 deletions

View File

@ -1,3 +1,13 @@
2016-10-06 Alan Modra <amodra@gmail.com>
* dlltool.c: Spell fall through comments consistently.
* objcopy.c: Likewise.
* readelf.c: Likewise.
* dwarf.c: Add missing fall through comments.
* elfcomm.c: Likewise.
* sysinfo.y: Likewise.
* readelf.c: Likewise. Also remove extraneous comments.
2016-10-06 Alan Modra <amodra@gmail.com>
* cxxfilt.c (usage): Add ATTRIBUTE_NORETURN.

View File

@ -2708,7 +2708,8 @@ make_one_lib_file (export_type *exp, int i, int delay)
sec->orelocation = rpp;
break;
}
/* else fall through */
/* Fall through. */
case IDATA4:
/* An idata$4 or idata$5 is one word long, and has an
rva to idata$6. */

View File

@ -1824,6 +1824,7 @@ read_and_display_attr_value (unsigned long attribute,
{
case DW_AT_frame_base:
have_frame_base = 1;
/* Fall through. */
case DW_AT_location:
case DW_AT_string_length:
case DW_AT_return_addr:
@ -2099,6 +2100,7 @@ read_and_display_attr_value (unsigned long attribute,
case DW_AT_frame_base:
have_frame_base = 1;
/* Fall through. */
case DW_AT_location:
case DW_AT_string_length:
case DW_AT_return_addr:
@ -7654,6 +7656,7 @@ dwarf_select_sections_by_letters (const char *letters)
case 'F':
do_debug_frames_interp = 1;
/* Fall through. */
case 'f':
do_debug_frames = 1;
break;

View File

@ -165,6 +165,7 @@ byte_get_little_endian (unsigned char *field, int size)
| (((unsigned long) (field[1])) << 8)
| (((unsigned long) (field[2])) << 16)
| (((unsigned long) (field[3])) << 24);
/* Fall through. */
case 6:
if (sizeof (elf_vma) == 8)
@ -182,6 +183,7 @@ byte_get_little_endian (unsigned char *field, int size)
| (((unsigned long) (field[1])) << 8)
| (((unsigned long) (field[2])) << 16)
| (((unsigned long) (field[3])) << 24);
/* Fall through. */
case 7:
if (sizeof (elf_vma) == 8)
@ -200,6 +202,7 @@ byte_get_little_endian (unsigned char *field, int size)
| (((unsigned long) (field[1])) << 8)
| (((unsigned long) (field[2])) << 16)
| (((unsigned long) (field[3])) << 24);
/* Fall through. */
case 8:
if (sizeof (elf_vma) == 8)
@ -219,6 +222,7 @@ byte_get_little_endian (unsigned char *field, int size)
| (((unsigned long) (field[1])) << 8)
| (((unsigned long) (field[2])) << 16)
| (((unsigned long) (field[3])) << 24);
/* Fall through. */
default:
error (_("Unhandled data length: %d\n"), size);
@ -265,6 +269,7 @@ byte_get_big_endian (unsigned char *field, int size)
| (((unsigned long) (field[1])) << 16)
| (((unsigned long) (field[0])) << 24);
}
/* Fall through. */
case 6:
if (sizeof (elf_vma) == 8)
@ -284,6 +289,7 @@ byte_get_big_endian (unsigned char *field, int size)
| (((unsigned long) (field[1])) << 16)
| (((unsigned long) (field[0])) << 24);
}
/* Fall through. */
case 7:
if (sizeof (elf_vma) == 8)
@ -304,6 +310,7 @@ byte_get_big_endian (unsigned char *field, int size)
| (((unsigned long) (field[1])) << 16)
| (((unsigned long) (field[0])) << 24);
}
/* Fall through. */
case 8:
if (sizeof (elf_vma) == 8)
@ -325,6 +332,7 @@ byte_get_big_endian (unsigned char *field, int size)
| (((unsigned long) (field[1])) << 16)
| (((unsigned long) (field[0])) << 24);
}
/* Fall through. */
default:
error (_("Unhandled data length: %d\n"), size);

View File

@ -4302,7 +4302,7 @@ copy_main (int argc, char *argv[])
{
case OPTION_CHANGE_SECTION_ADDRESS:
p->vma_val = val;
/* Drop through. */
/* Fall through. */
case OPTION_CHANGE_SECTION_LMA:
p->lma_val = val;

View File

@ -426,7 +426,7 @@ print_vma (bfd_vma vma, print_mode mode)
{
case FULL_HEX:
nc = printf ("0x");
/* Drop through. */
/* Fall through. */
case LONG_HEX:
#ifdef BFD64
@ -439,11 +439,11 @@ print_vma (bfd_vma vma, print_mode mode)
case DEC_5:
if (vma <= 99999)
return printf ("%5" BFD_VMA_FMT "d", vma);
/* Drop through. */
/* Fall through. */
case PREFIX_HEX:
nc = printf ("0x");
/* Drop through. */
/* Fall through. */
case HEX:
return nc + printf ("%" BFD_VMA_FMT "x", vma);
@ -1292,6 +1292,7 @@ dump_relocations (FILE * file,
rtype = elf_msp430x_reloc_type (type);
break;
}
/* Fall through. */
case EM_MSP430_OLD:
rtype = elf_msp430_reloc_type (type);
break;
@ -4450,7 +4451,7 @@ parse_args (int argc, char ** argv)
default:
/* xgettext:c-format */
error (_("Invalid option '-%c'\n"), c);
/* Drop through. */
/* Fall through. */
case '?':
usage (stderr);
}
@ -9679,6 +9680,7 @@ process_dynamic_section (FILE * file)
case DT_SYMENT :
case DT_RELENT :
dynamic_info[entry->d_tag] = entry->d_un.d_val;
/* Fall through. */
case DT_PLTPADSZ:
case DT_MOVEENT :
case DT_MOVESZ :
@ -11494,6 +11496,7 @@ target_specific_reloc_handling (Elf_Internal_Rela * reloc,
case 10: /* R_MSP430_SYM_DIFF */
if (uses_msp430x_relocs ())
break;
/* Fall through. */
case 21: /* R_MSP430X_SYM_DIFF */
saved_sym = symtab + get_reloc_symindex (reloc->r_info);
return TRUE;
@ -12011,6 +12014,7 @@ is_16bit_abs_reloc (unsigned int reloc_type)
case EM_MSP430:
if (uses_msp430x_relocs ())
return reloc_type == 2; /* R_MSP430_ABS16. */
/* Fall through. */
case EM_MSP430_OLD:
return reloc_type == 5; /* R_MSP430_16_BYTE. */
case EM_NDS32:
@ -14241,6 +14245,7 @@ process_attributes (FILE * file,
goto do_numlist;
case 3:
printf (_("Symbol Attributes:"));
/* Fall through. */
do_numlist:
for (;;)
{
@ -16418,8 +16423,8 @@ get_file_header (FILE * file)
/* Determine how to read the rest of the header. */
switch (elf_header.e_ident[EI_DATA])
{
default: /* fall through */
case ELFDATANONE: /* fall through */
default:
case ELFDATANONE:
case ELFDATA2LSB:
byte_get = byte_get_little_endian;
byte_put = byte_put_little_endian;

View File

@ -153,9 +153,8 @@ it:
break;
case 'g':
printf("\tchecksum(ffile,raw, idx, IT_%s_CODE);\n", it);
/* Fall through. */
case 'i':
case 'o':
case 'c':
printf("}\n");
@ -182,6 +181,7 @@ repeat_it_field: '(' REPEAT NAME
printf("\tprintf(\"repeat %%d\\n\", %s);\n",$3);
if (rdepth==2)
printf("\tprintf(\"repeat %%d\\n\", %s[n]);\n",$3);
/* Fall through. */
case 'i':
case 'g':
case 'o':