Re: [COMMITTERS] pgsql: Remove extra comma at end of enum list

From: Petr Jelinek <petr(at)2ndquadrant(dot)com>
To: Stephen Frost <sfrost(at)snowman(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Magnus Hagander <magnus(at)hagander(dot)net>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql: Remove extra comma at end of enum list
Date: 2016-10-24 20:53:27
Message-ID: 93404aed-5163-9844-2f47-d739e674ec82@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On 24/10/16 21:11, Stephen Frost wrote:
> * Tom Lane (tgl(at)sss(dot)pgh(dot)pa(dot)us) wrote:
>> Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> writes:
>>> Fabien COELHO wrote:
>>>> I find it annoying that "//" comments are not supported, or having to
>>>> declare variables at the beginning of a block instead of when first used...
>>
>>> I think some c99 features would be nice (variadic macros for one), but
>>> those particular two get a big "meh" from me.
>>
>> Yeah. Personally, I'd want to continue the rule against // comments just
>> as a matter of maintaining stylistic consistency. Code that's got a
>> random mishmash of // and /* comments looks messy, if you ask me.
>
> +1
>

+1

>> An alternative that would be worth considering is to adopt a uniform
>> rule of // for line-ending comments and /* for all other uses. We'd
>> have to teach pgindent about that, and I dunno how hard that is.
>
> I'm not sure it's worth the resulting mess.
>

I think the current commenting style stylistically nicer.

--
Petr Jelinek http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Fabien COELHO 2016-10-24 21:13:29 Re: [COMMITTERS] pgsql: Remove extra comma at end of enum list
Previous Message Fabien COELHO 2016-10-24 20:49:20 Re: [COMMITTERS] pgsql: Remove extra comma at end of enum list

Browse pgsql-hackers by date

  From Date Subject
Next Message Fabien COELHO 2016-10-24 21:13:29 Re: [COMMITTERS] pgsql: Remove extra comma at end of enum list
Previous Message Jonathan Katz 2016-10-24 20:50:53 Re: Press Release Draft - 2016-10-27 Cumulative Update