Re: Bug 52869 - [DR 1207] "this" not being allowed in noexcept clauses

[Marek Polacek <polacek@redhat.com>]

On Tue, Nov 13, 2018 at 11:49:55AM +0530, Umesh Kalappa wrote:
> Hi All,
> 
> the following patch fix the subjected issue
> 
> Index: gcc/cp/parser.c
> ===================================================================
> --- gcc/cp/parser.c     (revision 266026)
> +++ gcc/cp/parser.c     (working copy)
> @@ -24615,6 +24615,8 @@
>      {
>        tree expr;
>        cp_lexer_consume_token (parser->lexer);
> +
> +      inject_this_parameter (current_class_type, TYPE_UNQUALIFIED);
> 
>        if (cp_lexer_peek_token (parser->lexer)->type == CPP_OPEN_PAREN)
>         {
> 
> 
> ok to commit along the testcase with changelog update ?

Thanks for the patch.

Please also include the testcase along with the patch (and I think it should
also test noexcept in a template).  Please also include a ChangeLog entry
in the patch submission.

Can you describe how this patch has been tested?

Further, wouldn't it be better to call inject_this_parameter inside the
CPP_OPEN_PAREN block?  If noexcept doesn't have any expression, then it
can't refer to "this".

Marek