From 91c00e781a278286dab7ee322974fd2cc0aadffe Mon Sep 17 00:00:00 2001 From: Alex Kremer Date: Fri, 10 Jan 2025 15:47:48 +0000 Subject: [PATCH] fix: Silence expected use after move warnings (#1819) Fixes #1818 --- tests/unit/util/MoveTrackerTests.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/unit/util/MoveTrackerTests.cpp b/tests/unit/util/MoveTrackerTests.cpp index 366f5a49..29220686 100644 --- a/tests/unit/util/MoveTrackerTests.cpp +++ b/tests/unit/util/MoveTrackerTests.cpp @@ -35,7 +35,7 @@ TEST(MoveTrackerTests, SimpleChecks) EXPECT_FALSE(moveMe.wasMoved()); auto other = std::move(moveMe); - EXPECT_TRUE(moveMe.wasMoved()); + EXPECT_TRUE(moveMe.wasMoved()); // NOLINT(bugprone-use-after-move) EXPECT_FALSE(other.wasMoved()); } @@ -46,7 +46,7 @@ TEST(MoveTrackerTests, SupportReuse) original = std::move(other); EXPECT_FALSE(original.wasMoved()); - EXPECT_TRUE(other.wasMoved()); + EXPECT_TRUE(other.wasMoved()); // NOLINT(bugprone-use-after-move) } TEST(MoveTrackerTests, SelfMove) @@ -62,7 +62,8 @@ TEST(MoveTrackerTests, SelfMoveAfterWasMoved) auto original = MoveMe(); [[maybe_unused]] auto fake = std::move(original); + // NOLINTNEXTLINE(bugprone-use-after-move) [&](MoveMe& from) { original = std::move(from); }(original); // avoids the compiler catching self-move - EXPECT_TRUE(original.wasMoved()); + EXPECT_TRUE(original.wasMoved()); // NOLINT(bugprone-use-after-move) }