[C++ Patch] PR 47791

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

Hi,

this is a very old issue, with no impact on the functionality, which I 
already analyzed a bit a while ago: submitter noticed that 
finish_function & co could be cleaned up a bit wrt the constants passed 
in the flags parameter. When I had again a look today cleaning up the 
code appeared very straightforward:  just change the finish_function 
parameter to bool type and adjust the callers. Interestingly, as a 
consequence, cp_parser_function_definition_after_declarator can be also 
simplified, because ctor_initializer_p was only used for the 
finish_function call. Likewise the helper functions called by 
cp_parser_function_definition_after_declarator itself, which were just 
preparing ctor_initializer_p.

Tested x86_64-linux.

Thanks, Paolo.

////////////////////////

Attachment: CL_47791
Description: Text document

Attachment: patch_47791
Description: Text document