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: Sun, 31 Dec 2006 21:54:09 -0500
User-agent: Mutt/1.5.12-2006-07-14

On Sun, Dec 31, 2006 at 07:42:47PM -0500, Joel E. Denny wrote:
> On Tue, 26 Dec 2006, Joel E. Denny wrote:
> 
> > 1. For yacc.c, you can invoke yyparse multiple times.  For push.c's push 
> > mode, I haven't thought this through.  I guess yypush_parse should set 
> > yyps->yynew = 1 whenever it returns with a parse error or success.
> 
> I committed the following patch to implement this and a little cleanup.
> 
> As far as I can tell, it's time to mv push.c yacc.c.  However, that's a 
> major change that I should not make without some consensus.
> 
> Once we do that, I think we should add a new maintainer-check pass that 
> forces yacc.c to run in that special push mode that emulates pull mode.  
> (That pass should probably use Valgrind also.)  Otherwise, we'll have to 
> keep setting up that mode manually in the way I've previously described:

Yeah, automating it sounds right to me too. If it's going to be a long
time before push.c replaces yacc.c, I would think we should automate it
right away.

Bob Rossi




reply via email to

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