bison-patches
[Top][All Lists]
Advanced

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

Re: [GNU Bison 1.75b] testsuite.log: 7 failures


From: Paul Eggert
Subject: Re: [GNU Bison 1.75b] testsuite.log: 7 failures
Date: Tue, 19 Nov 2002 01:32:46 -0800 (PST)

> From: Akim Demaille <address@hidden>
> Date: Thu, 14 Nov 2002 19:49:28 +0100
> 
>  Paul> Any suggestions for how to debug this further?
> 
> I'm afraid I don't know :(  Maybe try to have a look at the routine
> that Paul left at the end for debugging?

I pursued the bug with thimbles, care, forks, and hope, and eventually
discovered that it was a minor coding error that I introduced two
weekends ago while removing a GCC warning.  Sorry about that.  I
installed this patch.

2002-11-18  Paul Eggert  <address@hidden>

        * data/glr.c (yygetLRActions): Replace `yyindex' with
        `yytable[yyindex]' to fix typo introduced in my 2002-11-09 patch.
        This fixes the regression with Sun ONE Studio 7 cc that I reported in
        <http://mail.gnu.org/pipermail/bug-bison/2002-November/001892.html>.

--- glr.c.~1.40.~       2002-11-18 00:21:58.000000000 -0800
+++ glr.c       2002-11-18 23:12:18.995522000 -0800
@@ -730,10 +730,10 @@ yydefaultAction (yyStateNum yystate)
   return yydefact[yystate];
 }
 
-#define yyis_table_ninf(yyindex) \
+#define yyis_table_ninf(yytable_value) \
   ]m4_if(m4_eval(b4_table_ninf < b4_table_min), 1,
         0,
-        ((yyindex) == YYTABLE_NINF))[
+        ((yytable_value) == YYTABLE_NINF))[
 
 /** Set *YYACTION to the action to take in YYSTATE on seeing YYTOKEN.
  *  Result R means
@@ -753,7 +753,7 @@ yygetLRActions (yyStateNum yystate, int 
       *yyaction = -yydefact[yystate];
       *yyconflicts = yyconfl;
     }
-  else if (! yyis_table_ninf (yyindex))
+  else if (! yyis_table_ninf (yytable[yyindex]))
     {
       *yyaction = yytable[yyindex];
       *yyconflicts = yyconfl + yyconflp[yyindex];




reply via email to

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