[cfe-commits] [PATCH] Rename matchers to better match AST nodes
Daniel Jasper
reviews at llvm-reviews.chandlerc.com
Wed Aug 22 10:16:25 PDT 2012
Hi klimek,
The matcher language is easiest to learn if it matches the AST as closely as possible. To this end, this rename removes unnecessary difference. The naming rules now are:
- Start with AST node name
- Leave out CXX prefix (unnecessary clutter, should we rename the AST nodes?)
- Leave out trailing "Expr" and "Decl" if possible (e.g. not for enumDecl) and unambiguous (e.g. not for memberExpr).
http://llvm-reviews.chandlerc.com/D21
Files:
include/clang/ASTMatchers/ASTMatchers.h
unittests/AST/DeclPrinterTest.cpp
unittests/ASTMatchers/ASTMatchersTest.cpp
unittests/Tooling/RefactoringCallbacksTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21.1.patch
Type: text/x-patch
Size: 73832 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120822/c695a672/attachment.bin>
More information about the cfe-commits
mailing list