don't use static objects containing bitmaps, this almost certainly results in problem...

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

diff --git a/src/common/dcbufcmn.cpp b/src/common/dcbufcmn.cpp
index f4f289ed08b2a285cda8c6c8a1fcb02a16e4da97..d57648f99a5454d627ddfae94d1c3e5302fc30ad 100644 (file)
--- a/src/common/dcbufcmn.cpp
+++ b/src/common/dcbufcmn.cpp
@@ -25,6 +25,7 @@
 #endif
 
 #ifndef WX_PRECOMP
+    #include "wx/module.h"
 #endif
 
 #include "wx/dcbuffer.h"
@@ -37,33 +38,34 @@
 // wxSharedDCBufferManager: helper class maintaining backing store bitmap
 // ----------------------------------------------------------------------------
 
-class wxSharedDCBufferManager
+class wxSharedDCBufferManager : public wxModule
 {
 public:
     wxSharedDCBufferManager() { }
-    ~wxSharedDCBufferManager() { }
 
-    wxBitmap GetBuffer(int w, int h)
+    virtual bool OnInit() { return true; }
+    virtual void OnExit() { wxDELETE(ms_buffer); }
+
+    static wxBitmap GetBuffer(int w, int h)
     {
-        if ( !m_buffer.IsOk() ||
-             w > m_buffer.GetWidth() ||
-             h > m_buffer.GetHeight() )
+        if ( !ms_buffer ||
+                w > ms_buffer->GetWidth() ||
+                    h > ms_buffer->GetHeight() )
         {
-            // Create slightly larger bitmap so we don't need to
-            // be reallocating constantly when the user enlarges
-            // the frame for the first time.
-            m_buffer = wxBitmap(w, h);
+            delete ms_buffer;
+            ms_buffer = new wxBitmap(w, h);
         }
 
-        return m_buffer;
+        return *ms_buffer;
     }
 
 private:
-    wxBitmap m_buffer;
-};
+    static wxBitmap *ms_buffer;
 
-static wxSharedDCBufferManager gs_sharedDCBufferManager;
+    DECLARE_DYNAMIC_CLASS(wxSharedDCBufferManager)
+};
 
+IMPLEMENT_DYNAMIC_CLASS(wxSharedDCBufferManager, wxModule)
 
 // ============================================================================
 // wxBufferedDC
@@ -76,7 +78,7 @@ void wxBufferedDC::UseBuffer(wxCoord w, wxCoord h)
         if ( w == -1 || h == -1 )
             m_dc->GetSize(&w, &h);
 
-        m_buffer = gs_sharedDCBufferManager.GetBuffer(w, h);
+        m_buffer = wxSharedDCBufferManager::GetBuffer(w, h);
     }
 
     SelectObject(m_buffer);