[cfe-commits] [PATCH] Adds the possibility to run ASTMatchFinder over arbitrary AST nodes.

[Manuel Klimek]

================
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