Revert 2009-12-30 change for undefined %prec token complaints.

That is, keep them as warnings because that should be sufficient
to satisfy POSIX without creating backward compatibility issues.
Suggested by Richard Stallman at
<http://lists.gnu.org/archive/html/bison-patches/2010-03/msg00033.html>.
* NEWS (2.5): Remove mention of complaint.
* src/reader.c (grammar_rule_check): Convert warning back to
complaint.
* tests/input.at (%prec's token must be defined): Update.

diff --git a/ChangeLog b/ChangeLog
index f16e05657dfcfb5d814d51e4d025654ce85da52f..c129357450fdf6a688b1a4a80ec9444c8d3ba2e9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2010-04-30  Joel E. Denny  <joeldenny@joeldenny.org>
+
+       Revert 2009-12-30 change for undefined %prec token complaints.
+       That is, keep them as warnings because that should be sufficient
+       to satisfy POSIX without creating backward compatibility issues.
+       Suggested by Richard Stallman at
+       <http://lists.gnu.org/archive/html/bison-patches/2010-03/msg00033.html>.
+       * NEWS (2.5): Remove mention of complaint.
+       * src/reader.c (grammar_rule_check): Convert warning back to
+       complaint.
+       * tests/input.at (%prec's token must be defined): Update.
+
 2010-04-25  Joel E. Denny  <joeldenny@joeldenny.org>
 
        build: don't require src/bison during bootstrap.

diff --git a/NEWS b/NEWS
index 4c4409b168ba22e404297c0c4262d9b120ac09ac..acdd5be18ffa179b589d6539dab03782280f4ae1 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -180,12 +180,6 @@ Bison News
   discussion of how to suppress C preprocessor warnings about YYFAIL
   being unused, see the Bison 2.4.2 NEWS entry.
 
-** `%prec IDENTIFIER' requires IDENTIFIER to be defined separately.
-
-  As promised in Bison 2.4.2's NEWS entry, it is now an error if a token
-  that appears after a %prec directive is not defined by %token, %left,
-  %right, or %nonassoc.  This is required by POSIX.
-
 ** Temporary hack for adding a semicolon to the user action.
 
   Previously, Bison appended a semicolon to every user action for

diff --git a/src/reader.c b/src/reader.c
index 03c5bae5cc74c5221f8878b3e9a528e2e3360d7f..f1be24b38c495c2334f262a6f08b3bfba5ce635f 100644 (file)
--- a/src/reader.c
+++ b/src/reader.c
@@ -327,8 +327,8 @@ grammar_rule_check (const symbol_list *r)
   if (r->ruleprec
       && r->ruleprec->tag[0] != '\'' && r->ruleprec->tag[0] != '"'
       && !r->ruleprec->declared && !r->ruleprec->prec)
-    complain_at (r->location, _("token for %%prec is not defined: %s"),
-                 r->ruleprec->tag);
+    warn_at (r->location, _("token for %%prec is not defined: %s"),
+             r->ruleprec->tag);
 }
 
 

diff --git a/tests/input.at b/tests/input.at
index 71fc3562392de0ad9e7aea7bc608d528f287e2f1..64a18093465c35065da9954479ac217ea6ca2b45 100644 (file)
--- a/tests/input.at
+++ b/tests/input.at
@@ -806,8 +806,8 @@ AT_DATA([[input.y]],
 start: %prec PREC ;
 ]])
 
-AT_BISON_CHECK([[input.y]], [[1]], [],
-[[input.y:2.8-17: token for %prec is not defined: PREC
+AT_BISON_CHECK([[input.y]], [[0]], [],
+[[input.y:2.8-17: warning: token for %prec is not defined: PREC
 ]])
 
 AT_CLEANUP