aaron.ballman added inline comments.
================ Comment at: unittests/ASTMatchers/Dynamic/ParserTest.cpp:188-190 + for (size_t i = 0, e = Sema.Errors.size(); i != e; ++i) { + EXPECT_EQ("", Sema.Errors[i]); + } ---------------- Why are empty errors created? I would have expected this to be `EXPECT_EQ(0, Sema.Errors.size())`. (If this formulation is desired, it should use a range-based for loop.) Repository: rC Clang https://reviews.llvm.org/D52751 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits