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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to