]> git.saurik.com Git - wxWidgets.git/blobdiff - src/msw/dc.cpp
Various small fixes
[wxWidgets.git] / src / msw / dc.cpp
index 674ca49a1933f6535b716a31654c184b12a1a030..73f33909521e59a5f224378bc019e6c523a04fd0 100644 (file)
@@ -6,7 +6,7 @@
 // Created:     01/02/97
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem
-// Licence:    wxWindows licence
+// Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
@@ -26,6 +26,8 @@
 #include "wx/utils.h"
 #include "wx/dialog.h"
 #include "wx/app.h"
+#include "wx/bitmap.h"
+#include "wx/dcmemory.h"
 #endif
 
 #include "wx/dcprint.h"
@@ -33,7 +35,6 @@
 
 #include <string.h>
 #include <math.h>
-#include <fstream.h>
 
 #if wxUSE_COMMON_DIALOGS
 #include <commdlg.h>
@@ -130,16 +131,10 @@ wxDC::~wxDC(void)
 // DC.
 void wxDC::SelectOldObjects(WXHDC dc)
 {
-#if WXDEBUG > 1
-  wxDebugMsg("wxDC::SelectOldObjects %X\n", this);
-#endif
   if (dc)
   {
     if (m_oldBitmap)
     {
-#if WXDEBUG > 1
-      wxDebugMsg("wxDC::SelectOldObjects: Selecting old HBITMAP %X\n", m_oldBitmap);
-#endif
       ::SelectObject((HDC) dc, (HBITMAP) m_oldBitmap);
       if (m_selectedBitmap.Ok())
       {
@@ -149,38 +144,23 @@ void wxDC::SelectOldObjects(WXHDC dc)
     m_oldBitmap = 0 ;
     if (m_oldPen)
     {
-#if WXDEBUG > 1
-      wxDebugMsg("wxDC::SelectOldObjects: Selecting old HPEN %X\n", m_oldPen);
-#endif
       ::SelectObject((HDC) dc, (HPEN) m_oldPen);
     }
     m_oldPen = 0 ;
     if (m_oldBrush)
     {
-#if WXDEBUG > 1
-      wxDebugMsg("wxDC::SelectOldObjects: Selecting old HBRUSH %X\n", m_oldBrush);
-#endif
       ::SelectObject((HDC) dc, (HBRUSH) m_oldBrush);
     }
     m_oldBrush = 0 ;
     if (m_oldFont)
     {
-#if WXDEBUG > 1
-      wxDebugMsg("wxDC::SelectOldObjects: Selecting old HFONT %X\n", m_oldFont);
-#endif
       ::SelectObject((HDC) dc, (HFONT) m_oldFont);
     }
     m_oldFont = 0 ;
     if (m_oldPalette)
     {
-#if WXDEBUG > 1
-      wxDebugMsg("wxDC::SelectOldObjects: Selecting old HPALETTE %X\n", m_oldPalette);
-#endif
       ::SelectPalette((HDC) dc, (HPALETTE) m_oldPalette, TRUE);
     }
-#if WXDEBUG > 1
-    wxDebugMsg("wxDC::SelectOldObjects: Done.\n");
-#endif
     m_oldPalette = 0 ;
   }
 
@@ -203,6 +183,26 @@ void wxDC::SetClippingRegion(long cx, long cy, long cw, long ch)
   DoClipping((WXHDC) m_hDC);
 }
 
+void wxDC::SetClippingRegion(const wxRegion& region)
+{
+  if (!region.GetHRGN())
+    return;
+
+  wxRect box = region.GetBox();
+
+  m_clipping = TRUE;
+  m_clipX1 = box.x;
+  m_clipY1 = box.y;
+  m_clipX2 = box.x + box.width;
+  m_clipY2 = box.y + box.height;
+
+#ifdef __WIN16__
+  SelectClipRgn((HDC) m_hDC, (HRGN) region.GetHRGN());
+#else
+  ExtSelectClipRgn((HDC) m_hDC, (HRGN) region.GetHRGN(), RGN_AND);
+#endif
+}
+
 void wxDC::DoClipping(WXHDC dc)
 {
   if (m_clipping && dc)
@@ -216,17 +216,14 @@ void wxDC::DestroyClippingRegion(void)
 {
   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.
     HRGN rgn = CreateRectRgn(0, 0, 32000, 32000);
-#if WXDEBUG > 1
-    wxDebugMsg("wxDC::DestroyClippingRegion: Selecting HRGN %X\n", rgn);
-#endif
     SelectClipRgn((HDC) m_hDC, rgn);
-#if WXDEBUG > 1
-    wxDebugMsg("wxDC::DestroyClippingRegion: Deleting HRGN %X\n", rgn);
-#endif
     DeleteObject(rgn);
-   }
-   m_clipping = FALSE;
+  }
+  m_clipping = FALSE;
 }
 
 bool wxDC::CanDrawBitmap(void) const
@@ -258,7 +255,7 @@ void wxDC::SetPalette(const wxPalette& palette)
     m_oldPalette = 0;
   }
 
