Re: [C++ Patch] PR 42056

2011-05-27 Thread Paolo Carlini
On 05/27/2011 03:57 PM, Jason Merrill wrote: OK, but please combine those tests into one file. Thanks. As applied. Paolo. // /cp 2011-05-27 Paolo Carlini PR c++/42056 * typeck2.c (build_functional_cast): Complain early for invalid uses of 'auto' and set

Re: [C++ Patch] PR 42056

2011-05-27 Thread Jason Merrill
OK, but please combine those tests into one file. Jason

Re: [C++ Patch] PR 42056

2011-05-27 Thread Paolo Carlini
Hi, On 05/26/2011 02:30 PM, Paolo Carlini wrote: We want to do that only when processing a template, because otherwise we get a duplicate diagnostic, see, eg, auto9.C Hmm, where's the error coming from in the non-template case? From cp_build_c_cast? In that case always giving the error in bu

Re: [C++ Patch] PR 42056

2011-05-26 Thread Jason Merrill
On 05/26/2011 02:30 PM, Paolo Carlini wrote: We want to do that only when processing a template, because otherwise we get a duplicate diagnostic, see, eg, auto9.C Hmm, where's the error coming from in the non-template case? From cp_build_c_cast? In that case always giving the error in build

Re: [C++ Patch] PR 42056

2011-05-26 Thread Paolo Carlini
On 05/26/2011 08:30 PM, Paolo Carlini wrote: ...also, not returning error_mark_node unconditionally, means a better diagnostic, without redundant "array bound is not an integer constant before...". Just noticed that outside a template, we do indeed emit the additional "array bound is not an int

[C++ Patch] PR 42056

2011-05-26 Thread Paolo Carlini
Hi, I have just regtested on x86_64-linux the below patchlet for a simple accepts-invalid, exploiting type_uses_auto, as suggested by Jason. We want to do that only when processing a template, because otherwise we get a duplicate diagnostic, see, eg, auto9.C; also, not returning error_mark_n