Fix a few -Wunreachable-code-aggressive warnings

No behavior change.

Bug: chromium:1066980
Change-Id: Ic9485f3244516a4196965d2ebb2bfbc2c62a91d7
Reviewed-on: https://chromium-review.googlesource.com/c/crashpad/crashpad/+/3054401
Commit-Queue: Nico Weber <thakis@chromium.org>
Commit-Queue: Mark Mentovai <mark@chromium.org>
Reviewed-by: Mark Mentovai <mark@chromium.org>
This commit is contained in:
Nico Weber 2021-07-27 09:56:27 -04:00 committed by Crashpad LUCI CQ
parent a62b654838
commit 058c1c2d9c
3 changed files with 0 additions and 7 deletions

View File

@ -398,7 +398,6 @@ class TestMachOImageAnnotationsReader final
case kCrashAbort: {
abort();
break;
}
case kCrashModuleInitialization: {
@ -411,7 +410,6 @@ class TestMachOImageAnnotationsReader final
// the FAIL() will fail the test.
ASSERT_NE(dl_handle, nullptr) << dlerror();
FAIL();
break;
}
case kCrashDyld: {

View File

@ -130,7 +130,6 @@ class ChildPortHandshakeTest : public Multiprocess {
case TestType::kServerDies:
// This was special-cased as an early return above.
FAIL();
break;
}
}

View File

@ -68,7 +68,6 @@ void CauseSignal(int sig) {
switch (sig) {
case SIGABRT: {
abort();
break;
}
case SIGALRM: {
@ -114,7 +113,6 @@ void CauseSignal(int sig) {
*mapped_file.addr_as<char*>() = 0;
_exit(kUnexpectedExitStatus);
break;
}
#if !defined(ARCH_CPU_ARM64)
@ -143,7 +141,6 @@ void CauseSignal(int sig) {
case SIGILL: {
// __builtin_trap() causes SIGTRAP on arm64 on Android.
__builtin_trap();
break;
}
#endif // defined(ARCH_CPU_X86_FAMILY) || defined(ARCH_CPU_ARMEL)
@ -208,7 +205,6 @@ void CauseSignal(int sig) {
default: {
LOG(ERROR) << "unexpected signal " << sig;
_exit(kUnexpectedExitStatus);
break;
}
}
}