From 40e01f4bfae3018984d947e07efc7a7c624a6eda Mon Sep 17 00:00:00 2001
From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Fri, 15 Feb 2008 14:16:32 +0000
Subject: [PATCH] compilation fix for bcc 5.82: don't use enum for value
 definition, Borland doesn't use the value correctly when the template is used
 later then

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@51809 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---
 include/wx/meta/movable.h | 22 ++++++++++++++++------
 1 file changed, 16 insertions(+), 6 deletions(-)

diff --git a/include/wx/meta/movable.h b/include/wx/meta/movable.h
index 5441bf586b..3583c98a59 100644
--- a/include/wx/meta/movable.h
+++ b/include/wx/meta/movable.h
@@ -14,6 +14,17 @@
 #include "wx/defs.h"
 #include "wx/string.h" // for wxIsMovable<wxString> specialization
 
+// This macro declares something called "value" inside a class declaration.
+//
+// It has to be used because VC6 doesn't handle initialization of the static
+// variables in the class declaration itself while BCC5.82 doesn't understand
+// enums (it compiles the template fine but can't use it later)
+#if defined(__VISUALC__) && !wxCHECK_VISUALC_VERSION(7)
+    #define wxDEFINE_TEMPLATE_BOOL_VALUE(val) enum { value = val }
+#else
+    #define wxDEFINE_TEMPLATE_BOOL_VALUE(val) static const bool value = val
+#endif
+
 // Helper to decide if an object of type T is "movable", i.e. if it can be
 // copied to another memory location using memmove() or realloc() C functions.
 // C++ only gurantees that POD types (including primitive types) are
@@ -21,8 +32,7 @@
 template<typename T>
 struct wxIsMovable
 {
-    // NB: enum, because VC6 can't handle "static const bool value = true;"
-    enum { value = false };
+    wxDEFINE_TEMPLATE_BOOL_VALUE(false);
 };
 
 // Macro to add wxIsMovable<T> specialization for given type that marks it
@@ -30,7 +40,7 @@ struct wxIsMovable
 #define WX_DECLARE_TYPE_MOVABLE(type)                       \
     template<> struct wxIsMovable<type>                     \
     {                                                       \
-        enum { value = true };                              \
+        wxDEFINE_TEMPLATE_BOOL_VALUE(true);                 \
     };
 
 WX_DECLARE_TYPE_MOVABLE(bool)
@@ -62,12 +72,13 @@ WX_DECLARE_TYPE_MOVABLE(wxULongLong_t)
 template<typename T>
 struct wxIsMovable<T*>
 {
-    enum { value = true };
+    static const bool value = true;
 };
+
 template<typename T>
 struct wxIsMovable<const T*>
 {
-    enum { value = true };
+    static const bool value = true;
 };
 
 #endif // !VC++ < 7
@@ -81,5 +92,4 @@ struct wxIsMovable<const T*>
 WX_DECLARE_TYPE_MOVABLE(wxString)
 #endif
 
-
 #endif // _WX_META_MOVABLE_H_