]> git.saurik.com Git - wxWidgets.git/blobdiff - src/msw/dc.cpp
Added dummy OnIdle to wxWindow in wxGTK; doc tweaks
[wxWidgets.git] / src / msw / dc.cpp
index 7ec41b1008baadf22a8d44d4f417306fae767d34..970a074492b8f92ca23c2885a7dbd0395eeb7389 100644 (file)
@@ -6,7 +6,7 @@
 // Created:     01/02/97
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem
 // Created:     01/02/97
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem
-// Licence:    wxWindows licence
+// Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
@@ -26,6 +26,8 @@
 #include "wx/utils.h"
 #include "wx/dialog.h"
 #include "wx/app.h"
 #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"
 #endif
 
 #include "wx/dcprint.h"
@@ -33,9 +35,8 @@
 
 #include <string.h>
 #include <math.h>
 
 #include <string.h>
 #include <math.h>
-#include <fstream.h>
 
 
-#if USE_COMMON_DIALOGS
+#if wxUSE_COMMON_DIALOGS
 #include <commdlg.h>
 #endif
 
 #include <commdlg.h>
 #endif
 
@@ -68,38 +69,17 @@ IMPLEMENT_ABSTRACT_CLASS(wxDC, wxObject)
 // Default constructor
 wxDC::wxDC(void)
 {
 // Default constructor
 wxDC::wxDC(void)
 {
-  // Stop internal GDI objects being found and pointers kept
-  // after these objects have been deleted.
-  // No - it's OK; these lists won't return them
-  // because their 'visible' status will be FALSE.
-/*
-  wxTheFontList->RemoveFont(&m_font);
-  wxThePenList->RemovePen(&m_pen);
-  wxTheBrushList->RemoveBrush(&m_brush);
-  wxTheBrushList->RemoveBrush(&m_backgroundBrush);
-*/
-
   m_minX = 0; m_minY = 0; m_maxX = 0; m_maxY = 0;
   m_clipping = FALSE;
   m_minX = 0; m_minY = 0; m_maxX = 0; m_maxY = 0;
   m_clipping = FALSE;
-  m_autoSetting = TRUE ;
-//  m_scaleGDI = TRUE;
 
   m_filename = "";
 
   m_filename = "";
-//  m_selectedBitmap = NULL;
   m_canvas = NULL;
   m_canvas = NULL;
-/*
-  cur_dc = NULL ;
-  cur_bk = 0 ;
-  cur_cpen = NULL ;
-  cur_cbrush = NULL ;
-*/
   m_oldBitmap = 0;
   m_oldPen = 0;
   m_oldBrush = 0;
   m_oldFont = 0;
   m_oldPalette = 0;
   m_minX = 0; m_minY = 0; m_maxX = 0; m_maxY = 0;
   m_oldBitmap = 0;
   m_oldPen = 0;
   m_oldBrush = 0;
   m_oldFont = 0;
   m_oldPalette = 0;
   m_minX = 0; m_minY = 0; m_maxX = 0; m_maxY = 0;
-//  m_font = NULL;
   m_logicalOriginX = 0;
   m_logicalOriginY = 0;
   m_deviceOriginX = 0;
   m_logicalOriginX = 0;
   m_logicalOriginY = 0;
   m_deviceOriginX = 0;
@@ -108,10 +88,11 @@ wxDC::wxDC(void)
   m_logicalScaleY = 1.0;
   m_userScaleX = 1.0;
   m_userScaleY = 1.0;
   m_logicalScaleY = 1.0;
   m_userScaleX = 1.0;
   m_userScaleY = 1.0;
+  m_signX = 1;
+  m_signY = 1;
   m_systemScaleX = 1.0;
   m_systemScaleY = 1.0;
   m_systemScaleX = 1.0;
   m_systemScaleY = 1.0;
-  m_mappingMode = MM_TEXT;
-//  m_dontDelete = FALSE;
+  m_mappingMode = wxMM_TEXT;
   m_bOwnsDC = FALSE;
   m_hDC = 0;
   m_clipping = FALSE;
   m_bOwnsDC = FALSE;
   m_hDC = 0;
   m_clipping = FALSE;
@@ -121,7 +102,6 @@ wxDC::wxDC(void)
   m_logicalFunction = -1;
 
   m_backgroundBrush = *wxWHITE_BRUSH;
   m_logicalFunction = -1;
 
   m_backgroundBrush = *wxWHITE_BRUSH;
-//  m_backgroundBrush.UseResource();
 
   m_textForegroundColour = *wxBLACK;
   m_textBackgroundColour = *wxWHITE;
 
   m_textForegroundColour = *wxBLACK;
   m_textBackgroundColour = *wxWHITE;
@@ -144,13 +124,6 @@ wxDC::~wxDC(void)
     }
   }
 
     }
   }
 
-/*
-  if (m_hDC)
-  {
-    SelectOldObjects(m_hDC);
-    ::DeleteDC((HDC) m_hDC);
-  }
-*/
 }
 
 // This will select current objects out of the DC,
 }
 
 // This will select current objects out of the DC,
