Re: [HACKERS] Minor style cleanup in tab-complete.c

2017-07-12 Thread Heikki Linnakangas 

On 07/12/2017 07:39 AM, Michael Paquier wrote:

On Wed, Jul 12, 2017 at 11:19 AM, Thomas Munro
 wrote:

From the triviality department: I noticed some branches in
tab-complete.c's gargantuan if statement, mostly brand new, that break
from the established brace style.  Should we fix that like this?


For consistency I think it does.


Agreed. Committed, thanks.

- Heikki



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

Re: [HACKERS] Minor style cleanup in tab-complete.c

2017-07-11 Thread Michael Paquier 
On Wed, Jul 12, 2017 at 11:19 AM, Thomas Munro
 wrote:
> From the triviality department: I noticed some branches in
> tab-complete.c's gargantuan if statement, mostly brand new, that break
> from the established brace style.  Should we fix that like this?

For consistency I think it does.
-- 
Michael


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

[HACKERS] Minor style cleanup in tab-complete.c

2017-07-11 Thread Thomas Munro 
Hi,

>From the triviality department: I noticed some branches in
tab-complete.c's gargantuan if statement, mostly brand new, that break
from the established brace style.  Should we fix that like this?

-- 
Thomas Munro
http://www.enterprisedb.com


remove-extraneous-braces-from-tab-complete.patch
Description: Binary data

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