This is the mail archive of the
cygwin-apps@cygwin.com
mailing list for the Cygwin project.
Re: setup (ini.cc) vs CVS mingw-runtime
- From: "Max Bowsher" <maxb at ukf dot net>
- To: "Danny Smith" <danny_r_smith_2001 at yahoo dot co dot nz>,"cygwin-apps" <cygwin-apps at cygwin dot com>
- Date: Mon, 31 Mar 2003 11:01:32 +0100
- Subject: Re: setup (ini.cc) vs CVS mingw-runtime
- References: <20030331001713.27279.qmail@web21414.mail.yahoo.com>
Danny Smith wrote:
> I've been expecting a bug report for mingw-runtime and ini.cc in setup for
> awhile, but haven't seen one, so I'll ask if the CVS mingw runtime is
> getting any testing with setup anymore.
Was it ever?
> The problem is in the default usage of _CRTIMP ( =
> __attribute__((dllimport)) ) macro on all mingw runtime functions
> imported from dll. This usage provides a small to midling performance
> benefit. However. it can cause both compile time ICE and/or linktime
> multiple definitions (depending on order of declaration/definition/usage
> and version of GCC) when standard CRT functions declared in system
> headers (ie. fprintf in setup/ini.cc) are redefined.
>
> I'll make the _CRTIMP macro a no-op by default if someone will confirm
> that there is a problem with setup.
I confirm your ICE.
However, the redefinition of fprintf in ini.cc, is as far as I can see,
totally unused, and can be removed.
Robert: Can you confirm this, and approve me to delete it from ini.cc ?
If so, this ceases to become a setup issue, and becomes one of mingw-runtime
policy - so, up to you, Danny.
Max.