Re: push parser

[Bob Rossi]

On Fri, Dec 08, 2006 at 04:00:08PM -0500, Joel E. Denny wrote:
> On Fri, 8 Dec 2006, Bob Rossi wrote:
> 
> > Here's a patch that renames all uses of pv to pstate. It changes all
> > uses of pvars to pstate. It also adds the prototypes for yypstateinit
> > and yypushparse to the bison generated header file.
> 
> Thanks, but I just noticed that your local variable pstate (formerly pv) 
> intrudes on the user namespace.  All Bison names should be in the yy (or 
> YY for most macros) namespace.  To eliminate another renaming patch, could 
> you just rewrite this one?  

Yup. I've got a patch coming. Is it ok to do this?
  struct yypstate *yypstate = ...;
and use yypstate as the type and variable name? If not, any suggestions?

Thanks,
Bob Rossi