Excerpts from Jim Nasby's message of mar ene 17 21:21:57 -0300 2012: > On Jan 15, 2012, at 4:42 PM, Peter Geoghegan wrote: > > Attached is a patch that myself and Simon Riggs collaborated on. I > > took the group commit patch that Simon posted to the list back in > > November, and partially rewrote it. > > Forgive me if this is a dumb question, but I noticed a few places doing > things like: > > if (...) > Blah(); > else > { > ... > } > > Is that allowed PG formatting? I thought that if braces were required on one > branch of an if they were supposed to go on both sides.
Yeah, we even used to have pg_indent remove braces around single statements, so if you had one statement in the "if" branch and more around the other one, pg_indent would have left things like that anyway. (This particular pg_indent behavior got removed because it messed up PG_TRY blocks.) -- Álvaro Herrera <alvhe...@commandprompt.com> The PostgreSQL Company - Command Prompt, Inc. PostgreSQL Replication, Consulting, Custom Development, 24x7 support -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers