Fix EXPECT_DEATH() and ASSERT_DEATH() triggering -Wcovered-switch-default

EXPECT_DEATH() and ASSERT_DEATH() have a switch case where every possible case is covered. This makes the default case unnecessary and triggers -Wcovered-switch-default. Due to these being macros, the lines are expanded in user code and are thus subject to warnings of the target codebase. Fixes #3456

Fix EXPECT_DEATH() and ASSERT_DEATH() triggering -Wcovered-switch-default
10088504 · Alex Karatarakis · 355d57d9 · 10088504
Commit 10088504 authored Jun 23, 2021 by Alex Karatarakis
Hide whitespace changes
Inline Side-by-side

Showing with 0 additions and 2 deletions

gtest-death-test-internal.h ...letest/include/gtest/internal/gtest-death-test-internal.h +0 -2

No files found.
--- a/googletest/include/gtest/internal/gtest-death-test-internal.h
+++ b/googletest/include/gtest/internal/gtest-death-test-internal.h
@@ -236,8 +236,6 @@ inline Matcher<const ::std::string&> MakeDeathTestMatcher(
          gtest_dt->Abort(::testing::internal::DeathTest::TEST_DID_NOT_DIE);   \
          break;                                                               \
        }                                                                      \
-        default:                                                               \
-          break;                                                               \
      }                                                                        \
    }                                                                          \
  } else                                                                       \