On 05/08/24 09:32 +0200, Franciszek Witt wrote:
Hi, could someone review this patch?

This is built on top of the v2 from
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101232 the only
difference is fix for error59.C

I have tested it on x86_64 Ubuntu 22 machine.


Regards
Franciszek

---

Author: Franciszek Witt <franek.w...@gmail.com>
Date:   Mon Aug 5 09:00:35 2024 +0200

   c++: [PR101232]

The line above should be the first line of your Git commit message,
which should be in the form "component: Summary of change [PRnnnn]"
The Subject: of your email seems like it should be repeated here
(although something shorter would be even nicer, if possible).
https://cbea.ms/git-commit/


           PR c++/101232

   gcc/cp/ChangeLog:

           * parser.cc (cp_parser_postfix_expression): Commit to the
parse in case we know its either a cast or invalid syntax.
           (cp_parser_braced_list): Add a heuristic to inform about
missing comma or operator.

The ChangeLog part of the commit message should follow the rules in
the GNU Coding Standards:
https://www.gnu.org/prep/standards/html_node/Style-of-Change-Logs.html

Specifically, they should be wrapped to fit in an 80 column terminal
window. I think wrapping around 76 columns is common. Please add line
breaks here, e.g.

        * parser.cc (cp_parser_postfix_expression): Commit to the
        parse in case we know its either a cast or invalid syntax.
        (cp_parser_braced_list): Add a heuristic to inform about
        missing comma or operator.

(but with a leading TAB, not spaces).

Reply via email to