curl-library
Re: MSVC and socklen_t and PSDK again, and now also include/curl/curl.h
Date: Tue, 23 Oct 2007 21:57:54 +0200 (CEST)
On Tue, 23 Oct 2007, Yang Tse wrote:
> This change
> http://cool.haxx.se/cvs.cgi/curl/lib/config-win32.h.diff?r1=1.52&r2=1.53
> breaks all msvc builds which have PSDK properly installed and are using a
> compiler previous to VS2005.
Does those headers define something that we can #ifdef on to check for an
existing socklen_t typedef?
> Since
> http://cool.haxx.se/cvs.cgi/curl/include/curl/curl.h.diff?r1=1.327&r2=1.328
> the interface of the library requires socklen_t being already defined since
> it is exposing struct curl_sockaddr.
>
> So the real question when using msvc is if libcurl should stop providing a
> socklen_t replacement or otherwise also provide it in inlude/curl/curl.h if
> not available for Windows target builds.
I think the latter, as I can't see how we can do the first without breaking
the build on some msvc compiles. Or did I missunderstand?
-- Commercial curl and libcurl Technical Support: http://haxx.se/curl.htmlReceived on 2007-10-23