@@ -158,16 +131,10 @@ wxDC::~wxDC(void)
 // DC.
 void wxDC::SelectOldObjects(WXHDC dc)
 {
 // DC.
 void wxDC::SelectOldObjects(WXHDC dc)
 {
-#if DEBUG > 1
-  wxDebugMsg("wxDC::SelectOldObjects %X\n", this);
-#endif
   if (dc)
   {
     if (m_oldBitmap)
     {
   if (dc)
   {
     if (m_oldBitmap)
     {
-#if DEBUG > 1
-      wxDebugMsg("wxDC::SelectOldObjects: Selecting old HBITMAP %X\n", m_oldBitmap);
-#endif
       ::SelectObject((HDC) dc, (HBITMAP) m_oldBitmap);
       if (m_selectedBitmap.Ok())
       {
       ::SelectObject((HDC) dc, (HBITMAP) m_oldBitmap);
       if (m_selectedBitmap.Ok())
       {
@@ -177,50 +144,25 @@ void wxDC::SelectOldObjects(WXHDC dc)
     m_oldBitmap = 0 ;
     if (m_oldPen)
     {
     m_oldBitmap = 0 ;
     if (m_oldPen)
     {
-#if DEBUG > 1
-      wxDebugMsg("wxDC::SelectOldObjects: Selecting old HPEN %X\n", m_oldPen);
-#endif
       ::SelectObject((HDC) dc, (HPEN) m_oldPen);
     }
     m_oldPen = 0 ;
     if (m_oldBrush)
     {
       ::SelectObject((HDC) dc, (HPEN) m_oldPen);
     }
     m_oldPen = 0 ;
     if (m_oldBrush)
     {
-#if DEBUG > 1
-      wxDebugMsg("wxDC::SelectOldObjects: Selecting old HBRUSH %X\n", m_oldBrush);
-#endif
       ::SelectObject((HDC) dc, (HBRUSH) m_oldBrush);
     }
     m_oldBrush = 0 ;
     if (m_oldFont)
     {
       ::SelectObject((HDC) dc, (HBRUSH) m_oldBrush);
     }
     m_oldBrush = 0 ;
     if (m_oldFont)
     {
-#if DEBUG > 1
-      wxDebugMsg("wxDC::SelectOldObjects: Selecting old HFONT %X\n", m_oldFont);
-#endif
       ::SelectObject((HDC) dc, (HFONT) m_oldFont);
     }
     m_oldFont = 0 ;
     if (m_oldPalette)
     {
       ::SelectObject((HDC) dc, (HFONT) m_oldFont);
     }
     m_oldFont = 0 ;
     if (m_oldPalette)
     {
-#if DEBUG > 1
-      wxDebugMsg("wxDC::SelectOldObjects: Selecting old HPALETTE %X\n", m_oldPalette);
-#endif
       ::SelectPalette((HDC) dc, (HPALETTE) m_oldPalette, TRUE);
     }
       ::SelectPalette((HDC) dc, (HPALETTE) m_oldPalette, TRUE);
     }
-#if DEBUG > 1
-    wxDebugMsg("wxDC::SelectOldObjects: Done.\n");
-#endif
     m_oldPalette = 0 ;
   }
     m_oldPalette = 0 ;
   }
-/*
-  if (m_font.Ok())
-    m_font.ReleaseResource();
-  if (m_pen.Ok())
-    m_pen.ReleaseResource();
-  if (m_brush.Ok())
-    m_brush.ReleaseResource();
-  if (m_backgroundBrush.Ok())
-    m_backgroundBrush.ReleaseResource();
-*/
 
   m_brush = wxNullBrush ;
   m_pen = wxNullPen;
 
   m_brush = wxNullBrush ;
   m_pen = wxNullPen;
@@ -238,11 +180,27 @@ void wxDC::SetClippingRegion(long cx, long cy, long cw, long ch)
   m_clipX2 = (int)(cx + cw);
   m_clipY2 = (int)(cy + ch);
 
   m_clipX2 = (int)(cx + cw);
   m_clipY2 = (int)(cy + ch);
 
-  BeginDrawing();
-
   DoClipping((WXHDC) m_hDC);
   DoClipping((WXHDC) m_hDC);
+}
 
 
-  EndDrawing();
+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)
 }
 
 void wxDC::DoClipping(WXHDC dc)
@@ -251,44 +209,21 @@ void wxDC::DoClipping(WXHDC dc)
   {
     IntersectClipRect((HDC) dc, XLOG2DEV(m_clipX1), YLOG2DEV(m_clipY1),
                           XLOG2DEV(m_clipX2), YLOG2DEV(m_clipY2));
   {
     IntersectClipRect((HDC) dc, XLOG2DEV(m_clipX1), YLOG2DEV(m_clipY1),
                           XLOG2DEV(m_clipX2), YLOG2DEV(m_clipY2));
-#if 0
-    int x_off = 0;
-    int y_off = 0;
-    if (m_canvas)
-    {
-      m_canvas->CalcScrolledPosition(0, 0, &x_off, &y_off);
-    }
-//    HRGN rgn = CreateRectRgn(XLOG2DEV(m_clipX1 + x_off), YLOG2DEV(m_clipY1 + y_off),
-//                          XLOG2DEV(m_clipX2 + x_off), YLOG2DEV(m_clipY2 + y_off));
-
-//    SelectClipRgn(dc, rgn);
-//    DeleteObject(rgn);
-    IntersectClipRect((HDC) dc, XLOG2DEV(m_clipX1 + x_off), YLOG2DEV(m_clipY1 + y_off),
-                          XLOG2DEV(m_clipX2 + x_off), YLOG2DEV(m_clipY2 + y_off));
-#endif
   }
 }
 
 void wxDC::DestroyClippingRegion(void)
 {
   }
 }
 
 void wxDC::DestroyClippingRegion(void)
 {
-  BeginDrawing();
-
   if (m_clipping && m_hDC)
   {
   if (m_clipping && m_hDC)
   {
-//        SelectClipRgn(dc, NULL);
+    // 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);
     HRGN rgn = CreateRectRgn(0, 0, 32000, 32000);
-#if DEBUG > 1
-    wxDebugMsg("wxDC::DestroyClippingRegion: Selecting HRGN %X\n", rgn);
-#endif
     SelectClipRgn((HDC) m_hDC, rgn);
     SelectClipRgn((HDC) m_hDC, rgn);
-#if DEBUG > 1
-    wxDebugMsg("wxDC::DestroyClippingRegion: Deleting HRGN %X\n", rgn);
-#endif
     DeleteObject(rgn);
     DeleteObject(rgn);
-   }
-   m_clipping = FALSE;
-
-   EndDrawing();
+  }
+  m_clipping = FALSE;
 }
 
 bool wxDC::CanDrawBitmap(void) const
 }
 
 bool wxDC::CanDrawBitmap(void) const
@@ -312,9 +247,15 @@ bool wxDC::CanGetTextExtent(void) const
 
 void wxDC::SetPalette(const wxPalette& palette)
 {
 
 void wxDC::SetPalette(const wxPalette& palette)
 {
-  BeginDrawing();
+  // Set the old object temporarily, in case the assignment deletes an object
+  // that's not yet selected out.
+  if (m_oldPalette)
+  {
+    ::SelectPalette((HDC) m_hDC, (HPALETTE) m_oldPalette, TRUE);
+    m_oldPalette = 0;
+  }
 
 
-  m_palette = m_palette;
+  m_palette = palette;
 
   if (!m_palette.Ok())
   {
 
   if (!m_palette.Ok())
   {
@@ -323,13 +264,10 @@ void wxDC::SetPalette(const wxPalette& palette)
     if (m_oldPalette)
     {
       ::SelectPalette((HDC) m_hDC, (HPALETTE) m_oldPalette, TRUE);
     if (m_oldPalette)
     {
       ::SelectPalette((HDC) m_hDC, (HPALETTE) m_oldPalette, TRUE);
-#if DEBUG > 1
-      wxDebugMsg("wxDC::SetPalette: set old palette %X\n", m_oldPalette);
-#endif
       m_oldPalette = 0;
     }
 
       m_oldPalette = 0;
     }
 
-       return;
+    return;
   }
     
   if (m_palette.Ok() && m_palette.GetHPALETTE())
   }
     
   if (m_palette.Ok() && m_palette.GetHPALETTE())
@@ -338,23 +276,12 @@ void wxDC::SetPalette(const wxPalette& palette)
     if (!m_oldPalette)
       m_oldPalette = (WXHPALETTE) oldPal;
 
     if (!m_oldPalette)
       m_oldPalette = (WXHPALETTE) oldPal;
 
-#if DEBUG > 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);
   }
     ::RealizePalette((HDC) m_hDC);
   }
-
-  EndDrawing();
 }
 
 void wxDC::Clear(void)
 {
 }
 
 void wxDC::Clear(void)
 {
-  BeginDrawing();
-
   RECT rect;
   if (m_canvas)
     GetClientRect((HWND) m_canvas->GetHWND(), &rect);
   RECT rect;
   if (m_canvas)
     GetClientRect((HWND) m_canvas->GetHWND(), &rect);
@@ -376,31 +303,16 @@ void wxDC::Clear(void)
   ::SetWindowExtEx((HDC) m_hDC, m_windowExtX, m_windowExtY, NULL);
   ::SetViewportOrgEx((HDC) m_hDC, (int)m_deviceOriginX, (int)m_deviceOriginY, NULL);
   ::SetWindowOrgEx((HDC) m_hDC, (int)m_logicalOriginX, (int)m_logicalOriginY, NULL);
   ::SetWindowExtEx((HDC) m_hDC, m_windowExtX, m_windowExtY, NULL);
   ::SetViewportOrgEx((HDC) m_hDC, (int)m_deviceOriginX, (int)m_deviceOriginY, NULL);
   ::SetWindowOrgEx((HDC) m_hDC, (int)m_logicalOriginX, (int)m_logicalOriginY, NULL);
-
-  EndDrawing();
 }
 
 }
 
-void wxDC::FloodFill(long x, long y, wxColour *col, int style)
+void wxDC::FloodFill(long x, long y, const wxColour& col, int style)
 {
 {
-//  int xx = (int)x;
-//  int yy = (int)y;
-
-  if (m_brush.Ok() && m_autoSetting)
-    SetBrush(m_brush);
-
-//  if (m_canvas)
-//    m_canvas->CalcScrolledPosition((int)x, (int)y, &xx, &yy);
-
-  BeginDrawing();
-
   (void)ExtFloodFill((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y),
   (void)ExtFloodFill((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y),
-                        col->GetPixel(),
+                        col.GetPixel(),
                         style==wxFLOOD_SURFACE?
                           FLOODFILLSURFACE:FLOODFILLBORDER
                         );
 
                         style==wxFLOOD_SURFACE?
                           FLOODFILLSURFACE:FLOODFILLBORDER
                         );
 
-  EndDrawing();
-
   CalcBoundingBox(x, y);
 }
 
   CalcBoundingBox(x, y);
 }
 
@@ -410,12 +322,6 @@ bool wxDC::GetPixel(long x, long y, wxColour *col) const
   // returns TRUE for pixels in the color of the current pen
   // and FALSE for all other pixels colors
   // if col is non-NULL return the color of the pixel
   // returns TRUE for pixels in the color of the current pen
   // and FALSE for all other pixels colors
   // if col is non-NULL return the color of the pixel
-/*
-  int xx1 = (int)x;
-  int yy1 = (int)y;
-  if (m_canvas)
-    m_canvas->CalcScrolledPosition((int)x, (int)y, &xx1, &yy1);
-*/
 
   // get the color of the pixel
   COLORREF pixelcolor = ::GetPixel((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y));
 
   // get the color of the pixel
   COLORREF pixelcolor = ::GetPixel((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y));
