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