bison-patches
[Top][All Lists]
Advanced

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

Re: Fix %error-verbose for conflicts resolved by %nonassoc.


From: Joel E. Denny
Subject: Re: Fix %error-verbose for conflicts resolved by %nonassoc.
Date: Wed, 26 Aug 2009 16:38:17 -0400 (EDT)
User-agent: Alpine 1.00 (DEB 882 2007-12-20)

On Wed, 26 Aug 2009, Akim Demaille wrote:

> > #define YYPACT_NINF ]b4_pact_ninf[
> > 
> > +#define yyis_pact_ninf(yystate) \
> > +  ]b4_table_value_equals([[pact]], [[yystate]], [b4_pact_ninf])[
> > +
> > #define YYTABLE_NINF ]b4_table_ninf[
> > 
> > +#define yyis_table_ninf(yytable_value) \
> > +  ]b4_table_value_equals([[table]], [[yytable_value]], [b4_table_ninf])[
> 
> We try to stick to CamelCase for macro argument names.  I case see that this
> way it really looks like a function, but it is not :)

Ah, thanks.  I had copied from glr.c, but I've noticed in the past that 
glr.c has its own style.

But what about making this a static inline function?  Or does this 
discussion need to wait until we abandon K&R?




reply via email to

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