@@ -437,31 +343,11 @@ bool wxDC::GetPixel(long x, long y, wxColour *col) const
 
 void wxDC::CrossHair(long x, long y)
 {
 
 void wxDC::CrossHair(long x, long y)
 {
-      if (m_pen.Ok() && m_autoSetting)
-        SetPen(m_pen);
-
       // We suppose that our screen is 2000x2000 max.
       long x1 = x-2000;
       long y1 = y-2000;
       long x2 = x+2000;
       long y2 = y+2000;
       // We suppose that our screen is 2000x2000 max.
       long x1 = x-2000;
       long y1 = y-2000;
       long x2 = x+2000;
       long y2 = y+2000;
-/*
-      long xx1 = x1 ;
-      long yy1 = y1 ;
-      long xx2 = x2 ;
-      long yy2 = y2 ;
-      long xx  = x ;
-      long yy  = y ;
-
-      if (m_canvas)
-      {
-        m_canvas->CalcScrolledPosition(x1, y1, &xx1, &yy1);
-        m_canvas->CalcScrolledPosition(x2, y2, &xx2, &yy2);
-        m_canvas->CalcScrolledPosition(x, y, &xx, &yy);
-      }
-*/
-
-      BeginDrawing();
 
       (void)MoveToEx((HDC) m_hDC, XLOG2DEV(x1), YLOG2DEV(y), NULL);
       (void)LineTo((HDC) m_hDC, XLOG2DEV(x2), YLOG2DEV(y));
 
       (void)MoveToEx((HDC) m_hDC, XLOG2DEV(x1), YLOG2DEV(y), NULL);
       (void)LineTo((HDC) m_hDC, XLOG2DEV(x2), YLOG2DEV(y));
@@ -469,31 +355,12 @@ void wxDC::CrossHair(long x, long y)
       (void)MoveToEx((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y1), NULL);
       (void)LineTo((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y2));
 
       (void)MoveToEx((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y1), NULL);
       (void)LineTo((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y2));
 
-      EndDrawing();
-
       CalcBoundingBox(x1, y1);
       CalcBoundingBox(x2, y2);
 }
 
 void wxDC::DrawLine(long x1, long y1, long x2, long y2)
 {
       CalcBoundingBox(x1, y1);
       CalcBoundingBox(x2, y2);
 }
 
 void wxDC::DrawLine(long x1, long y1, long x2, long y2)
 {
-// BUGBUG - is this necessary? YES YES YES YEs Yes yes ye....
-  if (m_pen.Ok() && m_autoSetting)
-    SetPen(m_pen);
-/*
-  int xx1 = (int)x1;
-  int yy1 = (int)y1;
-  int xx2 = (int)x2;
-  int yy2 = (int)y2;
-  if (m_canvas)
-  {
-    m_canvas->CalcScrolledPosition((int)x1, (int)y1, &xx1, &yy1);
-    m_canvas->CalcScrolledPosition((int)x2, (int)y2, &xx2, &yy2);
-  }
-*/
-
-  BeginDrawing();
-
   (void)MoveToEx((HDC) m_hDC, XLOG2DEV(x1), YLOG2DEV(y1), NULL);
   (void)LineTo((HDC) m_hDC, XLOG2DEV(x2), YLOG2DEV(y2));
 
   (void)MoveToEx((HDC) m_hDC, XLOG2DEV(x1), YLOG2DEV(y1), NULL);
   (void)LineTo((HDC) m_hDC, XLOG2DEV(x2), YLOG2DEV(y2));
 
@@ -502,47 +369,21 @@ void wxDC::DrawLine(long x1, long y1, long x2, long y2)
   (void)LineTo((HDC) m_hDC, XLOG2DEV(x2) + 1, YLOG2DEV(y2));
 #endif
 
   (void)LineTo((HDC) m_hDC, XLOG2DEV(x2) + 1, YLOG2DEV(y2));
 #endif
 
-  EndDrawing();
-
   CalcBoundingBox(x1, y1);
   CalcBoundingBox(x2, y2);
 }
 
   CalcBoundingBox(x1, y1);
   CalcBoundingBox(x2, y2);
 }
 
-void wxDC::DrawArc(long x1,long y1,long x2,long y2,double xc,double yc)
+void wxDC::DrawArc(long x1,long y1,long x2,long y2, long xc, long yc)
 {
 {
-/*
-  int xx1 = (int)x1;
-  int yy1 = (int)y1;
-  int xx2 = (int)x2;
-  int yy2 = (int)y2;
-  int xxc = (int)xc ;
-  int yyc = (int)yc;
-*/
-
   double dx = xc-x1 ;
   double dy = yc-y1 ;
   double radius = (double)sqrt(dx*dx+dy*dy) ;;
   if (x1==x2 && x2==y2)
   {
   double dx = xc-x1 ;
   double dy = yc-y1 ;
   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 ;
   }
 
     return ;
   }
 
-// BUGBUG - is this necessary?
-  if (m_pen.Ok() && m_autoSetting)
-    SetPen(m_pen) ;
-
-/*
-  if (m_canvas)
-  {
-    m_canvas->CalcScrolledPosition((int)x1, (int)y1, &xx1, &yy1);
-    m_canvas->CalcScrolledPosition((int)x2, (int)y2, &xx2, &yy2);
-    m_canvas->CalcScrolledPosition((int)xc, (int)yc, &xxc, &yyc);
-  }
-*/
-
-  BeginDrawing();
-
   long xx1 = XLOG2DEV(x1) ;
   long yy1 = YLOG2DEV(y1) ;
   long xx2 = XLOG2DEV(x2) ;
   long xx1 = XLOG2DEV(x1) ;
   long yy1 = YLOG2DEV(y1) ;
   long xx2 = XLOG2DEV(x2) ;
@@ -558,157 +399,98 @@ void wxDC::DrawArc(long x1,long y1,long x2,long y2,double xc,double yc)
   long yyy2 = (long) (yyc+ray);
   if (m_brush.Ok() && m_brush.GetStyle() !=wxTRANSPARENT)
   {
   long yyy2 = (long) (yyc+ray);
   if (m_brush.Ok() && m_brush.GetStyle() !=wxTRANSPARENT)
   {
-// BUGBUG - is this necessary?
-    if (m_brush.GetStyle()!=wxTRANSPARENT&&m_autoSetting)
-      SetBrush(m_brush) ;
-    Pie((HDC) m_hDC,xxx1,yyy1,xxx2,yyy2,
+    // 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((HDC) m_hDC,xxx1,yyy1,xxx2+1,yyy2+1,
         xx1,yy1,xx2,yy2) ;
   }
   else
     Arc((HDC) m_hDC,xxx1,yyy1,xxx2,yyy2,
         xx1,yy1,xx2,yy2) ;
 
         xx1,yy1,xx2,yy2) ;
   }
   else
     Arc((HDC) m_hDC,xxx1,yyy1,xxx2,yyy2,
         xx1,yy1,xx2,yy2) ;
 
-  EndDrawing();
-
   CalcBoundingBox((xc-radius), (yc-radius));
   CalcBoundingBox((xc+radius), (yc+radius));
 }
 
   CalcBoundingBox((xc-radius), (yc-radius));
   CalcBoundingBox((xc+radius), (yc+radius));
 }
 