-  m_palette = m_palette;
+  m_palette = palette;
 
   if (!m_palette.Ok())
   {
@@ -267,13 +264,10 @@ void wxDC::SetPalette(const wxPalette& palette)
     if (m_oldPalette)
     {
       ::SelectPalette((HDC) m_hDC, (HPALETTE) m_oldPalette, TRUE);
-#if WXDEBUG > 1
-      wxDebugMsg("wxDC::SetPalette: set old palette %X\n", m_oldPalette);
-#endif
       m_oldPalette = 0;
     }
 
-       return;
+    return;
   }
     
   if (m_palette.Ok() && m_palette.GetHPALETTE())
@@ -282,13 +276,6 @@ void wxDC::SetPalette(const wxPalette& palette)
     if (!m_oldPalette)
       m_oldPalette = (WXHPALETTE) oldPal;
 
-#if WXDEBUG > 1
-    wxDebugMsg("wxDC::SetPalette %X: selected palette %X\n", this, m_palette.GetHPALETTE());
-    if (oldPal)
-      wxDebugMsg("wxDC::SetPalette: oldPal was palette %X\n", oldPal);
-    if (m_oldPalette)
-      wxDebugMsg("wxDC::SetPalette: m_oldPalette is palette %X\n", m_oldPalette);
-#endif
     ::RealizePalette((HDC) m_hDC);
   }
 }
@@ -393,7 +380,7 @@ void wxDC::DrawArc(long x1,long y1,long x2,long y2, long xc, long yc)
   double radius = (double)sqrt(dx*dx+dy*dy) ;;
   if (x1==x2 && x2==y2)
   {
-    DrawEllipse(xc,yc,(double)(radius*2.0),(double)(radius*2)) ;
+    DrawEllipse(xc,yc,(long)(radius*2.0),(long)(radius*2.0)) ;
     return ;
   }
 
@@ -512,28 +499,28 @@ void wxDC::DrawRectangle(long x, long y, long width, long height)
   do_pen = m_pen.Ok() && m_pen.GetStyle() != wxTRANSPARENT;
 
   if (do_brush) {
-        HPEN orig_pen = NULL;
+     HPEN orig_pen = NULL;
 
-        if (do_pen || !m_pen.Ok())
-               orig_pen = (HPEN) ::SelectObject((HDC) m_hDC, (HPEN) ::GetStockObject(NULL_PEN));
+     if (do_pen || !m_pen.Ok())
+        orig_pen = (HPEN) ::SelectObject((HDC) m_hDC, (HPEN) ::GetStockObject(NULL_PEN));
 
-        (void)Rectangle((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y),
-                 XLOG2DEV(x2) + 1, YLOG2DEV(y2) + 1);
+     (void)Rectangle((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y),
+          XLOG2DEV(x2) + 1, YLOG2DEV(y2) + 1);
 
-        if (do_pen || !m_pen.Ok())
-               ::SelectObject((HDC) m_hDC , orig_pen);
+     if (do_pen || !m_pen.Ok())
+        ::SelectObject((HDC) m_hDC , orig_pen);
   }
   if (do_pen) {
-        HBRUSH orig_brush = NULL;
+     HBRUSH orig_brush = NULL;
 
-        if (do_brush || !m_brush.Ok())
-               orig_brush = (HBRUSH) ::SelectObject((HDC) m_hDC, (HBRUSH) ::GetStockObject(NULL_BRUSH));
+     if (do_brush || !m_brush.Ok())
+        orig_brush = (HBRUSH) ::SelectObject((HDC) m_hDC, (HBRUSH) ::GetStockObject(NULL_BRUSH));
 
-        (void)Rectangle((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y),
-                 XLOG2DEV(x2), YLOG2DEV(y2));
+     (void)Rectangle((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y),
+          XLOG2DEV(x2), YLOG2DEV(y2));
 
-        if (do_brush || !m_brush.Ok())
-               ::SelectObject((HDC) m_hDC, orig_brush);
+     if (do_brush || !m_brush.Ok())
+        ::SelectObject((HDC) m_hDC, orig_brush);
   }
 #else
   (void)Rectangle((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2), YLOG2DEV(y2));
