Daniel Stenberg wrote:
On Wed, 8 Apr 2009, Sukender wrote:
I'm not really sure about chat to remove/change. Do you have an idea?
Should we use a special flag that means "I'm using directly
mprintf.c, so don't set printf-like functions to be imported"?
I will commit my changes in include/curl/curl.h and mprintf.h and
such. They are not really intrusive. However, if you find the changes
unsuitable, I'll modify or rollback anything you want. Just tell me in
that case.
If you post your suggested changes here as a patch we can discuss them
before you commit anything. That's a better way...
In curl.h, line 100 the code looks like:
#if (defined(WIN32) || defined(_WIN32) || defined(__SYMBIAN32__)) && \
!defined(CURL_STATICLIB)
#if defined(BUILDING_LIBCURL)
#define CURL_EXTERN __declspec(dllexport)
#else
#define CURL_EXTERN __declspec(dllimport)
#endif
#else
#ifdef CURL_HIDDEN_SYMBOLS
#define CURL_EXTERN CURL_EXTERN_SYMBOL
#else
#define CURL_EXTERN
#endif
#endif
My question is: why don't CURL_HIDDEN_SYMBOLS is tested when we're in Win32
DLL? Isn't there something missing here?
And about the define to avoid DLL linkage problem, I suggest:
#ifdef USES_DIRECT_CURL_UTILITIES
#define CURL_UTILITY_EXTERN
#else
#define CURL_UTILITY_EXTERN CURL_EXTERN
#endif
And changing CURL_EXTERN to CURL_UTILITY_EXTERN when needed. For example with
curl_strequal, we would change:
CURL_EXTERN int (curl_strequal)(const char *s1, const char *s2);
to
CURL_UTILITY_EXTERN int (curl_strequal)(const char *s1, const char *s2);
But that's not clear if it sounds coherent with my first remark... Any idea?
--
Sukender
PVLE - Lightweight cross-platform game engine - http://pvle.sourceforge.net/