-void wxDC::DrawEllipticArc(long WXUNUSED(x),long WXUNUSED(y),long WXUNUSED(w),long WXUNUSED(h),double WXUNUSED(sa),double WXUNUSED(ea))
-{
-  // Not implemented
-}
-
 void wxDC::DrawPoint(long x, long y)
 {
 void wxDC::DrawPoint(long x, long y)
 {
-// BUGBUG - is this necessary?
-  if (m_pen.Ok() && m_autoSetting)
-    SetPen(m_pen) ;
-
-/*
-  int xx1 = (int)x;
-  int yy1 = (int)y;
-  if (m_canvas)
-    m_canvas->CalcScrolledPosition((int)x, (int)y, &xx1, &yy1);
-*/
-
-  BeginDrawing();
-
   COLORREF color = 0x00ffffff;
   if (m_pen.Ok())
   {
   COLORREF color = 0x00ffffff;
   if (m_pen.Ok())
   {
-//    m_pen.RealizeResource();
     color = m_pen.GetColour().GetPixel() ;
   }
 
     color = m_pen.GetColour().GetPixel() ;
   }
 
-/*
-    color = RGB(m_pen->GetColour().Red(),
-                m_pen->GetColour().Green(),
-                m_pen->GetColour().Blue());
-*/
-
   SetPixel((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), color);
 
   SetPixel((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), color);
 
-  EndDrawing();
-
   CalcBoundingBox(x, y);
 }
 
 void wxDC::DrawPolygon(int n, wxPoint points[], long xoffset, long yoffset,int fillStyle)
 {
   CalcBoundingBox(x, y);
 }
 
 void wxDC::DrawPolygon(int n, wxPoint points[], long xoffset, long yoffset,int fillStyle)
 {
-// BUGBUG - is this necessary?
-  if (m_pen.Ok() && m_autoSetting)
-    SetPen(m_pen) ;
-/*
-  int xoffset1 = 0;
-  int yoffset1 = 0;
-
-  if (m_canvas)
-    m_canvas->CalcScrolledPosition(0, 0, &xoffset1, &yoffset1);
-
-  xoffset1 += (int)xoffset; yoffset1 += (int)yoffset;
-*/
-
-  BeginDrawing();
-
-  POINT *cpoints = new POINT[n];
-  int i;
-  for (i = 0; i < n; i++)
+  // Do things less efficiently if we have offsets
+  if (xoffset != 0 || yoffset != 0)
   {
   {
-    cpoints[i].x = (int)(XLOG2DEV(points[i].x));
-    cpoints[i].y = (int)(YLOG2DEV(points[i].y));
+    POINT *cpoints = new POINT[n];
+    int i;
+    for (i = 0; i < n; i++)
+    {
+        cpoints[i].x = (int)(points[i].x + xoffset);
+        cpoints[i].y = (int)(points[i].y + yoffset);
 
 
-    CalcBoundingBox(points[i].x, points[i].y);
+        CalcBoundingBox(cpoints[i].x, cpoints[i].y);
+    }
+    int prev = SetPolyFillMode((HDC) m_hDC,fillStyle==wxODDEVEN_RULE?ALTERNATE:WINDING) ;
+    (void)Polygon((HDC) m_hDC, cpoints, n);
+    SetPolyFillMode((HDC) m_hDC,prev) ;
+    delete[] cpoints;
   }
   }
+  else
+  {
+    int i;
+    for (i = 0; i < n; i++)
+        CalcBoundingBox(points[i].x, points[i].y);
 
 
-  int prev = SetPolyFillMode((HDC) m_hDC,fillStyle==wxODDEVEN_RULE?ALTERNATE:WINDING) ;
-  (void)Polygon((HDC) m_hDC, cpoints, n);
-  SetPolyFillMode((HDC) m_hDC,prev) ;
-
-  EndDrawing();
-
-  delete[] cpoints;
+    int prev = SetPolyFillMode((HDC) m_hDC,fillStyle==wxODDEVEN_RULE?ALTERNATE:WINDING) ;
+    (void)Polygon((HDC) m_hDC, (POINT*) points, n);
+    SetPolyFillMode((HDC) m_hDC,prev) ;
+  }
 }
 
 void wxDC::DrawLines(int n, wxPoint points[], long xoffset, long yoffset)
 {
 }
 
 void wxDC::DrawLines(int n, wxPoint points[], long xoffset, long yoffset)
 {
-// BUGBUG - is this necessary?
-  if (m_pen.Ok() && m_autoSetting)
-    SetPen(m_pen) ;
-
-/*
-  int xoffset1 = 0;
-  int yoffset1 = 0;
-
-  if (m_canvas)
+  // Do things less efficiently if we have offsets
+  if (xoffset != 0 || yoffset != 0)
   {
   {
-    m_canvas->CalcScrolledPosition(0, 0, &xoffset1, &yoffset1);
-  }
-  xoffset1 += (int)xoffset; yoffset1 += (int)yoffset;
-*/
-
-  BeginDrawing();
+    POINT *cpoints = new POINT[n];
+    int i;
+    for (i = 0; i < n; i++)
+    {
+        cpoints[i].x = (int)(points[i].x + xoffset);
+        cpoints[i].y = (int)(points[i].y + yoffset);
 
 
-  POINT *cpoints = new POINT[n];
-  int i;
-  for (i = 0; i < n; i++)
+        CalcBoundingBox(cpoints[i].x, cpoints[i].y);
+    }
+    (void)Polyline((HDC) m_hDC, cpoints, n);
+    delete[] cpoints;
+  }
+  else
   {
   {
-    cpoints[i].x = (int)(XLOG2DEV(points[i].x));
-    cpoints[i].y = (int)(YLOG2DEV(points[i].y));
+    int i;
+    for (i = 0; i < n; i++)
+        CalcBoundingBox(points[i].x, points[i].y);
 
 
-    CalcBoundingBox(points[i].x, points[i].y);
+    (void)Polyline((HDC) m_hDC, (POINT*) points, n);
   }
   }
-
- (void)Polyline((HDC) m_hDC, cpoints, n);
-
-  EndDrawing();
-
-  delete[] cpoints;
 }
 
 void wxDC::DrawRectangle(long x, long y, long width, long height)
 {
 }
 
 void wxDC::DrawRectangle(long x, long y, long width, long height)
 {
-// BUGBUG - is this necessary?
-  if (m_pen.Ok() && m_autoSetting)
-    SetPen(m_pen) ;
-
-/*
-  int x1 = (int)x;
-  int y1 = (int)y;
-  int x2 = (int)(x+width);
-  int y2 = (int)(y+height);
-
-  if (m_canvas)
-  {
-    m_canvas->CalcScrolledPosition((int)x, (int)y, &x1, &y1);
-    m_canvas->CalcScrolledPosition((int)(x+width), (int)(y+height), &x2, &y2);
-  }
-*/
   long x2 = x + width;
   long y2 = y + height;
 
   long x2 = x + width;
   long y2 = y + height;
 
-  BeginDrawing();
-
 /* MATTHEW: [6] new normalization */
 #if WX_STANDARD_GRAPHICS
   bool do_brush, do_pen;
 /* MATTHEW: [6] new normalization */
 #if WX_STANDARD_GRAPHICS
   bool do_brush, do_pen;
@@ -717,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) {
   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 = ::SelectObject((HDC) m_hDC, ::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) {
   }
   if (do_pen) {
-        HBRUSH orig_brush = NULL;
+     HBRUSH orig_brush = NULL;
 
 
-        if (do_brush || !m_brush.Ok())
-               orig_brush = ::SelectObject((HDC) m_hDC, ::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));
   }
 #else
   (void)Rectangle((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2), YLOG2DEV(y2));
@@ -746,16 +528,10 @@ void wxDC::DrawRectangle(long x, long y, long width, long height)
 
   CalcBoundingBox(x, y);
   CalcBoundingBox(x2, y2);
 
   CalcBoundingBox(x, y);
   CalcBoundingBox(x2, y2);
-
-  EndDrawing();
 }
 
 void wxDC::DrawRoundedRectangle(long x, long y, long width, long height, double radius)
 {
 }
 
 void wxDC::DrawRoundedRectangle(long x, long y, long width, long height, double radius)
 {
-// BUGBUG - is this necessary?
-  if (m_pen.Ok() && m_autoSetting)
-    SetPen(m_pen) ;
-
   // 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'
 
@@ -769,87 +545,125 @@ void wxDC::DrawRoundedRectangle(long x, long y, long width, long height, double
     radius = (- radius * smallest);
   }
 
     radius = (- radius * smallest);
   }
 
-/*
-  int x1 = (int)x;
-  int y1 = (int)y;
-
-  if (m_canvas)
-  {
-    m_canvas->CalcScrolledPosition((int)x, (int)y, &x1, &y1);
-    m_canvas->CalcScrolledPosition((int)(x+width), (int)(y+height), &x2, &y2);
-  }
-*/
-
   long x2 = (x+width);
   long y2 = (y+height);
 
   long x2 = (x+width);
   long y2 = (y+height);
 
-  BeginDrawing();
-
   (void)RoundRect((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2),
                       YLOG2DEV(y2), 2*XLOG2DEV(radius), 2*YLOG2DEV(radius));
 
   CalcBoundingBox(x, y);
   CalcBoundingBox(x2, y2);
   (void)RoundRect((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2),
                       YLOG2DEV(y2), 2*XLOG2DEV(radius), 2*YLOG2DEV(radius));
 
   CalcBoundingBox(x, y);
   CalcBoundingBox(x2, y2);
-
-  EndDrawing();
 }
 
 void wxDC::DrawEllipse(long x, long y, long width, long height)
 {
 }
 
 void wxDC::DrawEllipse(long x, long y, long width, long height)
 {
-// BUGBUG - is this necessary?
-  if (m_pen.Ok() && m_autoSetting)
-    SetPen(m_pen) ;
-
-/*
-  int x1 = (int)x;
-  int y1 = (int)y;
-
-  if (m_canvas)
-  {
-    m_canvas->CalcScrolledPosition((int)x, (int)y, &x1, &y1);
-    m_canvas->CalcScrolledPosition((int)(x+width), (int)(y+height), &x2, &y2);
-  }
-*/
-
   long x2 = (x+width);
   long y2 = (y+height);
 
   long x2 = (x+width);
   long y2 = (y+height);
 
-  BeginDrawing();
-
   (void)Ellipse((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2), YLOG2DEV(y2));
 
   (void)Ellipse((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2), YLOG2DEV(y2));
 
-  EndDrawing();
-
   CalcBoundingBox(x, y);
   CalcBoundingBox(x2, y2);
 }
 
   CalcBoundingBox(x, y);
   CalcBoundingBox(x2, y2);
 }
 
