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: Wed, 27 Sep 2006 08:50:49 -0400
User-agent: Mutt/1.5.11

On Tue, Sep 26, 2006 at 10:28:15AM -0700, Paul Eggert wrote:
> That looks good.  Some minor comments:
> 
> The usual style is to give local variables lower-case names.
> The patch uses upper-caes names for local vars, which looks a bit strange.

Are you refering to the parameter names? I was copying this:
  ]b4_c_function_decl([yyparse], [int],
     [[void *YYPARSE_PARAM], [YYPARSE_PARAM]])[

Bob Rossi




reply via email to

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