MinGW fix: Use _MSC_VER instead of _WIN32 where appropriate

Use _MSC_VER (instead of _WIN32) for things that are specific for
Visual Studio.

Also remove #include <winsock2.h> from hiredis.h, as it leaks too
many symbols and defines into the global namespace, which is
undesirable for a public interface header. Anyone who uses the
the affected parts of the hiredis API needs to include the
appropriate headers anyway in order to declare struct timeval
variables.

1 files changed, 2 insertions, 4 deletions

diff --git a/async.c b/async.c
index 171fabd..e46573f 100644
--- a/async.c
+++ b/async.c
@@ -32,11 +32,8 @@
 #include "fmacros.h"
 #include <stdlib.h>
 #include <string.h>
-#ifndef _WIN32
+#ifndef _MSC_VER
 #include <strings.h>
-#else
-#define strcasecmp stricmp
-#define strncasecmp  strnicmp
 #endif
 #include <assert.h>
 #include <ctype.h>
@@ -46,6 +43,7 @@
 #include "dict.c"
 #include "sds.h"
 #include "sslio.h"
+#include "win32.h"
 
 #define _EL_ADD_READ(ctx)                                         \
     do {                                                          \