From: Christian Schmidbauer <ch.schmidbauer@gmail.com>
Date: Fri, 29 Apr 2016 19:53:30 +0000 (+0100)
Subject: st/nine: Clean up WINAPI definition
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=f5d6ed57022b7459bc0f63d9131528e47f4ec982;p=mesa.git

st/nine: Clean up WINAPI definition

As Emil pointed out, only gcc, clang and MSVC compatibility is required.
Hence the check for GNUC can be skipped, as __i386__ and __x86_64__ are
only defined for gcc/clang, not for MSVC.

Remove the #undef which has been there for historic reasons, when wine
dlls for nine have been built inside mesa. Instead use #ifndef in order
to avoid redefining WINAPI from MSVC's headers.

Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
Acked-by: Axel Davy <axel.davy@ens.fr>
---

diff --git a/include/D3D9/d3d9types.h b/include/D3D9/d3d9types.h
index e0b86523daa..88f22b97112 100644
--- a/include/D3D9/d3d9types.h
+++ b/include/D3D9/d3d9types.h
@@ -173,22 +173,16 @@ typedef struct _RGNDATA {
 #define D3DPRESENTFLAG_RESTRICTED_CONTENT              0x00000400
 #define D3DPRESENTFLAG_RESTRICT_SHARED_RESOURCE_DRIVER 0x00000800
 
-
-#ifdef WINAPI
-#undef WINAPI
-#endif /* WINAPI*/
-
-#ifdef __GNUC__
-  #if (defined(__x86_64__) && !defined(__ILP32__)) || defined(_M_X64)
+/* Windows calling convention */
+#ifndef WINAPI
+  #if defined(__x86_64__) && !defined(__ILP32__)
     #define WINAPI __attribute__((ms_abi))
-  #elif defined(__i386) || defined(_M_IX86)
+  #elif defined(__i386__)
     #define WINAPI __attribute__((__stdcall__))
   #else /* neither amd64 nor i386 */
     #define WINAPI
   #endif
-#else /* __GNUC__ */
-  #define WINAPI
-#endif
+#endif /* WINAPI */
 
 /* Implementation caps */
 #define D3DPRESENT_BACK_BUFFERS_MAX    3