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: Tue, 10 Oct 2006 10:57:34 -0400
User-agent: Mutt/1.5.11

On Mon, Oct 09, 2006 at 09:27:59AM -0700, Paul Eggert wrote:
> Bob Rossi <address@hidden> writes:
> 
> > That's odd, the test suite passes for me. I just did an update, and
> > reran the tests, and they still pass. I'm posting the patch again in 
> > case I messed something up the first time.
> 
> No, the patch is right: it matches what I have.
>  90: Calculator %push-parser %locations            FAILED (calc.at:574)
>  96: Calculator %push-parser %error-verbose %debug %locations %defines 
> %name-prefix=calc %verbose %yacc FAILED (calc.at:583)

OK, I understand now. It's because you have to 'cp push.c yacc.c' in
order for the new tests to pass. They will not work with yacc.c because
it doesn't support the push parser.

Should I modify the test suite somehow so that tests 90 and 96 
explicitly use the push.c skeleton?

Thanks,
Bob Rossi




reply via email to

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