diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-11-07 10:19:22 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-11-07 10:19:22 -0500 |
| commit | 33cb96ba1a84c612491fb5794674a649d1a6a4d6 (patch) | |
| tree | ac919f14dfaa786c6c9f2fbda86f3d572850f5d7 /src | |
| parent | 77517ba59f8d3a9d282c5e826bf19cbe195cd784 (diff) | |
| download | postgresql-33cb96ba1a84c612491fb5794674a649d1a6a4d6.tar.gz postgresql-33cb96ba1a84c612491fb5794674a649d1a6a4d6.zip | |
Revert "Provide DLLEXPORT markers for C functions via PG_FUNCTION_INFO_V1 macro."
This reverts commit c8ead2a3974d3eada145a0e18940150039493cc9.
Seems there is no way to do this that doesn't cause MSVC to give
warnings, so let's just go back to the way we've been doing it.
Discussion: <11843.1478358206@sss.pgh.pa.us>
Diffstat (limited to 'src')
| -rw-r--r-- | src/include/fmgr.h | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/include/fmgr.h b/src/include/fmgr.h index 3668ac3f6e9..0878418516e 100644 --- a/src/include/fmgr.h +++ b/src/include/fmgr.h @@ -350,11 +350,12 @@ typedef const Pg_finfo_record *(*PGFInfoFunction) (void); * * On Windows, the function and info function must be exported. Our normal * build processes take care of that via .DEF files or --export-all-symbols. - * Module authors using a different build process might do it differently, - * so we declare these functions PGDLLEXPORT for their convenience. + * Module authors using a different build process might need to manually + * declare the function PGDLLEXPORT. We do that automatically here for the + * info function, since authors shouldn't need to be explicitly aware of it. */ #define PG_FUNCTION_INFO_V1(funcname) \ -extern PGDLLEXPORT Datum funcname(PG_FUNCTION_ARGS); \ +extern Datum funcname(PG_FUNCTION_ARGS); \ extern PGDLLEXPORT const Pg_finfo_record * CppConcat(pg_finfo_,funcname)(void); \ const Pg_finfo_record * \ CppConcat(pg_finfo_,funcname) (void) \ |