[PATCH] D17163: [ASTMatchers] Add matcher hasAnyName.

[Samuel Benzaquen via cfe-commits]

sbenza added inline comments.

================
Comment at: include/clang/ASTMatchers/ASTMatchersInternal.h:644
@@ -643,2 +643,3 @@
   explicit HasNameMatcher(StringRef Name);
+  explicit HasNameMatcher(ArrayRef<const StringRef *> Names);
 
----------------
alexfh wrote:
> bkramer wrote:
> > alexfh wrote:
> > > Why not `ArrayRef<StringRef>`?
> > That's an artifact of how llvm::VariadicFunction works. It gives you an array ref of pointers to the arguments.
> Thanks! Good to know. Maybe a add a comment or is it just me who doesn't know this?
No reason to leak the VariadicFunction design into here.
I moved the transformation into std::vector<> to the caller.


http://reviews.llvm.org/D17163