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