Re: [BUGS] BUG #5394: invalid declspec for PG_MODULE_MAGIC - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: [BUGS] BUG #5394: invalid declspec for PG_MODULE_MAGIC
Date
Msg-id r2n9837222c1004061039n139ea2ccg2df2b99e8a9781e4@mail.gmail.com
Whole thread Raw
In response to Re: [BUGS] BUG #5394: invalid declspec for PG_MODULE_MAGIC  (Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp>)
Responses Re: [BUGS] BUG #5394: invalid declspec for PG_MODULE_MAGIC  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Mon, Mar 29, 2010 at 11:47 AM, Takahiro Itagaki
<itagaki.takahiro@oss.ntt.co.jp> wrote:
>
> "Vladimir Barzionov" <snego.barsik@gmail.com> wrote:
>
>> Same problem was already discussed for example here
>> http://dbaspot.com/forums/postgresql/393683-re-general-custom-c-function-palloc-broken.html
>>
>> Looks like the simplest way for correcting the issue is declaring additional
>> macro (something like PGMODULEEXPORT)
>
> Sure, I agree it is a longstanding bug in PostgreSQL. Developers who use
> MSVC (not mingw) always encounter the bug; machines in the buildfarm can
> build Windows binaries just because they have non-standard equipments.
>
> A patch attached. The name of "PGMODULEEXPORT" might be arguable.

I agree with this in principle, but won't this break every single
add-on module out there that supports Win32?


-- Magnus HaganderMe: http://www.hagander.net/Work: http://www.redpill-linpro.com/


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Proposal: Add JSON support
Next
From: Merlin Moncure
Date:
Subject: Re: SELECT constant; takes 15x longer on 9.0?