commit 49d9269b34f6c20ab7e1ba7e60cec4c1f3194701
author William A. Kennington III <wak@google.com> Tue Jun 06 14:02:01 2023 -0700
committer William A. Kennington III <wak@google.com> Tue Jun 06 14:11:55 2023 -0700
tree fdd190ea30a98dc87f37a35c88504fc08c65311c
parent 1cad7d7ffd9c6c3c811296d99583529122872364

treewide: Add NOLINT where needed to make clang-tidy happy

These are not real bugs, clang-tidy doesn't understand how we are
preventing UB.

Change-Id: I4305ab0b59aae07151d5978ba8642bece076139f
Signed-off-by: William A. Kennington III <wak@google.com>

test/handle/copyable.cpp

diff --git a/test/handle/copyable.cpp b/test/handle/copyable.cpp
index 161b808..fa95419 100644
--- a/test/handle/copyable.cpp
+++ b/test/handle/copyable.cpp
@@ -262,6 +262,7 @@
         StoreHandle h2(std::move(h1));
         EXPECT_TRUE(dropped.empty());
         EXPECT_FALSE(h1);
+        // NOLINTNEXTLINE(clang-analyzer-cplusplus.Move)
         EXPECT_THROW(h1.value(), std::bad_optional_access);
         EXPECT_TRUE(h2);
         EXPECT_EQ(expected, *h2);
@@ -285,6 +286,7 @@
         EXPECT_EQ(std::vector{expected2}, dropped);
         dropped.clear();
         EXPECT_FALSE(h1);
+        // NOLINTNEXTLINE(clang-analyzer-cplusplus.Move)
         EXPECT_THROW(h1.value(), std::bad_optional_access);
         EXPECT_TRUE(h2);
         EXPECT_EQ(expected, *h2);

test/handle/managed.cpp

diff --git a/test/handle/managed.cpp b/test/handle/managed.cpp
index 74816e5..0cdbe74 100644
--- a/test/handle/managed.cpp
+++ b/test/handle/managed.cpp
@@ -217,6 +217,7 @@
         StoreHandle h2(std::move(h1));
         EXPECT_TRUE(dropped.empty());
         EXPECT_FALSE(h1);
+        // NOLINTNEXTLINE(clang-analyzer-cplusplus.Move)
         EXPECT_THROW(h1.value(), std::bad_optional_access);
         EXPECT_TRUE(h2);
         EXPECT_EQ(expected, *h2);
@@ -240,6 +241,7 @@
         EXPECT_EQ(std::vector{expected2}, dropped);
         dropped.clear();
         EXPECT_FALSE(h1);
+        // NOLINTNEXTLINE(clang-analyzer-cplusplus.Move)
         EXPECT_THROW(h1.value(), std::bad_optional_access);
         EXPECT_TRUE(h2);
         EXPECT_EQ(expected, *h2);

test/pinned.cpp

diff --git a/test/pinned.cpp b/test/pinned.cpp
index dd83375..88c9bef 100644
--- a/test/pinned.cpp
+++ b/test/pinned.cpp
@@ -39,6 +39,7 @@
 TEST(Pinned, Fundamental)
 {
     Pinned pi = 4;
+    // NOLINTNEXTLINE(clang-analyzer-core.uninitialized.UndefReturn)
     EXPECT_EQ(4, [](int& f) { return f; }(pi));
     EXPECT_EQ(4, [](int f) { return f; }(pi));
 }