Re: [C++ Patch] PR 54377

2014-08-14 Thread Jason Merrill
OK. Jason

Re: [C++ Patch] PR 54377

2014-08-14 Thread Paolo Carlini
Hi, On 08/14/2014 04:18 PM, Jason Merrill wrote: On 08/14/2014 07:14 AM, Paolo Carlini wrote: + nparms -= variadic_p ? variadic_p : default_p; What if you have both default arguments and parameter packs? Right. Got distracted by the minor secondary issues... It seems to me that

Re: [C++ Patch] PR 54377

2014-08-14 Thread Jason Merrill
On 08/14/2014 07:14 AM, Paolo Carlini wrote: + nparms -= variadic_p ? variadic_p : default_p; What if you have both default arguments and parameter packs? Jason

[C++ Patch] PR 54377

2014-08-14 Thread Paolo Carlini
Hi, this is a diagnostic issue about the error message for a wrong number of template arguments vs default arguments. The fix is simple but also replacing the second error with a default, as recommended by Manuel in the audit trail, means that we have to adjust a few existing testcases. Note,