-void wxDC::DrawIcon(const wxIcon& icon, long x, long y)
+// Chris Breeze 20/5/98: first implementation of DrawEllipticArc on Windows
+void wxDC::DrawEllipticArc(long x,long y,long w,long h,double sa,double ea)
 {
 {
-/*
-  int x1 = (int)x;
-  int y1 = (int)y;
+  long x2 = (x+w);
+  long y2 = (y+h);
 
 
-  if (m_canvas)
-    m_canvas->CalcScrolledPosition(int)x, (int)y, &x1, &y1);
-*/
+  const double deg2rad = 3.14159265359 / 180.0;
+  int rx1 = XLOG2DEV(x+w/2);
+  int ry1 = YLOG2DEV(y+h/2);
+  int rx2 = rx1;
+  int ry2 = ry1;
+  rx1 += (int)(100.0 * abs(w) * cos(sa * deg2rad));
+  ry1 -= (int)(100.0 * abs(h) * m_signY * sin(sa * deg2rad));
+  rx2 += (int)(100.0 * abs(w) * cos(ea * deg2rad));
+  ry2 -= (int)(100.0 * abs(h) * m_signY * sin(ea * deg2rad));
+
+  // 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((HDC) m_hDC, (HPEN) ::GetStockObject(NULL_PEN));
+  if (m_signY > 0)
+  {
+    (void)Pie((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2)+1, YLOG2DEV(y2)+1,
+        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);
+  }
+  ::SelectObject((HDC) m_hDC, orig_pen);
+  (void)Arc((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), XLOG2DEV(x2), YLOG2DEV(y2),
+      rx1, ry1, rx2, ry2);
 
 
-  BeginDrawing();
+  CalcBoundingBox(x, y);
+  CalcBoundingBox(x2, y2);
+}
 
 
+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());
   ::DrawIcon((HDC) m_hDC, XLOG2DEV(x), YLOG2DEV(y), (HICON) icon.GetHICON());
+#endif
+
   CalcBoundingBox(x, y);
   CalcBoundingBox(x+icon.GetWidth(), y+icon.GetHeight());
   CalcBoundingBox(x, y);
   CalcBoundingBox(x+icon.GetWidth(), y+icon.GetHeight());
+}
 
 
-  EndDrawing();
+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)
 {
 }
 
 void wxDC::SetFont(const wxFont& the_font)
 {
-  // Release the current font from servitude (decrements the usage count)
-//  if (m_font.Ok())
-//    m_font.ReleaseResource();
-
-  BeginDrawing();
+  // Set the old object temporarily, in case the assignment deletes an object
+  // that's not yet selected out.
+  if (m_oldFont)
+  {
+    ::SelectObject((HDC) m_hDC, (HFONT) m_oldFont);
+    m_oldFont = 0;
+  }
 
   m_font = the_font;
 
 
   m_font = the_font;
 
@@ -862,22 +676,25 @@ void wxDC::SetFont(const wxFont& the_font)
 
   if (m_font.Ok() && m_font.GetResourceHandle())
   {
 
   if (m_font.Ok() && m_font.GetResourceHandle())
   {
-#if DEBUG > 1
-    wxDebugMsg("wxDC::SetFont: Selecting HFONT %X\n", m_font.GetResourceHandle());
-#endif
     HFONT f = (HFONT) ::SelectObject((HDC) m_hDC, (HFONT) m_font.GetResourceHandle());
     HFONT f = (HFONT) ::SelectObject((HDC) m_hDC, (HFONT) m_font.GetResourceHandle());
+    if (f == (HFONT) NULL)
+    {
+        wxDebugMsg("::SelectObject failed in wxDC::SetFont.");
+    }
     if (!m_oldFont)
       m_oldFont = (WXHFONT) f;
   }
     if (!m_oldFont)
       m_oldFont = (WXHFONT) f;
   }
-  EndDrawing();
 }
 
 void wxDC::SetPen(const wxPen& pen)
 {
 }
 
 void wxDC::SetPen(const wxPen& pen)
 {
-  BeginDrawing();
-
-//  if (m_pen.Ok())
-//    m_pen.ReleaseResource();
+  // Set the old object temporarily, in case the assignment deletes an object
+  // that's not yet selected out.
+  if (m_oldPen)
+  {
+    ::SelectObject((HDC) m_hDC, (HPEN) m_oldPen);
+    m_oldPen = 0;
+  }
 
   m_pen = pen;
 
 
   m_pen = pen;
 
@@ -890,8 +707,6 @@ void wxDC::SetPen(const wxPen& pen)
 
   if (m_pen.Ok())
   {
 
   if (m_pen.Ok())
   {
-//    m_pen.UseResource();
-//    m_pen.RealizeResource();
     if (m_pen.GetResourceHandle())
     {
       HPEN p = (HPEN) ::SelectObject((HDC) m_hDC, (HPEN)m_pen.GetResourceHandle()) ;
     if (m_pen.GetResourceHandle())
     {
       HPEN p = (HPEN) ::SelectObject((HDC) m_hDC, (HPEN)m_pen.GetResourceHandle()) ;
@@ -899,16 +714,17 @@ void wxDC::SetPen(const wxPen& pen)
         m_oldPen = (WXHPEN) p;
     }
   }
         m_oldPen = (WXHPEN) p;
     }
   }
-
-  EndDrawing();
 }
 
 void wxDC::SetBrush(const wxBrush& brush)
 {
 }
 
 void wxDC::SetBrush(const wxBrush& brush)
 {
-  BeginDrawing();
-
-//  if (m_brush.Ok())
-//    m_brush.ReleaseResource();
+  // Set the old object temporarily, in case the assignment deletes an object
+  // that's not yet selected out.
+  if (m_oldBrush)
+  {
+    ::SelectObject((HDC) m_hDC, (HBRUSH) m_oldBrush);
+    m_oldBrush = 0;
+  }
 
   m_brush = brush;
 
 
   m_brush = brush;
 
@@ -921,46 +737,32 @@ void wxDC::SetBrush(const wxBrush& brush)
 
   if (m_brush.Ok())
   {
 
   if (m_brush.Ok())
   {
-//    m_brush.UseResource();
-//    m_brush.RealizeResource();
-
     if (m_brush.GetResourceHandle())
     {
       HBRUSH b = 0;
     if (m_brush.GetResourceHandle())
     {
       HBRUSH b = 0;
-      b = ::SelectObject((HDC) m_hDC, (HBRUSH)m_brush.GetResourceHandle()) ;
+      b = (HBRUSH) ::SelectObject((HDC) m_hDC, (HBRUSH)m_brush.GetResourceHandle()) ;
       if (!m_oldBrush)
         m_oldBrush = (WXHBRUSH) b;
     }
   }
       if (!m_oldBrush)
         m_oldBrush = (WXHBRUSH) b;
     }
   }
-  EndDrawing();
 }
 
 void wxDC::DrawText(const wxString& text, long x, long y, bool use16bit)
 {
 }
 
 void wxDC::DrawText(const wxString& text, long x, long y, bool use16bit)
 {
-/*
-  int xx1 = (int)x;
-  int yy1 = (int)y;
-
-  if (m_canvas)
-    m_canvas->CalcScrolledPosition((int)x, (int)y, &xx1, &yy1);
-*/
-
-  BeginDrawing();
-
+    // Should be unnecessary: SetFont should have done this already.
+#if 0
   if (m_font.Ok() && m_font.GetResourceHandle())
   {
   if (m_font.Ok() && m_font.GetResourceHandle())
   {
-#if DEBUG > 1
-    wxDebugMsg("wxDC::DrawText: Selecting HFONT %X\n", m_font.GetResourceHandle());
-#endif
-    HFONT f = ::SelectObject((HDC) m_hDC, (HFONT) m_font.GetResourceHandle());
+    HFONT f = (HFONT) ::SelectObject((HDC) m_hDC, (HFONT) m_font.GetResourceHandle());
     if (!m_oldFont)
       m_oldFont = (WXHFONT) f;
   }
     if (!m_oldFont)
       m_oldFont = (WXHFONT) f;
   }
+#endif
 
   if (m_textForegroundColour.Ok())
     SetTextColor((HDC) m_hDC, m_textForegroundColour.GetPixel() ) ;
 
 
   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() ) ;
   if (m_textBackgroundColour.Ok())
   {
     old_background = SetBkColor((HDC) m_hDC, m_textBackgroundColour.GetPixel() ) ;
@@ -981,23 +783,15 @@ void wxDC::DrawText(const wxString& text, long x, long y, bool use16bit)
   long w, h;
   GetTextExtent(text, &w, &h);
   CalcBoundingBox((x + w), (y + h));
   long w, h;
   GetTextExtent(text, &w, &h);
   CalcBoundingBox((x + w), (y + h));
-
-  EndDrawing();
 }
 
 void wxDC::SetBackground(const wxBrush& brush)
 {
 }
 
 void wxDC::SetBackground(const wxBrush& brush)
 {
-//  if (m_backgroundBrush.Ok())
-//    m_backgroundBrush.ReleaseResource();
-
   m_backgroundBrush = brush;
 
   if (!m_backgroundBrush.Ok())
     return;
 
   m_backgroundBrush = brush;
 
   if (!m_backgroundBrush.Ok())
     return;
 
-//  m_backgroundBrush.UseResource();
-//  m_backgroundBrush.RealizeResource() ;
-
   if (m_canvas)
   {
     bool customColours = TRUE;
   if (m_canvas)
   {
     bool customColours = TRUE;
@@ -1008,52 +802,45 @@ void wxDC::SetBackground(const wxBrush& brush)
       
     if (customColours)
     {
       
     if (customColours)
     {
-//      HBRUSH br = (m_backgroundBrush.GetStyle()==wxTRANSPARENT) ?
-//                              GetStockObject(NULL_BRUSH) : (HBRUSH) m_backgroundBrush.GetResourceHandle();
       if (m_backgroundBrush.GetStyle()==wxTRANSPARENT)
       {
         m_canvas->m_backgroundTransparent = TRUE;
       }
       else
       {
       if (m_backgroundBrush.GetStyle()==wxTRANSPARENT)
       {
         m_canvas->m_backgroundTransparent = TRUE;
       }
       else
       {
-        m_canvas->SetBackgroundColour(m_backgroundBrush.GetColour());
+        // New behaviour, 10/2/99: setting the background brush of a DC
+        // doesn't affect the window background colour. However,
+        // I'm leaving in the transparency setting because it's needed by
+        // various controls (e.g. wxStaticText) to determine whether to draw
+        // transparently or not. TODO: maybe this should be a new function
+        // wxWindow::SetTransparency(). Should that apply to the child itself, or the
+        // parent?
+//        m_canvas->SetBackgroundColour(m_backgroundBrush.GetColour());
         m_canvas->m_backgroundTransparent = FALSE;
       }
     }
   }
         m_canvas->m_backgroundTransparent = FALSE;
       }
     }
   }
-  BeginDrawing();
-
   COLORREF new_color = m_backgroundBrush.GetColour().GetPixel() ;
   {
     (void)SetBkColor((HDC) m_hDC, new_color);
   }
   COLORREF new_color = m_backgroundBrush.GetColour().GetPixel() ;
   {
     (void)SetBkColor((HDC) m_hDC, new_color);
   }
-
-  EndDrawing();
 }
 
 void wxDC::SetBackgroundMode(int mode)
 {
   m_backgroundMode = mode;
 
 }
 
 void wxDC::SetBackgroundMode(int mode)
 {
   m_backgroundMode = mode;
 
-  BeginDrawing();
-
   if (m_backgroundMode == wxTRANSPARENT)
     ::SetBkMode((HDC) m_hDC, TRANSPARENT);
   else
     ::SetBkMode((HDC) m_hDC, OPAQUE);
   if (m_backgroundMode == wxTRANSPARENT)
     ::SetBkMode((HDC) m_hDC, TRANSPARENT);
   else
     ::SetBkMode((HDC) m_hDC, OPAQUE);
-
-  EndDrawing();
 }
 
 void wxDC::SetLogicalFunction(int function)
 {
   m_logicalFunction = function;
 
 }
 
 void wxDC::SetLogicalFunction(int function)
 {
   m_logicalFunction = function;
 
-  BeginDrawing();
-
   SetRop((WXHDC) m_hDC);
   SetRop((WXHDC) m_hDC);
-
-  EndDrawing();
 }
 
 void wxDC::SetRop(WXHDC dc)
 }
 
 void wxDC::SetRop(WXHDC dc)
