Re: [C++ Patch] PR 42021

2013-06-12 Thread Paolo Carlini
On 06/12/2013 04:05 PM, Paolo Carlini wrote: On 06/12/2013 03:38 PM, Jason Merrill wrote: OK, but please add a comment. Thanks. I added this: // cp_parser_lookup_name has the same diagnostic, // thus make sure to emit it almost once. As mentioned by Marc off-line,

Re: [C++ Patch] PR 42021

2013-06-12 Thread Paolo Carlini
On 06/12/2013 03:38 PM, Jason Merrill wrote: OK, but please add a comment. Thanks. I added this: // cp_parser_lookup_name has the same diagnostic, // thus make sure to emit it almost once. Paolo.

Re: [C++ Patch] PR 42021

2013-06-12 Thread Jason Merrill
OK, but please add a comment. Jason

[C++ Patch] PR 42021

2013-06-11 Thread Paolo Carlini
Hi, this issue, which originally was about spelling out in the error message the ambiguous candidates, is by and large fixed. Only, we produce the whole diagnostic twice for the testcase at issue: first in cp_parser_lookup_name and then again in cp_parser_nested_name_specifier_opt. The below