[PATCH] D15506: [ASTMatchers] Allow hasName() to look through inline namespaces

Samuel Benzaquen via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 20 13:37:00 PST 2016


sbenza added inline comments.

================
Comment at: lib/ASTMatchers/ASTMatchersInternal.cpp:351
@@ +350,3 @@
+
+  // First, match the name.
+  if (!MatchNodeName(Pattern, Node))
----------------
klimek wrote:
> Perhaps comment that Pattern will be changed to the prefix, so it's clear afterwards.
> (I wonder whether returning a tuple might be slightly easier to understand here)
Renamed from Match to Consume. Is that better?


http://reviews.llvm.org/D15506





More information about the cfe-commits mailing list