@@ -1091,31 +878,47 @@ bool wxDC::StartDoc(const wxString& message)
   if (!this->IsKindOf(CLASSINFO(wxPrinterDC)))
     return TRUE;
     
   if (!this->IsKindOf(CLASSINFO(wxPrinterDC)))
     return TRUE;
     
-  bool flag = FALSE;
-
   DOCINFO docinfo;
   docinfo.cbSize = sizeof(DOCINFO);
   docinfo.lpszDocName = (const char *)message;
   DOCINFO docinfo;
   docinfo.cbSize = sizeof(DOCINFO);
   docinfo.lpszDocName = (const char *)message;
-  docinfo.lpszOutput = (const char *)m_filename;
+
+  if (m_filename.IsEmpty())
+    docinfo.lpszOutput = NULL;
+  else
+    docinfo.lpszOutput = (const char *)m_filename;
+
 #if defined(__WIN95__)
   docinfo.lpszDatatype = NULL;
   docinfo.fwType = 0;
 #endif
 
 #if defined(__WIN95__)
   docinfo.lpszDatatype = NULL;
   docinfo.fwType = 0;
 #endif
 
-  if (m_hDC) flag = (SP_ERROR !=
+  if (!m_hDC)
+    return FALSE;
+
+  int ret =
 #ifndef __WIN32__
 #ifndef __WIN32__
-     ::StartDoc((HDC) m_hDC, &docinfo));
+     ::StartDoc((HDC) m_hDC, &docinfo);
 #else
 #ifdef UNICODE
 #else
 #ifdef UNICODE
-     ::StartDocW((HDC) m_hDC, &docinfo));
+     ::StartDocW((HDC) m_hDC, &docinfo);
+#else
+#ifdef __TWIN32__
+     ::StartDoc((HDC) m_hDC, &docinfo);
 #else
 #else
-     ::StartDocA((HDC) m_hDC, &docinfo));
+     ::StartDocA((HDC) m_hDC, &docinfo);
+#endif
 #endif
 #endif
 
 #endif
 #endif
 
-  else flag = FALSE;
+#ifndef __WIN16__
+  if (ret <= 0)
+  {
+    DWORD lastError = GetLastError();
+    wxDebugMsg("wxDC::StartDoc failed with error: %d\n", lastError);
+  }
+#endif
 
 
-  return flag;
+  return (ret > 0);
 }
 
 void wxDC::EndDoc(void)
 }
 
 void wxDC::EndDoc(void)
@@ -1172,8 +975,8 @@ void wxDC::GetTextExtent(const wxString& string, long *x, long *y,
   GetTextExtentPoint((HDC) m_hDC, (char *)(const char *) string, strlen((char *)(const char *) string), &sizeRect);
   GetTextMetrics((HDC) m_hDC, &tm);
 
   GetTextExtentPoint((HDC) m_hDC, (char *)(const char *) string, strlen((char *)(const char *) string), &sizeRect);
   GetTextMetrics((HDC) m_hDC, &tm);
 
-  *x = XDEV2LOGREL(sizeRect.cx);
-  *y = YDEV2LOGREL(sizeRect.cy);
+  if (x) *x = XDEV2LOGREL(sizeRect.cx);
+  if (y) *y = YDEV2LOGREL(sizeRect.cy);
   if (descent) *descent = tm.tmDescent;
   if (externalLeading) *externalLeading = tm.tmExternalLeading;
 }
   if (descent) *descent = tm.tmDescent;
   if (externalLeading) *externalLeading = tm.tmExternalLeading;
 }
@@ -1187,8 +990,6 @@ void wxDC::SetMapMode(int mode)
   int mm_width = 0;
   int mm_height = 0;
 
   int mm_width = 0;
   int mm_height = 0;
 
-  BeginDrawing();
-
   pixel_width = GetDeviceCaps((HDC) m_hDC, HORZRES);
   pixel_height = GetDeviceCaps((HDC) m_hDC, VERTRES);
   mm_width = GetDeviceCaps((HDC) m_hDC, HORZSIZE);
   pixel_width = GetDeviceCaps((HDC) m_hDC, HORZRES);
   pixel_height = GetDeviceCaps((HDC) m_hDC, VERTRES);
   mm_width = GetDeviceCaps((HDC) m_hDC, HORZSIZE);
@@ -1196,8 +997,6 @@ void wxDC::SetMapMode(int mode)
 
   if ((pixel_width == 0) || (pixel_height == 0) || (mm_width == 0) || (mm_height == 0))
   {
 
   if ((pixel_width == 0) || (pixel_height == 0) || (mm_width == 0) || (mm_height == 0))
   {
-//    if (!m_hDC && m_canvas)
-//      m_canvas->ReleaseHDC() ;
     return;
   }
 
     return;
   }
 
