cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: MSVC and socklen_t and PSDK again, and now also include/curl/curl.h

From: Daniel Stenberg <daniel_at_haxx.se>
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.html
Received on 2007-10-23