From: Danny S. <dan...@cl...> - 2003-02-06 09:02:16
|
----- Original Message ----- From: "Danny Smith" <dan...@cl...> To: "mingw-dvlpr" <min...@li...> Sent: Thursday, 6 February 2003 07:52 Subject: Re: [MinGW-dvlpr] _CRTIMP problem > > ----- Original Message ----- > From: "Wu Yongwei" <ad...@sh...> > To: <min...@li...> > Sent: Thursday, 6 February 2003 07:22 > Subject: Re: [MinGW-dvlpr] _CRTIMP problem > > > > Danny, > > > > I checked the code, and found some prototypes or inline definitions > > still lack __cdecl. I suppose they are not intentional? > ==snip===> > > Please check the list and tell me whether a patch over your patch is > > wanted. > > > No, I'll fix in my local patchset Done. Thanks for catching these. I'll repost the updated patchset to you, but first see below re workaround for warnings and apply to CVS as soon as > mingw-runtime version is bumped > > > > > Also, when I tested with the -mrtd option, gcc gave the following > > warning messages: > > > > e:/runtime/include/string.h:53: warning: conflicting types for > built-in > > function `memcmp' > > e:/runtime/include/string.h:54: warning: conflicting types for > built-in > > function `memcpy' > ==snip== > > > Any workarounds? > > I'll investigate. Could be even more serious in C++. > Doesn't raise warnings in C++ With gcc > 3.0, we can turn off the warnings with this at top of each header #if __GNUC__ >=3 #pragma GCC system_header #endif Then maybe go back to your _CRTAPI proposal with, #if defined __GNUC__ ... #if __GNUC >= 3 #define _CRTAPI __attribute__((cdecl)) #else #define _CRTAPI #endif ... #endif /* __GNUC__ */ in _mingw.h and use _CRTAPI throughout However, I'll file a GNATS PR for this because I think it needs to be documented and, hopefully, fixed. Danny > > > > Best regards, > > > > Wu Yongwei > > > > > > > > ------------------------------------------------------- > > This SF.NET email is sponsored by: > > SourceForge Enterprise Edition + IBM + LinuxWorld = Something 2 See! > > http://www.vasoftware.com > > _______________________________________________ > > MinGW-dvlpr mailing list > > Min...@li... > > https://lists.sourceforge.net/lists/listinfo/mingw-dvlpr > > > > > ------------------------------------------------------- > This SF.NET email is sponsored by: > SourceForge Enterprise Edition + IBM + LinuxWorld = Something 2 See! > http://www.vasoftware.com > _______________________________________________ > MinGW-dvlpr mailing list > Min...@li... > https://lists.sourceforge.net/lists/listinfo/mingw-dvlpr |