Re: Mark all GUC variable as PGDLLIMPORT
Julien Rouhaud <rjuju123@gmail.com>
From: Julien Rouhaud <rjuju123@gmail.com>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Chapman Flack <chap@anastigmatix.net>, Andres Freund <andres@anarazel.de>, Michael Paquier <michael@paquier.xyz>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2022-02-15T17:10:50Z
Lists: pgsql-hackers
Hi, On Mon, Feb 14, 2022 at 12:45:08PM -0500, Robert Haas wrote: > On Mon, Feb 14, 2022 at 12:34 PM Tom Lane <tgl@sss.pgh.pa.us> wrote: > > > * There's a moderately sizable subset of GUCs where the underlying > > variable is not visible at all because it's static in guc.c. > > Typically this is because that variable is only used for display > > and there's an assign hook that stores the real data somewhere else. > > I suppose what we want in such cases is for the "somewhere else" > > to be PGDLLIMPORT'd, but in a lot of cases those variables are also > > static in some other module. Does this proposal include exporting > > variables that currently aren't visible to extensions at all? > > I'm a little resistant to that. I can buy making sure that Windows > > has a level playing field, but that's as far as I want to go. > > I can live with that. If someone complains about those variables being > static-to-file instead of globally visible, we can address that > complaint on its merits when it is presented. Same here, if any third-party project had any use of such variable, they would have sent some patch for that already so I don't see any reason to change it now.
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