[PATCH] Re-design the convenience interfaces on MatchFinder.

Manuel Klimek klimek at google.com
Fri Feb 1 05:37:17 PST 2013



================
Comment at: include/clang/ASTMatchers/ASTMatchFinder.h:137
@@ -136,2 +136,3 @@
 
-  /// \brief Finds all matches on the given \c Node.
+  /// \brief Calls the registered callbacks on all matches on the given \c Node.
+  ///
----------------
Daniel Jasper wrote:
> \p ?
Done.

================
Comment at: include/clang/ASTMatchers/ASTMatchFinder.h:167
@@ -160,1 +166,3 @@
 
+/// \brief Returns the results of matching \c Matcher on \c Node.
+///
----------------
Daniel Jasper wrote:
> \p?
Done.

================
Comment at: lib/ASTMatchers/ASTMatchFinder.cpp:470
@@ -469,1 +469,3 @@
 
+  // Matches all registered matchers on the given node and calls the
+  // result callback for every node that matches.
----------------
Daniel Jasper wrote:
> maybe "on \p Node" ...
Note that this is not a doxycomment :)


http://llvm-reviews.chandlerc.com/D359

BRANCH
  clang-ast-matchers-match

ARCANIST PROJECT
  clang



More information about the cfe-commits mailing list