sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land.
This is pretty small and seems unlikely to be controversial. I think it's OK to commit at this point, it's been a few weeks. ================ Comment at: include/clang/Sema/CodeCompleteConsumer.h:1092 + const CodeCompleteConsumer::OverloadCandidate &Result, + unsigned CurrentArg); + ---------------- as in the other patch, I think `ArgIndex` is a better name for this parameter, when considering this function in isolation Repository: rC Clang https://reviews.llvm.org/D46001 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
