C++: avoid partial duplicate implementation of cp_parser_error
authorDavid Malcolm <dmalcolm@redhat.com>
Thu, 12 Oct 2017 17:29:15 +0000 (17:29 +0000)
committerDavid Malcolm <dmalcolm@gcc.gnu.org>
Thu, 12 Oct 2017 17:29:15 +0000 (17:29 +0000)
commite95c91292876830c8e36b0a55f0c03d1247aaccb
treeebedc33278fd940d2f13db951d60f5634e8eaf83
parent2c79bfcfc985e116ad1ef85579168428f14f0538
C++: avoid partial duplicate implementation of cp_parser_error

In r251026 (aka 3fe34694f0990d1d649711ede0326497f8a849dc,
"C/C++: show pertinent open token when missing a close token")
I copied part of cp_parser_error into cp_parser_required_error,
leading to duplication of code.

This patch eliminates this duplication by merging the two copies of the
code into a new cp_parser_error_1 subroutine.

Doing so removes an indentation level, making the patch appear to have
more churn than it really does.

The patch also undoes the change to g++.dg/parse/pragma2.C, as the
old behavior is restored.

From-SVN: r253686
gcc/cp/ChangeLog
gcc/cp/parser.c
gcc/testsuite/ChangeLog
gcc/testsuite/g++.dg/parse/pragma2.C