@@ -601,16 +588,16 @@ void wxDC::DrawEllipticArc(long x,long y,long w,long h,double sa,double ea)
   if (m_signY > 0)
   {
     (void)Pie((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2)+1, YLOG2DEV(y2)+1,
-           rx1, ry1, rx2, ry2);
+        rx1, ry1, rx2, ry2);
   }
   else
   {
     (void)Pie((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y)-1, XLOG2DEV(x2)+1, YLOG2DEV(y2),
-           rx1, ry1-1, rx2, ry2-1);
+        rx1, ry1-1, rx2, ry2-1);
   }
   ::SelectObject((HDC) m_hDC, orig_pen);
   (void)Arc((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2), YLOG2DEV(y2),
-         rx1, ry1, rx2, ry2);
+      rx1, ry1, rx2, ry2);
 
   CalcBoundingBox(x, y);
   CalcBoundingBox(x2, y2);
@@ -618,11 +605,56 @@ void wxDC::DrawEllipticArc(long x,long y,long w,long h,double sa,double ea)
 
 void wxDC::DrawIcon(const wxIcon& icon, long x, long y)
 {
+#if defined(__WIN32__) && !defined(__SC__) && !defined(__TWIN32__)
+  ::DrawIconEx((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), (HICON) icon.GetHICON(),
+      icon.GetWidth(), icon.GetHeight(), 0, 0, DI_NORMAL);
+#else
   ::DrawIcon((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), (HICON) icon.GetHICON());
+#endif
+
   CalcBoundingBox(x, y);
   CalcBoundingBox(x+icon.GetWidth(), y+icon.GetHeight());
 }
 
