"Joel E. Denny" <[EMAIL PROTECTED]> writes: > + m4_ifset([b4_parse_param], [,]) b4_parse_param)[
The patch overall looks fine to me, but one minor thing. In usages
like this, it's better to put the empty item inside the conditional.
That is:
m4_ifset([b4_parse_param], [, b4_parse_param]))[
This is the usual style in Bison and it's easier to follow.
