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