[PATCH] D18275: [ASTMatchers] Add own version of VariadicFunction.

[Alexander Kornienko via cfe-commits]

alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

LG


================
Comment at: include/clang/ASTMatchers/ASTMatchersInternal.h:80
@@ +79,3 @@
+  ResultT operator()(ArrayRef<ArgT> Args) const {
+    std::vector<const ArgT *> InnerArgs;
+    for (const ArgT &Arg : Args)
----------------
It's unfortunate that we need to create an array of the argument pointers here, but it seems there's no larger common denominator of the two ways this functions can be called.

One nit though: SmallVector will be a better container here.


http://reviews.llvm.org/D18275