compilation fixes (bug #672413)

[wxWidgets.git] / src / msw / dc.cpp
diff --git a/src/msw/dc.cpp b/src/msw/dc.cpp

index 737f7235e6c6648f061ead80fff7120767af5e3b..8a05efa95c76f6b68e7d0206eff9e4e6c8ea2ba5 100644 (file)
--- a/src/msw/dc.cpp
+++ b/src/msw/dc.cpp
@@ -40,14 +40,16 @@
      #include "wx/icon.h"
  #endif
  
      #include "wx/icon.h"
  #endif
  
+#include "wx/sysopt.h"
  #include "wx/dcprint.h"
  #include "wx/dcprint.h"
+#include "wx/module.h"
  
  #include <string.h>
  #include <math.h>
  
  #include "wx/msw/private.h" // needs to be before #include <commdlg.h>
  
  
  #include <string.h>
  #include <math.h>
  
  #include "wx/msw/private.h" // needs to be before #include <commdlg.h>
  
-#if wxUSE_COMMON_DIALOGS
+#if wxUSE_COMMON_DIALOGS && !defined(__WXMICROWIN__)
      #include <commdlg.h>
  #endif
  
      #include <commdlg.h>
  #endif
  
@@ -55,7 +57,12 @@
      #include <print.h>
  #endif
  
      #include <print.h>
  #endif
  
-IMPLEMENT_ABSTRACT_CLASS(wxDC, wxObject)
+/* Quaternary raster codes */
+#ifndef MAKEROP4
+#define MAKEROP4(fore,back) (DWORD)((((back) << 8) & 0xFF000000) | (fore))
+#endif
+
+IMPLEMENT_ABSTRACT_CLASS(wxDC, wxDCBase)
  
  // ---------------------------------------------------------------------------
  // constants
  
  // ---------------------------------------------------------------------------
  // constants
@@ -75,6 +82,25 @@ static const int MM_METRIC = 10;
  // MSDN docs for how this and other numbers in wxDC::Blit() are obtained)
  #define DSTCOPY 0x00AA0029      // a.k.a. NOP operation
  
  // MSDN docs for how this and other numbers in wxDC::Blit() are obtained)
  #define DSTCOPY 0x00AA0029      // a.k.a. NOP operation
  
+// ----------------------------------------------------------------------------
+// macros for logical <-> device coords conversion
+// ----------------------------------------------------------------------------
+
+/*
+   We currently let Windows do all the translations itself so these macros are
+   not really needed (any more) but keep them to enhance readability of the
+   code by allowing to see where are the logical and where are the device
+   coordinates used.
+ */
+
+// logical to device
+#define XLOG2DEV(x) (x)
+#define YLOG2DEV(y) (y)
+
+// device to logical
+#define XDEV2LOG(x) (x)
+#define YDEV2LOG(y) (y)
+
  // ---------------------------------------------------------------------------
  // private functions
  // ---------------------------------------------------------------------------
  // ---------------------------------------------------------------------------
  // private functions
  // ---------------------------------------------------------------------------
@@ -82,10 +108,112 @@ static const int MM_METRIC = 10;
  // convert degrees to radians
  static inline double DegToRad(double deg) { return (deg * M_PI) / 180.0; }
  
  // convert degrees to radians
  static inline double DegToRad(double deg) { return (deg * M_PI) / 180.0; }
  
+// ----------------------------------------------------------------------------
+// private classes
+// ----------------------------------------------------------------------------
+
+// instead of duplicating the same code which sets and then restores text
+// colours in each wxDC method working with wxSTIPPLE_MASK_OPAQUE brushes,
+// encapsulate this in a small helper class
+
+// wxColourChanger: changes the text colours in the ctor if required and
+//                  restores them in the dtor
+class wxColourChanger
+{
+public:
+    wxColourChanger(wxDC& dc);
+   ~wxColourChanger();
+
+private:
+    wxDC& m_dc;
+
+    COLORREF m_colFgOld, m_colBgOld;
+
+    bool m_changed;
+};
+
+// this class saves the old stretch blit mode during its life time
+class StretchBltModeChanger
+{
+public:
+    StretchBltModeChanger(HDC hdc, int mode)
+        : m_hdc(hdc)
+    {
+        m_modeOld = ::SetStretchBltMode(m_hdc, mode);
+        if ( !m_modeOld )
+            wxLogLastError(_T("SetStretchBltMode"));
+    }
+
+    ~StretchBltModeChanger()
+    {
+        if ( !::SetStretchBltMode(m_hdc, m_modeOld) )
+            wxLogLastError(_T("SetStretchBltMode"));
+    }
+
+private:
+    const HDC m_hdc;
+
+    int m_modeOld;
+};
+
  // ===========================================================================
  // implementation
  // ===========================================================================
  
  // ===========================================================================
  // implementation
  // ===========================================================================
  
+// ----------------------------------------------------------------------------
+// wxColourChanger
+// ----------------------------------------------------------------------------
+
+wxColourChanger::wxColourChanger(wxDC& dc) : m_dc(dc)
+{
+    const wxBrush& brush = dc.GetBrush();
+    if ( brush.Ok() && brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
+    {
+        HDC hdc = GetHdcOf(dc);
+        m_colFgOld = ::GetTextColor(hdc);
+        m_colBgOld = ::GetBkColor(hdc);
+
+        // note that Windows convention is opposite to wxWindows one, this is
+        // why text colour becomes the background one and vice versa
+        const wxColour& colFg = dc.GetTextForeground();
+        if ( colFg.Ok() )
+        {
+            ::SetBkColor(hdc, colFg.GetPixel());
+        }
+
+        const wxColour& colBg = dc.GetTextBackground();
+        if ( colBg.Ok() )
+        {
+            ::SetTextColor(hdc, colBg.GetPixel());
+        }
+
+        SetBkMode(hdc,
+                  dc.GetBackgroundMode() == wxTRANSPARENT ? TRANSPARENT
+                                                          : OPAQUE);
+
+        // flag which telsl us to undo changes in the dtor
+        m_changed = TRUE;
+    }
+    else
+    {
+        // nothing done, nothing to undo
+        m_changed = FALSE;
+    }
+}
+
+wxColourChanger::~wxColourChanger()
+{
+    if ( m_changed )
+    {
+        // restore the colours we changed
+        HDC hdc = GetHdcOf(m_dc);
+
+        ::SetBkMode(hdc, TRANSPARENT);
+        ::SetTextColor(hdc, m_colFgOld);
+        ::SetBkColor(hdc, m_colBgOld);
+    }
+}
+
  // ---------------------------------------------------------------------------
  // wxDC
  // ---------------------------------------------------------------------------
  // ---------------------------------------------------------------------------
  // wxDC
  // ---------------------------------------------------------------------------
@@ -99,30 +227,39 @@ wxDC::wxDC()
      m_oldPen = 0;
      m_oldBrush = 0;
      m_oldFont = 0;
      m_oldPen = 0;
      m_oldBrush = 0;
      m_oldFont = 0;
+#if wxUSE_PALETTE
      m_oldPalette = 0;
      m_oldPalette = 0;
+#endif // wxUSE_PALETTE
  
      m_bOwnsDC = FALSE;
      m_hDC = 0;
  
      m_bOwnsDC = FALSE;
      m_hDC = 0;
-
-    m_windowExtX = VIEWPORT_EXTENT;
-    m_windowExtY = VIEWPORT_EXTENT;
-
-    m_hDCCount = 0;
  }
  
  }
  
-
  wxDC::~wxDC()
  {
  wxDC::~wxDC()
  {
-    if ( m_hDC != 0 ) {
+    if ( m_hDC != 0 )
+    {
          SelectOldObjects(m_hDC);
          SelectOldObjects(m_hDC);
-        if ( m_bOwnsDC ) {
-            if ( m_canvas == NULL )
-                ::DeleteDC(GetHdc());
+
+        // if we own the HDC, we delete it, otherwise we just release it
+
+        if ( m_bOwnsDC )
+        {
+            ::DeleteDC(GetHdc());
+        }
+        else // we don't own our HDC
+        {
+            if (m_canvas)
+            {
+                ::ReleaseDC(GetHwndOf(m_canvas), GetHdc());
+            }
              else
              else
-                ::ReleaseDC((HWND)m_canvas->GetHWND(), GetHdc());
+            {
+                // Must have been a wxScreenDC
+                ::ReleaseDC((HWND) NULL, GetHdc());
+            }
          }
      }
          }
      }
-
  }
  
  // This will select current objects out of the DC,
  }
  
  // This will select current objects out of the DC,
@@ -156,16 +293,21 @@ void wxDC::SelectOldObjects(WXHDC dc)
              ::SelectObject((HDC) dc, (HFONT) m_oldFont);
          }
          m_oldFont = 0;
              ::SelectObject((HDC) dc, (HFONT) m_oldFont);
          }
          m_oldFont = 0;
+
+#if wxUSE_PALETTE
          if (m_oldPalette)
          {
          if (m_oldPalette)
          {
-            ::SelectPalette((HDC) dc, (HPALETTE) m_oldPalette, TRUE);
+            ::SelectPalette((HDC) dc, (HPALETTE) m_oldPalette, FALSE);
          }
          m_oldPalette = 0;
          }
          m_oldPalette = 0;
+#endif // wxUSE_PALETTE
      }
  
      m_brush = wxNullBrush;
      m_pen = wxNullPen;
      }
  
      m_brush = wxNullBrush;
      m_pen = wxNullPen;
+#if wxUSE_PALETTE
      m_palette = wxNullPalette;
      m_palette = wxNullPalette;
+#endif // wxUSE_PALETTE
      m_font = wxNullFont;
      m_backgroundBrush = wxNullBrush;
      m_selectedBitmap = wxNullBitmap;
      m_font = wxNullFont;
      m_backgroundBrush = wxNullBrush;
      m_selectedBitmap = wxNullBitmap;
@@ -175,52 +317,107 @@ void wxDC::SelectOldObjects(WXHDC dc)
  // clipping
  // ---------------------------------------------------------------------------
  
  // clipping
  // ---------------------------------------------------------------------------
  
-#define DO_SET_CLIPPING_BOX()                   \
-{                                               \
-    RECT rect;                                  \
-                                                \
-    GetClipBox(GetHdc(), &rect);                \
-                                                \
-    m_clipX1 = (wxCoord) XDEV2LOG(rect.left);   \
-    m_clipY1 = (wxCoord) YDEV2LOG(rect.top);    \
-    m_clipX2 = (wxCoord) XDEV2LOG(rect.right);  \
-    m_clipY2 = (wxCoord) YDEV2LOG(rect.bottom); \
+void wxDC::UpdateClipBox()
+{
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
+    RECT rect;
+    ::GetClipBox(GetHdc(), &rect);
+
+    m_clipX1 = (wxCoord) XDEV2LOG(rect.left);
+    m_clipY1 = (wxCoord) YDEV2LOG(rect.top);
+    m_clipX2 = (wxCoord) XDEV2LOG(rect.right);
+    m_clipY2 = (wxCoord) YDEV2LOG(rect.bottom);
  }
  
  }
  
-void wxDC::DoSetClippingRegion(wxCoord cx, wxCoord cy, wxCoord cw, wxCoord ch)
+// common part of DoSetClippingRegion() and DoSetClippingRegionAsRegion()
+void wxDC::SetClippingHrgn(WXHRGN hrgn)
  {
  {
+    wxCHECK_RET( hrgn, wxT("invalid clipping region") );
+
+#ifdef __WXMICROWIN__
+    if (!GetHdc()) return;
+#endif // __WXMICROWIN__
+
+    // note that we combine the new clipping region with the existing one: this
+    // is compatible with what the other ports do and is the documented
+    // behaviour now (starting with 2.3.3)
+#ifdef __WIN16__
+    RECT rectClip;
+    if ( !::GetClipBox(GetHdc(), &rectClip) )
+        return;
+
+    HRGN hrgnDest = ::CreateRectRgn(0, 0, 0, 0);
+    HRGN hrgnClipOld = ::CreateRectRgn(rectClip.left, rectClip.top,
+                                       rectClip.right, rectClip.bottom);
+
+    if ( ::CombineRgn(hrgnDest, hrgnClipOld, (HRGN)hrgn, RGN_AND) != ERROR )
+    {
+        ::SelectClipRgn(GetHdc(), hrgnDest);
+    }
+
+    ::DeleteObject(hrgnClipOld);
+    ::DeleteObject(hrgnDest);
+#else // Win32
+    if ( ::ExtSelectClipRgn(GetHdc(), (HRGN)hrgn, RGN_AND) == ERROR )
+    {
+        wxLogLastError(_T("ExtSelectClipRgn"));
+
+        return;
+    }
+#endif // Win16/32
+
      m_clipping = TRUE;
      m_clipping = TRUE;
-    IntersectClipRect(GetHdc(), XLOG2DEV(cx), YLOG2DEV(cy),
-                                XLOG2DEV(cx + cw), YLOG2DEV(cy + ch));
-    DO_SET_CLIPPING_BOX()
+
+    UpdateClipBox();
  }
  
  }
  
