* winlean.h: Make sure certain Windows macros are undefined again.

Add comment to explain why.
	* winsup.h: Include winlean.h from C sources as well.
	* libc/minires-os-if.c: Drop including ntdef.h.

	Fix previous ChangeLog entry.

winsup/cygwin/ChangeLog

@@ -1,6 +1,13 @@
 2012-07-12  Corinna Vinschen  <corinna@vinschen.de>
 
-	* pseudo-reloc.cc: Drop including wchar.h.
+	* winlean.h: Make sure certain Windows macros are undefined again.
+	Add comment to explain why.
+	* winsup.h: Include winlean.h from C sources as well.
+	* libc/minires-os-if.c: Drop including ntdef.h.
+
+2012-07-12  Corinna Vinschen  <corinna@vinschen.de>
+
+	* pseudo-reloc.cc: Drop including wchar.h and ntdef.h.
 	(__report_error): Define module as WCHAR.
 	* advapi.cc: Drop including wchar.h.
 	* kernel32.cc: Ditto.

winsup/cygwin/libc/minires-os-if.c

@@ -16,7 +16,6 @@ details. */
 #include <ws2tcpip.h>
 #include <iphlpapi.h>
 #include <windns.h>
-#include <ntdef.h>
 #include "ntdll.h"
 #undef h_errno
 #include "minires.h"

winsup/cygwin/winlean.h

@@ -35,11 +35,48 @@ details. */
 #define WINIMPM
 #define WINSOCK_API_LINKAGE
 
+/* Windows headers define a couple of annoyingly intrusive macros for the
+   sole purpose of inline documentation.  Since they are defined without
+   respect for the namespace and not undef'ed anymore, they tend to collide
+   with otherwise innocent definitions in the application.  We check if they
+   exist and if not, we undef them again after including the Windows headers. */
+#ifndef IN
+#define __undef_IN
+#endif
+#ifndef OUT
+#define __undef_OUT
+#endif
+#ifndef OPTIONAL
+#define __undef_OPTIONAL
+#endif
+#ifndef NOTHING
+#define __undef_NOTHING
+#endif
+#ifndef CRITICAL
+#define __undef_CRITICAL
+#endif
+
 #include <windows.h>
 #include <wincrypt.h>
 #include <lmcons.h>
 #include <ntdef.h>
 
+#ifdef __undef_IN
+#undef IN
+#endif
+#ifdef __undef_OUT
+#undef OUT
+#endif
+#ifdef __undef_OPTIONAL
+#undef OPTIONAL
+#endif
+#ifdef __undef_NOTHING
+#undef NOTHING
+#endif
+#ifdef __undef_CRITICAL
+#undef CRITICAL
+#endif
+
 #undef _WINGDI_H
 #undef _WINUSER_H
 #undef _WINNLS_H

winsup/cygwin/winsup.h

@@ -81,8 +81,12 @@ int fcntl64 (int fd, int cmd, ...);
 #define __WIDE(a) L ## a
 #define _WIDE(a) __WIDE(a)
 
+#include "winlean.h"
+
 #ifdef __cplusplus
 
+#include "wincap.h"
+
 extern const char case_folded_lower[];
 #define cyg_tolower(c) (case_folded_lower[(unsigned char)(c)])
 extern const char case_folded_upper[];
@@ -92,9 +96,6 @@ extern const char case_folded_upper[];
 #define cfree newlib_cfree_dont_use
 #endif
 
-#include "winlean.h"
-#include "wincap.h"
-
 /* Used as type by sys_wcstombs_alloc and sys_mbstowcs_alloc.  For a
    description see there. */
 #define HEAP_NOTHEAP -1