bison-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: bison test 012


From: Paul Eggert
Subject: Re: bison test 012
Date: Mon, 30 Jan 2006 01:01:10 -0800
User-agent: Gnus/5.1007 (Gnus v5.10.7) Emacs/21.4 (gnu/linux)

twlevo <address@hidden> writes:

> line 279 parse-gram.y should it not be `symbol_printer_set(list->sym,$1,@1);' 
> and see output from test 012

Thanks for reporting that.  I installed this fix:

2006-01-30  Paul Eggert  <address@hidden>

        * src/parse-gram.y (grammar_declaration): Location of printer
        symbol is @1, not list->location.  Bug reported by twlevo.
        * tests/input.at (Incompatible Aliases): Adjust to above change.

--- src/parse-gram.y    30 Jan 2006 07:25:59 -0000      1.72
+++ src/parse-gram.y    30 Jan 2006 08:58:02 -0000
@@ -287,7 +287,7 @@ grammar_declaration:
     {
       symbol_list *list;
       for (list = $2; list; list = list->next)
-       symbol_printer_set (list->sym, $1, list->location);
+       symbol_printer_set (list->sym, $1, @1);
       symbol_list_free ($2);
     }
 | "%default-prec"
--- tests/input.at      30 Jan 2006 07:51:58 -0000      1.41
+++ tests/input.at      30 Jan 2006 08:58:03 -0000
@@ -178,7 +178,7 @@ AT_CHECK([bison input.y], [1], [],
 input.y:3.7-11: first declaration
 input.y:10.13-17: %destructor redeclaration for foo
 input.y:5.13-17: first declaration
-input.y:9.19-23: %printer redeclaration for foo
+input.y:9.10-14: %printer redeclaration for foo
 input.y:10.13-17: first declaration
 input.y:11.1-5: %left redeclaration for foo
 input.y:6.1-5: first declaration




reply via email to

[Prev in Thread] Current Thread [Next in Thread]