From: Gilles Depeyrot <gilles_depeyrot@mac.com>
Date: Thu, 29 May 2003 14:03:37 +0000 (+0000)
Subject: improved deallocation fix for Visual C++ Multithreaded non DLL runtime
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/8ecf21b7db119d71e33e2651027570c4f91a690b

improved deallocation fix for Visual C++ Multithreaded non DLL runtime
so that it only has a performance impact on wxString in this configuration


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@20756 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/include/wx/string.h b/include/wx/string.h
index 852f67115b..229b69b2f2 100644
--- a/include/wx/string.h
+++ b/include/wx/string.h
@@ -190,17 +190,19 @@ struct WXDLLEXPORT wxStringData
   // lock/unlock
   void  Lock()   { if ( !IsEmpty() ) nRefs++;                    }
 
-  // VC++ will refuse to inline this function but profiling shows that it
-  // is wrong
+  // VC++ will refuse to inline Unlock but profiling shows that it is wrong
 #if defined(__VISUALC__) && (__VISUALC__ >= 1200)
   __forceinline
 #endif
-  void  Unlock() { if ( !IsEmpty() && --nRefs == 0) Free();  }
-
   // VC++ free must take place in same DLL as allocation when using non dll
   // run-time library (e.g. Multithreaded instead of Multithreaded DLL)
+#if defined(__VISUALC__) && defined(_MT) && !defined(_DLL)
+  void  Unlock() { if ( !IsEmpty() && --nRefs == 0) Free();  }
   // we must not inline deallocation since allocation is not inlined
   void  Free();
+#else
+  void  Unlock() { if ( !IsEmpty() && --nRefs == 0) free();  }
+#endif
 
   // if we had taken control over string memory (GetWriteBuf), it's
   // intentionally put in invalid state
diff --git a/src/common/string.cpp b/src/common/string.cpp
index d0b3facc6b..2a3ce18e41 100644
--- a/src/common/string.cpp
+++ b/src/common/string.cpp
@@ -164,10 +164,13 @@ wxSTD ostream& operator<<(wxSTD ostream& os, const wxString& str)
 // wxStringData class deallocation
 // ===========================================================================
 
+#if defined(__VISUALC__) && defined(_MT) && !defined(_DLL)
+#  pragma message (__FILE__ ": building with Multithreaded non DLL runtime has a performance impact on wxString!")
 void wxStringData::Free()
 {
     free(this);
 }
+#endif
 
 // ===========================================================================
 // wxString class core