Removed the SaveBG hack. The real incompatibility is that the MemoryDC

is not automatically cleared to the BG colour of the window when a
temporary bitmap is used.

Inlined the rest of the methods, there really is no reason for them not
to be.  (will remove dcbuffer.cpp as soon as I figure out the bakefiles)

Do device to logical transforms in UnMask.

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

diff --git a/include/wx/dcbuffer.h b/include/wx/dcbuffer.h
index 8385aa6d2b043a5cab8d42ab5831e6b37083727f..b29469ce2ac57b1067ec9c5c393c925ac6831b22 100644 (file)
--- a/include/wx/dcbuffer.h
+++ b/include/wx/dcbuffer.h
@@ -12,26 +12,9 @@
 #ifndef _WX_DCBUFFER_H_
 #define _WX_DCBUFFER_H_
 
-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-    #pragma interface "dcbuffer.h"
-#endif
-
 #include "wx/dcmemory.h"
 #include "wx/dcclient.h"
 
-// flags for wxBufferedDC ctor/Init()
-enum
-{
-    // this is more efficient and hence default
-    wxBUFFER_DC_OVERWRITE_BG = 0,
-
-    // preserve the old background: more time consuming
-    wxBUFFER_DC_PRESERVE_BG = 1,
-
-
-    // flags used by default
-    wxBUFFER_DC_DEFAULT = wxBUFFER_DC_OVERWRITE_BG
-};
 
 // ----------------------------------------------------------------------------
 // Double buffering helper.
@@ -46,22 +29,46 @@ public:
     }
 
     // Construct a wxBufferedDC using a user supplied buffer.
-    wxBufferedDC(wxDC *dc, const wxBitmap &buffer);
+    wxBufferedDC(wxDC *dc, const wxBitmap &buffer)
+        : m_dc( dc ),
+          m_buffer( buffer )
+    {
+        UseBuffer();
+    }
 
     // Construct a wxBufferedDC with an internal buffer of 'area'
     // (where area is usually something like the size of the window
     // being buffered)
-    wxBufferedDC(wxDC *dc, const wxSize &area, int flags = wxBUFFER_DC_DEFAULT);
+    wxBufferedDC(wxDC *dc, const wxSize &area)
+        : m_dc( dc ),
+          m_buffer( area.GetWidth(), area.GetHeight() )
+    {
+        UseBuffer();
+    }
 
     // default copy ctor ok.
 
     // The usually desired  action in the dtor is to blit the buffer.
-    virtual ~wxBufferedDC() { if ( m_dc ) UnMask(); }
+    virtual ~wxBufferedDC()
+    {
+        if ( m_dc ) UnMask();
+    }
 
     // These reimplement the actions of the ctors for two stage creation, but
     // are not used by the ctors themselves to save a few cpu cycles.
-    void Init(wxDC *dc, const wxBitmap &bitmap);
-    void Init(wxDC *dc, const wxSize &area, int flags = wxBUFFER_DC_DEFAULT);
+    void Init(wxDC *dc, const wxBitmap &buffer)
+    {
+        wxASSERT_MSG( m_dc == 0 && m_buffer == wxNullBitmap,
+                      _T("wxBufferedDC already initialised") );
+        m_dc = dc;
+        m_buffer = buffer;
+        UseBuffer();
+    }
+
+    void Init(wxDC *dc, const wxSize &area)
+    {
+        Init(dc, wxBitmap(area.GetWidth(), area.GetHeight()));
+    }
 
     // Blits the buffer to the dc, and detaches the dc from the buffer (so it
     // can be effectively used once only).
@@ -69,7 +76,16 @@ public:
     // Usually called in the dtor or by the dtor of derived classes if the
     // BufferedDC must blit before the derived class (which may own the dc it's
     // blitting to) is destroyed.
-    void UnMask();
+    void UnMask()
+    {
+        wxASSERT_MSG( m_dc != 0,
+                      _T("No underlying DC associated with wxBufferedDC (anymore)") );
+
+        m_dc->Blit( m_dc->DeviceToLogicalX(0), m_dc->DeviceToLogicalY(0),
+                    m_buffer.GetWidth(), m_buffer.GetHeight(), this,
+                    DeviceToLogicalX(0), DeviceToLogicalY(0) );
+        m_dc = NULL;
+    }
 
 private:
     // check that the bitmap is valid and use it
