diff options
| author | Tom Tromey <tom@tromey.com> | 2023-10-15 11:09:07 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2023-11-29 14:29:43 -0700 |
| commit | d182e39881061b11d1eb85426d9a6953e3171bf5 (patch) | |
| tree | c292f0615eee648207c86ae7035479cb518ea81a /gdb/remote.c | |
| parent | a0dc1f9a12a4394463b9dbf5927166f2ab8518a6 (diff) | |
| download | fsf-binutils-gdb-d182e39881061b11d1eb85426d9a6953e3171bf5.zip fsf-binutils-gdb-d182e39881061b11d1eb85426d9a6953e3171bf5.tar.gz fsf-binutils-gdb-d182e39881061b11d1eb85426d9a6953e3171bf5.tar.bz2 | |
Use C++17 [[fallthrough]] attribute
This changes gdb to use the C++17 [[fallthrough]] attribute rather
than special comments.
This was mostly done by script, but I neglected a few spellings and so
also fixed it up by hand.
I suspect this fixes the bug mentioned below, by switching to a
standard approach that, presumably, clang supports.
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=23159
Approved-By: John Baldwin <jhb@FreeBSD.org>
Approved-By: Luis Machado <luis.machado@arm.com>
Approved-By: Pedro Alves <pedro@palves.net>
Diffstat (limited to 'gdb/remote.c')
| -rw-r--r-- | gdb/remote.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/gdb/remote.c b/gdb/remote.c index 22215b5..8ccc126 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -8123,7 +8123,7 @@ Packet: '%s'\n"), if (event->ws.kind () != TARGET_WAITKIND_IGNORE) break; - /* fall through */ + [[fallthrough]]; case 'S': /* Old style status, just signal only. */ { int sig; @@ -8647,7 +8647,7 @@ remote_target::wait_as (ptid_t ptid, target_waitstatus *status, putpkt (buf); break; } - /* fallthrough */ + [[fallthrough]]; default: warning (_("Invalid remote reply: %s"), buf); break; @@ -10002,7 +10002,7 @@ remote_target::putpkt_binary (const char *buf, int cnt) return 1; case '-': remote_debug_printf_nofunc ("Received Nak"); - /* FALLTHROUGH */ + [[fallthrough]]; case SERIAL_TIMEOUT: tcount++; if (tcount > 3) |