remove declaration for ctor which is never defined

[wxWidgets.git] / src / common / string.cpp

diff --git a/src/common/string.cpp b/src/common/string.cpp
index c97125d2c46b1c6bd73fa3cd270ea54bae912297..82acf49d8640d430495f5ecf29f6dd435e771f67 100644 (file)
--- a/src/common/string.cpp
+++ b/src/common/string.cpp
@@ -59,7 +59,18 @@
 const size_t wxString::npos = (size_t) -1;
 
 #if wxUSE_STRING_POS_CACHE
 const size_t wxString::npos = (size_t) -1;
 
 #if wxUSE_STRING_POS_CACHE

-wxTLS_TYPE(wxString::Cache) wxString::ms_cache;
+
+struct wxStrCacheInitializer
+{
+    wxStrCacheInitializer()
+    {
+        // calling this function triggers s_cache initialization in it, and
+        // from now on it becomes safe to call from multiple threads
+        wxString::GetCache();
+    }
+};
+
+static wxStrCacheInitializer gs_stringCacheInit;

 
 // gdb seems to be unable to display thread-local variables correctly, at least
 // not my 6.4.98 version under amd64, so provide this debugging helper to do it
 
 // gdb seems to be unable to display thread-local variables correctly, at least
 // not my 6.4.98 version under amd64, so provide this debugging helper to do it


@@ -73,11 +84,11 @@ struct wxStrCacheDumper
         for ( unsigned n = 0; n < wxString::Cache::SIZE; n++ )
         {
             const wxString::Cache::Element&
         for ( unsigned n = 0; n < wxString::Cache::SIZE; n++ )
         {
             const wxString::Cache::Element&

-                c = wxString::ms_cache.cached[n];
+                c = wxString::GetCacheBegin()[n];

 
             printf("\t%u%s\t%p: pos=(%lu, %lu), len=%ld\n",
                    n,
 
             printf("\t%u%s\t%p: pos=(%lu, %lu), len=%ld\n",
                    n,

-                   n == wxString::ms_cache.lastUsed ? " [*]" : "",
+                   n == wxString::LastUsedCacheElement() ? " [*]" : "",

                    c.str,
                    (unsigned long)c.pos,
                    (unsigned long)c.impl,
                    c.str,
                    (unsigned long)c.pos,
                    (unsigned long)c.impl,


@@ -94,12 +105,9 @@ void wxDumpStrCache() { wxStrCacheDumper::ShowAll(); }
 
 wxString::CacheStats wxString::ms_cacheStats;
 
 
 wxString::CacheStats wxString::ms_cacheStats;
 

-namespace
-{
-
-struct ShowCacheStats
+struct wxStrCacheStatsDumper

 {
 {

-    ~ShowCacheStats()
+    ~wxStrCacheStatsDumper()

     {
         const wxString::CacheStats& stats = wxString::ms_cacheStats;
 
     {
         const wxString::CacheStats& stats = wxString::ms_cacheStats;
 


@@ -124,9 +132,9 @@ struct ShowCacheStats
                    stats.lentot, 100.*float(stats.lenhits)/stats.lentot);
         }
     }
                    stats.lentot, 100.*float(stats.lenhits)/stats.lentot);
         }
     }

-} s_showCacheStats;
+};

 
 

-} // anonymous namespace
+static wxStrCacheStatsDumper s_showCacheStats;

 
 #endif // wxPROFILE_STRING_CACHE
 
 
 #endif // wxPROFILE_STRING_CACHE