From: | Gurjeet Singh <singh(dot)gurjeet(at)gmail(dot)com> |
---|---|
To: | PGSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Patch to include c.h |
Date: | 2012-09-16 13:49:35 |
Message-ID: | CABwTF4VssPvF8rOP-u1BJ6KTQVgVxnNAza5VtO2Hg1HN1L440A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I noticed that xlog.h uses PGDLLIMPORT, but it does not include c.h
directly or indirectly. Also, in timestamp.h different code is enabled
depending on HAVE_INT64_TMESTAMP being defined, but even though that macro
is defined in pg_config.h, it does not automatically trickle down into this
file, as it is supposed to.
Apparently all .c files manage to include c.h (directly or indirectly)
before including these files, so the compilers do the right thing. But I
caught these since I tried using an IDE, and it grays out sections to, or
shows error-markers when it doesn't find a macro included directly or
indirectly.
I vaguely remember a discussion where (I think) Bruce mentioned that we do
or intend to do some kind of compilability check on individual h files. DO
we have something in place to catch such things? I am sure there are many
other places where such inclusions are omitted, but these are the ones I
found on my first attempts to use this IDE.
Best regards,
--
Gurjeet Singh
Attachment | Content-Type | Size |
---|---|---|
include_c.h.patch | application/octet-stream | 645 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2012-09-16 14:23:14 | Re: embedded list v2 |
Previous Message | Robert Haas | 2012-09-16 12:10:38 | Re: Re: [COMMITTERS] pgsql: Properly set relpersistence for fake relcache entries. |