Re: push parser implemenation

[Bob Rossi]

On Thu, Sep 14, 2006 at 11:36:36AM +0200, Akim Demaille wrote:
> >>> "Bob" == Bob Rossi <address@hidden> writes:
> 
>  > On Tue, Sep 12, 2006 at 02:54:22PM -0700, Paul Eggert wrote:
>  >> Bob Rossi <address@hidden> writes:
>  >> 
>  >> > I can make yyparse look like:
>  >> >   void yyparse (struct yypvars *YYPVARS);
>  >> >
>  >> > However, the type 'struct yypvars' will not be visible in the users
>  >> > parser if they simply include calc.h (the bison header file).
>  >> 
>  >> What's the harm of having Bison generate a line "struct yypvars;"
>  >> (with no body) into the Bison-generated header file?  It's a no-op if
>  >> the structure is already declared, and a forward reference otherwise.
> 
>  > OK, I like that idea. I'll try to get that working, and then everything
>  > else should fall into place. Thanks for the advice.
> 
> 
> Given that you filed the FSF papers, I suggest that we include push.c
> asap in the CVS repository, and that we now discuss patches on it.

This would be fine with me. Should I get temporary commit-after-approval
permission? or do I send the patch in, and wait for one of you to commit
it?

Thanks,
Bob Rossi