Andrew Dunstan escribió: > > On 11/15/2012 10:11 AM, Robert Haas wrote: > > >+ { > >+ sift_down(heap, i); > >+ } > > > >Project style is to omit braces for a single-line body. This comes up > >a few other places as well. > > I thought we modified that some years ago, although my memory of it > is a bit hazy.
No, we only modified pg_indent to not take the braces off, because of PG_TRY blocks. But we keep using single statements instead of compound in many places; but there is no hard rule about it. To me, using braces were they are not needed is pointless and ugly. We're very careful about ensuring that our macro definitions work nicely with single-statement if/else, for example. -- Álvaro Herrera http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers