Comments addressed in next revision.

================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:478
@@ +477,3 @@
+///
+/// Example: Given
+///   struct T {void func()};
----------------
Manuel Klimek wrote:
> Please use \code around the code blocks.
Done.

================
Comment at: unittests/ASTMatchers/ASTMatchersTest.cpp:1253
@@ +1252,3 @@
+TEST(MaterializeTempExpr, MatchesTemporary) {
+  StatementMatcher MaterializeTemp = materializeTempExpr();
+
----------------
Manuel Klimek wrote:
> Is there a reason not to just inline this?
Nope. Inlined.


http://llvm-reviews.chandlerc.com/D20
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to