On 09.06.26 19:03, Tom Lane wrote:
shaobo zhang <[email protected]> writes:
Why do we even need to care about that?
--
Michael

Agreed, it's purely cosmetic. If there's no interest in this cleanup,
I'm fine to withdraw it. Thanks both for looking.

My inner neatnik likes consistency, so in the abstract I'd rather have
the semicolon present.  But I agree with Michael that it's not worth
the overhead to do this sort of fix retail.  (BTW I've got little
confidence that there aren't other places needing the same fix.)

What I'd really like is an equivalent of pgindent for bison files,
so that we could enforce this sort of style rule automatically.
But I suppose that's more work than it's worth, too.  Unless someone
knows of an existing tool?

The release notes of bison 3.0 say

Missing semicolons at the end of actions are no longer added (as announced
  in the release 2.5).

But apparently that is not in fact happening?



Reply via email to