aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM with one minor correction (you can correct when committing it).


================
Comment at: unittests/ASTMatchers/ASTMatchersTest.cpp:2343
@@ +2342,3 @@
+
+TEST(DestructorDeclaration, MatchesVirtualDestructor) {
+  EXPECT_TRUE(matches("class Foo { virtual ~Foo(); };",
----------------
I think this test got duplicated accidentally; I don't see it removed as part 
of the diff.


http://reviews.llvm.org/D18584



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to