Horton

Thread

Commits

  1. Remove unnecessary braces, to match the surrounding style.

  1. Minor style cleanup in tab-complete.c

    Thomas Munro <thomas.munro@enterprisedb.com> — 2017-07-12T02:19:51Z

    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

  2. Re: Minor style cleanup in tab-complete.c

    Michael Paquier <michael.paquier@gmail.com> — 2017-07-12T04:39:48Z

    On Wed, Jul 12, 2017 at 11:19 AM, Thomas Munro
<thomas.munro@enterprisedb.com> 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

  3. Re: Minor style cleanup in tab-complete.c

    Heikki Linnakangas <hlinnaka@iki.fi> — 2017-07-12T09:32:58Z

    On 07/12/2017 07:39 AM, Michael Paquier wrote:
> On Wed, Jul 12, 2017 at 11:19 AM, Thomas Munro
> <thomas.munro@enterprisedb.com> 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