[PATCH] D70528: [NFC] ASSERT_EQ before accessing items in containers

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 21 03:11:29 PST 2019


ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.

LGTM, but please update the other diff as well!



================
Comment at: clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp:712
   for (const auto &D : Parsed.getDiagnostics()) {
-    EXPECT_EQ(D.Fixes.size(), 1u);
+    ASSERT_EQ(D.Fixes.size(), 1u);
     EXPECT_EQ(D.Fixes[0].Message,
----------------
Same here, actually. We should not break out of the loop.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70528/new/

https://reviews.llvm.org/D70528





More information about the cfe-commits mailing list