[cfe-commits] [PATCH] C++11 matchers

Manuel Klimek reviews at llvm-reviews.chandlerc.com
Thu Sep 20 07:43:32 PDT 2012



================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:805
@@ +804,3 @@
+  Stmt,
+  CXXNullPtrLiteralExpr> nullPtrLiteralExpr;
+
----------------
Daniel Jasper wrote:
> I wonder whether this should be nullPtrLiteral; for consistency with the other matchers. That would then (at a later point entail renaming the AST node). Manuel, any thoughts?
I'd rather keep as much as possible in common with the AST nodes, so I'd vote for keeping it as proposed here.


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



More information about the cfe-commits mailing list