Skip to content

Commit 1ba148f

Browse files
committed
[Impeller] Silence bugprone-unchecked-optional-access in renderer_golden_unittests
clang-tidy does not recognise gtest's ASSERT_TRUE as flow-affecting, so desc-> accesses after ASSERT_TRUE(desc.has_value()) trip bugprone-unchecked-optional-access. Use the same NOLINTBEGIN/NOLINTEND block as renderer_unittests.cc, referencing the same tracking issue.
1 parent 21e4d64 commit 1ba148f

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

engine/src/flutter/impeller/renderer/renderer_golden_unittests.cc

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@
2121
#include "impeller/renderer/render_pass.h"
2222
#include "impeller/renderer/vertex_buffer_builder.h"
2323

24+
// TODO(zanderso): https://github.com/flutter/flutter/issues/127701
25+
// NOLINTBEGIN(bugprone-unchecked-optional-access)
26+
2427
namespace impeller {
2528
namespace testing {
2629

@@ -81,4 +84,6 @@ TEST_P(RendererGoldenTest, BabysFirstTriangle) {
8184
} // namespace testing
8285
} // namespace impeller
8386

87+
// NOLINTEND(bugprone-unchecked-optional-access)
88+
8489
#endif // IMPELLER_GOLDEN_TESTS

0 commit comments

Comments
 (0)