Kevin Grittner <kgri...@ymail.com> writes: > Patch 1 pushed, with each fix back-patched as far as the error > exists in supported releases. I held off on patch 2 -- partly > because I spotted at least one case where things weren't quite > right, partly because there's so much I haven't had time to go over > it in sufficient detail, and partly because commas are omitted so > consistently where most style guides want them that I thought > someone might want to argue that this was an intentional style > choice and should be preserved.
I would argue against applying patch 2 at all. I think it's what John McIntyre (http://www.baltimoresun.com/news/language-blog/) would call peeverism. If there are any places where the extra commas/periods actually add anything to clarity, then sure, change those places --- but doing it only because some style guide tells you to is not the way to approach the issue. We're writing English not C code here, and so there is no single standard of correctness. regards, tom lane -- Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-docs
