[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