================
Comment at: docs/LibASTMatchersReference.html:2509
@@ -2497,3 +2508,3 @@
 function. e.g. various subtypes of clang::Type and various expressions.
 FIXME: Add all node types for which this is matcher is usable due to
 getDecl().
----------------
Manuel Klimek wrote:
> Perhaps fix the fixme? :)
Done.

================
Comment at: include/clang/ASTMatchers/ASTMatchersInternal.h:666
@@ +665,3 @@
+/// \brief All types that are supported by HasDeclarationMatcher above.
+typedef internal::TypeList<
+    internal::TypeList<CallExpr, CXXConstructExpr, DeclRefExpr, EnumType>,
----------------
Manuel Klimek wrote:
> Any reason we can't inline this into the method decl (and ideally parse it 
> for the doc-generator from there). Doesn't seem worse than the "usable as" 
> comment (and easier to keep in sync)
Mainly because I didn't want to add the macros all the way to _14().
Also, it is a pain having to write it twice (one in the function signature, and 
another in the return statement).


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

BRANCH
  has_decl

ARCANIST PROJECT
  clang
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to