================
Comment at: clang-tidy/misc/ArgumentCommentCheck.cpp:123
@@ +122,3 @@
+ if (auto Template = Callee->getTemplateInstantiationPattern()) {
+ if (Template->getNumParams() <= i ||
+ Template->getParamDecl(i)->isParameterPack()) {
----------------
klimek wrote:
> Can that one actually happen? Shouldn't that be an assert?
Sure it can. We're looking at template parameter packs here, which can then
instantiate to multiple parameters.
This is tested by the "variadic2(/*zzZ=*/0, /*xxx=*/1, /*yyy=*/2);" line. The
last argument of it.
http://reviews.llvm.org/D5168
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
