From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Move NON_EXEC_STATIC from c.h |
Date: | 2022-08-16 13:50:23 |
Message-ID: | 1118582.1660657823@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com> writes:
> Looking to tidy up c.h a bit, I think the NON_EXEC_STATIC #define
> doesn't need to be known globally, and it's not related to establishing
> a portable C environment, so I propose to move it to a more localized
> header, such as postmaster.h, as in the attached patch.
Hmm, postgres.h seems like a better choice, since in principle any
backend file might need this. This arrangement could require
postmaster.h to be included just for this macro.
Also, the macro was severely underdocumented already, and I don't
find "no comment at all" to be better. Can't we afford a couple
of lines of explanation?
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2022-08-16 13:55:14 | Re: SQL/JSON features for v15 |
Previous Message | Robert Haas | 2022-08-16 13:47:56 | Re: Propose a new function - list_is_empty |