From: Tom Tromey <tromey@adacore.com>
Date: Tue, 14 Nov 2023 17:37:55 +0000 (-0700)
Subject: Remove some redundant "break"s
X-Git-Tag: binutils-2_42~962
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c748dbd95bf3305539956053c2a25ecea7b442a5;p=thirdparty%2Fbinutils-gdb.git

Remove some redundant "break"s

I found some "break" statements that follow "return" or a call to a
noreturn function.  These aren't needed, and the compiler would warn
if they were.  So, this patch removes them.

Tested by rebuilding.
---

diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index fe09dbcbeee..a6b9b2402ed 100644
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -4977,13 +4977,11 @@ print_bp_stop_message (bpstat *bs)
     case print_it_noop:
       /* Nothing should be printed for this bpstat entry.  */
       return PRINT_UNKNOWN;
-      break;
 
     case print_it_done:
       /* We still want to print the frame, but we already printed the
 	 relevant messages.  */
       return PRINT_SRC_AND_LOC;
-      break;
 
     case print_it_normal:
       {
@@ -4997,11 +4995,9 @@ print_bp_stop_message (bpstat *bs)
 	/* Normal case.  Call the breakpoint's print_it method.  */
 	return b->print_it (bs);
       }
-      break;
 
     default:
       internal_error (_("print_bp_stop_message: unrecognized enum value"));
-      break;
     }
 }