[tor-dev] WIN32_WINNT in or/or.h

[Gisle Vanem]

I think that the values for 'WIN32_WINNT' and '_WIN32_WINNT'
should be protected against redefinement. 

Reason: In order for MingW to prototype getaddrinfo() and freeaddrinfo() 
correctly (in <ws2tcpip.h>), '_WIN32_WINNT' *must* be defined as 0x0501
or higher. or/or.h blindly defines them as 0x0400. So, building with 
-DHAVE_GETADDRINFO needs _WIN32_WINNT to be set to 0x0501.

As of now, MingW would simply fallback to use gethostbyname(). So I think this
little patch is in order:

--- ..\Git-latest\src\or\or.h   Mon Jun 20 23:58:06 2011
+++ or\or.h     Tue Jun 21 00:04:42 2011
@@ -23,8 +23,12 @@
 #endif

 #ifdef MS_WINDOWS
+#ifndef WIN32_WINNT
 #define WIN32_WINNT 0x400
+#endif
+#ifndef _WIN32_WINNT
 #define _WIN32_WINNT 0x400
+#endif
 #define WIN32_LEAN_AND_MEAN
 #endif

-----------------------------

What do you say?

--gv