[Top][All Lists]

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

Re: push parser

From: Joel E. Denny
Subject: Re: push parser
Date: Fri, 8 Dec 2006 16:35:19 -0500 (EST)

On Fri, 8 Dec 2006, Bob Rossi wrote:

> Yup. I've got a patch coming. Is it ok to do this?
>   struct yypstate *yypstate = ...;
> and use yypstate as the type and variable name? If not, any suggestions?

If we do this:

  typedef struct yypstate yypstate;

which seems to be the norm (for example, YYSTYPE and YYLTYPE), then that 
might get us in trouble.

In non-Bison code, I try to name types in UpperCamelCase.  The above 
situation is one of the reasons why.  glr.c is good about this internally.  
However, it doesn't seem to be the precedent anywhere else in Bison, so I 
guess we need something like this:

  yypstate *yyps;

reply via email to

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