@@ -80,15 +96,6 @@ private:
         SelectObject(m_buffer);
     }
 
-    // preserve the background if necessary
-    void SaveBg(const wxSize& area, int flags)
-    {
-        if ( flags & wxBUFFER_DC_PRESERVE_BG )
-        {
-            Blit(0, 0, area.GetWidth(), area.GetHeight(), m_dc, 0, 0);
-        }
-    }
-
     // the underlying DC to which we copy everything drawn on this one in
     // UnMask()
     //
@@ -112,22 +119,14 @@ private:
 class WXDLLIMPEXP_ADV wxBufferedPaintDC : public wxBufferedDC
 {
 public:
-    // this ctor creates a bitmap of the size of the window for buffering
-    wxBufferedPaintDC(wxWindow *window, int flags = wxBUFFER_DC_DEFAULT)
-        : m_paintdc(window)
-    {
-        Init(&m_paintdc, window->GetClientSize(), flags);
-
-        Prepare(window);
-    }
-
-    // the bitmap must be valid here
-    wxBufferedPaintDC(wxWindow *window, const wxBitmap& buffer)
+    // If no bitmap is supplied by the user, a temporary one wil; be created.
+    wxBufferedPaintDC(wxWindow *window, const wxBitmap& buffer = wxNullBitmap)
         : m_paintdc(window)
     {
-        Init(&m_paintdc, buffer);
-
-        Prepare(window);
+        if( buffer != wxNullBitmap )
+            Init(&m_paintdc, buffer);
+        else
+            Init(&m_paintdc, window->GetClientSize());
     }
 
     // default copy ctor ok.
@@ -140,12 +139,6 @@ public:
     }
 
 private:
-    // prepare the underlying DC
-    void Prepare(wxWindow *window)
-    {
-        window->PrepareDC(m_paintdc);
-    }
-
     wxPaintDC m_paintdc;
 
     DECLARE_NO_COPY_CLASS(wxBufferedPaintDC)

diff --git a/src/generic/dcbuffer.cpp b/src/generic/dcbuffer.cpp
index 6e5396a0eec4b85e6daca6c7b08cedf6b8da2ac7..3803cc2c53bf151a2aff6ddc83bcb09d247a7226 100644 (file)
--- a/src/generic/dcbuffer.cpp
+++ b/src/generic/dcbuffer.cpp
@@ -9,66 +9,3 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-    #pragma implementation "dcbuffer.h"
-#endif
-
-#include "wx/wxprec.h"
-
-#ifdef __BORLANDC__
-    #pragma hdrstop
-#endif
-
-#ifndef WX_PRECOMP
-    #include "wx/window.h"
-#endif
-
-#include "wx/dcbuffer.h"
-
-// ----------------------------------------------------------------------------
-// Double buffering helper.
-// ----------------------------------------------------------------------------
-
-wxBufferedDC::wxBufferedDC(wxDC *dc, const wxBitmap& buffer)
-            : m_dc( dc ),
-              m_buffer( buffer )
-{
-    UseBuffer();
-}
-
-wxBufferedDC::wxBufferedDC(wxDC *dc, const wxSize& area, int flags)
-            : m_dc( dc ),
-              m_buffer( area.GetWidth(), area.GetHeight() )
-{
-    UseBuffer();
-
-    SaveBg(area, flags);
-}
-
-void wxBufferedDC::Init( wxDC *dc, const wxBitmap &buffer )
-{
-    wxASSERT_MSG( m_dc == 0 && m_buffer == wxNullBitmap,
-                  _T("wxBufferedDC already initialised") );
-
-    m_dc = dc;
-    m_buffer = buffer;
-
-    UseBuffer();
-}
-
-void wxBufferedDC::Init(wxDC *dc, const wxSize& area, int flags)
-{
-    Init(dc, wxBitmap(area.GetWidth(), area.GetHeight()));
-
-    SaveBg(area, flags);
-}
-
-void wxBufferedDC::UnMask()
-{
-    wxASSERT_MSG( m_dc != 0,
-                  _T("No underlying DC associated with wxBufferedDC (anymore)") );
-
-    m_dc->Blit( 0, 0, m_buffer.GetWidth(), m_buffer.GetHeight(), this, 0, 0 );
-    m_dc = NULL;
-}
-