@@ -1206,32 +1005,32 @@ void wxDC::SetMapMode(int mode)
 
   switch (mode)
   {
 
   switch (mode)
   {
-    case MM_TWIPS:
+    case wxMM_TWIPS:
     {
       m_logicalScaleX = (twips2mm * mm2pixelsX);
       m_logicalScaleY = (twips2mm * mm2pixelsY);
       break;
     }
     {
       m_logicalScaleX = (twips2mm * mm2pixelsX);
       m_logicalScaleY = (twips2mm * mm2pixelsY);
       break;
     }
-    case MM_POINTS:
+    case wxMM_POINTS:
     {
       m_logicalScaleX = (pt2mm * mm2pixelsX);
       m_logicalScaleY = (pt2mm * mm2pixelsY);
       break;
     }
     {
       m_logicalScaleX = (pt2mm * mm2pixelsX);
       m_logicalScaleY = (pt2mm * mm2pixelsY);
       break;
     }
-    case MM_METRIC:
+    case wxMM_METRIC:
     {
       m_logicalScaleX = mm2pixelsX;
       m_logicalScaleY = mm2pixelsY;
       break;
     }
     {
       m_logicalScaleX = mm2pixelsX;
       m_logicalScaleY = mm2pixelsY;
       break;
     }
-    case MM_LOMETRIC:
+    case wxMM_LOMETRIC:
     {
       m_logicalScaleX = (mm2pixelsX/10.0);
       m_logicalScaleY = (mm2pixelsY/10.0);
       break;
     }
     default:
     {
       m_logicalScaleX = (mm2pixelsX/10.0);
       m_logicalScaleY = (mm2pixelsY/10.0);
       break;
     }
     default:
-    case MM_TEXT:
+    case wxMM_TEXT:
     {
       m_logicalScaleX = 1.0;
       m_logicalScaleY = 1.0;
     {
       m_logicalScaleX = 1.0;
       m_logicalScaleY = 1.0;
@@ -1248,8 +1047,6 @@ void wxDC::SetMapMode(int mode)
   ::SetWindowExtEx((HDC) m_hDC, m_windowExtX, m_windowExtY, NULL);
   ::SetViewportOrgEx((HDC) m_hDC, (int)m_deviceOriginX, (int)m_deviceOriginY, NULL);
   ::SetWindowOrgEx((HDC) m_hDC, (int)m_logicalOriginX, (int)m_logicalOriginY, NULL);
   ::SetWindowExtEx((HDC) m_hDC, m_windowExtX, m_windowExtY, NULL);
   ::SetViewportOrgEx((HDC) m_hDC, (int)m_deviceOriginX, (int)m_deviceOriginY, NULL);
   ::SetWindowOrgEx((HDC) m_hDC, (int)m_logicalOriginX, (int)m_logicalOriginY, NULL);
-
-  EndDrawing();
 }
 
 void wxDC::SetUserScale(double x, double y)
 }
 
 void wxDC::SetUserScale(double x, double y)
@@ -1260,6 +1057,14 @@ void wxDC::SetUserScale(double x, double y)
   SetMapMode(m_mappingMode);
 }
 
   SetMapMode(m_mappingMode);
 }
 
+void wxDC::SetAxisOrientation(bool xLeftRight, bool yBottomUp)
+{
+  m_signX = xLeftRight ? 1 : -1;
+  m_signY = yBottomUp ? -1 : 1;
+
+  SetMapMode(m_mappingMode);
+}
+
 void wxDC::SetSystemScale(double x, double y)
 {
   m_systemScaleX = x;
 void wxDC::SetSystemScale(double x, double y)
 {
   m_systemScaleX = x;
@@ -1273,11 +1078,7 @@ void wxDC::SetLogicalOrigin(long x, long y)
   m_logicalOriginX = x;
   m_logicalOriginY = y;
 
   m_logicalOriginX = x;
   m_logicalOriginY = y;
 
-  BeginDrawing();
-
   ::SetWindowOrgEx((HDC) m_hDC, (int)m_logicalOriginX, (int)m_logicalOriginY, NULL);
   ::SetWindowOrgEx((HDC) m_hDC, (int)m_logicalOriginX, (int)m_logicalOriginY, NULL);
-
-  EndDrawing();
 }
 
 void wxDC::SetDeviceOrigin(long x, long y)
 }
 
 void wxDC::SetDeviceOrigin(long x, long y)
@@ -1285,51 +1086,47 @@ void wxDC::SetDeviceOrigin(long x, long y)
   m_deviceOriginX = x;
   m_deviceOriginY = y;
 
   m_deviceOriginX = x;
   m_deviceOriginY = y;
 
-  BeginDrawing();
-
   ::SetViewportOrgEx((HDC) m_hDC, (int)m_deviceOriginX, (int)m_deviceOriginY, NULL);
   ::SetViewportOrgEx((HDC) m_hDC, (int)m_deviceOriginX, (int)m_deviceOriginY, NULL);
-
-  EndDrawing();
 }
 
 long wxDC::DeviceToLogicalX(long x) const
 {
 }
 
 long wxDC::DeviceToLogicalX(long x) const
 {
-       return (long) (((x) - m_deviceOriginX)/(m_logicalScaleX*m_userScaleX*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
 {
 }
 
 long wxDC::DeviceToLogicalXRel(long x) const
 {
-       return (long) ((x)/(m_logicalScaleX*m_userScaleX*m_systemScaleX)) ;
+    return (long) ((x)/(m_logicalScaleX*m_userScaleX*m_signX*m_systemScaleX)) ;
 }
 
 long wxDC::DeviceToLogicalY(long y) const
 {
 }
 
 long wxDC::DeviceToLogicalY(long y) const
 {
-       return (long) (((y) - m_deviceOriginY)/(m_logicalScaleY*m_userScaleY*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
 {
 }
 
 long wxDC::DeviceToLogicalYRel(long y) const
 {
-       return (long) ((y)/(m_logicalScaleY*m_userScaleY*m_systemScaleY)) ;
+    return (long) ((y)/(m_logicalScaleY*m_userScaleY*m_signY*m_systemScaleY)) ;
 }
 
 long wxDC::LogicalToDeviceX(long x) const
 {
 }
 
 long wxDC::LogicalToDeviceX(long x) const
 {
-       return (long) (floor((x) - m_logicalOriginX)*m_logicalScaleX*m_userScaleX*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
 {
 }
 
 long wxDC::LogicalToDeviceXRel(long x) const
 {
-       return (long) (floor(x)*m_logicalScaleX*m_userScaleX*m_systemScaleX) ;
+    return (long) (floor(x)*m_logicalScaleX*m_userScaleX*m_signX*m_systemScaleX) ;
 }
 
 long wxDC::LogicalToDeviceY(long y) const
 {
 }
 
 long wxDC::LogicalToDeviceY(long y) const
 {
-       return (long) (floor((y) - m_logicalOriginY)*m_logicalScaleY*m_userScaleY*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
 {
 }
 
 long wxDC::LogicalToDeviceYRel(long y) const
 {
-       return (long) (floor(y)*m_logicalScaleY*m_userScaleY*m_systemScaleY) ;
+    return (long) (floor(y)*m_logicalScaleY*m_userScaleY*m_signY*m_systemScaleY) ;
 }
 
 // This group of functions may not do any conversion
 }
 
 // This group of functions may not do any conversion
@@ -1338,63 +1135,73 @@ long wxDC::LogicalToDeviceYRel(long y) const
 
 long wxDC::ImplDeviceToLogicalX(long x) 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
 {
 }
 
 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
 {
 }
 
 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
 {
 }
 
 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
 {
 }
 
 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
 {
 }
 
 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
 {
 }
 
 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
 {
 }
 
 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,
                 wxDC *source, long xsrc, long ysrc, int rop, bool useMask)
 {
 }
 
 bool wxDC::Blit(long xdest, long ydest, long width, long height,
                 wxDC *source, long xsrc, long ysrc, int rop, bool useMask)
 {
-  BeginDrawing();
-  source->BeginDrawing();
-
   long xdest1 = xdest;
   long ydest1 = ydest;
   long xsrc1 = xsrc;
   long ysrc1 = ysrc;
 
   long xdest1 = xdest;
   long ydest1 = ydest;
   long xsrc1 = xsrc;
   long ysrc1 = ysrc;
 
+  // Chris Breeze 18/5/98: use text foreground/background colours
+  // when blitting from 1-bit bitmaps
+  COLORREF old_textground = ::GetTextColor((HDC)m_hDC);
+  COLORREF old_background = ::GetBkColor((HDC)m_hDC);
+  if (m_textForegroundColour.Ok())
+  {
+    ::SetTextColor((HDC) m_hDC, m_textForegroundColour.GetPixel() ) ;
+  }
+  if (m_textBackgroundColour.Ok())
+  {
+    ::SetBkColor((HDC) m_hDC, m_textBackgroundColour.GetPixel() ) ;
+  }
+
   DWORD dwRop = rop == wxCOPY ? SRCCOPY :
                 rop == wxCLEAR ? WHITENESS :
                 rop == wxSET ? BLACKNESS :
   DWORD dwRop = rop == wxCOPY ? SRCCOPY :
                 rop == wxCLEAR ? WHITENESS :
                 rop == wxSET ? BLACKNESS :
@@ -1409,12 +1216,12 @@ bool wxDC::Blit(long xdest, long ydest, long width, long height,
                 rop == wxSRC_AND ? SRCAND :
                 SRCCOPY;
 
                 rop == wxSRC_AND ? SRCAND :
                 SRCCOPY;
 
-  bool success;
+  bool success = TRUE;
   if (useMask && source->m_selectedBitmap.Ok() && source->m_selectedBitmap.GetMask())
   {
 
 #if 0 // __WIN32__
   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))
     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))
@@ -1426,65 +1233,119 @@ bool wxDC::Blit(long xdest, long ydest, long width, long height,
     {
         // Old code
 #if 0
     {
         // 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,
         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,
         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
 #endif
-        // New code from Chris Breeze, 8/5/98
-
-               // 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);
+        // 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);
+        }
     }
   }
   else
   {
     }
   }
   else
   {
+    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;
+          ::FillRect((HDC) m_hDC, &rect, brush);
+          ::DeleteObject(brush);
+        }
+      }
+    }
+    else
+    {
       success = (BitBlt((HDC) m_hDC, xdest1, ydest1, (int)width, (int)height, (HDC) source->m_hDC,
                             xsrc1, ysrc1, dwRop) != 0);
       success = (BitBlt((HDC) m_hDC, xdest1, ydest1, (int)width, (int)height, (HDC) source->m_hDC,
                             xsrc1, ysrc1, dwRop) != 0);
+    }
   }
   }
-  source->EndDrawing();
-  EndDrawing();
+  ::SetTextColor((HDC)m_hDC, old_textground);
+  ::SetBkColor((HDC)m_hDC, old_background);
 
   return success;
 }
 
   return success;
 }