+void wxDC::DrawBitmap( const wxBitmap &bmp, long x, long y, bool useMask )
+{
+    if (!bmp.Ok())
+        return;
+
+    // If we're not drawing transparently, and not drawing to a printer,
+    // optimize this function to use Windows functions.
+    if (!useMask && !IsKindOf(CLASSINFO(wxPrinterDC)))
+    {
+        HDC cdc = (HDC)m_hDC;
+        HDC memdc = ::CreateCompatibleDC( cdc );
+        HBITMAP hbitmap = (HBITMAP) bmp.GetHBITMAP( );
+        ::SelectObject( memdc, hbitmap );
+        ::BitBlt( cdc, x, y, bmp.GetWidth(), bmp.GetHeight(), memdc, 0, 0, SRCCOPY);
+        ::SelectObject( memdc, 0 );
+        ::DeleteDC( memdc );
+    }
+    else
+    {
+        // Rather than reproduce wxDC::Blit, let's do it at the wxWin API level
+        wxMemoryDC memDC;
+        memDC.SelectObject(bmp);
+
+        /* Not sure if we need this. The mask should leave the
+         * masked areas as per the original background of this DC.
+         */
+/*
+        // There might be transparent areas, so make these
+        // the same colour as this DC
+        memDC.SetBackground(* GetBackground());
+        memDC.Clear();
+*/
+
+        Blit(x, y, bmp.GetWidth(), bmp.GetHeight(), & memDC, 0, 0, wxCOPY, useMask);
+
+        memDC.SelectObject(wxNullBitmap);
+    }
+}
+
 void wxDC::SetFont(const wxFont& the_font)
 {
   // Set the old object temporarily, in case the assignment deletes an object
@@ -644,11 +676,8 @@ void wxDC::SetFont(const wxFont& the_font)
 
   if (m_font.Ok() && m_font.GetResourceHandle())
   {
-#if WXDEBUG > 1
-    wxDebugMsg("wxDC::SetFont: Selecting HFONT %X\n", m_font.GetResourceHandle());
-#endif
     HFONT f = (HFONT) ::SelectObject((HDC) m_hDC, (HFONT) m_font.GetResourceHandle());
-    if (f == NULL)
+    if (f == (HFONT) NULL)
     {
         wxDebugMsg("::SelectObject failed in wxDC::SetFont.");
     }
@@ -724,9 +753,6 @@ void wxDC::DrawText(const wxString& text, long x, long y, bool use16bit)
 #if 0
   if (m_font.Ok() && m_font.GetResourceHandle())
   {
-#if WXDEBUG > 1
-    wxDebugMsg("wxDC::DrawText: Selecting HFONT %X\n", m_font.GetResourceHandle());
-#endif
     HFONT f = (HFONT) ::SelectObject((HDC) m_hDC, (HFONT) m_font.GetResourceHandle());
     if (!m_oldFont)
       m_oldFont = (WXHFONT) f;
@@ -736,7 +762,7 @@ void wxDC::DrawText(const wxString& text, long x, long y, bool use16bit)
   if (m_textForegroundColour.Ok())
     SetTextColor((HDC) m_hDC, m_textForegroundColour.GetPixel() ) ;
 
-  DWORD old_background;
+  DWORD old_background = 0;
   if (m_textBackgroundColour.Ok())
   {
     old_background = SetBkColor((HDC) m_hDC, m_textBackgroundColour.GetPixel() ) ;
@@ -868,16 +894,23 @@ bool wxDC::StartDoc(const wxString& message)
 #else
 #ifdef UNICODE
      ::StartDocW((HDC) m_hDC, &docinfo);
+#else
+#ifdef __TWIN32__
+     ::StartDoc((HDC) m_hDC, &docinfo);
 #else
      ::StartDocA((HDC) m_hDC, &docinfo);
 #endif
 #endif
+#endif
 
+#ifndef __WIN16__
   if (ret <= 0)
   {
     DWORD lastError = GetLastError();
     wxDebugMsg("wxDC::StartDoc failed with error: %d\n", lastError);
   }
+#endif
+
   return (ret > 0);
 }
 
@@ -1051,42 +1084,42 @@ void wxDC::SetDeviceOrigin(long x, long y)
 
 long wxDC::DeviceToLogicalX(long x) const
 {
-       return (long) (((x) - m_deviceOriginX)/(m_logicalScaleX*m_userScaleX*m_signX*m_systemScaleX) - m_logicalOriginX) ;
+    return (long) (((x) - m_deviceOriginX)/(m_logicalScaleX*m_userScaleX*m_signX*m_systemScaleX) - m_logicalOriginX) ;
 }
 
 long wxDC::DeviceToLogicalXRel(long x) const
 {
-       return (long) ((x)/(m_logicalScaleX*m_userScaleX*m_signX*m_systemScaleX)) ;
+    return (long) ((x)/(m_logicalScaleX*m_userScaleX*m_signX*m_systemScaleX)) ;
 }
 
 long wxDC::DeviceToLogicalY(long y) const
 {
-       return (long) (((y) - m_deviceOriginY)/(m_logicalScaleY*m_userScaleY*m_signY*m_systemScaleY) - m_logicalOriginY) ;
+    return (long) (((y) - m_deviceOriginY)/(m_logicalScaleY*m_userScaleY*m_signY*m_systemScaleY) - m_logicalOriginY) ;
 }
 
 long wxDC::DeviceToLogicalYRel(long y) const
 {
-       return (long) ((y)/(m_logicalScaleY*m_userScaleY*m_signY*m_systemScaleY)) ;
+    return (long) ((y)/(m_logicalScaleY*m_userScaleY*m_signY*m_systemScaleY)) ;
 }
 
 long wxDC::LogicalToDeviceX(long x) const
 {
-       return (long) (floor((x) - m_logicalOriginX)*m_logicalScaleX*m_userScaleX*m_signX*m_systemScaleX + m_deviceOriginX) ;
+    return (long) (floor((x) - m_logicalOriginX)*m_logicalScaleX*m_userScaleX*m_signX*m_systemScaleX + m_deviceOriginX) ;
 }
 
 long wxDC::LogicalToDeviceXRel(long x) const
 {
-       return (long) (floor(x)*m_logicalScaleX*m_userScaleX*m_signX*m_systemScaleX) ;
+    return (long) (floor(x)*m_logicalScaleX*m_userScaleX*m_signX*m_systemScaleX) ;
 }
 
 long wxDC::LogicalToDeviceY(long y) const
 {
-       return (long) (floor((y) - m_logicalOriginY)*m_logicalScaleY*m_userScaleY*m_signY*m_systemScaleY + m_deviceOriginY);
+    return (long) (floor((y) - m_logicalOriginY)*m_logicalScaleY*m_userScaleY*m_signY*m_systemScaleY + m_deviceOriginY);
 }
 
 long wxDC::LogicalToDeviceYRel(long y) const
 {
-       return (long) (floor(y)*m_logicalScaleY*m_userScaleY*m_signY*m_systemScaleY) ;
+    return (long) (floor(y)*m_logicalScaleY*m_userScaleY*m_signY*m_systemScaleY) ;
 }
 
 // This group of functions may not do any conversion
@@ -1095,50 +1128,50 @@ long wxDC::LogicalToDeviceYRel(long y) const
 
 long wxDC::ImplDeviceToLogicalX(long x) const
 {
-//     return (m_scaleGDI ?  x :  DeviceToLogicalX(x));
-       return x;
+//  return (m_scaleGDI ?  x :  DeviceToLogicalX(x));
+    return x;
 }
 
 long wxDC::ImplDeviceToLogicalY(long y) const
 {
-//     return (m_scaleGDI ?  y :  DeviceToLogicalY(y));
-       return y;
+//  return (m_scaleGDI ?  y :  DeviceToLogicalY(y));
+    return y;
 }
 
 long wxDC::ImplDeviceToLogicalXRel(long x) const
 {
-//     return (m_scaleGDI ?  x :  DeviceToLogicalXRel(x));
-       return x;
+//  return (m_scaleGDI ?  x :  DeviceToLogicalXRel(x));
+    return x;
 }
 
 long wxDC::ImplDeviceToLogicalYRel(long y) const
 {
-//     return (m_scaleGDI ?  y :  DeviceToLogicalYRel(y));
-       return y;
+//  return (m_scaleGDI ?  y :  DeviceToLogicalYRel(y));
+    return y;
 }
 
 long wxDC::ImplLogicalToDeviceX(long x) const
 {
-//     return (m_scaleGDI ?  (floor(double(x))) :  LogicalToDeviceX(x));
-       return x;
+//  return (m_scaleGDI ?  (floor(double(x))) :  LogicalToDeviceX(x));
+    return x;
 }
 
 long wxDC::ImplLogicalToDeviceY(long y) const
 {
-//     return (m_scaleGDI ?  (floor(double(y))) :  LogicalToDeviceY(y));
-       return y;
+//  return (m_scaleGDI ?  (floor(double(y))) :  LogicalToDeviceY(y));
+    return y;
 }
 
 long wxDC::ImplLogicalToDeviceXRel(long x) const
 {
-//     return (m_scaleGDI ?  (floor(double(x))) :  LogicalToDeviceXRel(x));
-       return x;
+//  return (m_scaleGDI ?  (floor(double(x))) :  LogicalToDeviceXRel(x));
+    return x;
 }
 
 long wxDC::ImplLogicalToDeviceYRel(long y) const
 {
-//     return (m_scaleGDI ?  (floor(double(y))) :  LogicalToDeviceYRel(y));
-       return y;
+//  return (m_scaleGDI ?  (floor(double(y))) :  LogicalToDeviceYRel(y));
+    return y;
 }
 
 bool wxDC::Blit(long xdest, long ydest, long width, long height,
@@ -1155,11 +1188,11 @@ bool wxDC::Blit(long xdest, long ydest, long width, long height,
   COLORREF old_background = ::GetBkColor((HDC)m_hDC);
   if (m_textForegroundColour.Ok())
   {
-       ::SetTextColor((HDC) m_hDC, m_textForegroundColour.GetPixel() ) ;
+    ::SetTextColor((HDC) m_hDC, m_textForegroundColour.GetPixel() ) ;
   }
   if (m_textBackgroundColour.Ok())
   {
-       ::SetBkColor((HDC) m_hDC, m_textBackgroundColour.GetPixel() ) ;
+    ::SetBkColor((HDC) m_hDC, m_textBackgroundColour.GetPixel() ) ;
   }
 
   DWORD dwRop = rop == wxCOPY ? SRCCOPY :
@@ -1176,12 +1209,12 @@ bool wxDC::Blit(long xdest, long ydest, long width, long height,
                 rop == wxSRC_AND ? SRCAND :
                 SRCCOPY;
 
-  bool success;
+  bool success = TRUE;
   if (useMask && source->m_selectedBitmap.Ok() && source->m_selectedBitmap.GetMask())
   {
 
 #if 0 // __WIN32__
-       // Not implemented under Win95 (or maybe a specific device?)
+    // Not implemented under Win95 (or maybe a specific device?)
     if (MaskBlt((HDC) m_hDC, xdest1, ydest1, (int)width, (int)height,
             (HDC) source->m_hDC, xsrc1, ysrc1, (HBITMAP) source->m_selectedBitmap.GetMask()->GetMaskBitmap(),
             0, 0, 0xAACC0020))
@@ -1193,116 +1226,116 @@ bool wxDC::Blit(long xdest, long ydest, long width, long height,
     {
         // Old code
 #if 0
-           HDC dc_mask = CreateCompatibleDC((HDC) source->m_hDC);
-           ::SelectObject(dc_mask, (HBITMAP) source->m_selectedBitmap.GetMask()->GetMaskBitmap());
+        HDC dc_mask = CreateCompatibleDC((HDC) source->m_hDC);
+        ::SelectObject(dc_mask, (HBITMAP) source->m_selectedBitmap.GetMask()->GetMaskBitmap());
         success = (BitBlt((HDC) m_hDC, xdest1, ydest1, (int)width, (int)height,
-                                        dc_mask, xsrc1, ysrc1, 0x00220326 /* NOTSRCAND */) != 0);
+                     dc_mask, xsrc1, ysrc1, 0x00220326 /* NOTSRCAND */) != 0);
         success = (BitBlt((HDC) m_hDC, xdest1, ydest1, (int)width, (int)height,
-                                        (HDC) source->m_hDC, xsrc1, ysrc1, SRCPAINT) != 0);
-           ::SelectObject(dc_mask, 0);
-           ::DeleteDC(dc_mask);
+                     (HDC) source->m_hDC, xsrc1, ysrc1, SRCPAINT) != 0);
+        ::SelectObject(dc_mask, 0);
+        ::DeleteDC(dc_mask);
 #endif
         // New code from Chris Breeze, 15/7/98
-               // Blit bitmap with mask
-
-               if (IsKindOf(CLASSINFO(wxPrinterDC)))
-               {
-                       // If we are printing source colours are screen colours
-                       // not printer colours and so we need copy the bitmap
-                       // pixel by pixel.
-                       RECT rect;
-                       HDC dc_mask = ::CreateCompatibleDC((HDC) source->m_hDC);
-                       HDC dc_src = (HDC) source->m_hDC;
-
-                       ::SelectObject(dc_mask, (HBITMAP) source->m_selectedBitmap.GetMask()->GetMaskBitmap());
-                       for (int x = 0; x < width; x++)
-                       {
-                               for (int y = 0; y < height; y++)
-                               {
-                                       COLORREF cref = ::GetPixel(dc_mask, x, y);
-                                       if (cref)
-                                       {
-                                               HBRUSH brush = ::CreateSolidBrush(::GetPixel(dc_src, x, y));
-                                               rect.left = xdest1 + x; rect.right = rect.left + 1;
-                                               rect.top = ydest1 + y;  rect.bottom = rect.top + 1;
-                                               ::FillRect((HDC) m_hDC, &rect, brush);
-                                               ::DeleteObject(brush);
-                                       }
-                               }
-                       }
-                       ::SelectObject(dc_mask, 0);
-                       ::DeleteDC(dc_mask);
-               }
-               else
-               {
-                       // create a temp buffer bitmap and DCs to access it and the mask
-                       HDC dc_mask = ::CreateCompatibleDC((HDC) source->m_hDC);
-                       HDC dc_buffer = ::CreateCompatibleDC((HDC) m_hDC);
-                       HBITMAP buffer_bmap = ::CreateCompatibleBitmap((HDC) m_hDC, width, height);
-                       ::SelectObject(dc_mask, (HBITMAP) source->m_selectedBitmap.GetMask()->GetMaskBitmap());
-                       ::SelectObject(dc_buffer, buffer_bmap);
-
-                       // copy dest to buffer
-                       ::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
-                               (HDC) m_hDC, xdest1, ydest1, SRCCOPY);
-
-                       // copy src to buffer using selected raster op
-                       ::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
-                               (HDC) source->m_hDC, xsrc1, ysrc1, dwRop);
-
-                       // set masked area in buffer to BLACK (pixel value 0)
-                       COLORREF prevBkCol = ::SetBkColor((HDC) m_hDC, RGB(255, 255, 255));
-                       COLORREF prevCol = ::SetTextColor((HDC) m_hDC, RGB(0, 0, 0));
-                       ::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
-                               dc_mask, xsrc1, ysrc1, SRCAND);
-
-                       // set unmasked area in dest to BLACK
-                       ::SetBkColor((HDC) m_hDC, RGB(0, 0, 0));
-                       ::SetTextColor((HDC) m_hDC, RGB(255, 255, 255));
-                       ::BitBlt((HDC) m_hDC, xdest1, ydest1, (int)width, (int)height,
-                               dc_mask, xsrc1, ysrc1, SRCAND);
-                       ::SetBkColor((HDC) m_hDC, prevBkCol);   // restore colours to original values
-                       ::SetTextColor((HDC) m_hDC, prevCol);
-
-                       // OR buffer to dest
-                       success = (::BitBlt((HDC) m_hDC, xdest1, ydest1, (int)width, (int)height,
-                               dc_buffer, 0, 0, SRCPAINT) != 0);
-
-                       // tidy up temporary DCs and bitmap
-                       ::SelectObject(dc_mask, 0);
-                       ::DeleteDC(dc_mask);
-                       ::SelectObject(dc_buffer, 0);
-                       ::DeleteDC(dc_buffer);
-                       ::DeleteObject(buffer_bmap);
-               }
-       }
+        // Blit bitmap with mask
+
+        if (IsKindOf(CLASSINFO(wxPrinterDC)))
+        {
+            // If we are printing source colours are screen colours
+            // not printer colours and so we need copy the bitmap
+            // pixel by pixel.
+            RECT rect;
+            HDC dc_mask = ::CreateCompatibleDC((HDC) source->m_hDC);
+            HDC dc_src = (HDC) source->m_hDC;
+
+            ::SelectObject(dc_mask, (HBITMAP) source->m_selectedBitmap.GetMask()->GetMaskBitmap());
+            for (int x = 0; x < width; x++)
+            {
+                for (int y = 0; y < height; y++)
+                {
+                    COLORREF cref = ::GetPixel(dc_mask, x, y);
+                    if (cref)
+                    {
+                        HBRUSH brush = ::CreateSolidBrush(::GetPixel(dc_src, x, y));
+                        rect.left = xdest1 + x; rect.right = rect.left + 1;
+                        rect.top = ydest1 + y;  rect.bottom = rect.top + 1;
+                        ::FillRect((HDC) m_hDC, &rect, brush);
+                        ::DeleteObject(brush);
+                    }
+                }
+            }
+            ::SelectObject(dc_mask, 0);
+            ::DeleteDC(dc_mask);
+        }
+        else
+        {
+            // create a temp buffer bitmap and DCs to access it and the mask
+            HDC dc_mask = ::CreateCompatibleDC((HDC) source->m_hDC);
+            HDC dc_buffer = ::CreateCompatibleDC((HDC) m_hDC);
+            HBITMAP buffer_bmap = ::CreateCompatibleBitmap((HDC) m_hDC, width, height);
+            ::SelectObject(dc_mask, (HBITMAP) source->m_selectedBitmap.GetMask()->GetMaskBitmap());
+            ::SelectObject(dc_buffer, buffer_bmap);
+
+            // copy dest to buffer
+            ::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
+                (HDC) m_hDC, xdest1, ydest1, SRCCOPY);
+
+            // copy src to buffer using selected raster op
+            ::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
+                (HDC) source->m_hDC, xsrc1, ysrc1, dwRop);
+
+            // set masked area in buffer to BLACK (pixel value 0)
+            COLORREF prevBkCol = ::SetBkColor((HDC) m_hDC, RGB(255, 255, 255));
+            COLORREF prevCol = ::SetTextColor((HDC) m_hDC, RGB(0, 0, 0));
+            ::BitBlt(dc_buffer, 0, 0, (int)width, (int)height,
+                dc_mask, xsrc1, ysrc1, SRCAND);
+
+            // set unmasked area in dest to BLACK
+            ::SetBkColor((HDC) m_hDC, RGB(0, 0, 0));
+            ::SetTextColor((HDC) m_hDC, RGB(255, 255, 255));
+            ::BitBlt((HDC) m_hDC, xdest1, ydest1, (int)width, (int)height,
+                dc_mask, xsrc1, ysrc1, SRCAND);
+            ::SetBkColor((HDC) m_hDC, prevBkCol);   // restore colours to original values
+            ::SetTextColor((HDC) m_hDC, prevCol);
+
+            // OR buffer to dest
+            success = (::BitBlt((HDC) m_hDC, xdest1, ydest1, (int)width, (int)height,
+                dc_buffer, 0, 0, SRCPAINT) != 0);
+
+            // tidy up temporary DCs and bitmap
+            ::SelectObject(dc_mask, 0);
+            ::DeleteDC(dc_mask);
+            ::SelectObject(dc_buffer, 0);
+            ::DeleteDC(dc_buffer);
+            ::DeleteObject(buffer_bmap);
+        }
+    }
   }
   else
   {
-       if (IsKindOf(CLASSINFO(wxPrinterDC)))
-       {
-      // If we are printing source colours are screen colours
+    if (IsKindOf(CLASSINFO(wxPrinterDC)))
+    {
+      // If we are printing, source colours are screen colours
       // not printer colours and so we need copy the bitmap
       // pixel by pixel.
       HDC dc_src = (HDC) source->m_hDC;
       RECT rect;
       for (int x = 0; x < width; x++)
-         {
+      {
         for (int y = 0; y < height; y++)
-               {
+        {
           HBRUSH brush = ::CreateSolidBrush(::GetPixel(dc_src, x, y));
-          rect.left = xdest1 + x;      rect.right = rect.left + 1;
-          rect.top = ydest1 + y;       rect.bottom = rect.top + 1;
+          rect.left = xdest1 + x;   rect.right = rect.left + 1;
+          rect.top = ydest1 + y;    rect.bottom = rect.top + 1;
           ::FillRect((HDC) m_hDC, &rect, brush);
           ::DeleteObject(brush);
-               }
-         }
+        }
+      }
     }
-       else
-       {
+    else
+    {
       success = (BitBlt((HDC) m_hDC, xdest1, ydest1, (int)width, (int)height, (HDC) source->m_hDC,
                             xsrc1, ysrc1, dwRop) != 0);
-       }
+    }
   }
   ::SetTextColor((HDC)m_hDC, old_textground);
   ::SetBkColor((HDC)m_hDC, old_background);
@@ -1400,8 +1433,8 @@ void wxDC::GetTextExtent(const wxString& string, float *x, float *y,
                      wxFont *theFont, bool use16bit) const
 {
     long x1, y1, descent1, externalLeading1;
-       GetTextExtent(string, & x1, & y1, & descent1, & externalLeading1, theFont, use16bit);
-       *x = x1; *y = y1;
+    GetTextExtent(string, & x1, & y1, & descent1, & externalLeading1, theFont, use16bit);
+    *x = x1; *y = y1;
     if (descent)
         *descent = descent1;
     if (externalLeading)
@@ -1517,10 +1550,10 @@ void wx_draw_open_spline(wxDC *dc, wxSpline *spline)
     while ((node = node->Next()) != NULL)
     {
         p = (wxPoint *)node->Data();
-       x1 = x2;
-       y1 = y2;
-       x2 = p->x;
-       y2 = p->y;
+    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;
@@ -1528,8 +1561,8 @@ void wx_draw_open_spline(wxDC *dc, wxSpline *spline)
 
         wx_quadratic_spline(cx1, cy1, cx2, cy2, cx3, cy3, cx4, cy4);
 
-       cx1 = cx4;
-       cy1 = cy4;
+    cx1 = cx4;
+    cy1 = cy4;
         cx2 = (double)(cx1 + x2) / 2;
         cy2 = (double)(cy1 + y2) / 2;
     }
@@ -1543,23 +1576,23 @@ void wx_draw_open_spline(wxDC *dc, wxSpline *spline)
 
 /********************* CURVES FOR SPLINES *****************************
 
-       The following spline drawing routine is from
+    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.
+    "An Algorithm for High-Speed Curve Generation"
+    by George Merrill Chaikin,
+    Computer Graphics and Image Processing, 3, Academic Press,
+    1974, 346-349.
 
-       and
+    and
 
-       "On Chaikin's Algorithm" by R. F. Riesenfeld,
-       Computer Graphics and Image Processing, 4, Academic Press,
-       1975, 304-310.
+    "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
+#define     half(z1, z2)    ((z1+z2)/2.0)
+#define     THRESHOLD   5
 
 /* iterative version */
 
@@ -1575,16 +1608,16 @@ void wx_quadratic_spline(double a1, double b1, double a2, double b2, double a3,
     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) {
+    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 {
+    } 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);
-       }
+    }
     }
 }
 
@@ -1626,7 +1659,7 @@ 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);
+    return (0);
     wx_stack_top--;
     wx_stack_count--;
     *x1 = wx_stack_top->x1;
@@ -1651,7 +1684,7 @@ static bool wx_spline_add_point(double x, double y)
 
 static void wx_spline_draw_point_array(wxDC *dc)
 {
-  dc->DrawLines(&wx_spline_point_list, (double)0.0, (double)0.0);
+  dc->DrawLines(&wx_spline_point_list, 0, 0);
   wxNode *node = wx_spline_point_list.First();
   while (node)
   {