bison-patches
[Top][All Lists]
Advanced

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

Re: push parser


From: Bob Rossi
Subject: Re: push parser
Date: Wed, 6 Dec 2006 07:18:50 -0500
User-agent: Mutt/1.5.12-2006-07-14

On Wed, Dec 06, 2006 at 03:12:42AM -0500, Joel E. Denny wrote:
> On Tue, 5 Dec 2006, Bob Rossi wrote:
> 
> > On Tue, Dec 05, 2006 at 10:30:21PM -0500, Bob Rossi wrote:
> > > OK, I'm going to get the current patch reworked and resubmit it. If that
> > > is acceptable, we should take out these issues pretty quickly.
> > 
> > How is this?
> 
> Unfortunately, test cases 93 and 99 fail for me.  Could you look into that 
> first?  Be sure to use the --enable-gcc-warnings flag when you configure 
> Bison.  Thanks.

OK, here is another patch that fixes the problem. Just so you know,
tests 93 and 99 are push-parser tests and they explicitly use the
push-parser skeleton. 

Before I submit a patch, I make sure the test suite works. Then I do a
'cp data/push.c data/yacc.c' and do a make install and make check. This
ensures that the push parser still passes the test suite in normal and
pure mode. That should help the transition when we are ready to switch
from yacc.c to push.c

Thanks,
Bob Rossi

Attachment: push.c.diff
Description: Text document


reply via email to

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