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

From: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: 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 21:13:29
Message-ID: alpine.DEB.2.20.1610242257030.20001@lancre
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers


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

This is probably a matter of taste.

The question really to know when some C99 features (inline, mixing code &
declarations, flexible array members, slash-slash comments, compound
literals, variadic macros, restrict qualifier, ...), will be considered
okay for Pg sources, or if they should be kept C89 compliant for the next
27 years.

Well, it seems that Microsoft took time to implement C99 features, which
would mean that a recent "Visual C++" would be required to compile pg on
windows if pg would use C99 features.

> .oO( I wonder if it'd be possible to make ereport() an inline function ... )

Probaby not: ISTM that ereport relies on the fact that it is a macro to
avoid evaluating some arguments until necessary, but the compiler would
not do that on a function, whether inlined or not, because it would change
the semantics if the evaluation was to fail.

--
Fabien.

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2016-10-24 21:59:53 Re: [COMMITTERS] pgsql: Remove extra comma at end of enum list
Previous Message Petr Jelinek 2016-10-24 20:53:27 Re: [COMMITTERS] pgsql: Remove extra comma at end of enum list

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2016-10-24 21:59:53 Re: [COMMITTERS] pgsql: Remove extra comma at end of enum list
Previous Message Petr Jelinek 2016-10-24 20:53:27 Re: [COMMITTERS] pgsql: Remove extra comma at end of enum list