From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Extend & revamp pg_bswap.h infrastructure. |
Date: | 2017-09-30 16:28:51 |
Message-ID: | 29622.1506788931@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Andres Freund <andres(at)anarazel(dot)de> writes:
> On 2017-09-30 12:17:06 -0400, Tom Lane wrote:
>> c.h will already have included <stdlib.h>. There might be some
>> value in this if we anticipated allowing freestanding use of this
>> header, but that won't happen because it depends on configure symbols.
> Well, it's not that obvious where the _byteswap_* functions are coming
> from on msvc. I guess we can just leave the comment
> /* In all supported versions msvc provides _byteswap_* functions in stdlib.h */
> there, but I see no harm in the current form either.
I'm OK with a comment like that, but I think we have a general style
guideline against headers duplicating stuff from c.h/postgres.h.
It just promotes confusion, IMO.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-09-30 17:41:08 | pgsql: Support arrays over domains. |
Previous Message | Andres Freund | 2017-09-30 16:20:40 | Re: pgsql: Extend & revamp pg_bswap.h infrastructure. |