================
Comment at: unittests/ASTMatchers/ASTMatchersTest.cpp:3193
@@ +3192,3 @@
+
+TEST(MatchFinder, CanMatchDeclarationsRecursively) {
+ EXPECT_TRUE(matchAndVerifyResultTrue("class X { class Y {}; };",
----------------
Daniel Jasper wrote:
> Could you create one negative test case as a smoke test?
Done.
================
Comment at: include/clang/ASTMatchers/ASTMatchFinder.h:126
@@ -125,1 +125,3 @@
+ /// \brief Finds all matches on the given node.
+ ///
----------------
Daniel Jasper wrote:
> nit: maybe ".. on the given \c Node."
Done.
http://llvm-reviews.chandlerc.com/D75
BRANCH
match-in-match
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
