On Mon, Oct 31, 2011 at 1:34 PM, Daniel Stenberg <daniel_at_haxx.se> wrote:
> On Mon, 31 Oct 2011, Ivan Pechorin wrote:
> Thanks for this quote. I think it backs up our version.
> "The Winsock2.h header file internally includes core elements from the
>> Windows.h header file, so there is not usually an #include line for
>> the Windows.h header file in Winsock applications. If an #include line
>> is needed for the Windows.h header file
> Note this last 'if'. If windows.h is needed. We don't need it, we don't
> add an include for it...
> So, as far as I understand, the solution is :
>> 1) either include winsock2.h before windows.h
>> 2) or #define WIN32_LEAN_AND_MEAN, then include windows.h, then
>> include winsock2.h
> Right *if* we would want windows.h sure, but why do we have to add it now?
> If we haven't had to do it up until now with windows versions of libssh2
> working just fine, why do mingw's new headers suddenly force us to?
Btw, it's just a warning. Also, you ido nclude windows.h:
in libssh2_priv.h (line 47):
later line 95:
(which includes winsock2.h)
so a fix would be (line 47):
# define WIN32_LEAN_AND_MEAN
(note that I added a test on WIN32_LEAN_AND_MEAN because some stupid public
headers define it, but do not undef it. I experienced that)
Received on 2011-10-31