[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: push parser

From: Bob Rossi
Subject: Re: push parser
Date: Fri, 8 Dec 2006 10:46:17 -0500
User-agent: Mutt/1.5.12-2006-07-14

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.

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/ (_AT_DATA_CALC_Y): Use newly named functions.

Attachment: push.c.diff
Description: Text document

reply via email to

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