Re: Headers dependencies cleanup - Mailing list pgsql-patches

From Tom Lane
Subject Re: Headers dependencies cleanup
Date
Msg-id 13665.1207621346@sss.pgh.pa.us
Whole thread Raw
In response to Re: Headers dependencies cleanup  (Zdenek Kotala <Zdenek.Kotala@Sun.COM>)
List pgsql-patches
Zdenek Kotala <Zdenek.Kotala@Sun.COM> writes:
> Alvaro Herrera napsal(a):
>> Not all compilers like (== support) inline macros apparently.

> Is it your assumption or do you mean some specific compiler? IIRC, inline is
> defined in C99 and my assumption :-) is that it should be supported by all
> compilers today.

The problem is (1) not all compilers support inline, (2) the ones that
do have divergent ideas on its semantics, and (3) the semantics
specified by C99 utterly suck (cf tuplesort.c).  gcc's traditional
semantics for inline are far more usable.  We have done "#ifdef gcc"
inlines in one or two places where the performance argument for adding
such clutter was compelling.  We can do that some more, but you'll need
equally compelling arguments.  Do not bother submitting patches to
create a very large number of inlines.

            regards, tom lane

pgsql-patches by date:

Previous
From: Alvaro Herrera
Date:
Subject: explicit tracking of ActiveSnapshot
Next
From: Heikki Linnakangas
Date:
Subject: Indexam API changes