Re: push parser

[Bob Rossi]

On Mon, Dec 04, 2006 at 06:15:46PM -0500, Joel E. Denny wrote:
> Well, I found a little time much earlier than I thought....
> 
> 1. How was the name yypvars chosen?  What does ctx stand for?  Context?  
> It seems we have about three terms for the same thing: parser state, vars, 
> and context.  This will make the documentation a little confusing I fear.  
> I think we should pick one term and stick with it.  I prefer parser state, 
> which implies renaming yypvars to something like yypstate.

Hi Joel,

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.

I'm not really sure how to write a ChangeLog entry whening doing mass
renames, so hopefully this is OK.

Thanks,
Bob Rossi

2006-12-08  Bob Rossi  <address@hidden>

        * data/push.c (yypvarsinit, yypvars, struct yypvars): Renamed from
        yypvars to yypstate.
        (pv): Renamed all occurences of pv to pstate.
        (yypstateinit, yypushparse): Declare in bison generated header file.
        * tests/calc.at (_AT_DATA_CALC_Y): Use newly named functions.

push.c.diff
Description: Text document