================
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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to