fix: Silence expected use after move warnings (#1819)

Fixes #1818
This commit is contained in:
Alex Kremer
2025-01-10 15:47:48 +00:00
committed by GitHub
parent c0d52723c9
commit 91c00e781a

View File

@@ -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)
}