| From: | Andres Freund <andres(at)anarazel(dot)de> |
|---|---|
| To: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Declaration fixes |
| Date: | 2022-05-12 16:45:13 |
| Message-ID: | 20220512164513.vaheofqp2q24l65r@alap3.anarazel.de |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
I was experimenting with specifying symbol visiblity for functions explicitly,
i.e. adding PGDLLIMPORT markers for them, with the goal of getting rid of
src/tools/msvc/gendef.pl (and similar AIX stuff). While doing that I compared
the set of exported symbols before / after, leading me to find a few
pre-existing "issues".
I think the attached patches are all a good idea and trivial enought that I
think we should apply them now.
The changes are sufficiently obvious and/or explained in the commit messages.
Comments?
Greetings,
Andres Freund
| Attachment | Content-Type | Size |
|---|---|---|
| v1-0001-Add-missing-extern-to-function-prototypes.patch | text/x-diff | 16.0 KB |
| v1-0002-Add-static-to-file-local-variables-missing-it.patch | text/x-diff | 5.6 KB |
| v1-0003-Add-missing-binary_upgrade.h-where-includes.patch | text/x-diff | 1.5 KB |
| v1-0004-Remove-function-declaration-for-function-in-pg_pr.patch | text/x-diff | 1.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2022-05-12 17:18:05 | Re: Declaration fixes |
| Previous Message | Andres Freund | 2022-05-12 16:33:05 | Re: Crash in new pgstats code |