From d64ca34f1b6f34e86adefc7f735b4eff8e6d4a35 Mon Sep 17 00:00:00 2001
From: Lasse Collin <lasse.collin@tukaani.org>
Date: Sun, 1 Feb 2009 00:10:07 +0200
Subject: [PATCH] Use __cdecl also for function pointers in liblzma API when on
 Windows.

---
 src/liblzma/api/lzma.h      | 18 +++++++++++-------
 src/liblzma/api/lzma/base.h |  4 ++--
 src/liblzma/common/common.h | 18 +++++++-----------
 3 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/src/liblzma/api/lzma.h b/src/liblzma/api/lzma.h
index bb61721..d4c6932 100644
--- a/src/liblzma/api/lzma.h
+++ b/src/liblzma/api/lzma.h
@@ -154,15 +154,19 @@
  ******************/
 
 /*
- * Some systems require that the functions are declared specially
- * in the headers.
+ * Some systems require (or at least recommend) that the functions and
+ * function pointers are declared specially in the headers.
  */
+#ifdef _WIN32
+#	define LZMA_API_CALL __cdecl
+#	define LZMA_API_IMPORT __declspec(dllimport)
+#else
+#	define LZMA_API_CALL
+#	define LZMA_API_IMPORT
+#endif
+
 #ifndef LZMA_API
-#	if defined(_WIN32)
-#		define LZMA_API __cdecl __declspec(dllimport)
-#	else
-#		define LZMA_API
-#	endif
+#	define LZMA_API LZMA_API_CALL LZMA_API_IMPORT
 #endif
 
 
diff --git a/src/liblzma/api/lzma/base.h b/src/liblzma/api/lzma/base.h
index 9fe3ca6..4944e69 100644
--- a/src/liblzma/api/lzma/base.h
+++ b/src/liblzma/api/lzma/base.h
@@ -376,7 +376,7 @@ typedef struct {
 	 *              returned NULL if some function from liblzma
 	 *              returns LZMA_MEM_ERROR.
 	 */
-	void *(*alloc)(void *opaque, size_t nmemb, size_t size);
+	void *(LZMA_API_CALL *alloc)(void *opaque, size_t nmemb, size_t size);
 
 	/**
 	 * \brief       Pointer to a custom memory freeing function
@@ -390,7 +390,7 @@ typedef struct {
 	 *                      or when it is set to NULL, a pointer returned
 	 *                      by the standard malloc().
 	 */
-	void (*free)(void *opaque, void *ptr);
+	void (LZMA_API_CALL *free)(void *opaque, void *ptr);
 
 	/**
 	 * \brief       Pointer passed to .alloc() and .free()
diff --git a/src/liblzma/common/common.h b/src/liblzma/common/common.h
index 2642c5b..4d5b773 100644
--- a/src/liblzma/common/common.h
+++ b/src/liblzma/common/common.h
@@ -24,20 +24,16 @@
 #include "../../common/mythread.h"
 #include "../../common/integer.h"
 
-// Don't use ifdef...
-#if HAVE_VISIBILITY
-#	define LZMA_VISIBILITY __attribute__((__visibility__("default")))
+#if defined(_WIN32)
+#	define LZMA_API_EXPORT __declspec(dllexport)
+// Don't use ifdef or defined() below.
+#elif HAVE_VISIBILITY
+#	define LZMA_API_EXPORT __attribute__((__visibility__("default")))
 #else
-#	define LZMA_VISIBILITY
+#	define LZMA_API_EXPORT
 #endif
 
-#ifdef _WIN32
-#	define LZMA_EXPORT __cdecl __declspec(dllexport)
-#else
-#	define LZMA_EXPORT
-#endif
-
-#define LZMA_API LZMA_VISIBILITY LZMA_EXPORT
+#define LZMA_API LZMA_API_CALL LZMA_API_EXPORT
 
 #include "lzma.h"
 
-- 
2.7.4