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?

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. -- Micha

[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 D