[prev in list] [next in list] [prev in thread] [next in thread] 

List:       postgresql-general
Subject:    Re: [HACKERS] gincostestimate
From:       Robert Haas <robertmhaas () gmail ! com>
Date:       2010-07-30 18:03:59
Message-ID: AANLkTiki9N+7gASPdi78BGULYs4ZKJPHZMmj8NpPhMz4 () mail ! gmail ! com
[Download RAW message or body]

On Fri, Jul 30, 2010 at 1:19 PM, Jan Urbański <wulczer@wulczer.org> wrote:
> The patch has lots of statements like if ( GinPageIsLeaf(page) ), that is
> with extra space between the outer parenthesis and the condition, which
> AFAIK is not the project style. I guess pgindent fixes that, so it's no big
> deal.

It's better if these get cleaned up.  pgindent will fix it eventually,
but the less stuff pgindent has to touch, the less likelihood there is
of breaking outstanding patches when it's run.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

[prev in list] [next in list] [prev in thread] [next in thread]