Re: Mark all GUC variable as PGDLLIMPORT
Chapman Flack <chap@anastigmatix.net>
From: Chapman Flack <chap@anastigmatix.net>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Peter Eisentraut <peter.eisentraut@enterprisedb.com>,
Julien Rouhaud <rjuju123@gmail.com>, Bruce Momjian <bruce@momjian.us>,
Tom Lane <tgl@sss.pgh.pa.us>, Michael Paquier <michael@paquier.xyz>,
PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-08-24T21:06:54Z
Lists: pgsql-hackers
On 08/24/21 16:31, Robert Haas wrote: > about adding PGDLLIMPORT, which ought to be totally uncontroversial, The thing is, I think I have somewhere a list of all the threads on this topic that I've read through since the first time I had to come with my own hat in hand asking for a PGDLLIMPORT on something, years ago now, and I don't think I have ever seen one where it was as uncontroversial as you suggest. In each iteration, I think I've also seen a countervailing view expressed in favor of looking into whether globals visibility could be further /reduced/. Maybe that view is slowly losing adherents? I don't know; it did get advanced again by some in this thread. The positions seem to routinely fall into two boxes: * Let's look into that, it would be a step toward having a more defined API * No, what we have now is so far from a defined API that there's nothing worth stepping toward. The situation seems intermediate to me. The current condition is clearly something that organically grew without a strong emphasis on defining API. On the other hand, there are many corners of it that show evidence of thought about encapsulation and API by the developers of those corners. It seems to me like the kind of setting where incrementalists can find room for small moves. Regards, -Chap
Commits
-
Remove PGDLLIMPORT marker from __pg_log_level
- 8d3341266508 15.0 landed
-
Mark a few 'bbsink' related functions / variables static.
- b5f44225b833 15.0 landed
-
Add some missing PGDLLIMPORT markings
- 5edeb574285e 15.0 landed
-
Apply PGDLLIMPORT markings broadly.
- 8ec569479fc2 15.0 landed
-
Helper script to apply PGDLLIMPORT markings.
- 80900d469091 15.0 landed
-
Simplify declaring variables exported from libpgcommon and libpgport.
- e04a8059a74c 15.0 cited