On Tue, 19 Dec 2006, Paul Eggert wrote:

> "Joel E. Denny" <[EMAIL PROTECTED]> writes:
> 
> > -    int yytoken;
> > +    int yytoken]b4_push_if([], [[ = 0]])[;
> 
> How about if we leave this line alone, and put an initialization
> statement 'yytoken = 0;' elsewhere?  That sounds a bit cleaner.
> Similarly for the other initializations like this.
> 
> Other than that, it looks OK to me.

Actually that's my plan.  Somehow the above ugliness just made this diff a 
little easier to read.  If I promise that a future patch will clean this 
up, is it ok to commit this one?


Reply via email to