-void wxDC::DoSetClippingRegionAsRegion(const wxRegion& region)
+void wxDC::DoSetClippingRegion(wxCoord x, wxCoord y, wxCoord w, wxCoord h)
  {
  {
-    wxCHECK_RET( region.GetHRGN(), wxT("invalid clipping region") );
-
-    m_clipping = TRUE;
+    // the region coords are always the device ones, so do the translation
+    // manually
+    //
+    // FIXME: possible +/-1 error here, to check!
+    HRGN hrgn = ::CreateRectRgn(LogicalToDeviceX(x),
+                                LogicalToDeviceY(y),
+                                LogicalToDeviceX(x + w),
+                                LogicalToDeviceY(y + h));
+    if ( !hrgn )
+    {
+        wxLogLastError(_T("CreateRectRgn"));
+    }
+    else
+    {
+        SetClippingHrgn((WXHRGN)hrgn);
  
  
-#ifdef __WIN16__
-    SelectClipRgn(GetHdc(), (HRGN) region.GetHRGN());
-#else
-    ExtSelectClipRgn(GetHdc(), (HRGN) region.GetHRGN(), RGN_AND);
-#endif
+        ::DeleteObject(hrgn);
+    }
+}
  
  
-    DO_SET_CLIPPING_BOX()
+void wxDC::DoSetClippingRegionAsRegion(const wxRegion& region)
+{
+    SetClippingHrgn(region.GetHRGN());
  }
  
  void wxDC::DestroyClippingRegion()
  {
  }
  
  void wxDC::DestroyClippingRegion()
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      if (m_clipping && m_hDC)
      {
          // TODO: this should restore the previous clipping region,
      if (m_clipping && m_hDC)
      {
          // TODO: this should restore the previous clipping region,
-        // so that OnPaint processing works correctly, and the update clipping region
-        // doesn't get destroyed after the first DestroyClippingRegion.
+        //       so that OnPaint processing works correctly, and the update
+        //       clipping region doesn't get destroyed after the first
+        //       DestroyClippingRegion.
          HRGN rgn = CreateRectRgn(0, 0, 32000, 32000);
          HRGN rgn = CreateRectRgn(0, 0, 32000, 32000);
-        SelectClipRgn(GetHdc(), rgn);
-        DeleteObject(rgn);
+        ::SelectClipRgn(GetHdc(), rgn);
+        ::DeleteObject(rgn);
      }
      }
+
      m_clipping = FALSE;
  }
  
      m_clipping = FALSE;
  }
  
@@ -235,14 +432,23 @@ bool wxDC::CanDrawBitmap() const
  
  bool wxDC::CanGetTextExtent() const
  {
  
  bool wxDC::CanGetTextExtent() const
  {
+#ifdef __WXMICROWIN__
+    // TODO Extend MicroWindows' GetDeviceCaps function
+    return TRUE;
+#else
      // What sort of display is it?
      int technology = ::GetDeviceCaps(GetHdc(), TECHNOLOGY);
  
      return (technology == DT_RASDISPLAY) || (technology == DT_RASPRINTER);
      // What sort of display is it?
      int technology = ::GetDeviceCaps(GetHdc(), TECHNOLOGY);
  
      return (technology == DT_RASDISPLAY) || (technology == DT_RASPRINTER);
+#endif
  }
  
  int wxDC::GetDepth() const
  {
  }
  
  int wxDC::GetDepth() const
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return 16;
+#endif
+
      return (int)::GetDeviceCaps(GetHdc(), BITSPIXEL);
  }
  
      return (int)::GetDeviceCaps(GetHdc(), BITSPIXEL);
  }
  
