public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Daniel Black <daniel.black@au1.ibm.com>
To: binutils@sourceware.org
Cc: Daniel Black <daniel.black@au1.ibm.com>
Subject: [PATCH 4/4] fix fallthough statements to allow gcc-7 compile
Date: Thu, 23 Feb 2017 00:37:00 -0000	[thread overview]
Message-ID: <1487810145-3339-4-git-send-email-daniel.black@au.ibm.com> (raw)
In-Reply-To: <1487810145-3339-1-git-send-email-daniel.black@au.ibm.com>

Signed-off-by: Daniel Black <daniel.black@au.ibm.com>
---
 binutils/dwarf.c     | 3 +++
 gas/config/obj-elf.c | 1 +
 gas/depend.c         | 5 +++--
 gas/dw2gencfi.c      | 1 +
 gas/expr.c           | 1 +
 5 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/binutils/dwarf.c b/binutils/dwarf.c
index 282e069..091921f 100644
--- a/binutils/dwarf.c
+++ b/binutils/dwarf.c
@@ -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:
@@ -7580,6 +7582,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;
diff --git a/gas/config/obj-elf.c b/gas/config/obj-elf.c
index 8af563f..4b85b52 100644
--- a/gas/config/obj-elf.c
+++ b/gas/config/obj-elf.c
@@ -813,6 +813,7 @@ obj_elf_parse_section_letters (char *str, size_t len, bfd_boolean *is_clone)
 		}
 	      break;
 	    }
+	  /* fallthrough */
 	default:
 	  {
 	    const char *bad_msg = _("unrecognized .section attribute:"
diff --git a/gas/depend.c b/gas/depend.c
index 7203e59..86d0566 100644
--- a/gas/depend.c
+++ b/gas/depend.c
@@ -121,8 +121,9 @@ quote_string_for_make (FILE *file, const char *src)
 	  if (file)
 	    putc (c, file);
 	  i++;
-	  /* Fall through.  This can mishandle things like "$(" but
-	     there's no easy fix.  */
+	  /* This can mishandle things like "$(" but
+	     there's no easy fix so */
+	  /* fall through */
 	default:
 	ordinary_char:
 	  /* This can mishandle characters in the string "\0\n%*?[\\~";
diff --git a/gas/dw2gencfi.c b/gas/dw2gencfi.c
index fb3e302..a362a57 100644
--- a/gas/dw2gencfi.c
+++ b/gas/dw2gencfi.c
@@ -1153,6 +1153,7 @@ dot_cfi_val_encoded_addr (int ignored ATTRIBUTE_UNUSED)
     case O_constant:
       if ((encoding & 0x70) != DW_EH_PE_pcrel)
 	break;
+      /* fallthrough */
     default:
       encoding = DW_EH_PE_omit;
       break;
diff --git a/gas/expr.c b/gas/expr.c
index b1cdb38..3611f2a 100644
--- a/gas/expr.c
+++ b/gas/expr.c
@@ -1008,6 +1008,7 @@ operand (expressionS *expressionP, enum expr_mode mode)
       /* '~' is permitted to start a label on the Delta.  */
       if (is_name_beginner (c))
 	goto isname;
+      /* fallthrough */
     case '!':
     case '-':
     case '+':
-- 
2.7.4

  parent reply	other threads:[~2017-02-23  0:37 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-23  0:37 [PATCH 1/4] Fix format-overflow errors Daniel Black
2017-02-23  0:37 ` [PATCH 2/4] Fix incorrect indirection Daniel Black
2017-02-27 12:49   ` Nick Clifton
2017-02-27 22:07     ` Daniel Black
2017-02-28 23:57       ` Alan Modra
2017-02-23  0:37 ` Daniel Black [this message]
2017-02-27 12:54   ` [PATCH 4/4] fix fallthough statements to allow gcc-7 compile Nick Clifton
2017-02-23  0:37 ` [PATCH 3/4] Add missing break statement Daniel Black
2017-02-27 12:50   ` Nick Clifton
2017-02-27 12:47 ` [PATCH 1/4] Fix format-overflow errors Nick Clifton

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1487810145-3339-4-git-send-email-daniel.black@au.ibm.com \
    --to=daniel.black@au1.ibm.com \
    --cc=binutils@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).