[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