Re: [C++ Patch] PR 51404

[Paolo Carlini <paolo dot carlini at oracle dot com>]

Hi,
> Hmm, I think giving the additional error was intentional, which is why 
> we check for error_operand_p after giving it.  But I suppose I'm not 
> attached to that.
>
> We could avoid reformatting by adding
>
> if (error_operand_p (bounds))
>   /* Already gave an error.  */;
>
> OK either way.
Thanks. Thus I'm going to apply the below.

By the way, something I didn't like about the simpler patch not tweaking 
cp_parser_direct_declarator is that we would handle inconsistently the 
two tests in auto25.C, producing the more verbose diagnostic only for 
the first one. A little more work would be needed.

Paolo.

Attachment: CL_51404_2
Description: Text document

Attachment: patch_51404_2
Description: Text document