@@ -252,6 +458,10 @@ int wxDC::GetDepth() const
  
  void wxDC::Clear()
  {
  
  void wxDC::Clear()
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      RECT rect;
      if ( m_canvas )
      {
      RECT rect;
      if ( m_canvas )
      {
@@ -272,24 +482,32 @@ void wxDC::Clear()
  
      (void) ::SetMapMode(GetHdc(), MM_TEXT);
  
  
      (void) ::SetMapMode(GetHdc(), MM_TEXT);
  
-    DWORD colour = GetBkColor(GetHdc());
-    HBRUSH brush = CreateSolidBrush(colour);
-    FillRect(GetHdc(), &rect, brush);
-    DeleteObject(brush);
+    DWORD colour = ::GetBkColor(GetHdc());
+    HBRUSH brush = ::CreateSolidBrush(colour);
+    ::FillRect(GetHdc(), &rect, brush);
+    ::DeleteObject(brush);
+
+    int width = DeviceToLogicalXRel(VIEWPORT_EXTENT)*m_signX,
+        height = DeviceToLogicalYRel(VIEWPORT_EXTENT)*m_signY;
  
      ::SetMapMode(GetHdc(), MM_ANISOTROPIC);
      ::SetViewportExtEx(GetHdc(), VIEWPORT_EXTENT, VIEWPORT_EXTENT, NULL);
  
      ::SetMapMode(GetHdc(), MM_ANISOTROPIC);
      ::SetViewportExtEx(GetHdc(), VIEWPORT_EXTENT, VIEWPORT_EXTENT, NULL);
-    ::SetWindowExtEx(GetHdc(), m_windowExtX, m_windowExtY, NULL);
+    ::SetWindowExtEx(GetHdc(), width, height, NULL);
      ::SetViewportOrgEx(GetHdc(), (int)m_deviceOriginX, (int)m_deviceOriginY, NULL);
      ::SetWindowOrgEx(GetHdc(), (int)m_logicalOriginX, (int)m_logicalOriginY, NULL);
  }
  
      ::SetViewportOrgEx(GetHdc(), (int)m_deviceOriginX, (int)m_deviceOriginY, NULL);
      ::SetWindowOrgEx(GetHdc(), (int)m_logicalOriginX, (int)m_logicalOriginY, NULL);
  }
  
-void wxDC::DoFloodFill(wxCoord x, wxCoord y, const wxColour& col, int style)
+bool wxDC::DoFloodFill(wxCoord x, wxCoord y, const wxColour& col, int style)
  {
  {
-    if ( !::ExtFloodFill(GetHdc(), XLOG2DEV(x), YLOG2DEV(y),
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return FALSE;
+#endif
+
+    bool success = (0 != ::ExtFloodFill(GetHdc(), XLOG2DEV(x), YLOG2DEV(y),
                           col.GetPixel(),
                           style == wxFLOOD_SURFACE ? FLOODFILLSURFACE
                           col.GetPixel(),
                           style == wxFLOOD_SURFACE ? FLOODFILLSURFACE
-                                                  : FLOODFILLBORDER) )
+                                                  : FLOODFILLBORDER) ) ;
+    if (!success)
      {
          // quoting from the MSDN docs:
          //
      {
          // quoting from the MSDN docs:
          //
@@ -303,45 +521,36 @@ void wxDC::DoFloodFill(wxCoord x, wxCoord y, const wxColour& col, int style)
          //      * The point is outside the clipping region that is, it is not
          //        visible on the device.
          //
          //      * The point is outside the clipping region that is, it is not
          //        visible on the device.
          //
-        wxLogLastError("ExtFloodFill");
+        wxLogLastError(wxT("ExtFloodFill"));
      }
  
      CalcBoundingBox(x, y);
      }
  
      CalcBoundingBox(x, y);
+    
+    return success;
  }
  
  bool wxDC::DoGetPixel(wxCoord x, wxCoord y, wxColour *col) const
  {
  }
  
  bool wxDC::DoGetPixel(wxCoord x, wxCoord y, wxColour *col) const
  {
-    // get the color of the pixel
-    COLORREF pixelcolor = ::GetPixel(GetHdc(), XLOG2DEV(x), YLOG2DEV(y));
-
-    // JACS: what was this for?
-#if 0
-    // get the color of the pen
-    COLORREF pencolor = 0x00ffffff;
-    if (m_pen.Ok())
-    {
-        pencolor = m_pen.GetColour().GetPixel();
-    }
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return FALSE;
  #endif
  
  #endif
  
-    // return the color of the pixel
-    if( col )
-    {
-        col->Set(GetRValue(pixelcolor),
-                 GetGValue(pixelcolor),
-                 GetBValue(pixelcolor));
-    }
+    wxCHECK_MSG( col, FALSE, _T("NULL colour parameter in wxDC::GetPixel") );
  
  
-    // check, if color of the pixels is the same as the color of the current
-    // pen and return TRUE if it is, FALSE otherwise
-    // JACS, 24/02/2000: can't understand the reason for this, so returning TRUE instead.
-    // return pixelcolor == pencolor;
+    // get the color of the pixel
+    COLORREF pixelcolor = ::GetPixel(GetHdc(), XLOG2DEV(x), YLOG2DEV(y));
+
+    wxRGBToColour(*col, pixelcolor);
  
      return TRUE;
  }
  
  void wxDC::DoCrossHair(wxCoord x, wxCoord y)
  {
  
      return TRUE;
  }
  
  void wxDC::DoCrossHair(wxCoord x, wxCoord y)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      wxCoord x1 = x-VIEWPORT_EXTENT;
      wxCoord y1 = y-VIEWPORT_EXTENT;
      wxCoord x2 = x+VIEWPORT_EXTENT;
      wxCoord x1 = x-VIEWPORT_EXTENT;
      wxCoord y1 = y-VIEWPORT_EXTENT;
      wxCoord x2 = x+VIEWPORT_EXTENT;
@@ -359,50 +568,38 @@ void wxDC::DoCrossHair(wxCoord x, wxCoord y)
  
  void wxDC::DoDrawLine(wxCoord x1, wxCoord y1, wxCoord x2, wxCoord y2)
  {
  
  void wxDC::DoDrawLine(wxCoord x1, wxCoord y1, wxCoord x2, wxCoord y2)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      (void)MoveToEx(GetHdc(), XLOG2DEV(x1), YLOG2DEV(y1), NULL);
      (void)LineTo(GetHdc(), XLOG2DEV(x2), YLOG2DEV(y2));
  
      (void)MoveToEx(GetHdc(), XLOG2DEV(x1), YLOG2DEV(y1), NULL);
      (void)LineTo(GetHdc(), XLOG2DEV(x2), YLOG2DEV(y2));
  
-    // Normalization: Windows doesn't draw the last point of the line.
-    // But apparently neither does GTK+, so we take it out again.
-//    (void)LineTo(GetHdc(), XLOG2DEV(x2) + 1, YLOG2DEV(y2));
-
      CalcBoundingBox(x1, y1);
      CalcBoundingBox(x2, y2);
  }
  
      CalcBoundingBox(x1, y1);
      CalcBoundingBox(x2, y2);
  }
  
-void wxDC::DoDrawArc(wxCoord x1,wxCoord y1,wxCoord x2,wxCoord y2, wxCoord xc, wxCoord yc)
+// Draws an arc of a circle, centred on (xc, yc), with starting point (x1, y1)
+// and ending at (x2, y2)
+void wxDC::DoDrawArc(wxCoord x1, wxCoord y1,
+                     wxCoord x2, wxCoord y2,
+                     wxCoord xc, wxCoord yc)
  {
  {
-    COLORREF colFgOld = 0,
-             colBgOld = 0;
-
-    if (m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE)
-    {
-       colFgOld = ::GetTextColor(GetHdc());
-       colBgOld = ::GetBkColor(GetHdc());
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
  
  
-       if (m_textForegroundColour.Ok())
-       {   //just the oposite from what is expected see help on pattern brush
-           // 1 in mask becomes bk color
-           ::SetBkColor(GetHdc(), m_textForegroundColour.GetPixel() );
-       }
-       if (m_textBackgroundColour.Ok())
-       {   //just the oposite from what is expected
-           // 0 in mask becomes text color
-           ::SetTextColor(GetHdc(), m_textBackgroundColour.GetPixel() );
-       }
+    wxColourChanger cc(*this); // needed for wxSTIPPLE_MASK_OPAQUE handling
  
  
-       if (m_backgroundMode == wxTRANSPARENT)
-           SetBkMode(GetHdc(), TRANSPARENT);
-       else
-           SetBkMode(GetHdc(), OPAQUE);
-    }
+    double dx = xc - x1;
+    double dy = yc - y1;
+    double radius = (double)sqrt(dx*dx+dy*dy);
+    wxCoord r = (wxCoord)radius;
  
  
-    double dx = xc-x1;
-    double dy = yc-y1;
-    double radius = (double)sqrt(dx*dx+dy*dy) ;;
-    if (x1==x2 && x2==y2)
+    // treat the special case of full circle separately
+    if ( x1 == x2 && y1 == y2 )
      {
      {
-        DrawEllipse(xc,yc,(wxCoord)(radius*2.0),(wxCoord)(radius*2.0));
+        DrawEllipse(xc - r, yc - r, 2*r, 2*r);
          return;
      }
  
          return;
      }
  
@@ -414,44 +611,40 @@ void wxDC::DoDrawArc(wxCoord x1,wxCoord y1,wxCoord x2,wxCoord y2, wxCoord xc, wx
      wxCoord yyc = YLOG2DEV(yc);
      wxCoord ray = (wxCoord) sqrt(double((xxc-xx1)*(xxc-xx1)+(yyc-yy1)*(yyc-yy1)));
  
      wxCoord yyc = YLOG2DEV(yc);
      wxCoord ray = (wxCoord) sqrt(double((xxc-xx1)*(xxc-xx1)+(yyc-yy1)*(yyc-yy1)));
  
-    (void)MoveToEx(GetHdc(), (int) xx1, (int) yy1, NULL);
      wxCoord xxx1 = (wxCoord) (xxc-ray);
      wxCoord yyy1 = (wxCoord) (yyc-ray);
      wxCoord xxx2 = (wxCoord) (xxc+ray);
      wxCoord yyy2 = (wxCoord) (yyc+ray);
      wxCoord xxx1 = (wxCoord) (xxc-ray);
      wxCoord yyy1 = (wxCoord) (yyc-ray);
      wxCoord xxx2 = (wxCoord) (xxc+ray);
      wxCoord yyy2 = (wxCoord) (yyc+ray);
-    if (m_brush.Ok() && m_brush.GetStyle() !=wxTRANSPARENT)
+
+    if ( m_brush.Ok() && m_brush.GetStyle() != wxTRANSPARENT )
      {
          // Have to add 1 to bottom-right corner of rectangle
          // to make semi-circles look right (crooked line otherwise).
          // Unfortunately this is not a reliable method, depends
          // on the size of shape.
          // TODO: figure out why this happens!
      {
          // Have to add 1 to bottom-right corner of rectangle
          // to make semi-circles look right (crooked line otherwise).
          // Unfortunately this is not a reliable method, depends
          // on the size of shape.
          // TODO: figure out why this happens!
-        Pie(GetHdc(),xxx1,yyy1,xxx2+1,yyy2+1,
-            xx1,yy1,xx2,yy2);
+        Pie(GetHdc(),xxx1,yyy1,xxx2+1,yyy2+1, xx1,yy1,xx2,yy2);
      }
      else
      }
      else
-        Arc(GetHdc(),xxx1,yyy1,xxx2,yyy2,
-        xx1,yy1,xx2,yy2);
-
-    CalcBoundingBox((wxCoord)(xc-radius), (wxCoord)(yc-radius));
-    CalcBoundingBox((wxCoord)(xc+radius), (wxCoord)(yc+radius));
-
-    if ( m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
      {
      {
-        // restore the colours we changed
-        ::SetBkMode(GetHdc(), TRANSPARENT);
-        ::SetTextColor(GetHdc(), colFgOld);
-        ::SetBkColor(GetHdc(), colBgOld);
+        Arc(GetHdc(),xxx1,yyy1,xxx2,yyy2, xx1,yy1,xx2,yy2);
      }
      }
+
+    CalcBoundingBox(xc - r, yc - r);
+    CalcBoundingBox(xc + r, yc + r);
  }
  
  void wxDC::DoDrawCheckMark(wxCoord x1, wxCoord y1,
                             wxCoord width, wxCoord height)
  {
  }
  
  void wxDC::DoDrawCheckMark(wxCoord x1, wxCoord y1,
                             wxCoord width, wxCoord height)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      wxCoord x2 = x1 + width,
              y2 = y1 + height;
  
      wxCoord x2 = x1 + width,
              y2 = y1 + height;
  
-#if defined(__WIN32__) && !defined(__SC__)
+#if defined(__WIN32__) && !defined(__SC__) && !defined(__WXMICROWIN__)
      RECT rect;
      rect.left   = x1;
      rect.top    = y1;
      RECT rect;
      rect.left   = x1;
      rect.top    = y1;
@@ -467,9 +660,9 @@ void wxDC::DoDrawCheckMark(wxCoord x1, wxCoord y1,
      HPEN hBrushOld = (HPEN)::SelectObject(GetHdc(), whiteBrush);
      ::SetROP2(GetHdc(), R2_COPYPEN);
      Rectangle(GetHdc(), x1, y1, x2, y2);
      HPEN hBrushOld = (HPEN)::SelectObject(GetHdc(), whiteBrush);
      ::SetROP2(GetHdc(), R2_COPYPEN);
      Rectangle(GetHdc(), x1, y1, x2, y2);
-    MoveTo(GetHdc(), x1, y1);
+    MoveToEx(GetHdc(), x1, y1, NULL);
      LineTo(GetHdc(), x2, y2);
      LineTo(GetHdc(), x2, y2);
-    MoveTo(GetHdc(), x2, y1);
+    MoveToEx(GetHdc(), x2, y1, NULL);
      LineTo(GetHdc(), x1, y2);
      ::SelectObject(GetHdc(), hPenOld);
      ::SelectObject(GetHdc(), hBrushOld);
      LineTo(GetHdc(), x1, y2);
      ::SelectObject(GetHdc(), hPenOld);
      ::SelectObject(GetHdc(), hBrushOld);
@@ -482,6 +675,10 @@ void wxDC::DoDrawCheckMark(wxCoord x1, wxCoord y1,
  
  void wxDC::DoDrawPoint(wxCoord x, wxCoord y)
  {
  
  void wxDC::DoDrawPoint(wxCoord x, wxCoord y)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      COLORREF color = 0x00ffffff;
      if (m_pen.Ok())
      {
      COLORREF color = 0x00ffffff;
      if (m_pen.Ok())
      {
@@ -495,30 +692,11 @@ void wxDC::DoDrawPoint(wxCoord x, wxCoord y)
  
  void wxDC::DoDrawPolygon(int n, wxPoint points[], wxCoord xoffset, wxCoord yoffset,int fillStyle)
  {
  
  void wxDC::DoDrawPolygon(int n, wxPoint points[], wxCoord xoffset, wxCoord yoffset,int fillStyle)
  {
-    COLORREF colFgOld = 0,
-             colBgOld = 0;
-
-    if (m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE)
-    {
-       colFgOld = ::GetTextColor(GetHdc());
-       colBgOld = ::GetBkColor(GetHdc());
-
-       if (m_textForegroundColour.Ok())
-       {   //just the oposite from what is expected see help on pattern brush
-           // 1 in mask becomes bk color
-           ::SetBkColor(GetHdc(), m_textForegroundColour.GetPixel() );
-       }
-       if (m_textBackgroundColour.Ok())
-       {   //just the oposite from what is expected
-           // 0 in mask becomes text color
-           ::SetTextColor(GetHdc(), m_textBackgroundColour.GetPixel() );
-       }
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
  
  
-       if (m_backgroundMode == wxTRANSPARENT)
-           SetBkMode(GetHdc(), TRANSPARENT);
-       else
-           SetBkMode(GetHdc(), OPAQUE);
-    }
+    wxColourChanger cc(*this); // needed for wxSTIPPLE_MASK_OPAQUE handling
  
      // Do things less efficiently if we have offsets
      if (xoffset != 0 || yoffset != 0)
  
      // Do things less efficiently if we have offsets
      if (xoffset != 0 || yoffset != 0)
@@ -547,18 +725,14 @@ void wxDC::DoDrawPolygon(int n, wxPoint points[], wxCoord xoffset, wxCoord yoffs
          (void)Polygon(GetHdc(), (POINT*) points, n);
          SetPolyFillMode(GetHdc(),prev);
      }
          (void)Polygon(GetHdc(), (POINT*) points, n);
          SetPolyFillMode(GetHdc(),prev);
      }
-
-    if ( m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
-    {
-        // restore the colours we changed
-        ::SetBkMode(GetHdc(), TRANSPARENT);
-        ::SetTextColor(GetHdc(), colFgOld);
-        ::SetBkColor(GetHdc(), colBgOld);
-    }
  }
  
  void wxDC::DoDrawLines(int n, wxPoint points[], wxCoord xoffset, wxCoord yoffset)
  {
  }
  
  void wxDC::DoDrawLines(int n, wxPoint points[], wxCoord xoffset, wxCoord yoffset)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      // Do things less efficiently if we have offsets
      if (xoffset != 0 || yoffset != 0)
      {
      // Do things less efficiently if we have offsets
      if (xoffset != 0 || yoffset != 0)
      {
@@ -586,31 +760,11 @@ void wxDC::DoDrawLines(int n, wxPoint points[], wxCoord xoffset, wxCoord yoffset
  
  void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
  {
  
  void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
  {
-    COLORREF colFgOld = 0,
-             colBgOld = 0;
-
-    if ( m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
-    {
-        colFgOld = ::GetTextColor(GetHdc());
-        colBgOld = ::GetBkColor(GetHdc());
-
-        if ( m_textForegroundColour.Ok() )
-        {
-            // just the oposite from what is expected see help on pattern brush
-            // 1 in mask becomes bk color
-            ::SetBkColor(GetHdc(), m_textForegroundColour.GetPixel());
-        }
-
-        if ( m_textBackgroundColour.Ok() )
-        {
-            // 0 in mask becomes text color
-            ::SetTextColor(GetHdc(), m_textBackgroundColour.GetPixel());
-        }
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
  
  
-        // VZ: IMHO this does strictly nothing here
-        SetBkMode(GetHdc(), m_backgroundMode == wxTRANSPARENT ? TRANSPARENT
-                                                              : OPAQUE);
-    }
+    wxColourChanger cc(*this); // needed for wxSTIPPLE_MASK_OPAQUE handling
  
      wxCoord x2 = x + width;
      wxCoord y2 = y + height;
  
      wxCoord x2 = x + width;
      wxCoord y2 = y + height;
@@ -628,7 +782,7 @@ void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
      {
          // Windows draws the filled rectangles without outline (i.e. drawn with a
          // transparent pen) one pixel smaller in both directions and we want them
      {
          // Windows draws the filled rectangles without outline (i.e. drawn with a
          // transparent pen) one pixel smaller in both directions and we want them
-           // to have the same size regardless of which pen is used - adjust
+        // to have the same size regardless of which pen is used - adjust
  
          // I wonder if this shouldn´t be done after the LOG2DEV() conversions. RR.
          if ( m_pen.GetStyle() == wxTRANSPARENT )
  
          // I wonder if this shouldn´t be done after the LOG2DEV() conversions. RR.
          if ( m_pen.GetStyle() == wxTRANSPARENT )
@@ -637,49 +791,21 @@ void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
              y2++;
          }
  
              y2++;
          }
  
-           (void)Rectangle(GetHdc(), XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2), YLOG2DEV(y2));
+        (void)Rectangle(GetHdc(), XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2), YLOG2DEV(y2));
      }
  
  
      CalcBoundingBox(x, y);
      CalcBoundingBox(x2, y2);
      }
  
  
      CalcBoundingBox(x, y);
      CalcBoundingBox(x2, y2);
-
-    if ( m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
-    {
-        // restore the colours we changed
-        ::SetBkMode(GetHdc(), TRANSPARENT);
-        ::SetTextColor(GetHdc(), colFgOld);
-        ::SetBkColor(GetHdc(), colBgOld);
-    }
  }
  
  void wxDC::DoDrawRoundedRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height, double radius)
  {
  }
  
  void wxDC::DoDrawRoundedRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height, double radius)
  {
-    COLORREF colFgOld = 0,
-             colBgOld = 0;
-
-    if ( m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
-    {
-        colFgOld = ::GetTextColor(GetHdc());
-        colBgOld = ::GetBkColor(GetHdc());
-
-        if ( m_textForegroundColour.Ok() )
-        {
-            // just the oposite from what is expected see help on pattern brush
-            // 1 in mask becomes bk color
-            ::SetBkColor(GetHdc(), m_textForegroundColour.GetPixel());
-        }
-
-        if ( m_textBackgroundColour.Ok() )
-        {
-            // 0 in mask becomes text color
-            ::SetTextColor(GetHdc(), m_textBackgroundColour.GetPixel());
-        }
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
  
  
-        // VZ: IMHO this does strictly nothing here
-        SetBkMode(GetHdc(), m_backgroundMode == wxTRANSPARENT ? TRANSPARENT
-                                                              : OPAQUE);
-    }
+    wxColourChanger cc(*this); // needed for wxSTIPPLE_MASK_OPAQUE handling
  
      // Now, a negative radius value is interpreted to mean
      // 'the proportion of the smallest X or Y dimension'
  
      // Now, a negative radius value is interpreted to mean
      // 'the proportion of the smallest X or Y dimension'
@@ -711,43 +837,15 @@ void wxDC::DoDrawRoundedRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord h
  
      CalcBoundingBox(x, y);
      CalcBoundingBox(x2, y2);
  
      CalcBoundingBox(x, y);
      CalcBoundingBox(x2, y2);
-
-    if ( m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
-    {
-        // restore the colours we changed
-        ::SetBkMode(GetHdc(), TRANSPARENT);
-        ::SetTextColor(GetHdc(), colFgOld);
-        ::SetBkColor(GetHdc(), colBgOld);
-    }
  }
  
  void wxDC::DoDrawEllipse(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
  {
  }
  
  void wxDC::DoDrawEllipse(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
  {
-    COLORREF colFgOld = 0,
-             colBgOld = 0;
-
-    if ( m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
-    {
-        colFgOld = ::GetTextColor(GetHdc());
-        colBgOld = ::GetBkColor(GetHdc());
-
-        if ( m_textForegroundColour.Ok() )
-        {
-            // just the oposite from what is expected see help on pattern brush
-            // 1 in mask becomes bk color
-            ::SetBkColor(GetHdc(), m_textForegroundColour.GetPixel());
-        }
-
-        if ( m_textBackgroundColour.Ok() )
-        {
-            // 0 in mask becomes text color
-            ::SetTextColor(GetHdc(), m_textBackgroundColour.GetPixel());
-        }
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
  
  
-        // VZ: IMHO this does strictly nothing here
-        SetBkMode(GetHdc(), m_backgroundMode == wxTRANSPARENT ? TRANSPARENT
-                                                              : OPAQUE);
-    }
+    wxColourChanger cc(*this); // needed for wxSTIPPLE_MASK_OPAQUE handling
  
      wxCoord x2 = (x+width);
      wxCoord y2 = (y+height);
  
      wxCoord x2 = (x+width);
      wxCoord y2 = (y+height);
@@ -756,47 +854,19 @@ void wxDC::DoDrawEllipse(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
  
      CalcBoundingBox(x, y);
      CalcBoundingBox(x2, y2);
  
      CalcBoundingBox(x, y);
      CalcBoundingBox(x2, y2);
-
-    if ( m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
-    {
-        // restore the colours we changed
-        ::SetBkMode(GetHdc(), TRANSPARENT);
-        ::SetTextColor(GetHdc(), colFgOld);
-        ::SetBkColor(GetHdc(), colBgOld);
-    }
  }
  
  // Chris Breeze 20/5/98: first implementation of DrawEllipticArc on Windows
  void wxDC::DoDrawEllipticArc(wxCoord x,wxCoord y,wxCoord w,wxCoord h,double sa,double ea)
  {
  }
  
  // Chris Breeze 20/5/98: first implementation of DrawEllipticArc on Windows
  void wxDC::DoDrawEllipticArc(wxCoord x,wxCoord y,wxCoord w,wxCoord h,double sa,double ea)
  {
-    COLORREF colFgOld = 0,
-             colBgOld = 0;
-
-    if ( m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
-    {
-        colFgOld = ::GetTextColor(GetHdc());
-        colBgOld = ::GetBkColor(GetHdc());
-
-        if ( m_textForegroundColour.Ok() )
-        {
-            // just the oposite from what is expected see help on pattern brush
-            // 1 in mask becomes bk color
-            ::SetBkColor(GetHdc(), m_textForegroundColour.GetPixel());
-        }
-
-        if ( m_textBackgroundColour.Ok() )
-        {
-            // 0 in mask becomes text color
-            ::SetTextColor(GetHdc(), m_textBackgroundColour.GetPixel());
-        }
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
  
  
-        // VZ: IMHO this does strictly nothing here
-        SetBkMode(GetHdc(), m_backgroundMode == wxTRANSPARENT ? TRANSPARENT
-                                                              : OPAQUE);
-    }
+    wxColourChanger cc(*this); // needed for wxSTIPPLE_MASK_OPAQUE handling
  
  
-    wxCoord x2 = (x+w);
-    wxCoord y2 = (y+h);
+    wxCoord x2 = x + w;
+    wxCoord y2 = y + h;
  
      int rx1 = XLOG2DEV(x+w/2);
      int ry1 = YLOG2DEV(y+h/2);
  
      int rx1 = XLOG2DEV(x+w/2);
      int ry1 = YLOG2DEV(y+h/2);
@@ -813,38 +883,40 @@ void wxDC::DoDrawEllipticArc(wxCoord x,wxCoord y,wxCoord w,wxCoord h,double sa,d
  
      // draw pie with NULL_PEN first and then outline otherwise a line is
      // drawn from the start and end points to the centre
  
      // draw pie with NULL_PEN first and then outline otherwise a line is
      // drawn from the start and end points to the centre
-    HPEN orig_pen = (HPEN) ::SelectObject(GetHdc(), (HPEN) ::GetStockObject(NULL_PEN));
+    HPEN hpenOld = (HPEN) ::SelectObject(GetHdc(), (HPEN) ::GetStockObject(NULL_PEN));
      if (m_signY > 0)
      {
          (void)Pie(GetHdc(), XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2)+1, YLOG2DEV(y2)+1,
      if (m_signY > 0)
      {
          (void)Pie(GetHdc(), XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2)+1, YLOG2DEV(y2)+1,
-            rx1, ry1, rx2, ry2);
+                  rx1, ry1, rx2, ry2);
      }
      else
      {
          (void)Pie(GetHdc(), XLOG2DEV(x), YLOG2DEV(y)-1, XLOG2DEV(x2)+1, YLOG2DEV(y2),
      }
      else
      {
          (void)Pie(GetHdc(), XLOG2DEV(x), YLOG2DEV(y)-1, XLOG2DEV(x2)+1, YLOG2DEV(y2),
-            rx1, ry1-1, rx2, ry2-1);
+                  rx1, ry1-1, rx2, ry2-1);
      }
      }
-    ::SelectObject(GetHdc(), orig_pen);
+
+    ::SelectObject(GetHdc(), hpenOld);
+
      (void)Arc(GetHdc(), XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2), YLOG2DEV(y2),
      (void)Arc(GetHdc(), XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2), YLOG2DEV(y2),
-        rx1, ry1, rx2, ry2);
+              rx1, ry1, rx2, ry2);
  
      CalcBoundingBox(x, y);
      CalcBoundingBox(x2, y2);
  
      CalcBoundingBox(x, y);
      CalcBoundingBox(x2, y2);
-
-    if ( m_brush.GetStyle() == wxSTIPPLE_MASK_OPAQUE )
-    {
-        // restore the colours we changed
-        ::SetBkMode(GetHdc(), TRANSPARENT);
-        ::SetTextColor(GetHdc(), colFgOld);
-        ::SetBkColor(GetHdc(), colBgOld);
-    }
  }
  
  void wxDC::DoDrawIcon(const wxIcon& icon, wxCoord x, wxCoord y)
  {
  }
  
  void wxDC::DoDrawIcon(const wxIcon& icon, wxCoord x, wxCoord y)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      wxCHECK_RET( icon.Ok(), wxT("invalid icon in DrawIcon") );
  
      wxCHECK_RET( icon.Ok(), wxT("invalid icon in DrawIcon") );
  
+#ifdef __WIN32__
+    ::DrawIconEx(GetHdc(), XLOG2DEV(x), YLOG2DEV(y), GetHiconOf(icon), icon.GetWidth(), icon.GetHeight(), 0, NULL, DI_NORMAL);
+#else
      ::DrawIcon(GetHdc(), XLOG2DEV(x), YLOG2DEV(y), GetHiconOf(icon));
      ::DrawIcon(GetHdc(), XLOG2DEV(x), YLOG2DEV(y), GetHiconOf(icon));
+#endif
  
      CalcBoundingBox(x, y);
      CalcBoundingBox(x + icon.GetWidth(), y + icon.GetHeight());
  
      CalcBoundingBox(x, y);
      CalcBoundingBox(x + icon.GetWidth(), y + icon.GetHeight());
@@ -852,6 +924,10 @@ void wxDC::DoDrawIcon(const wxIcon& icon, wxCoord x, wxCoord y)
  
  void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask )
  {
  
  void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask )
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      wxCHECK_RET( bmp.Ok(), _T("invalid bitmap in wxDC::DrawBitmap") );
  
      int width = bmp.GetWidth(),
      wxCHECK_RET( bmp.Ok(), _T("invalid bitmap in wxDC::DrawBitmap") );
  
      int width = bmp.GetWidth(),
@@ -859,6 +935,10 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
  
      HBITMAP hbmpMask = 0;
  
  
      HBITMAP hbmpMask = 0;
  
+#if wxUSE_PALETTE
+    HPALETTE oldPal = 0;
+#endif // wxUSE_PALETTE
+
      if ( useMask )
      {
          wxMask *mask = bmp.GetMask();
      if ( useMask )
      {
          wxMask *mask = bmp.GetMask();
@@ -872,20 +952,44 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
              useMask = FALSE;
          }
      }
              useMask = FALSE;
          }
      }
-
      if ( useMask )
      {
  #ifdef __WIN32__
      if ( useMask )
      {
  #ifdef __WIN32__
-        HDC hdcMem = ::CreateCompatibleDC(GetHdc());
-        ::SelectObject(hdcMem, GetHbitmapOf(bmp));
-
          // use MaskBlt() with ROP which doesn't do anything to dst in the mask
          // points
          // use MaskBlt() with ROP which doesn't do anything to dst in the mask
          // points
-        bool ok = ::MaskBlt(GetHdc(), x, y, width, height,
+        // On some systems, MaskBlt succeeds yet is much much slower
+        // than the wxWindows fall-back implementation. So we need
+        // to be able to switch this on and off at runtime.
+        bool ok = FALSE;
+#if wxUSE_SYSTEM_OPTIONS
+        if (wxSystemOptions::GetOptionInt(wxT("no-maskblt")) == 0)
+#endif
+        {
+            HDC cdc = GetHdc();
+            HDC hdcMem = ::CreateCompatibleDC(GetHdc());
+            HGDIOBJ hOldBitmap = ::SelectObject(hdcMem, GetHbitmapOf(bmp));
+#if wxUSE_PALETTE
+            wxPalette *pal = bmp.GetPalette();
+            if ( pal && ::GetDeviceCaps(cdc,BITSPIXEL) <= 8 )
+            {
+                oldPal = ::SelectPalette(hdcMem, GetHpaletteOf(*pal), FALSE);
+                ::RealizePalette(hdcMem);
+            }
+#endif // wxUSE_PALETTE
+
+            ok = ::MaskBlt(cdc, x, y, width, height,
                              hdcMem, 0, 0,
                              hbmpMask, 0, 0,
                              MAKEROP4(SRCCOPY, DSTCOPY)) != 0;
                              hdcMem, 0, 0,
                              hbmpMask, 0, 0,
                              MAKEROP4(SRCCOPY, DSTCOPY)) != 0;
-        ::DeleteDC(hdcMem);
+
+#if wxUSE_PALETTE
+            if (oldPal)
+                ::SelectPalette(hdcMem, oldPal, FALSE);
+#endif // wxUSE_PALETTE
+
+            ::SelectObject(hdcMem, hOldBitmap);
+            ::DeleteDC(hdcMem);
+        }
  
          if ( !ok )
  #endif // Win32
  
          if ( !ok )
  #endif // Win32
@@ -919,8 +1023,24 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
              ::SetBkColor(GetHdc(), m_textBackgroundColour.GetPixel() );
          }
  
              ::SetBkColor(GetHdc(), m_textBackgroundColour.GetPixel() );
          }
  
-        ::SelectObject( memdc, hbitmap );
+#if wxUSE_PALETTE
+        wxPalette *pal = bmp.GetPalette();
+        if ( pal && ::GetDeviceCaps(cdc,BITSPIXEL) <= 8 )
+        {
+            oldPal = ::SelectPalette(memdc, GetHpaletteOf(*pal), FALSE);
+            ::RealizePalette(memdc);
+        }
+#endif // wxUSE_PALETTE
+
+        HGDIOBJ hOldBitmap = ::SelectObject( memdc, hbitmap );
          ::BitBlt( cdc, x, y, width, height, memdc, 0, 0, SRCCOPY);
          ::BitBlt( cdc, x, y, width, height, memdc, 0, 0, SRCCOPY);
+
+#if wxUSE_PALETTE
+        if (oldPal)
+            ::SelectPalette(memdc, oldPal, FALSE);
+#endif // wxUSE_PALETTE
+
+        ::SelectObject( memdc, hOldBitmap );
          ::DeleteDC( memdc );
  
          ::SetTextColor(GetHdc(), old_textground);
          ::DeleteDC( memdc );
  
          ::SetTextColor(GetHdc(), old_textground);
@@ -930,6 +1050,10 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
  
  void wxDC::DoDrawText(const wxString& text, wxCoord x, wxCoord y)
  {
  
  void wxDC::DoDrawText(const wxString& text, wxCoord x, wxCoord y)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      DrawAnyText(text, x, y);
  
      // update the bounding box
      DrawAnyText(text, x, y);
  
      // update the bounding box
@@ -942,6 +1066,10 @@ void wxDC::DoDrawText(const wxString& text, wxCoord x, wxCoord y)
  
  void wxDC::DrawAnyText(const wxString& text, wxCoord x, wxCoord y)
  {
  
  void wxDC::DrawAnyText(const wxString& text, wxCoord x, wxCoord y)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      // prepare for drawing the text
      if ( m_textForegroundColour.Ok() )
          SetTextColor(GetHdc(), m_textForegroundColour.GetPixel());
      // prepare for drawing the text
      if ( m_textForegroundColour.Ok() )
          SetTextColor(GetHdc(), m_textForegroundColour.GetPixel());
@@ -958,7 +1086,7 @@ void wxDC::DrawAnyText(const wxString& text, wxCoord x, wxCoord y)
      if ( ::TextOut(GetHdc(), XLOG2DEV(x), YLOG2DEV(y),
                     text.c_str(), text.length()) == 0 )
      {
      if ( ::TextOut(GetHdc(), XLOG2DEV(x), YLOG2DEV(y),
                     text.c_str(), text.length()) == 0 )
      {
-        wxLogLastError("TextOut");
+        wxLogLastError(wxT("TextOut"));
      }
  
      // restore the old parameters (text foreground colour may be left because
      }
  
      // restore the old parameters (text foreground colour may be left because
@@ -974,6 +1102,10 @@ void wxDC::DoDrawRotatedText(const wxString& text,
                               wxCoord x, wxCoord y,
                               double angle)
  {
                               wxCoord x, wxCoord y,
                               double angle)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      // we test that we have some font because otherwise we should still use the
      // "else" part below to avoid that DrawRotatedText(angle = 180) and
      // DrawRotatedText(angle = 0) use different fonts (we can't use the default
      // we test that we have some font because otherwise we should still use the
      // "else" part below to avoid that DrawRotatedText(angle = 180) and
      // DrawRotatedText(angle = 0) use different fonts (we can't use the default
@@ -982,16 +1114,18 @@ void wxDC::DoDrawRotatedText(const wxString& text,
      {
          DoDrawText(text, x, y);
      }
      {
          DoDrawText(text, x, y);
      }
+#ifndef __WXMICROWIN__
      else
      {
      else
      {
-        // NB: don't take DEFAULT_GUI_FONT because it's not TrueType and so
-        //     can't have non zero orientation/escapement
-        wxFont font = m_font.Ok() ? m_font : *wxNORMAL_FONT;
+        // NB: don't take DEFAULT_GUI_FONT (a.k.a. wxSYS_DEFAULT_GUI_FONT)
+        //     because it's not TrueType and so can't have non zero
+        //     orientation/escapement under Win9x
+        wxFont font = m_font.Ok() ? m_font : *wxSWISS_FONT;
          HFONT hfont = (HFONT)font.GetResourceHandle();
          LOGFONT lf;
          if ( ::GetObject(hfont, sizeof(lf), &lf) == 0 )
          {
          HFONT hfont = (HFONT)font.GetResourceHandle();
          LOGFONT lf;
          if ( ::GetObject(hfont, sizeof(lf), &lf) == 0 )
          {
-            wxLogLastError("GetObject(hfont)");
+            wxLogLastError(wxT("GetObject(hfont)"));
          }
  
          // GDI wants the angle in tenth of degree
          }
  
          // GDI wants the angle in tenth of degree
@@ -1002,7 +1136,7 @@ void wxDC::DoDrawRotatedText(const wxString& text,
          hfont = ::CreateFontIndirect(&lf);
          if ( !hfont )
          {
          hfont = ::CreateFontIndirect(&lf);
          if ( !hfont )
          {
-            wxLogLastError("CreateFont");
+            wxLogLastError(wxT("CreateFont"));
          }
          else
          {
          }
          else
          {
@@ -1024,57 +1158,86 @@ void wxDC::DoDrawRotatedText(const wxString& text,
  
          // "upper left" and "upper right"
          CalcBoundingBox(x, y);
  
          // "upper left" and "upper right"
          CalcBoundingBox(x, y);
-        CalcBoundingBox(x + w*cos(rad), y - h*sin(rad));
+        CalcBoundingBox(x + wxCoord(w*cos(rad)), y - wxCoord(h*sin(rad)));
  
          // "bottom left" and "bottom right"
          x += (wxCoord)(h*sin(rad));
          y += (wxCoord)(h*cos(rad));
          CalcBoundingBox(x, y);
  
          // "bottom left" and "bottom right"
          x += (wxCoord)(h*sin(rad));
          y += (wxCoord)(h*cos(rad));
          CalcBoundingBox(x, y);
-        CalcBoundingBox(x + h*sin(rad), y + h*cos(rad));
+        CalcBoundingBox(x + wxCoord(h*sin(rad)), y + wxCoord(h*cos(rad)));
      }
      }
+#endif
  }
  
  // ---------------------------------------------------------------------------
  // set GDI objects
  // ---------------------------------------------------------------------------
  
  }
  
  // ---------------------------------------------------------------------------
  // set GDI objects
  // ---------------------------------------------------------------------------
  
-void wxDC::SetPalette(const wxPalette& palette)
+#if wxUSE_PALETTE
+
+void wxDC::DoSelectPalette(bool realize)
  {
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      // Set the old object temporarily, in case the assignment deletes an object
      // that's not yet selected out.
      if (m_oldPalette)
      {
      // Set the old object temporarily, in case the assignment deletes an object
      // that's not yet selected out.
      if (m_oldPalette)
      {
-        ::SelectPalette(GetHdc(), (HPALETTE) m_oldPalette, TRUE);
+        ::SelectPalette(GetHdc(), (HPALETTE) m_oldPalette, FALSE);
          m_oldPalette = 0;
      }
  
          m_oldPalette = 0;
      }
  
-    m_palette = palette;
-
-    if (!m_palette.Ok())
+    if ( m_palette.Ok() )
      {
      {
-        // Setting a NULL colourmap is a way of restoring
-        // the original colourmap
-        if (m_oldPalette)
-        {
-            ::SelectPalette(GetHdc(), (HPALETTE) m_oldPalette, TRUE);
-            m_oldPalette = 0;
-        }
+        HPALETTE oldPal = ::SelectPalette(GetHdc(),
+                                          GetHpaletteOf(m_palette),
+                                          FALSE);
+        if (!m_oldPalette)
+            m_oldPalette = (WXHPALETTE) oldPal;
  
  
-        return;
+        if (realize)
+            ::RealizePalette(GetHdc());
      }
      }
+}
  
  
-    if (m_palette.Ok() && m_palette.GetHPALETTE())
+void wxDC::SetPalette(const wxPalette& palette)
+{
+    if ( palette.Ok() )
      {
      {
-        HPALETTE oldPal = ::SelectPalette(GetHdc(), (HPALETTE) m_palette.GetHPALETTE(), TRUE);
-        if (!m_oldPalette)
-            m_oldPalette = (WXHPALETTE) oldPal;
+        m_palette = palette;
+        DoSelectPalette(TRUE);
+    }
+}
  
  
-        ::RealizePalette(GetHdc());
+void wxDC::InitializePalette()
+{
+    if ( wxDisplayDepth() <= 8 )
+    {
+        // look for any window or parent that has a custom palette. If any has
+        // one then we need to use it in drawing operations
+        wxWindow *win = m_canvas->GetAncestorWithCustomPalette();
+
+        m_hasCustomPalette = win && win->HasCustomPalette();
+        if ( m_hasCustomPalette )
+        {
+            m_palette = win->GetPalette();
+
+            // turn on MSW translation for this palette
+            DoSelectPalette();
+        }
      }
  }
  
      }
  }
  
+#endif // wxUSE_PALETTE
+
  void wxDC::SetFont(const wxFont& the_font)
  {
  void wxDC::SetFont(const wxFont& the_font)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      // Set the old object temporarily, in case the assignment deletes an object
      // that's not yet selected out.
      if (m_oldFont)
      // Set the old object temporarily, in case the assignment deletes an object
      // that's not yet selected out.
      if (m_oldFont)
@@ -1106,6 +1269,10 @@ void wxDC::SetFont(const wxFont& the_font)
  
  void wxDC::SetPen(const wxPen& pen)
  {
  
  void wxDC::SetPen(const wxPen& pen)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      // Set the old object temporarily, in case the assignment deletes an object
      // that's not yet selected out.
      if (m_oldPen)
      // Set the old object temporarily, in case the assignment deletes an object
      // that's not yet selected out.
      if (m_oldPen)
@@ -1136,6 +1303,10 @@ void wxDC::SetPen(const wxPen& pen)
  
  void wxDC::SetBrush(const wxBrush& brush)
  {
  
  void wxDC::SetBrush(const wxBrush& brush)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      // Set the old object temporarily, in case the assignment deletes an object
      // that's not yet selected out.
      if (m_oldBrush)
      // Set the old object temporarily, in case the assignment deletes an object
      // that's not yet selected out.
      if (m_oldBrush)
@@ -1160,12 +1331,12 @@ void wxDC::SetBrush(const wxBrush& brush)
          if ( stipple && stipple->Ok() )
          {
  #ifdef __WIN32__
          if ( stipple && stipple->Ok() )
          {
  #ifdef __WIN32__
-               ::SetBrushOrgEx(GetHdc(),
+            ::SetBrushOrgEx(GetHdc(),
                              m_deviceOriginX % stipple->GetWidth(),
                              m_deviceOriginY % stipple->GetHeight(),
                              NULL);  // don't need previous brush origin
  #else
                              m_deviceOriginX % stipple->GetWidth(),
                              m_deviceOriginY % stipple->GetHeight(),
                              NULL);  // don't need previous brush origin
  #else
-               ::SetBrushOrg(GetHdc(),
+            ::SetBrushOrg(GetHdc(),
                              m_deviceOriginX % stipple->GetWidth(),
                              m_deviceOriginY % stipple->GetHeight());
  #endif
                              m_deviceOriginX % stipple->GetWidth(),
                              m_deviceOriginY % stipple->GetHeight());
  #endif
@@ -1183,6 +1354,10 @@ void wxDC::SetBrush(const wxBrush& brush)
  
  void wxDC::SetBackground(const wxBrush& brush)
  {
  
  void wxDC::SetBackground(const wxBrush& brush)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      m_backgroundBrush = brush;
  
      if (!m_backgroundBrush.Ok())
      m_backgroundBrush = brush;
  
      if (!m_backgroundBrush.Ok())
@@ -1224,21 +1399,22 @@ void wxDC::SetBackground(const wxBrush& brush)
  
  void wxDC::SetBackgroundMode(int mode)
  {
  
  void wxDC::SetBackgroundMode(int mode)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      m_backgroundMode = mode;
  
      // SetBackgroundColour now only refers to text background
      // and m_backgroundMode is used there
      m_backgroundMode = mode;
  
      // SetBackgroundColour now only refers to text background
      // and m_backgroundMode is used there
-
-/*
-    if (m_backgroundMode == wxTRANSPARENT)
-        ::SetBkMode(GetHdc(), TRANSPARENT);
-    else
-        ::SetBkMode(GetHdc(), OPAQUE);
-*/
  }
  
  void wxDC::SetLogicalFunction(int function)
  {
  }
  
  void wxDC::SetLogicalFunction(int function)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      m_logicalFunction = function;
  
      SetRop(m_hDC);
      m_logicalFunction = function;
  
      SetRop(m_hDC);
@@ -1278,7 +1454,7 @@ void wxDC::SetRop(WXHDC dc)
      SetROP2(GetHdc(), rop);
  }
  
      SetROP2(GetHdc(), rop);
  }
  
-bool wxDC::StartDoc(const wxString& message)
+bool wxDC::StartDoc(const wxString& WXUNUSED(message))
  {
      // We might be previewing, so return TRUE to let it continue.
      return TRUE;
  {
      // We might be previewing, so return TRUE to let it continue.
      return TRUE;
@@ -1302,112 +1478,158 @@ void wxDC::EndPage()
  
  wxCoord wxDC::GetCharHeight() const
  {
  
  wxCoord wxDC::GetCharHeight() const
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return 0;
+#endif
+
      TEXTMETRIC lpTextMetric;
  
      GetTextMetrics(GetHdc(), &lpTextMetric);
  
      TEXTMETRIC lpTextMetric;
  
      GetTextMetrics(GetHdc(), &lpTextMetric);
  
-    return YDEV2LOGREL(lpTextMetric.tmHeight);
+    return lpTextMetric.tmHeight;
  }
  
  wxCoord wxDC::GetCharWidth() const
  {
  }
  
  wxCoord wxDC::GetCharWidth() const
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return 0;
+#endif
+
      TEXTMETRIC lpTextMetric;
  
      GetTextMetrics(GetHdc(), &lpTextMetric);
  
      TEXTMETRIC lpTextMetric;
  
      GetTextMetrics(GetHdc(), &lpTextMetric);
  
-    return XDEV2LOGREL(lpTextMetric.tmAveCharWidth);
+    return lpTextMetric.tmAveCharWidth;
  }
  
  void wxDC::DoGetTextExtent(const wxString& string, wxCoord *x, wxCoord *y,
                             wxCoord *descent, wxCoord *externalLeading,
  }
  
  void wxDC::DoGetTextExtent(const wxString& string, wxCoord *x, wxCoord *y,
                             wxCoord *descent, wxCoord *externalLeading,
-                           wxFont *theFont) const
+                           wxFont *font) const
  {
  {
-    wxFont *fontToUse = (wxFont*) theFont;
-    if (!fontToUse)
-        fontToUse = (wxFont*) &m_font;
+#ifdef __WXMICROWIN__
+    if (!GetHDC())
+    {
+        if (x) *x = 0;
+        if (y) *y = 0;
+        if (descent) *descent = 0;
+        if (externalLeading) *externalLeading = 0;
+        return;
+    }
+#endif // __WXMICROWIN__
+
+    HFONT hfontOld;
+    if ( font )
+    {
+        wxASSERT_MSG( font->Ok(), _T("invalid font in wxDC::GetTextExtent") );
+
+        hfontOld = (HFONT)::SelectObject(GetHdc(), GetHfontOf(*font));
+    }
+    else // don't change the font
+    {
+        hfontOld = 0;
+    }
  
      SIZE sizeRect;
      TEXTMETRIC tm;
  
  
      SIZE sizeRect;
      TEXTMETRIC tm;
  
-    GetTextExtentPoint(GetHdc(), WXSTRINGCAST string, wxStrlen(WXSTRINGCAST string), &sizeRect);
+    GetTextExtentPoint(GetHdc(), string, string.length(), &sizeRect);
      GetTextMetrics(GetHdc(), &tm);
  
      GetTextMetrics(GetHdc(), &tm);
  
-    if (x) *x = XDEV2LOGREL(sizeRect.cx);
-    if (y) *y = YDEV2LOGREL(sizeRect.cy);
-    if (descent) *descent = tm.tmDescent;
-    if (externalLeading) *externalLeading = tm.tmExternalLeading;
+    if (x)
+        *x = sizeRect.cx;
+    if (y)
+        *y = sizeRect.cy;
+    if (descent)
+        *descent = tm.tmDescent;
+    if (externalLeading)
+        *externalLeading = tm.tmExternalLeading;
+
+    if ( hfontOld )
+    {
+        ::SelectObject(GetHdc(), hfontOld);
+    }
  }
  
  void wxDC::SetMapMode(int mode)
  {
  }
  
  void wxDC::SetMapMode(int mode)
  {
-    m_mappingMode = mode;
-
-    int pixel_width = 0;
-    int pixel_height = 0;
-    int mm_width = 0;
-    int mm_height = 0;
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
  
  
-    pixel_width = GetDeviceCaps(GetHdc(), HORZRES);
-    pixel_height = GetDeviceCaps(GetHdc(), VERTRES);
-    mm_width = GetDeviceCaps(GetHdc(), HORZSIZE);
-    mm_height = GetDeviceCaps(GetHdc(), VERTSIZE);
+    m_mappingMode = mode;
  
  
-    if ((pixel_width == 0) || (pixel_height == 0) || (mm_width == 0) || (mm_height == 0))
+    if ( mode == wxMM_TEXT )
      {
      {
-        return;
+        m_logicalScaleX =
+        m_logicalScaleY = 1.0;
      }
      }
-
-    double mm2pixelsX = pixel_width/mm_width;
-    double mm2pixelsY = pixel_height/mm_height;
-
-    switch (mode)
+    else // need to do some calculations
      {
      {
-    case wxMM_TWIPS:
-        {
-            m_logicalScaleX = (twips2mm * mm2pixelsX);
-            m_logicalScaleY = (twips2mm * mm2pixelsY);
-            break;
-        }
-    case wxMM_POINTS:
-        {
-            m_logicalScaleX = (pt2mm * mm2pixelsX);
-            m_logicalScaleY = (pt2mm * mm2pixelsY);
-            break;
-        }
-    case wxMM_METRIC:
-        {
-            m_logicalScaleX = mm2pixelsX;
-            m_logicalScaleY = mm2pixelsY;
-            break;
-        }
-    case wxMM_LOMETRIC:
+        int pixel_width = ::GetDeviceCaps(GetHdc(), HORZRES),
+            pixel_height = ::GetDeviceCaps(GetHdc(), VERTRES),
+            mm_width = ::GetDeviceCaps(GetHdc(), HORZSIZE),
+            mm_height = ::GetDeviceCaps(GetHdc(), VERTSIZE);
+
+        if ( (mm_width == 0) || (mm_height == 0) )
          {
          {
-            m_logicalScaleX = (mm2pixelsX/10.0);
-            m_logicalScaleY = (mm2pixelsY/10.0);
-            break;
+            // we can't calculate mm2pixels[XY] then!
+            return;
          }
          }
-    default:
-    case wxMM_TEXT:
+
+        double mm2pixelsX = pixel_width / mm_width,
+               mm2pixelsY = pixel_height / mm_height;
+
+        switch (mode)
          {
          {
-            m_logicalScaleX = 1.0;
-            m_logicalScaleY = 1.0;
-            break;
+            case wxMM_TWIPS:
+                m_logicalScaleX = twips2mm * mm2pixelsX;
+                m_logicalScaleY = twips2mm * mm2pixelsY;
+                break;
+
+            case wxMM_POINTS:
+                m_logicalScaleX = pt2mm * mm2pixelsX;
+                m_logicalScaleY = pt2mm * mm2pixelsY;
+                break;
+
+            case wxMM_METRIC:
+                m_logicalScaleX = mm2pixelsX;
+                m_logicalScaleY = mm2pixelsY;
+                break;
+
+            case wxMM_LOMETRIC:
+                m_logicalScaleX = mm2pixelsX / 10.0;
+                m_logicalScaleY = mm2pixelsY / 10.0;
+                break;
+
+            default:
+                wxFAIL_MSG( _T("unknown mapping mode in SetMapMode") );
          }
      }
  
          }
      }
  
-    if (::GetMapMode(GetHdc()) != MM_ANISOTROPIC)
-        ::SetMapMode(GetHdc(), MM_ANISOTROPIC);
+    // VZ: it seems very wasteful to always use MM_ANISOTROPIC when in 99% of
+    //     cases we could do with MM_TEXT and in the remaining 0.9% with
+    //     MM_ISOTROPIC (TODO!)
+    ::SetMapMode(GetHdc(), MM_ANISOTROPIC);
  
  
-    SetViewportExtEx(GetHdc(), VIEWPORT_EXTENT, VIEWPORT_EXTENT, NULL);
-    m_windowExtX = (int)MS_XDEV2LOGREL(VIEWPORT_EXTENT);
-    m_windowExtY = (int)MS_YDEV2LOGREL(VIEWPORT_EXTENT);
-    ::SetWindowExtEx(GetHdc(), m_windowExtX, m_windowExtY, NULL);
-    ::SetViewportOrgEx(GetHdc(), (int)m_deviceOriginX, (int)m_deviceOriginY, NULL);
-    ::SetWindowOrgEx(GetHdc(), (int)m_logicalOriginX, (int)m_logicalOriginY, NULL);
+    int width = DeviceToLogicalXRel(VIEWPORT_EXTENT)*m_signX,
+        height = DeviceToLogicalYRel(VIEWPORT_EXTENT)*m_signY;
+
+    ::SetViewportExtEx(GetHdc(), VIEWPORT_EXTENT, VIEWPORT_EXTENT, NULL);
+    ::SetWindowExtEx(GetHdc(), width, height, NULL);
+
+    ::SetViewportOrgEx(GetHdc(), m_deviceOriginX, m_deviceOriginY, NULL);
+    ::SetWindowOrgEx(GetHdc(), m_logicalOriginX, m_logicalOriginY, NULL);
  }
  
  void wxDC::SetUserScale(double x, double y)
  {
  }
  
  void wxDC::SetUserScale(double x, double y)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
+    if ( x == m_userScaleX && y == m_userScaleY )
+        return;
+
      m_userScaleX = x;
      m_userScaleY = y;
  
      m_userScaleX = x;
      m_userScaleY = y;
  
@@ -1416,14 +1638,31 @@ void wxDC::SetUserScale(double x, double y)
  
  void wxDC::SetAxisOrientation(bool xLeftRight, bool yBottomUp)
  {
  
  void wxDC::SetAxisOrientation(bool xLeftRight, bool yBottomUp)
  {
-    m_signX = xLeftRight ? 1 : -1;
-    m_signY = yBottomUp ? -1 : 1;
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
  
  
-    SetMapMode(m_mappingMode);
+    int signX = xLeftRight ? 1 : -1,
+        signY = yBottomUp ? -1 : 1;
+
+    if ( signX != m_signX || signY != m_signY )
+    {
+        m_signX = signX;
+        m_signY = signY;
+
+        SetMapMode(m_mappingMode);
+    }
  }
  
  void wxDC::SetSystemScale(double x, double y)
  {
  }
  
  void wxDC::SetSystemScale(double x, double y)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
+    if ( x == m_scaleX && y == m_scaleY )
+        return;
+
      m_scaleX = x;
      m_scaleY = y;
  
      m_scaleX = x;
      m_scaleY = y;
  
@@ -1432,6 +1671,13 @@ void wxDC::SetSystemScale(double x, double y)
  
  void wxDC::SetLogicalOrigin(wxCoord x, wxCoord y)
  {
  
  void wxDC::SetLogicalOrigin(wxCoord x, wxCoord y)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
+    if ( x == m_logicalOriginX && y == m_logicalOriginY )
+        return;
+
      m_logicalOriginX = x;
      m_logicalOriginY = y;
  
      m_logicalOriginX = x;
      m_logicalOriginY = y;
  
@@ -1440,6 +1686,13 @@ void wxDC::SetLogicalOrigin(wxCoord x, wxCoord y)
  
  void wxDC::SetDeviceOrigin(wxCoord x, wxCoord y)
  {
  
  void wxDC::SetDeviceOrigin(wxCoord x, wxCoord y)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
+    if ( x == m_deviceOriginX && y == m_deviceOriginY )
+        return;
+
      m_deviceOriginX = x;
      m_deviceOriginY = y;
  
      m_deviceOriginX = x;
      m_deviceOriginY = y;
  
@@ -1452,42 +1705,46 @@ void wxDC::SetDeviceOrigin(wxCoord x, wxCoord y)
  
  wxCoord wxDCBase::DeviceToLogicalX(wxCoord x) const
  {
  
  wxCoord wxDCBase::DeviceToLogicalX(wxCoord x) const
  {
-    return (wxCoord) (((x) - m_deviceOriginX)/(m_logicalScaleX*m_userScaleX*m_signX*m_scaleX) - m_logicalOriginX);
+    return DeviceToLogicalXRel(x - m_deviceOriginX)*m_signX + m_logicalOriginX;
  }
  
  wxCoord wxDCBase::DeviceToLogicalXRel(wxCoord x) const
  {
  }
  
  wxCoord wxDCBase::DeviceToLogicalXRel(wxCoord x) const
  {
-    return (wxCoord) ((x)/(m_logicalScaleX*m_userScaleX*m_signX*m_scaleX));
+    // axis orientation is not taken into account for conversion of a distance
+    return (wxCoord)(x / (m_logicalScaleX*m_userScaleX*m_scaleX));
  }
  
  wxCoord wxDCBase::DeviceToLogicalY(wxCoord y) const
  {
  }
  
  wxCoord wxDCBase::DeviceToLogicalY(wxCoord y) const
  {
-    return (wxCoord) (((y) - m_deviceOriginY)/(m_logicalScaleY*m_userScaleY*m_signY*m_scaleY) - m_logicalOriginY);
+    return DeviceToLogicalYRel(y - m_deviceOriginY)*m_signY + m_logicalOriginY;
  }
  
  wxCoord wxDCBase::DeviceToLogicalYRel(wxCoord y) const
  {
  }
  
  wxCoord wxDCBase::DeviceToLogicalYRel(wxCoord y) const
  {
-    return (wxCoord) ((y)/(m_logicalScaleY*m_userScaleY*m_signY*m_scaleY));
+    // axis orientation is not taken into account for conversion of a distance
+    return (wxCoord)( y / (m_logicalScaleY*m_userScaleY*m_scaleY));
  }
  
  wxCoord wxDCBase::LogicalToDeviceX(wxCoord x) const
  {
  }
  
  wxCoord wxDCBase::LogicalToDeviceX(wxCoord x) const
  {
-    return (wxCoord) ((x - m_logicalOriginX)*m_logicalScaleX*m_userScaleX*m_signX*m_scaleX + m_deviceOriginX);
+    return LogicalToDeviceXRel(x - m_logicalOriginX)*m_signX + m_deviceOriginX;
  }
  
  wxCoord wxDCBase::LogicalToDeviceXRel(wxCoord x) const
  {
  }
  
  wxCoord wxDCBase::LogicalToDeviceXRel(wxCoord x) const
  {
-    return (wxCoord) (x*m_logicalScaleX*m_userScaleX*m_signX*m_scaleX);
+    // axis orientation is not taken into account for conversion of a distance
+    return (wxCoord) (x*m_logicalScaleX*m_userScaleX*m_scaleX);
  }
  
  wxCoord wxDCBase::LogicalToDeviceY(wxCoord y) const
  {
  }
  
  wxCoord wxDCBase::LogicalToDeviceY(wxCoord y) const
  {
-    return (wxCoord) ((y - m_logicalOriginY)*m_logicalScaleY*m_userScaleY*m_signY*m_scaleY + m_deviceOriginY);
+    return LogicalToDeviceYRel(y - m_logicalOriginY)*m_signY + m_deviceOriginY;
  }
  
  wxCoord wxDCBase::LogicalToDeviceYRel(wxCoord y) const
  {
  }
  
  wxCoord wxDCBase::LogicalToDeviceYRel(wxCoord y) const
  {
-    return (wxCoord) (y*m_logicalScaleY*m_userScaleY*m_signY*m_scaleY);
+    // axis orientation is not taken into account for conversion of a distance
+    return (wxCoord) (y*m_logicalScaleY*m_userScaleY*m_scaleY);
  }
  
  // ---------------------------------------------------------------------------
  }
  
  // ---------------------------------------------------------------------------
@@ -1497,8 +1754,13 @@ wxCoord wxDCBase::LogicalToDeviceYRel(wxCoord y) const
  bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest,
                    wxCoord width, wxCoord height,
                    wxDC *source, wxCoord xsrc, wxCoord ysrc,
  bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest,
                    wxCoord width, wxCoord height,
                    wxDC *source, wxCoord xsrc, wxCoord ysrc,
-                  int rop, bool useMask)
+                  int rop, bool useMask,
+                  wxCoord xsrcMask, wxCoord ysrcMask)
  {
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return FALSE;
+#endif
+
      wxMask *mask = NULL;
      if ( useMask )
      {
      wxMask *mask = NULL;
      if ( useMask )
      {
@@ -1513,6 +1775,11 @@ bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest,
          }
      }
  
          }
      }
  
+    if (xsrcMask == -1 && ysrcMask == -1)
+    {
+        xsrcMask = xsrc; ysrcMask = ysrc;
+    }
+
      COLORREF old_textground = ::GetTextColor(GetHdc());
      COLORREF old_background = ::GetBkColor(GetHdc());
      if (m_textForegroundColour.Ok())
      COLORREF old_textground = ::GetTextColor(GetHdc());
      COLORREF old_background = ::GetBkColor(GetHdc());
      if (m_textForegroundColour.Ok())
@@ -1548,7 +1815,7 @@ bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest,
             return FALSE;
      }
  
             return FALSE;
      }
  
-    bool success;
+    bool success = FALSE;
  
      if (useMask)
      {
  
      if (useMask)
      {
@@ -1557,53 +1824,85 @@ bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest,
          // transparent, so use "DSTCOPY" ROP for the mask points (the usual
          // meaning of fg and bg is inverted which corresponds to wxWin notion
          // of the mask which is also contrary to the Windows one)
          // transparent, so use "DSTCOPY" ROP for the mask points (the usual
          // meaning of fg and bg is inverted which corresponds to wxWin notion
          // of the mask which is also contrary to the Windows one)
-        success = ::MaskBlt(GetHdc(), xdest, ydest, width, height,
-                            GetHdcOf(*source), xsrc, ysrc,
-                            (HBITMAP)mask->GetMaskBitmap(), 0, 0,
-                            MAKEROP4(dwRop, DSTCOPY)) != 0;
+
+        // On some systems, MaskBlt succeeds yet is much much slower
+        // than the wxWindows fall-back implementation. So we need
+        // to be able to switch this on and off at runtime.
+#if wxUSE_SYSTEM_OPTIONS
+        if (wxSystemOptions::GetOptionInt(wxT("no-maskblt")) == 0)
+#endif
+        {
+           success = ::MaskBlt
+                       (
+                            GetHdc(),
+                            xdest, ydest, width, height,
+                            GetHdcOf(*source),
+                            xsrc, ysrc,
+                            (HBITMAP)mask->GetMaskBitmap(),
+                            xsrcMask, ysrcMask,
+                            MAKEROP4(dwRop, DSTCOPY)
+                        ) != 0;
+        }
  
          if ( !success )
  #endif // Win32
          {
              // Blit bitmap with mask
  
          if ( !success )
  #endif // Win32
          {
              // Blit bitmap with mask
+            HDC dc_mask ;
+            HDC  dc_buffer ;
+            HBITMAP buffer_bmap ;
  
  
+#if wxUSE_DC_CACHEING
              // create a temp buffer bitmap and DCs to access it and the mask
              // create a temp buffer bitmap and DCs to access it and the mask
-            HDC dc_mask = ::CreateCompatibleDC(GetHdcOf(*source));
-            HDC dc_buffer = ::CreateCompatibleDC(GetHdc());
-            HBITMAP buffer_bmap = ::CreateCompatibleBitmap(GetHdc(), width, height);
-            ::SelectObject(dc_mask, (HBITMAP) mask->GetMaskBitmap());
-            ::SelectObject(dc_buffer, buffer_bmap);
+            wxDCCacheEntry* dcCacheEntry1 = FindDCInCache(NULL, source->GetHDC());
+            dc_mask = (HDC) dcCacheEntry1->m_dc;
+
+            wxDCCacheEntry* dcCacheEntry2 = FindDCInCache(dcCacheEntry1, GetHDC());
+            dc_buffer = (HDC) dcCacheEntry2->m_dc;
+
+            wxDCCacheEntry* bitmapCacheEntry = FindBitmapInCache(GetHDC(),
+                width, height);
+
+            buffer_bmap = (HBITMAP) bitmapCacheEntry->m_bitmap;
+#else // !wxUSE_DC_CACHEING
+            // create a temp buffer bitmap and DCs to access it and the mask
+            dc_mask = ::CreateCompatibleDC(GetHdcOf(*source));
+            dc_buffer = ::CreateCompatibleDC(GetHdc());
+            buffer_bmap = ::CreateCompatibleBitmap(GetHdc(), width, height);
+#endif // wxUSE_DC_CACHEING/!wxUSE_DC_CACHEING
+            HGDIOBJ hOldMaskBitmap = ::SelectObject(dc_mask, (HBITMAP) mask->GetMaskBitmap());
+            HGDIOBJ hOldBufferBitmap = ::SelectObject(dc_buffer, buffer_bmap);
  
              // copy dest to buffer
              if ( !::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
                             GetHdc(), xdest, ydest, SRCCOPY) )
              {
  
              // copy dest to buffer
              if ( !::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
                             GetHdc(), xdest, ydest, SRCCOPY) )
              {
-                wxLogLastError("BitBlt");
+                wxLogLastError(wxT("BitBlt"));
              }
  
              // copy src to buffer using selected raster op
              if ( !::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
                             GetHdcOf(*source), xsrc, ysrc, dwRop) )
              {
              }
  
              // copy src to buffer using selected raster op
              if ( !::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
                             GetHdcOf(*source), xsrc, ysrc, dwRop) )
              {
-                wxLogLastError("BitBlt");
+                wxLogLastError(wxT("BitBlt"));
              }
  
              // set masked area in buffer to BLACK (pixel value 0)
              COLORREF prevBkCol = ::SetBkColor(GetHdc(), RGB(255, 255, 255));
              COLORREF prevCol = ::SetTextColor(GetHdc(), RGB(0, 0, 0));
              if ( !::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
              }
  
              // set masked area in buffer to BLACK (pixel value 0)
              COLORREF prevBkCol = ::SetBkColor(GetHdc(), RGB(255, 255, 255));
              COLORREF prevCol = ::SetTextColor(GetHdc(), RGB(0, 0, 0));
              if ( !::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
-                           dc_mask, xsrc, ysrc, SRCAND) )
+                           dc_mask, xsrcMask, ysrcMask, SRCAND) )
              {
              {
-                wxLogLastError("BitBlt");
+                wxLogLastError(wxT("BitBlt"));
              }
  
              // set unmasked area in dest to BLACK
              ::SetBkColor(GetHdc(), RGB(0, 0, 0));
              ::SetTextColor(GetHdc(), RGB(255, 255, 255));
              if ( !::BitBlt(GetHdc(), xdest, ydest, (int)width, (int)height,
              }
  
              // set unmasked area in dest to BLACK
              ::SetBkColor(GetHdc(), RGB(0, 0, 0));
              ::SetTextColor(GetHdc(), RGB(255, 255, 255));
              if ( !::BitBlt(GetHdc(), xdest, ydest, (int)width, (int)height,
-                           dc_mask, xsrc, ysrc, SRCAND) )
+                           dc_mask, xsrcMask, ysrcMask, SRCAND) )
              {
              {
-                wxLogLastError("BitBlt");
+                wxLogLastError(wxT("BitBlt"));
              }
              ::SetBkColor(GetHdc(), prevBkCol);   // restore colours to original values
              ::SetTextColor(GetHdc(), prevCol);
              }
              ::SetBkColor(GetHdc(), prevBkCol);   // restore colours to original values
              ::SetTextColor(GetHdc(), prevCol);
@@ -1614,27 +1913,57 @@ bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest,
                                 dc_buffer, 0, 0, SRCPAINT) != 0;
              if ( !success )
              {
                                 dc_buffer, 0, 0, SRCPAINT) != 0;
              if ( !success )
              {
-                wxLogLastError("BitBlt");
+                wxLogLastError(wxT("BitBlt"));
              }
  
              // tidy up temporary DCs and bitmap
              }
  
              // tidy up temporary DCs and bitmap
-            ::SelectObject(dc_mask, 0);
-            ::DeleteDC(dc_mask);
-            ::SelectObject(dc_buffer, 0);
-            ::DeleteDC(dc_buffer);
-            ::DeleteObject(buffer_bmap);
+            ::SelectObject(dc_mask, hOldMaskBitmap);
+            ::SelectObject(dc_buffer, hOldBufferBitmap);
+
+#if !wxUSE_DC_CACHEING
+            {
+                ::DeleteDC(dc_mask);
+                ::DeleteDC(dc_buffer);
+                ::DeleteObject(buffer_bmap);
+            }
+#endif
          }
      }
      else // no mask, just BitBlt() it
      {
          }
      }
      else // no mask, just BitBlt() it
      {
-        success = ::BitBlt(GetHdc(), xdest, ydest,
-                           (int)width, (int)height,
-                           GetHdcOf(*source), xsrc, ysrc, dwRop) != 0;
+        // use StretchBlt() if available
+        if ( ::GetDeviceCaps(GetHdc(), RASTERCAPS) & RC_STRETCHBLT )
+        {
+            StretchBltModeChanger changeMode(GetHdc(), COLORONCOLOR);
+
+            success = ::StretchBlt
+                        (
+                            GetHdc(),
+                            xdest, ydest, width, height,
+                            GetHdcOf(*source),
+                            xsrc, ysrc, width, height,
+                            dwRop
+                        ) != 0;
+        }
+        else
+        {
+            success = ::BitBlt
+                        (
+                            GetHdc(),
+                            xdest, ydest,
+                            (int)width, (int)height,
+                            GetHdcOf(*source),
+                            xsrc, ysrc,
+                            dwRop
+                        ) != 0;
+        }
+
          if ( !success )
          {
          if ( !success )
          {
-            wxLogLastError("BitBlt");
+            wxLogLastError(wxT("BitBlt/StretchBlt"));
          }
      }
          }
      }
+
      ::SetTextColor(GetHdc(), old_textground);
      ::SetBkColor(GetHdc(), old_background);
  
      ::SetTextColor(GetHdc(), old_textground);
      ::SetBkColor(GetHdc(), old_background);
  
@@ -1643,18 +1972,52 @@ bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest,
  
  void wxDC::DoGetSize(int *w, int *h) const
  {
  
  void wxDC::DoGetSize(int *w, int *h) const
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      if ( w ) *w = ::GetDeviceCaps(GetHdc(), HORZRES);
      if ( h ) *h = ::GetDeviceCaps(GetHdc(), VERTRES);
  }
  
  void wxDC::DoGetSizeMM(int *w, int *h) const
  {
      if ( w ) *w = ::GetDeviceCaps(GetHdc(), HORZRES);
      if ( h ) *h = ::GetDeviceCaps(GetHdc(), VERTRES);
  }
  
  void wxDC::DoGetSizeMM(int *w, int *h) const
  {
-    if ( w ) *w = ::GetDeviceCaps(GetHdc(), HORZSIZE);
-    if ( h ) *h = ::GetDeviceCaps(GetHdc(), VERTSIZE);
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
+    // if we implement it in terms of DoGetSize() instead of directly using the
+    // results returned by GetDeviceCaps(HORZ/VERTSIZE) as was done before, it
+    // will also work for wxWindowDC and wxClientDC even though their size is
+    // not the same as the total size of the screen
+    int wPixels, hPixels;
+    DoGetSize(&wPixels, &hPixels);
+
+    if ( w )
+    {
+        int wTotal = ::GetDeviceCaps(GetHdc(), HORZRES);
+
+        wxCHECK_RET( wTotal, _T("0 width device?") );
+
+        *w = (wPixels * ::GetDeviceCaps(GetHdc(), HORZSIZE)) / wTotal;
+    }
+
+    if ( h )
+    {
+        int hTotal = ::GetDeviceCaps(GetHdc(), VERTRES);
+
+        wxCHECK_RET( hTotal, _T("0 height device?") );
+
+        *h = (hPixels * ::GetDeviceCaps(GetHdc(), VERTSIZE)) / hTotal;
+    }
  }
  
  wxSize wxDC::GetPPI() const
  {
  }
  
  wxSize wxDC::GetPPI() const
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return wxSize();
+#endif
+
      int x = ::GetDeviceCaps(GetHdc(), LOGPIXELSX);
      int y = ::GetDeviceCaps(GetHdc(), LOGPIXELSY);
  
      int x = ::GetDeviceCaps(GetHdc(), LOGPIXELSX);
      int y = ::GetDeviceCaps(GetHdc(), LOGPIXELSY);
  
@@ -1664,258 +2027,149 @@ wxSize wxDC::GetPPI() const
  // For use by wxWindows only, unless custom units are required.
  void wxDC::SetLogicalScale(double x, double y)
  {
  // For use by wxWindows only, unless custom units are required.
  void wxDC::SetLogicalScale(double x, double y)
  {
+#ifdef __WXMICROWIN__
+    if (!GetHDC()) return;
+#endif
+
      m_logicalScaleX = x;
      m_logicalScaleY = y;
  }
  
      m_logicalScaleX = x;
      m_logicalScaleY = y;
  }
  
-#if WXWIN_COMPATIBILITY
-void wxDC::DoGetTextExtent(const wxString& string, float *x, float *y,
-                         float *descent, float *externalLeading,
-                         wxFont *theFont, bool use16bit) const
-{
-    wxCoord x1, y1, descent1, externalLeading1;
-    GetTextExtent(string, & x1, & y1, & descent1, & externalLeading1, theFont, use16bit);
-    *x = x1; *y = y1;
-    if (descent)
-        *descent = descent1;
-    if (externalLeading)
-        *externalLeading = externalLeading1;
-}
-#endif
+#if wxUSE_DC_CACHEING
  
  
-// ---------------------------------------------------------------------------
-// spline drawing code
-// ---------------------------------------------------------------------------
+/*
+ * This implementation is a bit ugly and uses the old-fashioned wxList class, so I will
+ * improve it in due course, either using arrays, or simply storing pointers to one
+ * entry for the bitmap, and two for the DCs. -- JACS
+ */
  
  
-#if wxUSE_SPLINES
+wxList wxDC::sm_bitmapCache;
+wxList wxDC::sm_dcCache;
  
  
-class wxSpline: public wxObject
+wxDCCacheEntry::wxDCCacheEntry(WXHBITMAP hBitmap, int w, int h, int depth)
  {
  {
-public:
-    int type;
-    wxList *points;
-
-    wxSpline(wxList *list);
-    void DeletePoints();
-
-    // Doesn't delete points
-    ~wxSpline();
-};
-
-void wx_draw_open_spline(wxDC *dc, wxSpline *spline);
-
-void wx_quadratic_spline(double a1, double b1, double a2, double b2,
-                         double a3, double b3, double a4, double b4);
-void wx_clear_stack();
-int wx_spline_pop(double *x1, double *y1, double *x2, double *y2, double *x3,
-                  double *y3, double *x4, double *y4);
-void wx_spline_push(double x1, double y1, double x2, double y2, double x3, double y3,
-                    double x4, double y4);
-static bool wx_spline_add_point(double x, double y);
-static void wx_spline_draw_point_array(wxDC *dc);
-wxSpline *wx_make_spline(int x1, int y1, int x2, int y2, int x3, int y3);
+    m_bitmap = hBitmap;
+    m_dc = 0;
+    m_width = w;
+    m_height = h;
+    m_depth = depth;
+}
  
  
-void wxDC::DoDrawSpline(wxList *list)
+wxDCCacheEntry::wxDCCacheEntry(WXHDC hDC, int depth)
  {
  {
-    wxSpline spline(list);
-
-    wx_draw_open_spline(this, &spline);
+    m_bitmap = 0;
+    m_dc = hDC;
+    m_width = 0;
+    m_height = 0;
+    m_depth = depth;
  }
  
  }
  
-wxList wx_spline_point_list;
-
-void wx_draw_open_spline(wxDC *dc, wxSpline *spline)
+wxDCCacheEntry::~wxDCCacheEntry()
  {
  {
-    wxPoint *p;
-    double           cx1, cy1, cx2, cy2, cx3, cy3, cx4, cy4;
-    double           x1, y1, x2, y2;
-
-    wxNode *node = spline->points->First();
-    p = (wxPoint *)node->Data();
-
-    x1 = p->x;
-    y1 = p->y;
-
-    node = node->Next();
-    p = (wxPoint *)node->Data();
-
-    x2 = p->x;
-    y2 = p->y;
-    cx1 = (double)((x1 + x2) / 2);
-    cy1 = (double)((y1 + y2) / 2);
-    cx2 = (double)((cx1 + x2) / 2);
-    cy2 = (double)((cy1 + y2) / 2);
-
-    wx_spline_add_point(x1, y1);
-
-    while ((node = node->Next()) != NULL)
-    {
-        p = (wxPoint *)node->Data();
-        x1 = x2;
-        y1 = y2;
-        x2 = p->x;
-        y2 = p->y;
-        cx4 = (double)(x1 + x2) / 2;
-        cy4 = (double)(y1 + y2) / 2;
-        cx3 = (double)(x1 + cx4) / 2;
-        cy3 = (double)(y1 + cy4) / 2;
-
-        wx_quadratic_spline(cx1, cy1, cx2, cy2, cx3, cy3, cx4, cy4);
-
-        cx1 = cx4;
-        cy1 = cy4;
-        cx2 = (double)(cx1 + x2) / 2;
-        cy2 = (double)(cy1 + y2) / 2;
-    }
-
-    wx_spline_add_point((double)wx_round(cx1), (double)wx_round(cy1));
-    wx_spline_add_point(x2, y2);
-
-    wx_spline_draw_point_array(dc);
-
+    if (m_bitmap)
+        ::DeleteObject((HBITMAP) m_bitmap);
+    if (m_dc)
+        ::DeleteDC((HDC) m_dc);
  }
  
  }
  
-/********************* CURVES FOR SPLINES *****************************
-
-  The following spline drawing routine is from
-
-    "An Algorithm for High-Speed Curve Generation"
-    by George Merrill Chaikin,
-    Computer Graphics and Image Processing, 3, Academic Press,
-    1974, 346-349.
-
-      and
-
-        "On Chaikin's Algorithm" by R. F. Riesenfeld,
-        Computer Graphics and Image Processing, 4, Academic Press,
-        1975, 304-310.
-
-***********************************************************************/
-
-#define     half(z1, z2)    ((z1+z2)/2.0)
-#define     THRESHOLD   5
-
-/* iterative version */
-
-void wx_quadratic_spline(double a1, double b1, double a2, double b2, double a3, double b3, double a4,
-                         double b4)
+wxDCCacheEntry* wxDC::FindBitmapInCache(WXHDC dc, int w, int h)
  {
  {
-    register double  xmid, ymid;
-    double           x1, y1, x2, y2, x3, y3, x4, y4;
-
-    wx_clear_stack();
-    wx_spline_push(a1, b1, a2, b2, a3, b3, a4, b4);
+    int depth = ::GetDeviceCaps((HDC) dc, PLANES) * ::GetDeviceCaps((HDC) dc, BITSPIXEL);
+    wxNode* node = sm_bitmapCache.First();
+    while (node)
+    {
+        wxDCCacheEntry* entry = (wxDCCacheEntry*) node->Data();
  
  
-    while (wx_spline_pop(&x1, &y1, &x2, &y2, &x3, &y3, &x4, &y4)) {
-        xmid = (double)half(x2, x3);
-        ymid = (double)half(y2, y3);
-        if (fabs(x1 - xmid) < THRESHOLD && fabs(y1 - ymid) < THRESHOLD &&
-            fabs(xmid - x4) < THRESHOLD && fabs(ymid - y4) < THRESHOLD) {
-            wx_spline_add_point((double)wx_round(x1), (double)wx_round(y1));
-            wx_spline_add_point((double)wx_round(xmid), (double)wx_round(ymid));
-        } else {
-            wx_spline_push(xmid, ymid, (double)half(xmid, x3), (double)half(ymid, y3),
-                (double)half(x3, x4), (double)half(y3, y4), x4, y4);
-            wx_spline_push(x1, y1, (double)half(x1, x2), (double)half(y1, y2),
-                (double)half(x2, xmid), (double)half(y2, ymid), xmid, ymid);
+        if (entry->m_depth == depth)
+        {
+            if (entry->m_width < w || entry->m_height < h)
+            {
+                ::DeleteObject((HBITMAP) entry->m_bitmap);
+                entry->m_bitmap = (WXHBITMAP) ::CreateCompatibleBitmap((HDC) dc, w, h);
+                if ( !entry->m_bitmap)
+                {
+                    wxLogLastError(wxT("CreateCompatibleBitmap"));
+                }
+                entry->m_width = w; entry->m_height = h;
+                return entry;
+            }
+            return entry;
          }
          }
-    }
-}
-
-
-/* utilities used by spline drawing routines */
-
  
  
-typedef struct wx_spline_stack_struct {
-    double           x1, y1, x2, y2, x3, y3, x4, y4;
+        node = node->Next();
+    }
+    WXHBITMAP hBitmap = (WXHBITMAP) ::CreateCompatibleBitmap((HDC) dc, w, h);
+    if ( !hBitmap)
+    {
+        wxLogLastError(wxT("CreateCompatibleBitmap"));
+    }
+    wxDCCacheEntry* entry = new wxDCCacheEntry(hBitmap, w, h, depth);
+    AddToBitmapCache(entry);
+    return entry;
  }
  }
-Stack;
-
-#define         SPLINE_STACK_DEPTH             20
-static Stack    wx_spline_stack[SPLINE_STACK_DEPTH];
-static Stack   *wx_stack_top;
-static int      wx_stack_count;
  
  
-void wx_clear_stack()
+wxDCCacheEntry* wxDC::FindDCInCache(wxDCCacheEntry* notThis, WXHDC dc)
  {
  {
-    wx_stack_top = wx_spline_stack;
-    wx_stack_count = 0;
-}
+    int depth = ::GetDeviceCaps((HDC) dc, PLANES) * ::GetDeviceCaps((HDC) dc, BITSPIXEL);
+    wxNode* node = sm_dcCache.First();
+    while (node)
+    {
+        wxDCCacheEntry* entry = (wxDCCacheEntry*) node->Data();
  
  
-void wx_spline_push(double x1, double y1, double x2, double y2, double x3, double y3, double x4, double y4)
-{
-    wx_stack_top->x1 = x1;
-    wx_stack_top->y1 = y1;
-    wx_stack_top->x2 = x2;
-    wx_stack_top->y2 = y2;
-    wx_stack_top->x3 = x3;
-    wx_stack_top->y3 = y3;
-    wx_stack_top->x4 = x4;
-    wx_stack_top->y4 = y4;
-    wx_stack_top++;
-    wx_stack_count++;
-}
+        // Don't return the same one as we already have
+        if (!notThis || (notThis != entry))
+        {
+            if (entry->m_depth == depth)
+            {
+                return entry;
+            }
+        }
  
  
-int wx_spline_pop(double *x1, double *y1, double *x2, double *y2,
-                  double *x3, double *y3, double *x4, double *y4)
-{
-    if (wx_stack_count == 0)
-        return (0);
-    wx_stack_top--;
-    wx_stack_count--;
-    *x1 = wx_stack_top->x1;
-    *y1 = wx_stack_top->y1;
-    *x2 = wx_stack_top->x2;
-    *y2 = wx_stack_top->y2;
-    *x3 = wx_stack_top->x3;
-    *y3 = wx_stack_top->y3;
-    *x4 = wx_stack_top->x4;
-    *y4 = wx_stack_top->y4;
-    return (1);
+        node = node->Next();
+    }
+    WXHDC hDC = (WXHDC) ::CreateCompatibleDC((HDC) dc);
+    if ( !hDC)
+    {
+        wxLogLastError(wxT("CreateCompatibleDC"));
+    }
+    wxDCCacheEntry* entry = new wxDCCacheEntry(hDC, depth);
+    AddToDCCache(entry);
+    return entry;
  }
  
  }
  
-static bool wx_spline_add_point(double x, double y)
+void wxDC::AddToBitmapCache(wxDCCacheEntry* entry)
  {
  {
-    wxPoint *point = new wxPoint;
-    point->x = (int) x;
-    point->y = (int) y;
-    wx_spline_point_list.Append((wxObject*)point);
-    return TRUE;
+    sm_bitmapCache.Append(entry);
  }
  
  }
  
-static void wx_spline_draw_point_array(wxDC *dc)
+void wxDC::AddToDCCache(wxDCCacheEntry* entry)
  {
  {
-    dc->DrawLines(&wx_spline_point_list, 0, 0);
-    wxNode *node = wx_spline_point_list.First();
-    while (node)
-    {
-        wxPoint *point = (wxPoint *)node->Data();
-        delete point;
-        delete node;
-        node = wx_spline_point_list.First();
-    }
+    sm_dcCache.Append(entry);
  }
  
  }
  
-wxSpline::wxSpline(wxList *list)
+void wxDC::ClearCache()
  {
  {
-    points = list;
+    sm_dcCache.DeleteContents(TRUE);
+    sm_dcCache.Clear();
+    sm_dcCache.DeleteContents(FALSE);
+    sm_bitmapCache.DeleteContents(TRUE);
+    sm_bitmapCache.Clear();
+    sm_bitmapCache.DeleteContents(FALSE);
  }
  
  }
  
-wxSpline::~wxSpline()
+// Clean up cache at app exit
+class wxDCModule : public wxModule
  {
  {
-}
+public:
+    virtual bool OnInit() { return TRUE; }
+    virtual void OnExit() { wxDC::ClearCache(); }
  
  
-void wxSpline::DeletePoints()
-{
-    for(wxNode *node = points->First(); node; node = points->First())
-    {
-        wxPoint *point = (wxPoint *)node->Data();
-        delete point;
-        delete node;
-    }
-    delete points;
-}
+private:
+    DECLARE_DYNAMIC_CLASS(wxDCModule)
+};
  
  
+IMPLEMENT_DYNAMIC_CLASS(wxDCModule, wxModule)
  
  
-#endif // wxUSE_SPLINES
+#endif
+    // wxUSE_DC_CACHEING