@@ -1505,20 +1366,6 @@ void wxDC::GetSizeMM(long *width, long *height) const
   *height = h;
 }
 
   *height = h;
 }
 
-/*
-#if USE_SPLINES
-# if USE_XFIG_SPLINE_CODE
-#  include "../common/xfspline.inc"
-# else
-#  include "../common/wxspline.inc"
-# endif
-#endif // USE_SPLINES
-*/
-
-#if USE_SPLINES
-#include "xfspline.inc"
-#endif // USE_SPLINES
-
 void wxDC::DrawPolygon(wxList *list, long xoffset, long yoffset,int fillStyle)
 {
   int n = list->Number();
 void wxDC::DrawPolygon(wxList *list, long xoffset, long yoffset,int fillStyle)
 {
   int n = list->Number();
@@ -1559,7 +1406,56 @@ void wxDC::SetTextBackground(const wxColour& colour)
     m_textBackgroundColour = colour;
 }
 
     m_textBackgroundColour = colour;
 }
 
-#if USE_SPLINES
+// For use by wxWindows only, unless custom units are required.
+void wxDC::SetLogicalScale(double x, double y)
+{
+  m_logicalScaleX = x;
+  m_logicalScaleY = y;
+}
+
+void wxDC::CalcBoundingBox(long x, long y)
+{
+  if (x < m_minX) m_minX = x;
+  if (y < m_minY) m_minY = y;
+  if (x > m_maxX) m_maxX = x;
+  if (y > m_maxY) m_maxY = y;
+}
+
+void wxDC::GetClippingBox(long *x,long *y,long *w,long *h) const
+{
+  if (m_clipping)
+  {
+    *x = m_clipX1 ;
+    *y = m_clipY1 ;
+    *w = (m_clipX2 - m_clipX1) ;
+    *h = (m_clipY2 - m_clipY1) ;
+  }
+  else
+   *x = *y = *w = *h = 0 ;
+}
+
+#if WXWIN_COMPATIBILITY
+void wxDC::GetTextExtent(const wxString& string, float *x, float *y,
+                     float *descent, float *externalLeading,
+                     wxFont *theFont, bool use16bit) const
+{
+    long 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
+
+int wxDC::GetDepth(void) const
+{
+  return (int) ::GetDeviceCaps((HDC) m_hDC,BITSPIXEL);
+}
+
+#if wxUSE_SPLINES
+
 // Make a 3-point spline
 void wxDC::DrawSpline(long x1, long y1, long x2, long y2, long x3, long y3)
 {
 // Make a 3-point spline
 void wxDC::DrawSpline(long x1, long y1, long x2, long y2, long x3, long y3)
 {
@@ -1584,55 +1480,248 @@ void wxDC::DrawSpline(long x1, long y1, long x2, long y2, long x3, long y3)
     delete p;
   }
   delete point_list;
     delete p;
   }
   delete point_list;
-/*
-  wxSpline spline(point_list);
+}
+
+////#define     wx_round(a)    (int)((a)+.5)
+//#define     wx_round(a)    (a)
+
+class wxSpline: public wxObject
+{
+ public:
+  int type;
+  wxList *points;
+
+  wxSpline(wxList *list);
+  void DeletePoints(void);
+
+  // Doesn't delete points
+  ~wxSpline(void);
+};
+
+void wxDC::DrawSpline(int n, wxPoint points[])
+{
+  wxList list;
+  int i;
+  for (i =0; i < n; i++)
+    list.Append((wxObject*)&points[i]);
+  DrawSpline((wxList *)&list);
+}
+
+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(void);
+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);
+
+void wxDC::DrawSpline(wxList *list)
+{
+  wxSpline spline(list);
 
   wx_draw_open_spline(this, &spline);
 
   wx_draw_open_spline(this, &spline);
-  spline.DeletePoints();
- */
 }
 }
-#endif
 
 
-// For use by wxWindows only, unless custom units are required.
-void wxDC::SetLogicalScale(double x, double y)
+
+wxList wx_spline_point_list;
+
+void wx_draw_open_spline(wxDC *dc, wxSpline *spline)
 {
 {
-  m_logicalScaleX = x;
-  m_logicalScaleY = y;
+    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);
+
 }
 
 }
 
-void wxDC::CalcBoundingBox(long x, long y)
+/********************* 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)
 {
 {
-  if (x < m_minX) m_minX = x;
-  if (y < m_minY) m_minY = y;
-  if (x > m_maxX) m_maxX = x;
-  if (y > m_maxY) m_maxY = y;
+    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);
+
+    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);
+    }
+    }
 }
 
 }
 
-void wxDC::GetClippingBox(long *x,long *y,long *w,long *h) const
+
+/* utilities used by spline drawing routines */
+
+
+typedef struct wx_spline_stack_struct {
+    double           x1, y1, x2, y2, x3, y3, x4, y4;
+}
+                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(void)
 {
 {
-  if (m_clipping)
+    wx_stack_top = wx_spline_stack;
+    wx_stack_count = 0;
+}
+
+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++;
+}
+
+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);
+}
+
+static bool wx_spline_add_point(double x, double y)
+{
+  wxPoint *point = new wxPoint ;
+  point->x = (int) x;
+  point->y = (int) y;
+  wx_spline_point_list.Append((wxObject*)point);
+  return TRUE;
+}
+
+static void wx_spline_draw_point_array(wxDC *dc)
+{
+  dc->DrawLines(&wx_spline_point_list, 0, 0);
+  wxNode *node = wx_spline_point_list.First();
+  while (node)
   {
   {
-    *x = m_clipX1 ;
-    *y = m_clipY1 ;
-    *w = (m_clipX2 - m_clipX1) ;
-    *h = (m_clipY2 - m_clipY1) ;
+    wxPoint *point = (wxPoint *)node->Data();
+    delete point;
+    delete node;
+    node = wx_spline_point_list.First();
   }
   }
-  else
-   *x = *y = *w = *h = 0 ;
 }
 
 }
 
-#if WXWIN_COMPATIBILITY
-void wxDC::GetTextExtent(const wxString& string, float *x, float *y,
-                     float *descent, float *externalLeading,
-                     wxFont *theFont, bool use16bit) const
+wxSpline::wxSpline(wxList *list)
 {
 {
-    long x1, y1, descent1, externalLeading1;
-       GetTextExtent(string, & x1, & y1, & descent1, & externalLeading1, theFont, use16bit);
-       *x = x1; *y = y1;
-    if (descent)
-        *descent = descent1;
-    if (externalLeading)
-        *externalLeading = externalLeading1;
+  points = list;
 }
 }
-#endif
+
+wxSpline::~wxSpline(void)
+{
+}
+
+void wxSpline::DeletePoints(void)
+{
+  for(wxNode *node = points->First(); node; node = points->First())
+  {
+    wxPoint *point = (wxPoint *)node->Data();
+    delete point;
+    delete node;
+  }
+  delete points;
+}
+
+
+#endif // wxUSE_SPLINES