fixing osx_carbon

[wxWidgets.git] / src / os2 / dc.cpp

diff --git a/src/os2/dc.cpp b/src/os2/dc.cpp
index dfdcab5f4cbc6468e83ee28f1525b522ce83f196..14d766ba38c74bf0bc916d3f19a4d426d3f0ea0b 100644 (file)
--- a/src/os2/dc.cpp
+++ b/src/os2/dc.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        dc.cpp
+// Name:        src/os2/dc.cpp

 // Purpose:     wxDC class
 // Author:      David Webster
 // Modified by:
 // Purpose:     wxDC class
 // Author:      David Webster
 // Modified by:


@@ -23,38 +23,36 @@
     #include "wx/log.h"
     #include "wx/icon.h"
     #include "wx/msgdlg.h"
     #include "wx/log.h"
     #include "wx/icon.h"
     #include "wx/msgdlg.h"

-#if wxUSE_STATUSBAR
+    #include "wx/dcprint.h"

     #include "wx/statusbr.h"
     #include "wx/statusbr.h"

+    #include "wx/module.h"

 #endif
 #endif

-#endif
-
-#include "wx/module.h"
-#include "wx/dcprint.h"

 
 #include <string.h>
 
 #include <string.h>

-#include <math.h>

 
 

+#include "wx/os2/dc.h"
+#include "wx/os2/dcclient.h"

 #include "wx/os2/private.h"
 
 #include "wx/os2/private.h"
 

-    IMPLEMENT_ABSTRACT_CLASS(wxDC, wxObject)
+IMPLEMENT_ABSTRACT_CLASS(wxPMDCImpl, wxDCImpl)

 
 //
 
 //

-// wxWindows uses the Microsoft convention that the origin is the UPPER left.
+// wxWidgets uses the Microsoft convention that the origin is the UPPER left.

 // Native OS/2 however in the GPI and PM define the origin as the LOWER left.
 // Native OS/2 however in the GPI and PM define the origin as the LOWER left.

-// In order to map OS/2 GPI/PM y coordinates to wxWindows coordinates we must
+// In order to map OS/2 GPI/PM y coordinates to wxWidgets coordinates we must

 // perform the following transformation:
 //
 // Parent object height:     POBJHEIGHT
 // Desried origin:           WXORIGINY
 // Object to place's height: OBJHEIGHT
 //
 // perform the following transformation:
 //
 // Parent object height:     POBJHEIGHT
 // Desried origin:           WXORIGINY
 // Object to place's height: OBJHEIGHT
 //

-// To get the OS2 position from the wxWindows one:
+// To get the OS2 position from the wxWidgets one:

 //
 // OS2Y = POBJHEIGHT - (WXORIGINY + OBJHEIGHT)
 //
 // For OS/2 wxDC's we will always determine m_vRclPaint as the size of the
 // OS/2 Presentation Space associated with the device context.  y is the
 //
 // OS2Y = POBJHEIGHT - (WXORIGINY + OBJHEIGHT)
 //
 // For OS/2 wxDC's we will always determine m_vRclPaint as the size of the
 // OS/2 Presentation Space associated with the device context.  y is the

-// desired application's y coordinate of the origin in wxWindows space.
+// desired application's y coordinate of the origin in wxWidgets space.

 // objy is the height of the object we are going to draw.
 //
 #define OS2Y(y, objy) ((m_vRclPaint.yTop - m_vRclPaint.yBottom) - (y + objy))
 // objy is the height of the object we are going to draw.
 //
 #define OS2Y(y, objy) ((m_vRclPaint.yTop - m_vRclPaint.yBottom) - (y + objy))


@@ -68,11 +66,6 @@ static const int VIEWPORT_EXTENT = 1000;
 static const int MM_POINTS = 9;
 static const int MM_METRIC = 10;
 
 static const int MM_POINTS = 9;
 static const int MM_METRIC = 10;
 

-// usually this is defined in math.h
-#ifndef M_PI
-    static const double M_PI = 3.14159265358979323846;
-#endif // M_PI
-

 // ---------------------------------------------------------------------------
 // private functions
 // ---------------------------------------------------------------------------
 // ---------------------------------------------------------------------------
 // private functions
 // ---------------------------------------------------------------------------


@@ -165,8 +158,8 @@ int SetBkMode(
 // wxDCCacheEntry
 // ---------------------------------------------------------------------------
 
 // wxDCCacheEntry
 // ---------------------------------------------------------------------------
 

-wxList wxDC::m_svBitmapCache;
-wxList wxDC::m_svDCCache;
+wxList wxPMDCImpl::m_svBitmapCache;
+wxList wxPMDCImpl::m_svDCCache;

 
 wxDCCacheEntry::wxDCCacheEntry(
   WXHBITMAP                         hBitmap
 
 wxDCCacheEntry::wxDCCacheEntry(
   WXHBITMAP                         hBitmap


@@ -202,7 +195,7 @@ wxDCCacheEntry::~wxDCCacheEntry()
         ::GpiDestroyPS(m_hPS);
 } // end of wxDCCacheEntry::~wxDCCacheEntry
 
         ::GpiDestroyPS(m_hPS);
 } // end of wxDCCacheEntry::~wxDCCacheEntry
 

-wxDCCacheEntry* wxDC::FindBitmapInCache(
+wxDCCacheEntry* wxPMDCImpl::FindBitmapInCache(

   HPS                               hPS
 , int                               nWidth
 , int                               nHeight
   HPS                               hPS
 , int                               nWidth
 , int                               nHeight


@@ -227,7 +220,7 @@ wxDCCacheEntry* wxDC::FindBitmapInCache(
                 vBmpHdr.cx        = nWidth;
                 vBmpHdr.cy        = nHeight;
                 vBmpHdr.cPlanes   = 1;
                 vBmpHdr.cx        = nWidth;
                 vBmpHdr.cy        = nHeight;
                 vBmpHdr.cPlanes   = 1;

-                vBmpHdr.cBitCount = nDepth;
+                vBmpHdr.cBitCount = (USHORT)nDepth;

 
                 pEntry->m_hBitmap = (WXHBITMAP) ::GpiCreateBitmap( hPS
                                                                   ,&vBmpHdr
 
                 pEntry->m_hBitmap = (WXHBITMAP) ::GpiCreateBitmap( hPS
                                                                   ,&vBmpHdr


@@ -250,7 +243,7 @@ wxDCCacheEntry* wxDC::FindBitmapInCache(
     vBmpHdr.cx        = nWidth;
     vBmpHdr.cy        = nHeight;
     vBmpHdr.cPlanes   = 1;
     vBmpHdr.cx        = nWidth;
     vBmpHdr.cy        = nHeight;
     vBmpHdr.cPlanes   = 1;

-    vBmpHdr.cBitCount = nDepth;
+    vBmpHdr.cBitCount = (USHORT)nDepth;

 
     WXHBITMAP                       hBitmap = (WXHBITMAP) ::GpiCreateBitmap( hPS
                                                                             ,&vBmpHdr
 
     WXHBITMAP                       hBitmap = (WXHBITMAP) ::GpiCreateBitmap( hPS
                                                                             ,&vBmpHdr


@@ -269,7 +262,7 @@ wxDCCacheEntry* wxDC::FindBitmapInCache(
     return pEntry;
 } // end of FindBitmapInCache
 
     return pEntry;
 } // end of FindBitmapInCache
 

-wxDCCacheEntry* wxDC::FindDCInCache(
+wxDCCacheEntry* wxPMDCImpl::FindDCInCache(

   wxDCCacheEntry*                   pNotThis
 , HPS                               hPS
 )
   wxDCCacheEntry*                   pNotThis
 , HPS                               hPS
 )


@@ -298,38 +291,38 @@ wxDCCacheEntry* wxDC::FindDCInCache(
                                                                );
     AddToDCCache(pEntry);
     return pEntry;
                                                                );
     AddToDCCache(pEntry);
     return pEntry;

-} // end of wxDC::FindDCInCache
+} // end of wxPMDCImpl::FindDCInCache

 
 

-void wxDC::AddToBitmapCache(
+void wxPMDCImpl::AddToBitmapCache(

   wxDCCacheEntry*                   pEntry
 )
 {
     m_svBitmapCache.Append(pEntry);
   wxDCCacheEntry*                   pEntry
 )
 {
     m_svBitmapCache.Append(pEntry);

-} // end of wxDC::AddToBitmapCache
+} // end of wxPMDCImpl::AddToBitmapCache

 
 

-void wxDC::AddToDCCache(
+void wxPMDCImpl::AddToDCCache(

   wxDCCacheEntry*                   pEntry
 )
 {
     m_svDCCache.Append(pEntry);
   wxDCCacheEntry*                   pEntry
 )
 {
     m_svDCCache.Append(pEntry);

-} // end of wxDC::AddToDCCache
+} // end of wxPMDCImpl::AddToDCCache

 
 

-void wxDC::ClearCache()
+void wxPMDCImpl::ClearCache()

 {
 {

-    m_svBitmapCache.DeleteContents(TRUE);
+    m_svBitmapCache.DeleteContents(true);

     m_svBitmapCache.Clear();
     m_svBitmapCache.Clear();

-    m_svBitmapCache.DeleteContents(FALSE);
-    m_svDCCache.DeleteContents(TRUE);
+    m_svBitmapCache.DeleteContents(false);
+    m_svDCCache.DeleteContents(true);

     m_svDCCache.Clear();
     m_svDCCache.Clear();

-    m_svDCCache.DeleteContents(FALSE);
-} // end of wxDC::ClearCache
+    m_svDCCache.DeleteContents(false);
+} // end of wxPMDCImpl::ClearCache

 
 // Clean up cache at app exit
 class wxDCModule : public wxModule
 {
 public:
 
 // Clean up cache at app exit
 class wxDCModule : public wxModule
 {
 public:

-    virtual bool OnInit() { return TRUE; }
-    virtual void OnExit() { wxDC::ClearCache(); }
+    virtual bool OnInit() { return true; }
+    virtual void OnExit() { wxPMDCImpl::ClearCache(); }

 
 private:
     DECLARE_DYNAMIC_CLASS(wxDCModule)
 
 private:
     DECLARE_DYNAMIC_CLASS(wxDCModule)


@@ -343,31 +336,14 @@ IMPLEMENT_DYNAMIC_CLASS(wxDCModule, wxModule)
 // wxDC
 // ---------------------------------------------------------------------------
 
 // wxDC
 // ---------------------------------------------------------------------------
 

-wxDC::wxDC(void)
+wxPMDCImpl::wxPMDCImpl( wxDC *owner, WXHDC hDC ) :
+    wxDCImpl( owner )

 {
 {

-    wxColour                        vColor;
-
-    m_pCanvas      = NULL;
-
-    m_hOldBitmap   = 0;
-    m_hOldPen      = 0;
-    m_hOldBrush    = 0;
-    m_hOldFont     = 0;
-    m_hOldPalette  = 0;
+    Init();
+    m_hDC          = hDC;
+} // end of wxPMDCImpl::wxPMDCImpl

 
 

-    m_bOwnsDC      = FALSE;
-    m_hDC          = 0;
-    m_hOldPS       = NULL;
-    m_hPS          = NULL;
-    m_bIsPaintTime = FALSE; // True at Paint Time
-
-    vColor.InitFromName("BLACK");
-    m_pen.SetColour(vColor);
-    vColor.Set("WHITE");
-    m_brush.SetColour(vColor);
-} // end of wxDC::wxDC
-
-wxDC::~wxDC(void)
+wxPMDCImpl::~wxPMDCImpl(void)

 {
     if ( m_hDC != 0 )
     {
 {
     if ( m_hDC != 0 )
     {


@@ -396,12 +372,12 @@ wxDC::~wxDC(void)
             }
         }
     }
             }
         }
     }

-} // end of wxDC::~wxDC
+} // end of wxPMDCImpl::~wxDC

 
 // This will select current objects out of the DC,
 // which is what you have to do before deleting the
 // DC.
 
 // This will select current objects out of the DC,
 // which is what you have to do before deleting the
 // DC.

-void wxDC::SelectOldObjects(
+void wxPMDCImpl::SelectOldObjects(

   WXHDC                             hPS
 )
 {
   WXHDC                             hPS
 )
 {


@@ -410,7 +386,7 @@ void wxDC::SelectOldObjects(
         if (m_hOldBitmap)
         {
             ::GpiSetBitmap(hPS, (HBITMAP) m_hOldBitmap);
         if (m_hOldBitmap)
         {
             ::GpiSetBitmap(hPS, (HBITMAP) m_hOldBitmap);

-            if (m_vSelectedBitmap.Ok())
+            if (m_vSelectedBitmap.IsOk())

             {
                 m_vSelectedBitmap.SetSelectedInto(NULL);
             }
             {
                 m_vSelectedBitmap.SetSelectedInto(NULL);
             }


@@ -431,7 +407,7 @@ void wxDC::SelectOldObjects(
     m_font            = wxNullFont;
     m_backgroundBrush = wxNullBrush;
     m_vSelectedBitmap = wxNullBitmap;
     m_font            = wxNullFont;
     m_backgroundBrush = wxNullBrush;
     m_vSelectedBitmap = wxNullBitmap;

-} // end of wxDC::SelectOldObjects
+} // end of wxPMDCImpl::SelectOldObjects

 
 // ---------------------------------------------------------------------------
 // clipping
 
 // ---------------------------------------------------------------------------
 // clipping


@@ -449,7 +425,7 @@ void wxDC::SelectOldObjects(
     m_clipY2 = (wxCoord) YDEV2LOG(rect.yBottom); \
 }
 
     m_clipY2 = (wxCoord) YDEV2LOG(rect.yBottom); \
 }
 

-void wxDC::DoSetClippingRegion(
+void wxPMDCImpl::DoSetClippingRegion(

   wxCoord                           vX
 , wxCoord                           vY
 , wxCoord                           vWidth
   wxCoord                           vX
 , wxCoord                           vY
 , wxCoord                           vWidth


@@ -459,31 +435,31 @@ void wxDC::DoSetClippingRegion(
     RECTL                           vRect;
 
     vY = OS2Y(vY,vHeight);
     RECTL                           vRect;
 
     vY = OS2Y(vY,vHeight);

-    m_clipping    = TRUE;
+    m_clipping    = true;

     vRect.xLeft   = vX;
     vRect.yTop    = vY + vHeight;
     vRect.xRight  = vX + vWidth;
     vRect.yBottom = vY;
     ::GpiIntersectClipRectangle(m_hPS, &vRect);
     DO_SET_CLIPPING_BOX()
     vRect.xLeft   = vX;
     vRect.yTop    = vY + vHeight;
     vRect.xRight  = vX + vWidth;
     vRect.yBottom = vY;
     ::GpiIntersectClipRectangle(m_hPS, &vRect);
     DO_SET_CLIPPING_BOX()

-} // end of wxDC::DoSetClippingRegion
+} // end of wxPMDCImpl::DoSetClippingRegion

 
 

-void wxDC::DoSetClippingRegionAsRegion(
+void wxPMDCImpl::DoSetDeviceClippingRegion(

   const wxRegion&                   rRegion
 )
 {
      wxCHECK_RET(rRegion.GetHRGN(), wxT("invalid clipping region"));
      HRGN                           hRgnOld;
 
   const wxRegion&                   rRegion
 )
 {
      wxCHECK_RET(rRegion.GetHRGN(), wxT("invalid clipping region"));
      HRGN                           hRgnOld;
 

-     m_clipping = TRUE;
+     m_clipping = true;

      ::GpiSetClipRegion( m_hPS
                         ,(HRGN)rRegion.GetHRGN()
                         ,&hRgnOld
                        );
     DO_SET_CLIPPING_BOX()
      ::GpiSetClipRegion( m_hPS
                         ,(HRGN)rRegion.GetHRGN()
                         ,&hRgnOld
                        );
     DO_SET_CLIPPING_BOX()

-} // end of wxDC::DoSetClippingRegionAsRegion
+} // end of wxPMDCImpl::DoSetDeviceClippingRegion

 
 

-void wxDC::DestroyClippingRegion(void)
+void wxPMDCImpl::DestroyClippingRegion(void)

 {
     if (m_clipping && m_hPS)
     {
 {
     if (m_clipping && m_hPS)
     {


@@ -503,47 +479,47 @@ void wxDC::DestroyClippingRegion(void)
 
          ::GpiSetClipRegion(m_hPS, hRgn, &hRgnOld);
      }
 
          ::GpiSetClipRegion(m_hPS, hRgn, &hRgnOld);
      }

-      m_clipping = FALSE;
-} // end of wxDC::DestroyClippingRegion
+    ResetClipping();
+} // end of wxPMDCImpl::DestroyClippingRegion

 
 // ---------------------------------------------------------------------------
 // query capabilities
 // ---------------------------------------------------------------------------
 
 
 // ---------------------------------------------------------------------------
 // query capabilities
 // ---------------------------------------------------------------------------
 

-bool wxDC::CanDrawBitmap() const
+bool wxPMDCImpl::CanDrawBitmap() const

 {
 {

-    return TRUE;
+    return true;

 }
 
 }
 

-bool wxDC::CanGetTextExtent() const
+bool wxPMDCImpl::CanGetTextExtent() const

 {
     LONG                            lTechnology = 0L;
 
     ::DevQueryCaps(GetHDC(), CAPS_TECHNOLOGY, 1L, &lTechnology);
     return (lTechnology == CAPS_TECH_RASTER_DISPLAY) || (lTechnology == CAPS_TECH_RASTER_PRINTER);
 {
     LONG                            lTechnology = 0L;
 
     ::DevQueryCaps(GetHDC(), CAPS_TECHNOLOGY, 1L, &lTechnology);
     return (lTechnology == CAPS_TECH_RASTER_DISPLAY) || (lTechnology == CAPS_TECH_RASTER_PRINTER);

-} // end of wxDC::CanGetTextExtent
+} // end of wxPMDCImpl::CanGetTextExtent

 
 

-int wxDC::GetDepth() const
+int wxPMDCImpl::GetDepth() const

 {
 {

-    LONG                            lArray[CAPS_COLOR_BITCOUNT];
-    int                             nBitsPerPixel;
+    LONG                            lCapsColorBitcount;
+    int                             nBitsPerPixel = 0;

 
     if(::DevQueryCaps( GetHDC()
 
     if(::DevQueryCaps( GetHDC()

-                      ,CAPS_FAMILY

                       ,CAPS_COLOR_BITCOUNT
                       ,CAPS_COLOR_BITCOUNT

-                      ,lArray
+                      ,1L
+                      ,&lCapsColorBitcount

                      ))
     {
                      ))
     {

-        nBitsPerPixel = (int)lArray[CAPS_COLOR_BITCOUNT];
+        nBitsPerPixel = (int)lCapsColorBitcount;

     }
     return nBitsPerPixel;
     }
     return nBitsPerPixel;

-} // end of wxDC::GetDepth
+} // end of wxPMDCImpl::GetDepth

 
 // ---------------------------------------------------------------------------
 // drawing
 // ---------------------------------------------------------------------------
 
 
 // ---------------------------------------------------------------------------
 // drawing
 // ---------------------------------------------------------------------------
 

-void wxDC::Clear()
+void wxPMDCImpl::Clear()

 {
     //
     // If this is a canvas DC then just fill with the background color
 {
     //
     // If this is a canvas DC then just fill with the background color


@@ -558,20 +534,20 @@ void wxDC::Clear()
     }
     else
     ::GpiErase(m_hPS);
     }
     else
     ::GpiErase(m_hPS);

-} // end of wxDC::Clear
+} // end of wxPMDCImpl::Clear

 
 

-bool wxDC::DoFloodFill(
+bool wxPMDCImpl::DoFloodFill(

   wxCoord                           vX
 , wxCoord                           vY
 , const wxColour&                   rCol
   wxCoord                           vX
 , wxCoord                           vY
 , const wxColour&                   rCol

-, int                               nStyle
+, wxFloodFillStyle                  nStyle

 )
 {
     POINTL                          vPtlPos;
     LONG                            lColor;
     LONG                            lOptions;
     LONG                            lHits;
 )
 {
     POINTL                          vPtlPos;
     LONG                            lColor;
     LONG                            lOptions;
     LONG                            lHits;

-    bool                            bSuccess = FALSE;
+    bool                            bSuccess = false;

 
     vPtlPos.x = vX;             // Loads x-coordinate
     vPtlPos.y = OS2Y(vY,0);     // Loads y-coordinate
 
     vPtlPos.x = vX;             // Loads x-coordinate
     vPtlPos.y = OS2Y(vY,0);     // Loads y-coordinate


@@ -582,11 +558,12 @@ bool wxDC::DoFloodFill(
         lOptions = FF_SURFACE;
 
     if ((lHits = ::GpiFloodFill(m_hPS, lOptions, lColor)) != GPI_ERROR)
         lOptions = FF_SURFACE;
 
     if ((lHits = ::GpiFloodFill(m_hPS, lOptions, lColor)) != GPI_ERROR)

-        bSuccess = TRUE;
-    return TRUE;
-} // end of wxDC::DoFloodFill
+        bSuccess = true;

 
 

-bool wxDC::DoGetPixel(
+    return bSuccess;
+} // end of wxPMDCImpl::DoFloodFill
+
+bool wxPMDCImpl::DoGetPixel(

   wxCoord                           vX
 , wxCoord                           vY
 , wxColour*                         pCol
   wxCoord                           vX
 , wxCoord                           vY
 , wxColour*                         pCol


@@ -597,17 +574,7 @@ bool wxDC::DoGetPixel(
 
     vPoint.x = vX;
     vPoint.y = OS2Y(vY,0);
 
     vPoint.x = vX;
     vPoint.y = OS2Y(vY,0);

-    lColor = ::GpiSetPel(m_hPS, &vPoint);
-
-    //
-    // Get the color of the pen
-    //
-    LONG                            lPencolor = 0x00ffffff;
-
-    if (m_pen.Ok())
-    {
-        lPencolor = m_pen.GetColour().GetPixel();
-    }
+    lColor = ::GpiQueryPel(m_hPS, &vPoint);

 
     //
     // return the color of the pixel
 
     //
     // return the color of the pixel


@@ -617,10 +584,10 @@ bool wxDC::DoGetPixel(
                   ,GetGValue(lColor)
                   ,GetBValue(lColor)
                  );
                   ,GetGValue(lColor)
                   ,GetBValue(lColor)
                  );

-    return(lColor == lPencolor);
-} // end of wxDC::DoGetPixel
+    return true;
+} // end of wxPMDCImpl::DoGetPixel

 
 

-void wxDC::DoCrossHair(
+void wxPMDCImpl::DoCrossHair(

   wxCoord                           vX
 , wxCoord                           vY
 )
   wxCoord                           vX
 , wxCoord                           vY
 )


@@ -652,9 +619,9 @@ void wxDC::DoCrossHair(
     ::GpiLine(m_hPS, &vPoint[3]);
     CalcBoundingBox(vX1, vY1);
     CalcBoundingBox(vX2, vY2);
     ::GpiLine(m_hPS, &vPoint[3]);
     CalcBoundingBox(vX1, vY1);
     CalcBoundingBox(vX2, vY2);

-} // end of wxDC::DoCrossHair
+} // end of wxPMDCImpl::DoCrossHair

 
 

-void wxDC::DoDrawLine(
+void wxPMDCImpl::DoDrawLine(

   wxCoord                           vX1
 , wxCoord                           vY1
 , wxCoord                           vX2
   wxCoord                           vX1
 , wxCoord                           vY1
 , wxCoord                           vX2


@@ -677,7 +644,7 @@ void wxDC::DoDrawLine(
     }
     else
     {
     }
     else
     {

-        if (m_vSelectedBitmap != wxNullBitmap)
+        if (m_vSelectedBitmap.IsOk())

         {
             m_vRclPaint.yTop = m_vSelectedBitmap.GetHeight();
             m_vRclPaint.xRight = m_vSelectedBitmap.GetWidth();
         {
             m_vRclPaint.yTop = m_vSelectedBitmap.GetHeight();
             m_vRclPaint.xRight = m_vSelectedBitmap.GetWidth();


@@ -689,7 +656,7 @@ void wxDC::DoDrawLine(
     vPoint[0].y = vY1;
     vPoint[1].x = vX2;
     vPoint[1].y = vY2;
     vPoint[0].y = vY1;
     vPoint[1].x = vX2;
     vPoint[1].y = vY2;

-    if (m_pen.Ok())
+    if (m_pen.IsOk())

     {
         vColor = m_pen.GetColour().GetPixel();
     }
     {
         vColor = m_pen.GetColour().GetPixel();
     }


@@ -698,7 +665,7 @@ void wxDC::DoDrawLine(
     ::GpiLine(m_hPS, &vPoint[1]);
     CalcBoundingBox(vX1, vY1);
     CalcBoundingBox(vX2, vY2);
     ::GpiLine(m_hPS, &vPoint[1]);
     CalcBoundingBox(vX1, vY1);
     CalcBoundingBox(vX2, vY2);

-} // end of wxDC::DoDrawLine
+} // end of wxPMDCImpl::DoDrawLine

 
 //////////////////////////////////////////////////////////////////////////////
 // Draws an arc of a circle, centred on (xc, yc), with starting point (x1, y1)
 
 //////////////////////////////////////////////////////////////////////////////
 // Draws an arc of a circle, centred on (xc, yc), with starting point (x1, y1)


@@ -706,7 +673,7 @@ void wxDC::DoDrawLine(
 // current brush for filling the shape. The arc is drawn in an anticlockwise
 // direction from the start point to the end point.
 //////////////////////////////////////////////////////////////////////////////
 // current brush for filling the shape. The arc is drawn in an anticlockwise
 // direction from the start point to the end point.
 //////////////////////////////////////////////////////////////////////////////

-void wxDC::DoDrawArc(
+void wxPMDCImpl::DoDrawArc(

   wxCoord                           vX1
 , wxCoord                           vY1
 , wxCoord                           vX2
   wxCoord                           vX1
 , wxCoord                           vY1
 , wxCoord                           vX2


@@ -717,8 +684,6 @@ void wxDC::DoDrawArc(
 {
      POINTL                         vPtlPos;
      POINTL                         vPtlArc[2]; // Structure for current position
 {
      POINTL                         vPtlPos;
      POINTL                         vPtlArc[2]; // Structure for current position

-     int                            nDx;
-     int                            nDy;

      double                         dRadius;
      double                         dAngl1;
      double                         dAngl2;
      double                         dRadius;
      double                         dAngl1;
      double                         dAngl2;


@@ -755,8 +720,8 @@ void wxDC::DoDrawArc(
         // Medium point
         //
         dAnglmid = (dAngl1 + dAngl2)/2. + M_PI;
         // Medium point
         //
         dAnglmid = (dAngl1 + dAngl2)/2. + M_PI;

-        vXm      = vXc + dRadius * cos(dAnglmid);
-        vYm      = vYc + dRadius * sin(dAnglmid);
+        vXm      = (wxCoord)(vXc + dRadius * cos(dAnglmid));
+        vYm      = (wxCoord)(vYc + dRadius * sin(dAnglmid));

         DoDrawArc( vX1, vY1
                   ,vXm, vYm
                   ,vXc, vYc
         DoDrawArc( vX1, vY1
                   ,vXm, vYm
                   ,vXc, vYc


@@ -772,8 +737,8 @@ void wxDC::DoDrawArc(
     // Medium point
     //
     dAnglmid = (dAngl1 + dAngl2)/2.;
     // Medium point
     //
     dAnglmid = (dAngl1 + dAngl2)/2.;

-    vXm      = vXc + dRadius * cos(dAnglmid);
-    vYm      = vYc + dRadius * sin(dAnglmid);
+    vXm      = (wxCoord)(vXc + dRadius * cos(dAnglmid));
+    vYm      = (wxCoord)(vYc + dRadius * sin(dAnglmid));

 
     //
     // Ellipse main axis (r,q), (p,s) with center at (0,0) */
 
     //
     // Ellipse main axis (r,q), (p,s) with center at (0,0) */


@@ -792,15 +757,15 @@ void wxDC::DoDrawArc(
     vPtlArc[1].x = vX2;
     vPtlArc[1].y = vY2;
     ::GpiPointArc(m_hPS, vPtlArc); // Draws the arc
     vPtlArc[1].x = vX2;
     vPtlArc[1].y = vY2;
     ::GpiPointArc(m_hPS, vPtlArc); // Draws the arc

-    CalcBoundingBox( (vXc - dRadius)
-                    ,(vYc - dRadius)
+    CalcBoundingBox( (wxCoord)(vXc - dRadius)
+                    ,(wxCoord)(vYc - dRadius)

                    );
                    );

-    CalcBoundingBox( (vXc + dRadius)
-                    ,(vYc + dRadius)
+    CalcBoundingBox( (wxCoord)(vXc + dRadius)
+                    ,(wxCoord)(vYc + dRadius)

                    );
                    );

-} // end of wxDC::DoDrawArc
+} // end of wxPMDCImpl::DoDrawArc

 
 

-void wxDC::DoDrawCheckMark(
+void wxPMDCImpl::DoDrawCheckMark(

   wxCoord                           vX1
 , wxCoord                           vY1
 , wxCoord                           vWidth
   wxCoord                           vX1
 , wxCoord                           vY1
 , wxCoord                           vWidth


@@ -847,9 +812,9 @@ void wxDC::DoDrawCheckMark(
     CalcBoundingBox( vX2
                     ,vY2
                    );
     CalcBoundingBox( vX2
                     ,vY2
                    );

-} // end of wxDC::DoDrawCheckMark
+} // end of wxPMDCImpl::DoDrawCheckMark

 
 

-void wxDC::DoDrawPoint(
+void wxPMDCImpl::DoDrawPoint(

   wxCoord                           vX
 , wxCoord                           vY
 )
   wxCoord                           vX
 , wxCoord                           vY
 )


@@ -857,7 +822,7 @@ void wxDC::DoDrawPoint(
     POINTL                          vPoint;
     COLORREF                        vColor = 0x00ffffff;
 
     POINTL                          vPoint;
     COLORREF                        vColor = 0x00ffffff;
 

-    if (m_pen.Ok())
+    if (m_pen.IsOk())

     {
         vColor = m_pen.GetColour().GetPixel();
     }
     {
         vColor = m_pen.GetColour().GetPixel();
     }


@@ -868,49 +833,46 @@ void wxDC::DoDrawPoint(
     CalcBoundingBox( vX
                     ,vY
                    );
     CalcBoundingBox( vX
                     ,vY
                    );

-} // end of wxDC::DoDrawPoint
+} // end of wxPMDCImpl::DoDrawPoint

 
 

-void wxDC::DoDrawPolygon(
-  int                               n
-, wxPoint                           vPoints[]
-, wxCoord                           vXoffset
-, wxCoord                           vYoffset
-, int                               nFillStyle
-)
+void wxPMDCImpl::DoDrawPolygon( int n,
+                          wxPoint vPoints[],
+                          wxCoord vXoffset,
+                          wxCoord vYoffset,
+                          wxPolygonFillMode nFillStyle )

 {
 {

-    ULONG                           ulCount = 1;    // Number of polygons.
-    POLYGON                         vPlgn;          // polygon.
-    ULONG                           flOptions = 0L; // Drawing options.
+    ULONG     ulCount = 1;    // Number of polygons.
+    POLYGON   vPlgn;          // polygon.
+    ULONG     flOptions = 0L; // Drawing options.

 
 

-//////////////////////////////////////////////////////////////////////////////
-// This contains fields of option bits... to draw boundary lines as well as
-// the area interior.
-//
-// Drawing boundary lines:
-//   POLYGON_NOBOUNDARY              Does not draw boundary lines.
-//   POLYGON_BOUNDARY                Draws boundary lines (the default).
-//
-// Construction of the area interior:
-//   POLYGON_ALTERNATE               Constructs interior in alternate mode
-//                                   (the default).
-//   POLYGON_WINDING                 Constructs interior in winding mode.
-//////////////////////////////////////////////////////////////////////////////
-
-    ULONG                           flModel = 0L; // Drawing model.
-
-//////////////////////////////////////////////////////////////////////////////
-// Drawing model.
-//   POLYGON_INCL  Fill is inclusive of bottom right (the default).
-//   POLYGON_EXCL  Fill is exclusive of bottom right.
-//       This is provided to aid migration from other graphics models.
-//////////////////////////////////////////////////////////////////////////////
-
-    LONG                            lHits = 0L; // Correlation/error indicator.
-    POINTL                          vPoint;
-    int                             i;
-    int                             nIsTRANSPARENT = 0;
-    LONG                            lBorderColor = 0L;
-    LONG                            lColor = 0L;
+    //////////////////////////////////////////////////////////////////////////////
+    // This contains fields of option bits... to draw boundary lines as well as
+    // the area interior.
+    //
+    // Drawing boundary lines:
+    //   POLYGON_NOBOUNDARY              Does not draw boundary lines.
+    //   POLYGON_BOUNDARY                Draws boundary lines (the default).
+    //
+    // Construction of the area interior:
+    //   POLYGON_ALTERNATE               Constructs interior in alternate mode
+    //                                   (the default).
+    //   POLYGON_WINDING                 Constructs interior in winding mode.
+    //////////////////////////////////////////////////////////////////////////////
+
+    ULONG     flModel = POLYGON_INCL; // Drawing model.
+
+    //////////////////////////////////////////////////////////////////////////////
+    // Drawing model.
+    //   POLYGON_INCL  Fill is inclusive of bottom right (the default).
+    //   POLYGON_EXCL  Fill is exclusive of bottom right.
+    //       This is provided to aid migration from other graphics models.
+    //////////////////////////////////////////////////////////////////////////////
+
+    LONG      lHits = 0L; // Correlation/error indicator.
+    int       i;
+    int       nIsTRANSPARENT = 0;
+    LONG      lBorderColor = 0L;
+    LONG      lColor = 0L;

 
     lBorderColor = m_pen.GetColour().GetPixel();
     lColor       = m_brush.GetColour().GetPixel();
 
     lBorderColor = m_pen.GetColour().GetPixel();
     lColor       = m_brush.GetColour().GetPixel();


@@ -924,25 +886,22 @@ void wxDC::DoDrawPolygon(
 
     for(i = 0; i < n; i++)
     {
 
     for(i = 0; i < n; i++)
     {

-        vPlgn.aPointl[i].x = vPoints[i].x;         // +xoffset;
-        vPlgn.aPointl[i].y = OS2Y(vPoints[i].y,0); // +yoffset;
+        vPlgn.aPointl[i].x = vPoints[i].x+vXoffset;
+        vPlgn.aPointl[i].y = OS2Y(vPoints[i].y+vYoffset,0);

     }
     }

-    flModel = POLYGON_BOUNDARY;
+    flOptions = POLYGON_BOUNDARY;

     if(nFillStyle == wxWINDING_RULE)
     if(nFillStyle == wxWINDING_RULE)

-        flModel |= POLYGON_WINDING;
+        flOptions |= POLYGON_WINDING;

     else
     else

-        flModel |= POLYGON_ALTERNATE;
-
-    vPoint.x = vXoffset;
-    vPoint.y = OS2Y(vYoffset,0);
+        flOptions |= POLYGON_ALTERNATE;

 
     ::GpiSetColor(m_hPS, lBorderColor);
 
     ::GpiSetColor(m_hPS, lBorderColor);

-    ::GpiMove(m_hPS, &vPoint);
+    ::GpiMove(m_hPS, &vPlgn.aPointl[0]);

     lHits = ::GpiPolygons(m_hPS, ulCount, &vPlgn, flOptions, flModel);
     free(vPlgn.aPointl);
     lHits = ::GpiPolygons(m_hPS, ulCount, &vPlgn, flOptions, flModel);
     free(vPlgn.aPointl);

-} // end of wxDC::DoDrawPolygon
+} // end of wxPMDCImpl::DoDrawPolygon

 
 

-void wxDC::DoDrawLines(
+void wxPMDCImpl::DoDrawLines(

   int                               n
 , wxPoint                           vPoints[]
 , wxCoord                           vXoffset
   int                               n
 , wxPoint                           vPoints[]
 , wxCoord                           vXoffset


@@ -973,8 +932,8 @@ void wxDC::DoDrawLines(
     {
         int                         i;
 
     {
         int                         i;
 

-        CalcBoundingBox( vPoints[i].x
-                        ,vPoints[i].y
+        CalcBoundingBox( vPoints[0].x
+                        ,vPoints[0].y

                        );
         vPoint.x = vPoints[0].x;
         vPoint.y = OS2Y(vPoints[0].y,0);
                        );
         vPoint.x = vPoints[0].x;
         vPoint.y = OS2Y(vPoints[0].y,0);


@@ -990,9 +949,9 @@ void wxDC::DoDrawLines(
             ::GpiLine(m_hPS, &vPoint);
         }
     }
             ::GpiLine(m_hPS, &vPoint);
         }
     }

-} // end of wxDC::DoDrawLines
+} // end of wxPMDCImpl::DoDrawLines

 
 

-void wxDC::DoDrawRectangle(
+void wxPMDCImpl::DoDrawRectangle(

   wxCoord                           vX
 , wxCoord                           vY
 , wxCoord                           vWidth
   wxCoord                           vX
 , wxCoord                           vY
 , wxCoord                           vWidth


@@ -1015,7 +974,7 @@ void wxDC::DoDrawRectangle(
         vY = OS2Y(vY,vHeight);
     else
     {
         vY = OS2Y(vY,vHeight);
     else
     {

-        if (m_vSelectedBitmap != wxNullBitmap)
+        if (m_vSelectedBitmap.IsOk())

         {
             m_vRclPaint.yTop = m_vSelectedBitmap.GetHeight();
             m_vRclPaint.xRight = m_vSelectedBitmap.GetWidth();
         {
             m_vRclPaint.yTop = m_vSelectedBitmap.GetHeight();
             m_vRclPaint.xRight = m_vSelectedBitmap.GetWidth();


@@ -1079,9 +1038,9 @@ void wxDC::DoDrawRectangle(
     }
     CalcBoundingBox(vX, vY);
     CalcBoundingBox(vX2, vY2);
     }
     CalcBoundingBox(vX, vY);
     CalcBoundingBox(vX2, vY2);

-} // end of wxDC::DoDrawRectangle
+} // end of wxPMDCImpl::DoDrawRectangle

 
 

-void wxDC::DoDrawRoundedRectangle(
+void wxPMDCImpl::DoDrawRoundedRectangle(

   wxCoord                           vX
 , wxCoord                           vY
 , wxCoord                           vWidth
   wxCoord                           vX
 , wxCoord                           vY
 , wxCoord                           vWidth


@@ -1105,7 +1064,7 @@ void wxDC::DoDrawRoundedRectangle(
         vY = OS2Y(vY,vHeight);
     else
     {
         vY = OS2Y(vY,vHeight);
     else
     {

-        if (m_vSelectedBitmap != wxNullBitmap)
+        if (m_vSelectedBitmap.IsOk())

         {
             m_vRclPaint.yTop = m_vSelectedBitmap.GetHeight();
             m_vRclPaint.xRight = m_vSelectedBitmap.GetWidth();
         {
             m_vRclPaint.yTop = m_vSelectedBitmap.GetHeight();
             m_vRclPaint.xRight = m_vSelectedBitmap.GetWidth();


@@ -1172,10 +1131,10 @@ void wxDC::DoDrawRoundedRectangle(
 
     CalcBoundingBox(vX, vY);
     CalcBoundingBox(vX2, vY2);
 
     CalcBoundingBox(vX, vY);
     CalcBoundingBox(vX2, vY2);

-} // end of wxDC::DoDrawRoundedRectangle
+} // end of wxPMDCImpl::DoDrawRoundedRectangle

 
 // Draw Ellipse within box (x,y) - (x+width, y+height)
 
 // Draw Ellipse within box (x,y) - (x+width, y+height)

-void wxDC::DoDrawEllipse(
+void wxPMDCImpl::DoDrawEllipse(

   wxCoord                           vX
 , wxCoord                           vY
 , wxCoord                           vWidth
   wxCoord                           vX
 , wxCoord                           vY
 , wxCoord                           vWidth


@@ -1215,9 +1174,9 @@ void wxDC::DoDrawEllipse(
 
     CalcBoundingBox(vX, vY);
     CalcBoundingBox(vX2, vY2);
 
     CalcBoundingBox(vX, vY);
     CalcBoundingBox(vX2, vY2);

-} // end of wxDC::DoDrawEllipse
+} // end of wxPMDCImpl::DoDrawEllipse

 
 

-void wxDC::DoDrawEllipticArc(
+void wxPMDCImpl::DoDrawEllipticArc(

   wxCoord                           vX
 , wxCoord                           vY
 , wxCoord                           vWidth
   wxCoord                           vX
 , wxCoord                           vY
 , wxCoord                           vWidth


@@ -1233,7 +1192,6 @@ void wxDC::DoDrawEllipticArc(
     FIXED                           vFSweepa; // Start angle, sweep angle
     double                          dIntPart;
     double                          dFractPart;
     FIXED                           vFSweepa; // Start angle, sweep angle
     double                          dIntPart;
     double                          dFractPart;

-    double                          dRadius;

 
     vY = OS2Y(vY,vHeight);
 
 
     vY = OS2Y(vY,vHeight);
 


@@ -1250,8 +1208,8 @@ void wxDC::DoDrawEllipticArc(
     vArcp.lP = vWidth/2;
     vArcp.lS = 0;
     ::GpiSetArcParams(m_hPS, &vArcp); // Sets parameters to default
     vArcp.lP = vWidth/2;
     vArcp.lS = 0;
     ::GpiSetArcParams(m_hPS, &vArcp); // Sets parameters to default

-    vPtlPos.x = vX + vWidth/2  * (1. + cos(DegToRad(dSa))); // Loads x-coordinate
-    vPtlPos.y = vY + vHeight/2 * (1. + sin(DegToRad(dSa))); // Loads y-coordinate
+    vPtlPos.x = (wxCoord)(vX + vWidth/2  * (1. + cos(DegToRad(dSa)))); // Loads x-coordinate
+    vPtlPos.y = (wxCoord)(vY + vHeight/2 * (1. + sin(DegToRad(dSa)))); // Loads y-coordinate

     ::GpiMove(m_hPS, &vPtlPos); // Sets current position
 
     //
     ::GpiMove(m_hPS, &vPtlPos); // Sets current position
 
     //


@@ -1275,9 +1233,9 @@ void wxDC::DoDrawEllipticArc(
 
     CalcBoundingBox(vX, vY);
     CalcBoundingBox(vX2, vY2);
 
     CalcBoundingBox(vX, vY);
     CalcBoundingBox(vX2, vY2);

-} // end of wxDC::DoDrawEllipticArc
+} // end of wxPMDCImpl::DoDrawEllipticArc

 
 

-void wxDC::DoDrawIcon(
+void wxPMDCImpl::DoDrawIcon(

   const wxIcon&                     rIcon
 , wxCoord                           vX
 , wxCoord                           vY
   const wxIcon&                     rIcon
 , wxCoord                           vX
 , wxCoord                           vY


@@ -1286,33 +1244,35 @@ void wxDC::DoDrawIcon(
     //
     // Need to copy back into a bitmap.  ::WinDrawPointer uses device coords
     // and I don't feel like figuring those out for scrollable windows so
     //
     // Need to copy back into a bitmap.  ::WinDrawPointer uses device coords
     // and I don't feel like figuring those out for scrollable windows so

-    // just convert to a bitmap then let the DoDrawBitmap routing display it
+    // just convert to a bitmap then let the DoDrawBitmap routine display it

     //
     if (rIcon.IsXpm())
     {
     //
     if (rIcon.IsXpm())
     {

-        DoDrawBitmap(rIcon.GetXpmSrc(), vX, vY, TRUE);
+        DoDrawBitmap(rIcon.GetXpmSrc(), vX, vY, true);

     }
     else
     {
         wxBitmap                        vBitmap(rIcon);
 
     }
     else
     {
         wxBitmap                        vBitmap(rIcon);
 

-        DoDrawBitmap(vBitmap, vX, vY, FALSE);
+        DoDrawBitmap(vBitmap, vX, vY, false);

     }
     CalcBoundingBox(vX, vY);
     CalcBoundingBox(vX + rIcon.GetWidth(), vY + rIcon.GetHeight());
     }
     CalcBoundingBox(vX, vY);
     CalcBoundingBox(vX + rIcon.GetWidth(), vY + rIcon.GetHeight());

-} // end of wxDC::DoDrawIcon
+} // end of wxPMDCImpl::DoDrawIcon

 
 

-void wxDC::DoDrawBitmap(
+void wxPMDCImpl::DoDrawBitmap(

   const wxBitmap&                   rBmp
 , wxCoord                           vX
 , wxCoord                           vY
 , bool                              bUseMask
 )
 {
   const wxBitmap&                   rBmp
 , wxCoord                           vX
 , wxCoord                           vY
 , bool                              bUseMask
 )
 {

+#if wxUSE_PRINTING_ARCHITECTURE

     if (!IsKindOf(CLASSINFO(wxPrinterDC)))
     if (!IsKindOf(CLASSINFO(wxPrinterDC)))

+#endif

     {
         HBITMAP                         hBitmap =  (HBITMAP)rBmp.GetHBITMAP();
     {
         HBITMAP                         hBitmap =  (HBITMAP)rBmp.GetHBITMAP();

-        HBITMAP                         hBitmapOld;
+        HBITMAP                         hBitmapOld = NULLHANDLE;

         POINTL                          vPoint[4];
 
         vY = OS2Y(vY,rBmp.GetHeight());
         POINTL                          vPoint[4];
 
         vY = OS2Y(vY,rBmp.GetHeight());


@@ -1393,19 +1353,19 @@ void wxDC::DoDrawBitmap(
                 //
                 // Need to get a background color for mask blitting
                 //
                 //
                 // Need to get a background color for mask blitting
                 //

-                if (IsKindOf(CLASSINFO(wxWindowDC)))
+                if (IsKindOf(CLASSINFO(wxWindowDCImpl)))

                 {
                 {

-                    wxWindowDC*             pWindowDC = wxDynamicCast(this, wxWindowDC);
+                    wxWindowDCImpl*  pWindowDC = wxDynamicCast(this, wxWindowDCImpl);

 
                     lColor = pWindowDC->m_pCanvas->GetBackgroundColour().GetPixel();
                 }
 
                     lColor = pWindowDC->m_pCanvas->GetBackgroundColour().GetPixel();
                 }

-                else if (GetBrush() != wxNullBrush)
+                else if (GetBrush().IsOk())

                     lColor = GetBrush().GetColour().GetPixel();
                 else
                     lColor = m_textBackgroundColour.GetPixel();
 
                 //
                     lColor = GetBrush().GetColour().GetPixel();
                 else
                     lColor = m_textBackgroundColour.GetPixel();
 
                 //

-                // Bitmap must be ina double-word alligned address so we may
+                // Bitmap must be in a double-word aligned address so we may

                 // have some padding to worry about
                 //
                 if (nLineBoundary > 0)
                 // have some padding to worry about
                 //
                 if (nLineBoundary > 0)


@@ -1563,16 +1523,16 @@ void wxDC::DoDrawBitmap(
         }
         else
         {
         }
         else
         {

-            LONG                        lOldForeGround = ::GpiQueryColor((HPS)GetHPS());
-            LONG                        lOldBackGround = ::GpiQueryBackColor((HPS)GetHPS());
+            ULONG                       lOldForeGround = ::GpiQueryColor((HPS)GetHPS());
+            ULONG                       lOldBackGround = ::GpiQueryBackColor((HPS)GetHPS());

 
 

-            if (m_textForegroundColour.Ok())
+            if (m_textForegroundColour.IsOk())

             {
                 ::GpiSetColor( (HPS)GetHPS()
                                ,m_textForegroundColour.GetPixel()
                               );
             }
             {
                 ::GpiSetColor( (HPS)GetHPS()
                                ,m_textForegroundColour.GetPixel()
                               );
             }

-            if (m_textBackgroundColour.Ok())
+            if (m_textBackgroundColour.IsOk())

             {
                 ::GpiSetBackColor( (HPS)GetHPS()
                                   ,m_textBackgroundColour.GetPixel()
             {
                 ::GpiSetBackColor( (HPS)GetHPS()
                                   ,m_textBackgroundColour.GetPixel()


@@ -1619,6 +1579,7 @@ void wxDC::DoDrawBitmap(
                 {
                     vError = ::WinGetLastError(vHabmain);
                     sError = wxPMErrorToStr(vError);
                 {
                     vError = ::WinGetLastError(vHabmain);
                     sError = wxPMErrorToStr(vError);

+                    delete [] pucBits;

                     return;
                 }
                 if ((lScans = ::GpiQueryBitmapBits( hPS
                     return;
                 }
                 if ((lScans = ::GpiQueryBitmapBits( hPS


@@ -1630,16 +1591,13 @@ void wxDC::DoDrawBitmap(
                 {
                     vError = ::WinGetLastError(vHabmain);
                     sError = wxPMErrorToStr(vError);
                 {
                     vError = ::WinGetLastError(vHabmain);
                     sError = wxPMErrorToStr(vError);

+                    delete [] pucBits;

                     return;
                 }
                 unsigned char           cOldRedFore   = (unsigned char)(lOldForeGround >> 16);
                 unsigned char           cOldGreenFore = (unsigned char)(lOldForeGround >> 8);
                 unsigned char           cOldBlueFore  = (unsigned char)lOldForeGround;
 
                     return;
                 }
                 unsigned char           cOldRedFore   = (unsigned char)(lOldForeGround >> 16);
                 unsigned char           cOldGreenFore = (unsigned char)(lOldForeGround >> 8);
                 unsigned char           cOldBlueFore  = (unsigned char)lOldForeGround;
 

-                unsigned char           cOldRedBack   = (unsigned char)(lOldBackGround >> 16);
-                unsigned char           cOldGreenBack = (unsigned char)(lOldBackGround >> 8);
-                unsigned char           cOldBlueBack  = (unsigned char)lOldBackGround;
-

                 unsigned char           cRedFore   = (unsigned char)(lForeGround >> 16);
                 unsigned char           cGreenFore = (unsigned char)(lForeGround >> 8);
                 unsigned char           cBlueFore  = (unsigned char)lForeGround;
                 unsigned char           cRedFore   = (unsigned char)(lForeGround >> 16);
                 unsigned char           cGreenFore = (unsigned char)(lForeGround >> 8);
                 unsigned char           cBlueFore  = (unsigned char)lForeGround;


@@ -1707,9 +1665,9 @@ void wxDC::DoDrawBitmap(
             ::GpiSetBackColor((HPS)GetHPS(), lOldBackGround);
         }
     }
             ::GpiSetBackColor((HPS)GetHPS(), lOldBackGround);
         }
     }

-} // end of wxDC::DoDrawBitmap
+} // end of wxPMDCImpl::DoDrawBitmap

 
 

-void wxDC::DoDrawText(
+void wxPMDCImpl::DoDrawText(

   const wxString&                   rsText
 , wxCoord                           vX
 , wxCoord                           vY
   const wxString&                   rsText
 , wxCoord                           vX
 , wxCoord                           vY


@@ -1724,15 +1682,13 @@ void wxDC::DoDrawText(
                );
 
     CalcBoundingBox(vX, vY);
                );
 
     CalcBoundingBox(vX, vY);

-    GetTextExtent(rsText, &vWidth, &vHeight);
+    GetOwner()->GetTextExtent(rsText, &vWidth, &vHeight);

     CalcBoundingBox((vX + vWidth), (vY + vHeight));
     CalcBoundingBox((vX + vWidth), (vY + vHeight));

-} // end of wxDC::DoDrawText
+} // end of wxPMDCImpl::DoDrawText

 
 

-void wxDC::DrawAnyText(
-  const wxString&                   rsText
-, wxCoord                           vX
-, wxCoord                           vY
-)
+void wxPMDCImpl::DrawAnyText( const wxString& rsText,
+                        wxCoord vX,
+                        wxCoord vY )

 {
     int                             nOldBackground = 0;
     POINTL                          vPtlStart;
 {
     int                             nOldBackground = 0;
     POINTL                          vPtlStart;


@@ -1747,14 +1703,14 @@ void wxDC::DrawAnyText(
     //
     // Set text color attributes
     //
     //
     // Set text color attributes
     //

-    if (m_textForegroundColour.Ok())
+    if (m_textForegroundColour.IsOk())

     {
         SetTextColor( m_hPS
                      ,(int)m_textForegroundColour.GetPixel()
                     );
     }
 
     {
         SetTextColor( m_hPS
                      ,(int)m_textForegroundColour.GetPixel()
                     );
     }
 

-    if (m_textBackgroundColour.Ok())
+    if (m_textBackgroundColour.IsOk())

     {
         nOldBackground = SetTextBkColor( m_hPS
                                         ,(int)m_textBackgroundColour.GetPixel()
     {
         nOldBackground = SetTextBkColor( m_hPS
                                         ,(int)m_textBackgroundColour.GetPixel()


@@ -1763,45 +1719,34 @@ void wxDC::DrawAnyText(
     SetBkMode( m_hPS
               ,m_backgroundMode
              );
     SetBkMode( m_hPS
               ,m_backgroundMode
              );

-    GetTextExtent( rsText
-                  ,&vTextX
-                  ,&vTextY
-                 );
+    GetOwner()->GetTextExtent( rsText
+                              ,&vTextX
+                              ,&vTextY
+                             );

     vPtlStart.x = vX;
     if (!(m_vRclPaint.yTop == 0 &&
           m_vRclPaint.yBottom == 0 &&
           m_vRclPaint.xRight == 0 &&
           m_vRclPaint.xLeft == 0))
     {
     vPtlStart.x = vX;
     if (!(m_vRclPaint.yTop == 0 &&
           m_vRclPaint.yBottom == 0 &&
           m_vRclPaint.xRight == 0 &&
           m_vRclPaint.xLeft == 0))
     {

-        //
-        // Position Text a little differently in the Statusbar from other panels
-        //
-        if (m_pCanvas && m_pCanvas->IsKindOf(CLASSINFO(wxStatusBar)))
-            vPtlStart.y = OS2Y(vY,vTextY);
-        else
-            vPtlStart.y = OS2Y(vY,vTextY/1.5); // Full extent is a bit much
+        vPtlStart.y = OS2Y(vY,vTextY);

     }
     else
     {
     }
     else
     {

-        if (m_vSelectedBitmap != wxNullBitmap)
+        if (m_vSelectedBitmap.IsOk())

         {
             m_vRclPaint.yTop = m_vSelectedBitmap.GetHeight();
             m_vRclPaint.xRight = m_vSelectedBitmap.GetWidth();
         {
             m_vRclPaint.yTop = m_vSelectedBitmap.GetHeight();
             m_vRclPaint.xRight = m_vSelectedBitmap.GetWidth();

-            if (m_pCanvas && m_pCanvas->IsKindOf(CLASSINFO(wxStatusBar)))
-                vPtlStart.y = OS2Y(vY,vTextY);
-            else
-                vPtlStart.y = OS2Y(vY,vTextY/1.5);
+            vPtlStart.y = OS2Y(vY,vTextY);

         }
         else
             vPtlStart.y = vY;
     }
 
         }
         else
             vPtlStart.y = vY;
     }
 

-    PCH                             pzStr = (PCH)rsText.c_str();
-

     ::GpiMove(m_hPS, &vPtlStart);
     lHits = ::GpiCharString( m_hPS
                             ,rsText.length()
     ::GpiMove(m_hPS, &vPtlStart);
     lHits = ::GpiCharString( m_hPS
                             ,rsText.length()

-                            ,pzStr
+                            ,rsText.char_str()

                            );
     if (lHits != GPI_OK)
     {
                            );
     if (lHits != GPI_OK)
     {


@@ -1813,7 +1758,7 @@ void wxDC::DrawAnyText(
     // it never is set to anything else, but background should remain
     // transparent even if we just drew an opaque string)
     //
     // it never is set to anything else, but background should remain
     // transparent even if we just drew an opaque string)
     //

-    if (m_textBackgroundColour.Ok())
+    if (m_textBackgroundColour.IsOk())

             SetTextBkColor( m_hPS
                            ,nOldBackground
                           );
             SetTextBkColor( m_hPS
                            ,nOldBackground
                           );


@@ -1822,7 +1767,7 @@ void wxDC::DrawAnyText(
              );
 }
 
              );
 }
 

-void wxDC::DoDrawRotatedText(
+void wxPMDCImpl::DoDrawRotatedText(

   const wxString&                   rsText
 , wxCoord                           vX
 , wxCoord                           vY
   const wxString&                   rsText
 , wxCoord                           vX
 , wxCoord                           vY


@@ -1893,9 +1838,7 @@ void wxDC::DoDrawRotatedText(
 // set GDI objects
 // ---------------------------------------------------------------------------
 
 // set GDI objects
 // ---------------------------------------------------------------------------
 

-void wxDC::DoSelectPalette(
-  bool                              bRealize
-)
+void wxPMDCImpl::DoSelectPalette( bool WXUNUSED(bRealize) )

 {
     //
     // Set the old object temporarily, in case the assignment deletes an object
 {
     //
     // Set the old object temporarily, in case the assignment deletes an object


@@ -1906,7 +1849,7 @@ void wxDC::DoSelectPalette(
         m_hOldPalette = 0;
     }
 
         m_hOldPalette = 0;
     }
 

-    if (m_palette.Ok())
+    if (m_palette.IsOk())

     {
         HPALETTE                    hOldPal;
 
     {
         HPALETTE                    hOldPal;
 


@@ -1914,9 +1857,9 @@ void wxDC::DoSelectPalette(
         if (!m_hOldPalette)
             m_hOldPalette = (WXHPALETTE)hOldPal;
     }
         if (!m_hOldPalette)
             m_hOldPalette = (WXHPALETTE)hOldPal;
     }

-} // end of wxDC::DoSelectPalette
+} // end of wxPMDCImpl::DoSelectPalette

 
 

-void wxDC::InitializePalette()
+void wxPMDCImpl::InitializePalette()

 {
     if (wxDisplayDepth() <= 8 )
     {
 {
     if (wxDisplayDepth() <= 8 )
     {


@@ -1937,9 +1880,9 @@ void wxDC::InitializePalette()
             DoSelectPalette();
         }
     }
             DoSelectPalette();
         }
     }

-} // end of wxDC::InitializePalette
+} // end of wxPMDCImpl::InitializePalette

 
 

-void wxDC::SetPalette(
+void wxPMDCImpl::SetPalette(

   const wxPalette&                  rPalette
 )
 {
   const wxPalette&                  rPalette
 )
 {


@@ -1948,7 +1891,7 @@ void wxDC::SetPalette(
         m_hOldFont = 0;
     }
     m_palette = rPalette;
         m_hOldFont = 0;
     }
     m_palette = rPalette;

-    if (!rPalette.Ok())
+    if (!rPalette.IsOk())

     {
         if (m_hOldFont)
         {
     {
         if (m_hOldFont)
         {


@@ -1958,9 +1901,9 @@ void wxDC::SetPalette(
     HPALETTE                    hOldPal = ::GpiSelectPalette((HDC) m_hPS, (HPALETTE) m_palette.GetHPALETTE());
     if (!m_hOldPalette)
         m_hOldPalette = (WXHPALETTE)hOldPal;
     HPALETTE                    hOldPal = ::GpiSelectPalette((HDC) m_hPS, (HPALETTE) m_palette.GetHPALETTE());
     if (!m_hOldPalette)
         m_hOldPalette = (WXHPALETTE)hOldPal;

-} // end of wxDC::SetPalette
+} // end of wxPMDCImpl::SetPalette

 
 

-void wxDC::SetFont(
+void wxPMDCImpl::SetFont(

   const wxFont&                     rFont
 )
 {
   const wxFont&                     rFont
 )
 {


@@ -1973,42 +1916,40 @@ void wxDC::SetFont(
         m_hOldFont = 0;
     }
     m_font = rFont;
         m_hOldFont = 0;
     }
     m_font = rFont;

-    if (!rFont.Ok())
+    if (!rFont.IsOk())

     {
         m_hOldFont = 0;
     }
 
     m_font.SetPS(m_hPS); // this will realize the font
 
     {
         m_hOldFont = 0;
     }
 
     m_font.SetPS(m_hPS); // this will realize the font
 

-    if (m_font.Ok())
+    if (m_font.IsOk())

     {
         HFONT                       hFont = m_font.GetResourceHandle();
         if (hFont == (HFONT) NULL)
         {
     {
         HFONT                       hFont = m_font.GetResourceHandle();
         if (hFont == (HFONT) NULL)
         {

-            wxLogDebug(wxT("::SelectObject failed in wxDC::SetFont."));
+            wxLogDebug(wxT("::SelectObject failed in wxPMDCImpl::SetFont."));

         }
         if (!m_hOldFont)
             m_hOldFont = (WXHFONT) hFont;
     }
         }
         if (!m_hOldFont)
             m_hOldFont = (WXHFONT) hFont;
     }

-} // end of wxDC::SetFont
+} // end of wxPMDCImpl::SetFont

 
 

-void wxDC::SetPen(
+void wxPMDCImpl::SetPen(

   const wxPen&                      rPen
 )
 {
   const wxPen&                      rPen
 )
 {

-    wxCHECK_RET( Ok(), wxT("invalid window dc") );
-

     if (m_pen == rPen)
         return;
     m_pen = rPen;
     if (m_pen == rPen)
         return;
     m_pen = rPen;

-    if (!m_pen.Ok())
+    if (!m_pen.IsOk())

         return;
 
     if (m_hOldPen)
         m_hOldPen = 0L;
     m_pen = rPen;
 
         return;
 
     if (m_hOldPen)
         m_hOldPen = 0L;
     m_pen = rPen;
 

-    if (!m_pen.Ok())
+    if (!m_pen.IsOk())

     {
         if (m_hOldPen)
         {
     {
         if (m_hOldPen)
         {


@@ -2017,7 +1958,7 @@ void wxDC::SetPen(
         m_hOldPen = 0L;
     }
 
         m_hOldPen = 0L;
     }
 

-    if (m_pen.Ok())
+    if (m_pen.IsOk())

     {
         if (m_pen.GetResourceHandle())
         {
     {
         if (m_pen.GetResourceHandle())
         {


@@ -2029,23 +1970,21 @@ void wxDC::SetPen(
     }
 }
 
     }
 }
 

-void wxDC::SetBrush(
+void wxPMDCImpl::SetBrush(

   const wxBrush&                    rBrush
 )
 {
   const wxBrush&                    rBrush
 )
 {

-    wxCHECK_RET( Ok(), wxT("invalid window dc") );
-

     if (m_hOldBrush)
         m_hOldBrush = 0L;
     m_brush = rBrush;
     if (m_hOldBrush)
         m_hOldBrush = 0L;
     m_brush = rBrush;

-    if (!m_brush.Ok())
+    if (!m_brush.IsOk())

     if (m_brush == rBrush)
         return;
     if (m_brush == rBrush)
         return;

-    if (!m_brush.Ok())
+    if (!m_brush.IsOk())

         if (m_hOldBrush)
             m_hOldBrush = 0L;
 
         if (m_hOldBrush)
             m_hOldBrush = 0L;
 

-    if (!m_brush.Ok())
+    if (!m_brush.IsOk())

     {
         if (m_hOldBrush)
         {
     {
         if (m_hOldBrush)
         {


@@ -2054,7 +1993,7 @@ void wxDC::SetBrush(
         m_hOldBrush = 0L;
     }
 
         m_hOldBrush = 0L;
     }
 

-    if (m_brush.Ok())
+    if (m_brush.IsOk())

     {
         if (m_brush.GetResourceHandle())
         {
     {
         if (m_brush.GetResourceHandle())
         {


@@ -2063,75 +2002,35 @@ void wxDC::SetBrush(
                 m_hOldBrush = (WXHWND)m_brush.GetPS();
         }
     }
                 m_hOldBrush = (WXHWND)m_brush.GetPS();
         }
     }

-} // end of wxDC::SetBrush
+} // end of wxPMDCImpl::SetBrush

 
 

-void wxDC::SetBackground(
-  const wxBrush&                    rBrush
-)
+void wxPMDCImpl::SetBackground(const wxBrush& rBrush)

 {
     m_backgroundBrush = rBrush;
 {
     m_backgroundBrush = rBrush;

-    if (!m_backgroundBrush.Ok())
-        return;
-    if (m_pCanvas)
-    {
-        bool                        bCustomColours = TRUE;

 
 

-        //
-        // If we haven't specified wxUSER_COLOURS, don't allow the panel/dialog box to
-        // change background colours from the control-panel specified colours.
-        //
-        if (m_pCanvas->IsKindOf(CLASSINFO(wxWindow)) &&
-            ((m_pCanvas->GetWindowStyleFlag() & wxUSER_COLOURS) != wxUSER_COLOURS))
-            bCustomColours = FALSE;
-        if (bCustomColours)
-        {
-            if (m_backgroundBrush.GetStyle()==wxTRANSPARENT)
-            {
-                m_pCanvas->SetTransparent(TRUE);
-            }
-            else
-            {
-                //
-                // 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_pCanvas->SetTransparent(FALSE);
-            }
-        }
+    if (m_backgroundBrush.IsOk())
+    {
+        (void)::GpiSetBackColor((HPS)m_hPS, m_backgroundBrush.GetColour().GetPixel());

     }
     }

-    COLORREF                        vNewColor = m_backgroundBrush.GetColour().GetPixel();
-    (void)::GpiSetBackColor((HPS)m_hPS, (LONG)vNewColor);
-} // end of wxDC::SetBackground
+} // end of wxPMDCImpl::SetBackground

 
 

-void wxDC::SetBackgroundMode(
-  int                               nMode
-)
+void wxPMDCImpl::SetBackgroundMode(int nMode)

 {
     m_backgroundMode = nMode;
 {
     m_backgroundMode = nMode;

-} // end of wxDC::SetBackgroundMode
+} // end of wxPMDCImpl::SetBackgroundMode

 
 

-void wxDC::SetLogicalFunction(
-  int                               nFunction
-)
+void wxPMDCImpl::SetLogicalFunction(wxRasterOperationMode nFunction)

 {
     m_logicalFunction = nFunction;
     SetRop((WXHDC)m_hDC);
 {
     m_logicalFunction = nFunction;
     SetRop((WXHDC)m_hDC);

-} // wxDC::SetLogicalFunction
+} // wxPMDCImpl::SetLogicalFunction

 
 

-void wxDC::SetRop(
-  WXHDC                             hDC
-)
+void wxPMDCImpl::SetRop(WXHDC hDC)

 {
     if (!hDC || m_logicalFunction < 0)
         return;
 
 {
     if (!hDC || m_logicalFunction < 0)
         return;
 

-    LONG                            lCRop;
+    LONG lCRop;

     switch (m_logicalFunction)
     {
         case wxXOR:
     switch (m_logicalFunction)
     {
         case wxXOR:


@@ -2186,33 +2085,31 @@ void wxDC::SetRop(
             break;
     }
     ::GpiSetMix((HPS)hDC, lCRop);
             break;
     }
     ::GpiSetMix((HPS)hDC, lCRop);

-} // end of wxDC::SetRop
+} // end of wxPMDCImpl::SetRop

 
 

-bool wxDC::StartDoc(
-  const wxString&                   rsMessage
-)
+bool wxPMDCImpl::StartDoc( const wxString& WXUNUSED(rsMessage) )

 {
 {

-    // We might be previewing, so return TRUE to let it continue.
-    return TRUE;
-} // end of wxDC::StartDoc
+    // We might be previewing, so return true to let it continue.
+    return true;
+} // end of wxPMDCImpl::StartDoc

 
 

-void wxDC::EndDoc()
+void wxPMDCImpl::EndDoc()

 {
 {

-} // end of wxDC::EndDoc
+} // end of wxPMDCImpl::EndDoc

 
 

-void wxDC::StartPage()
+void wxPMDCImpl::StartPage()

 {
 {

-} // end of wxDC::StartPage
+} // end of wxPMDCImpl::StartPage

 
 

-void wxDC::EndPage()
+void wxPMDCImpl::EndPage()

 {
 {

-} // end of wxDC::EndPage
+} // end of wxPMDCImpl::EndPage

 
 // ---------------------------------------------------------------------------
 // text metrics
 // ---------------------------------------------------------------------------
 
 
 // ---------------------------------------------------------------------------
 // text metrics
 // ---------------------------------------------------------------------------
 

-wxCoord wxDC::GetCharHeight() const
+wxCoord wxPMDCImpl::GetCharHeight() const

 {
     FONTMETRICS                     vFM; // metrics structure
 
 {
     FONTMETRICS                     vFM; // metrics structure
 


@@ -2223,7 +2120,7 @@ wxCoord wxDC::GetCharHeight() const
     return YDEV2LOGREL(vFM.lXHeight);
 }
 
     return YDEV2LOGREL(vFM.lXHeight);
 }
 

-wxCoord wxDC::GetCharWidth() const
+wxCoord wxPMDCImpl::GetCharWidth() const

 {
     FONTMETRICS                     vFM; // metrics structure
 
 {
     FONTMETRICS                     vFM; // metrics structure
 


@@ -2234,13 +2131,13 @@ wxCoord wxDC::GetCharWidth() const
     return XDEV2LOGREL(vFM.lAveCharWidth);
 }
 
     return XDEV2LOGREL(vFM.lAveCharWidth);
 }
 

-void wxDC::DoGetTextExtent(
+void wxPMDCImpl::DoGetTextExtent(

   const wxString&                   rsString
 , wxCoord*                          pvX
 , wxCoord*                          pvY
 , wxCoord*                          pvDescent
 , wxCoord*                          pvExternalLeading
   const wxString&                   rsString
 , wxCoord*                          pvX
 , wxCoord*                          pvY
 , wxCoord*                          pvDescent
 , wxCoord*                          pvExternalLeading

-, wxFont*                           pTheFont
+, const wxFont*                     pTheFont

 ) const
 {
     POINTL                          avPoint[TXTBOX_COUNT];
 ) const
 {
     POINTL                          avPoint[TXTBOX_COUNT];


@@ -2250,24 +2147,30 @@ void wxDC::DoGetTextExtent(
     int                             l;
     FONTMETRICS                     vFM; // metrics structure
     BOOL                            bRc;
     int                             l;
     FONTMETRICS                     vFM; // metrics structure
     BOOL                            bRc;

-    char*                           pStr;

     ERRORID                         vErrorCode; // last error id code
     wxFont*                         pFontToUse = (wxFont*)pTheFont;
 
     ERRORID                         vErrorCode; // last error id code
     wxFont*                         pFontToUse = (wxFont*)pTheFont;
 

-    char                            zMsg[128]; // DEBUG
+    wxChar                          zMsg[128]; // DEBUG

     wxString                        sError;
 
     if (!pFontToUse)
         pFontToUse = (wxFont*)&m_font;
     l = rsString.length();
     wxString                        sError;
 
     if (!pFontToUse)
         pFontToUse = (wxFont*)&m_font;
     l = rsString.length();

-    pStr = (PCH) rsString.c_str();

 
     //
     // In world coordinates.
     //
 
     //
     // In world coordinates.
     //

+    if (!m_hPS)
+    {
+    (void)wxMessageBox( wxT("wxWidgets core library")
+                ,"Using uninitialized DC for measuring text!\n"
+                ,wxICON_INFORMATION
+                );
+    }
+

     bRc = ::GpiQueryTextBox( m_hPS
                             ,l
     bRc = ::GpiQueryTextBox( m_hPS
                             ,l

-                            ,pStr
+                            ,rsString.char_str()

                             ,TXTBOX_COUNT // return maximum information
                             ,avPoint      // array of coordinates points
                            );
                             ,TXTBOX_COUNT // return maximum information
                             ,avPoint      // array of coordinates points
                            );


@@ -2276,8 +2179,8 @@ void wxDC::DoGetTextExtent(
        vErrorCode = ::WinGetLastError(wxGetInstance());
        sError = wxPMErrorToStr(vErrorCode);
        // DEBUG
        vErrorCode = ::WinGetLastError(wxGetInstance());
        sError = wxPMErrorToStr(vErrorCode);
        // DEBUG

-       sprintf(zMsg, "GpiQueryTextBox for %s: failed with Error: %x - %s", pStr, vErrorCode, sError.c_str());
-       (void)wxMessageBox( "wxWindows Menu sample"
+       wxSprintf(zMsg, wxT("GpiQueryTextBox for %s: failed with Error: %lx - %s"), rsString.c_str(), vErrorCode, sError.c_str());
+       (void)wxMessageBox( wxT("wxWidgets core library")

                           ,zMsg
                           ,wxICON_INFORMATION
                          );
                           ,zMsg
                           ,wxICON_INFORMATION
                          );


@@ -2309,21 +2212,21 @@ void wxDC::DoGetTextExtent(
         *pvExternalLeading = vFM.lExternalLeading;
 }
 
         *pvExternalLeading = vFM.lExternalLeading;
 }
 

-void wxDC::SetMapMode(
-  int                               nMode
+void wxPMDCImpl::SetMapMode(
+  wxMappingMode                     nMode

 )
 {
     int                             nPixelWidth = 0;
     int                             nPixelHeight = 0;
     int                             nMmWidth = 1;
     int                             nMmHeight = 1;
 )
 {
     int                             nPixelWidth = 0;
     int                             nPixelHeight = 0;
     int                             nMmWidth = 1;
     int                             nMmHeight = 1;

-    LONG                            lArray[CAPS_VERTICAL_RESOLUTION];
+    LONG                            lArray[CAPS_VERTICAL_RESOLUTION+1];

 
     m_mappingMode = nMode;
 
     if(::DevQueryCaps( m_hDC
 
     m_mappingMode = nMode;
 
     if(::DevQueryCaps( m_hDC

-                      ,CAPS_FAMILY
-                      ,CAPS_VERTICAL_RESOLUTION
+                      ,CAPS_FAMILY                  // id of first item
+                      ,CAPS_VERTICAL_RESOLUTION+1   // number of items wanted

                       ,lArray
                      ))
     {
                       ,lArray
                      ))
     {


@@ -2335,15 +2238,15 @@ void wxDC::SetMapMode(
         lHorzRes  = lArray[CAPS_HORIZONTAL_RESOLUTION]; // returns pel/meter
         lVertRes  = lArray[CAPS_VERTICAL_RESOLUTION];   // returns pel/meter
         nMmWidth  = (lHorzRes/1000) * nPixelWidth;
         lHorzRes  = lArray[CAPS_HORIZONTAL_RESOLUTION]; // returns pel/meter
         lVertRes  = lArray[CAPS_VERTICAL_RESOLUTION];   // returns pel/meter
         nMmWidth  = (lHorzRes/1000) * nPixelWidth;

-        nMmWidth = (lVertRes/1000) * nPixelHeight;
+        nMmHeight = (lVertRes/1000) * nPixelHeight;

     }
     if ((nPixelWidth == 0) || (nPixelHeight == 0) || (nMmWidth == 0) || (nMmHeight == 0))
     {
         return;
     }
 
     }
     if ((nPixelWidth == 0) || (nPixelHeight == 0) || (nMmWidth == 0) || (nMmHeight == 0))
     {
         return;
     }
 

-    double                          dMm2pixelsX = nPixelWidth/nMmWidth;
-    double                          dMm2pixelsY = nPixelHeight/nMmHeight;
+    double dMm2pixelsX = nPixelWidth/(double)nMmWidth;
+    double dMm2pixelsY = nPixelHeight/(double)nMmHeight;

 
     switch (nMode)
     {
 
     switch (nMode)
     {


@@ -2373,6 +2276,7 @@ void wxDC::SetMapMode(
             m_logicalScaleY = 1.0;
             break;
     }
             m_logicalScaleY = 1.0;
             break;
     }

+

     SIZEL                           vSize;
     ULONG                           ulOptions;
 
     SIZEL                           vSize;
     ULONG                           ulOptions;
 


@@ -2382,45 +2286,29 @@ void wxDC::SetMapMode(
         ulOptions = PU_ARBITRARY | GPIF_DEFAULT;
         ::GpiSetPS(m_hPS, &vSize, ulOptions);
     }
         ulOptions = PU_ARBITRARY | GPIF_DEFAULT;
         ::GpiSetPS(m_hPS, &vSize, ulOptions);
     }

-    m_nWindowExtX = (int)MS_XDEV2LOG(VIEWPORT_EXTENT);
-    m_nWindowExtY = (int)MS_YDEV2LOG(VIEWPORT_EXTENT);
-    // ????
-}; // end of wxDC::SetMapMode
+    ComputeScaleAndOrigin();

 
 

-void wxDC::SetUserScale(
-  double                            dX
-, double                            dY
-)
+}; // end of wxPMDCImpl::SetMapMode
+
+void wxPMDCImpl::SetUserScale( double dX,
+                         double dY )

 {
     m_userScaleX = dX;
     m_userScaleY = dY;
 
     SetMapMode(m_mappingMode);
 {
     m_userScaleX = dX;
     m_userScaleY = dY;
 
     SetMapMode(m_mappingMode);

-} // end of wxDC::SetUserScale
+} // end of wxPMDCImpl::SetUserScale

 
 

-void wxDC::SetAxisOrientation(
-  bool                              bXLeftRight
-, bool                              bYBottomUp
-)
+void wxPMDCImpl::SetAxisOrientation( bool bXLeftRight,
+                               bool bYBottomUp )

 {
     m_signX = bXLeftRight ? 1 : -1;
     m_signY = bYBottomUp ? -1 : 1;
 
     SetMapMode(m_mappingMode);
 {
     m_signX = bXLeftRight ? 1 : -1;
     m_signY = bYBottomUp ? -1 : 1;
 
     SetMapMode(m_mappingMode);

-} // end of wxDC::SetAxisOrientation
-
-void wxDC::SetSystemScale(
-  double                            dX
-, double                            dY
-)
-{
-    m_scaleX = dX;
-    m_scaleY = dY;
-
-    SetMapMode(m_mappingMode);
-} // end of wxDC::SetSystemScale
+} // end of wxPMDCImpl::SetAxisOrientation

 
 

-void wxDC::SetLogicalOrigin(
+void wxPMDCImpl::SetLogicalOrigin(

   wxCoord                           vX
 , wxCoord                           vY
 )
   wxCoord                           vX
 , wxCoord                           vY
 )


@@ -2437,9 +2325,9 @@ void wxDC::SetLogicalOrigin(
     ::GpiSetPageViewport( m_hPS
                          ,&vRect
                         );
     ::GpiSetPageViewport( m_hPS
                          ,&vRect
                         );

-}; // end of wxDC::SetLogicalOrigin
+}; // end of wxPMDCImpl::SetLogicalOrigin

 
 

-void wxDC::SetDeviceOrigin(
+void wxPMDCImpl::SetDeviceOrigin(

   wxCoord                           vX
 , wxCoord                           vY
 )
   wxCoord                           vX
 , wxCoord                           vY
 )


@@ -2458,87 +2346,46 @@ void wxDC::SetDeviceOrigin(
     ::GpiSetPageViewport( m_hPS
                          ,&vRect
                         );
     ::GpiSetPageViewport( m_hPS
                          ,&vRect
                         );

-}; // end of wxDC::SetDeviceOrigin
-
-// ---------------------------------------------------------------------------
-// coordinates transformations
-// ---------------------------------------------------------------------------
-
-wxCoord wxDCBase::DeviceToLogicalX(wxCoord x) const
-{
-    return (wxCoord) (((x) - m_deviceOriginX)/(m_logicalScaleX*m_userScaleX*m_signX*m_scaleX) - m_logicalOriginX);
-}
-
-wxCoord wxDCBase::DeviceToLogicalXRel(wxCoord x) const
-{
-       // axis orientation is not taken into account for conversion of a distance
-    return (wxCoord) ((x)/(m_logicalScaleX*m_userScaleX*m_scaleX));
-}
-
-wxCoord wxDCBase::DeviceToLogicalY(wxCoord y) const
-{
-    return (wxCoord) (((y) - m_deviceOriginY)/(m_logicalScaleY*m_userScaleY*m_signY*m_scaleY) - m_logicalOriginY);
-}
-
-wxCoord wxDCBase::DeviceToLogicalYRel(wxCoord y) const
-{
-       // axis orientation is not taken into account for conversion of a distance
-    return (wxCoord) ((y)/(m_logicalScaleY*m_userScaleY*m_scaleY));
-}
-
-wxCoord wxDCBase::LogicalToDeviceX(wxCoord x) const
-{
-    return (wxCoord) ((x - m_logicalOriginX)*m_logicalScaleX*m_userScaleX*m_signX*m_scaleX + m_deviceOriginX);
-}
-
-wxCoord wxDCBase::LogicalToDeviceXRel(wxCoord x) const
-{
-       // axis orientation is not taken into account for conversion of a distance
-    return (wxCoord) (x*m_logicalScaleX*m_userScaleX*m_scaleX);
-}
-
-wxCoord wxDCBase::LogicalToDeviceY(wxCoord y) const
-{
-    return (wxCoord) ((y - m_logicalOriginY)*m_logicalScaleY*m_userScaleY*m_signY*m_scaleY + m_deviceOriginY);
-}
-
-wxCoord wxDCBase::LogicalToDeviceYRel(wxCoord y) const
-{
-       // axis orientation is not taken into account for conversion of a distance
-    return (wxCoord) (y*m_logicalScaleY*m_userScaleY*m_scaleY);
-}
+}; // end of wxPMDCImpl::SetDeviceOrigin

 
 // ---------------------------------------------------------------------------
 // bit blit
 // ---------------------------------------------------------------------------
 
 
 // ---------------------------------------------------------------------------
 // bit blit
 // ---------------------------------------------------------------------------
 

-bool wxDC::DoBlit(
-  wxCoord                           vXdest
-, wxCoord                           vYdest
-, wxCoord                           vWidth
-, wxCoord                           vHeight
-, wxDC*                             pSource
-, wxCoord                           vXsrc
-, wxCoord                           vYsrc
-, int                               nRop
-, bool                              bUseMask
-, wxCoord                           vXsrcMask
-, wxCoord                           vYsrcMask
-)
+bool wxPMDCImpl::DoBlit( wxCoord vXdest,
+                   wxCoord vYdest,
+                   wxCoord vWidth,
+                   wxCoord vHeight,
+                   wxDC*   pSource,
+                   wxCoord vXsrc,
+                   wxCoord vYsrc,
+                   wxRasterOperationMode     nRop,
+                   bool    bUseMask,
+                   wxCoord WXUNUSED(vXsrcMask),
+                   wxCoord WXUNUSED(vYsrcMask) )

 {
     wxMask*                         pMask = NULL;
     CHARBUNDLE                      vCbnd;
     COLORREF                        vOldTextColor;
     COLORREF                        vOldBackground = ::GpiQueryBackColor(m_hPS);
 
 {
     wxMask*                         pMask = NULL;
     CHARBUNDLE                      vCbnd;
     COLORREF                        vOldTextColor;
     COLORREF                        vOldBackground = ::GpiQueryBackColor(m_hPS);
 

+    wxDCImpl *impl = pSource->GetImpl();
+    wxPMDCImpl *pm_impl = wxDynamicCast( impl, wxPMDCImpl );
+    if (!pm_impl)
+    {
+        // TODO: Do we want to be able to blit
+        //       from other DCs too?
+        return false;
+    }
+

     if (bUseMask)
     {
     if (bUseMask)
     {

-        const wxBitmap&             rBmp = pSource->m_vSelectedBitmap;
+        const wxBitmap&             rBmp = pm_impl->GetSelectedBitmap();

 
         pMask = rBmp.GetMask();
 
         pMask = rBmp.GetMask();

-        if (!(rBmp.Ok() && pMask && pMask->GetMaskBitmap()))
+        if (!(rBmp.IsOk() && pMask && pMask->GetMaskBitmap()))

         {
         {

-            bUseMask = FALSE;
+            bUseMask = false;

         }
     }
 
         }
     }
 


@@ -2549,7 +2396,7 @@ bool wxDC::DoBlit(
                    );
     vOldTextColor = (COLORREF)vCbnd.lColor;
 
                    );
     vOldTextColor = (COLORREF)vCbnd.lColor;
 

-    if (m_textForegroundColour.Ok())
+    if (m_textForegroundColour.IsOk())

     {
         vCbnd.lColor = (LONG)m_textForegroundColour.GetPixel();
         ::GpiSetAttrs( m_hPS           // presentation-space handle
     {
         vCbnd.lColor = (LONG)m_textForegroundColour.GetPixel();
         ::GpiSetAttrs( m_hPS           // presentation-space handle


@@ -2559,7 +2406,7 @@ bool wxDC::DoBlit(
                       ,&vCbnd          // buffer for attributes.
                      );
     }
                       ,&vCbnd          // buffer for attributes.
                      );
     }

-    if (m_textBackgroundColour.Ok())
+    if (m_textBackgroundColour.IsOk())

     {
         ::GpiSetBackColor(m_hPS, (LONG)m_textBackgroundColour.GetPixel());
     }
     {
         ::GpiSetBackColor(m_hPS, (LONG)m_textBackgroundColour.GetPixel());
     }


@@ -2586,7 +2433,7 @@ bool wxDC::DoBlit(
         case wxNOR:          lRop = ROP_NOTSRCCOPY;       break;
         default:
            wxFAIL_MSG( wxT("unsupported logical function") );
         case wxNOR:          lRop = ROP_NOTSRCCOPY;       break;
         default:
            wxFAIL_MSG( wxT("unsupported logical function") );

-           return FALSE;
+           return false;

     }
 
     bool                            bSuccess;
     }
 
     bool                            bSuccess;


@@ -2618,13 +2465,12 @@ bool wxDC::DoBlit(
         vBmpHdr.cBitCount = 24;
 
 #if wxUSE_DC_CACHEING
         vBmpHdr.cBitCount = 24;
 
 #if wxUSE_DC_CACHEING

-        if (TRUE)

         {
             //
             // create a temp buffer bitmap and DCs to access it and the mask
             //
             wxDCCacheEntry*         pDCCacheEntry1    = FindDCInCache( NULL
         {
             //
             // create a temp buffer bitmap and DCs to access it and the mask
             //
             wxDCCacheEntry*         pDCCacheEntry1    = FindDCInCache( NULL

-                                                                      ,pSource->GetHPS()
+                                                                      ,pm_impl->GetHPS()

                                                                      );
             wxDCCacheEntry*         pDCCacheEntry2    = FindDCInCache( pDCCacheEntry1
                                                                       ,GetHPS()
                                                                      );
             wxDCCacheEntry*         pDCCacheEntry2    = FindDCInCache( pDCCacheEntry1
                                                                       ,GetHPS()


@@ -2637,9 +2483,9 @@ bool wxDC::DoBlit(
             hPSMask = pDCCacheEntry1->m_hPS;
             hDCBuffer = (HDC)pDCCacheEntry2->m_hPS;
             hBufBitmap = (HBITMAP)pBitmapCacheEntry->m_hBitmap;
             hPSMask = pDCCacheEntry1->m_hPS;
             hDCBuffer = (HDC)pDCCacheEntry2->m_hPS;
             hBufBitmap = (HBITMAP)pBitmapCacheEntry->m_hBitmap;

+            wxUnusedVar(hDCMask);

         }
         }

-        else
-#endif
+#else

         {
             hDCMask = ::DevOpenDC(vHabmain, OD_MEMORY, "*", 5L, (PDEVOPENDATA)&vDOP, NULLHANDLE);
             hDCBuffer = ::DevOpenDC(vHabmain, OD_MEMORY, "*", 5L, (PDEVOPENDATA)&vDOP, NULLHANDLE);
         {
             hDCMask = ::DevOpenDC(vHabmain, OD_MEMORY, "*", 5L, (PDEVOPENDATA)&vDOP, NULLHANDLE);
             hDCBuffer = ::DevOpenDC(vHabmain, OD_MEMORY, "*", 5L, (PDEVOPENDATA)&vDOP, NULLHANDLE);


@@ -2647,26 +2493,27 @@ bool wxDC::DoBlit(
             hPSBuffer = ::GpiCreatePS(vHabmain, hDCBuffer, &vSize, PU_PELS | GPIT_MICRO | GPIA_ASSOC);
             hBufBitmap = ::GpiCreateBitmap(GetHPS(), &vBmpHdr, 0L, NULL, NULL);
         }
             hPSBuffer = ::GpiCreatePS(vHabmain, hDCBuffer, &vSize, PU_PELS | GPIT_MICRO | GPIA_ASSOC);
             hBufBitmap = ::GpiCreateBitmap(GetHPS(), &vBmpHdr, 0L, NULL, NULL);
         }

+#endif

 
         POINTL                          aPoint1[4] = { {0, 0}
 
         POINTL                          aPoint1[4] = { {0, 0}

-                                                     ,{vWidth, vHeight}
-                                                     ,{vXdest, vYdest}
-                                                     ,{vXdest + vWidth, vYdest + vHeight}
+                              ,{vWidth, vHeight}
+                              ,{vXdest, vYdest}
+                              ,{vXdest + vWidth, vYdest + vHeight}

                                                      };
         POINTL                          aPoint2[4] = { {0, 0}
                                                      };
         POINTL                          aPoint2[4] = { {0, 0}

-                                                     ,{vWidth, vHeight}
-                                                     ,{vXsrc, vYsrc}
-                                                     ,{vXsrc + vWidth, vYsrc + vHeight}
+                              ,{vWidth, vHeight}
+                              ,{vXsrc, vYsrc}
+                              ,{vXsrc + vWidth, vYsrc + vHeight}

                                                      };
         POINTL                          aPoint3[4] = { {vXdest, vYdest}
                                                      };
         POINTL                          aPoint3[4] = { {vXdest, vYdest}

-                                                     ,{vXdest + vWidth, vYdest + vHeight}
-                                                     ,{vXsrc, vYsrc}
-                                                     ,{vXsrc + vWidth, vYsrc + vHeight}
+                              ,{vXdest + vWidth, vYdest + vHeight}
+                              ,{vXsrc, vYsrc}
+                              ,{vXsrc + vWidth, vYsrc + vHeight}

                                                      };
         POINTL                          aPoint4[4] = { {vXdest, vYdest}
                                                      };
         POINTL                          aPoint4[4] = { {vXdest, vYdest}

-                                                     ,{vXdest + vWidth, vYdest + vHeight}
-                                                     ,{0, 0}
-                                                     ,{vWidth, vHeight}
+                              ,{vXdest + vWidth, vYdest + vHeight}
+                              ,{0, 0}
+                              ,{vWidth, vHeight}

                                                      };
         ::GpiSetBitmap(hPSMask, (HBITMAP) pMask->GetMaskBitmap());
         ::GpiSetBitmap(hPSBuffer, (HBITMAP) hBufBitmap);
                                                      };
         ::GpiSetBitmap(hPSMask, (HBITMAP) pMask->GetMaskBitmap());
         ::GpiSetBitmap(hPSBuffer, (HBITMAP) hBufBitmap);


@@ -2757,7 +2604,7 @@ bool wxDC::DoBlit(
                         );
         if (rc == GPI_ERROR)
         {
                         );
         if (rc == GPI_ERROR)
         {

-            bSuccess = FALSE;
+            bSuccess = false;

             wxLogLastError(wxT("BitBlt"));
         }
 
             wxLogLastError(wxT("BitBlt"));
         }
 


@@ -2773,18 +2620,18 @@ bool wxDC::DoBlit(
         ::DevCloseDC(hDCBuffer);
         ::GpiDeleteBitmap(hBufBitmap);
 #endif
         ::DevCloseDC(hDCBuffer);
         ::GpiDeleteBitmap(hBufBitmap);
 #endif

-        bSuccess = TRUE;
+        bSuccess = true;

     }
     else // no mask, just BitBlt() it
     {
       POINTL                          aPoint[4] = { {vXdest, vYdest}
     }
     else // no mask, just BitBlt() it
     {
       POINTL                          aPoint[4] = { {vXdest, vYdest}

-                                                  ,{vXdest + vWidth, vYdest + vHeight}
-                                                  ,{vXsrc, vYsrc}
-                                                  ,{vXsrc + vWidth, vYsrc + vHeight}
+                           ,{vXdest + vWidth, vYdest + vHeight}
+                           ,{vXsrc, vYsrc}
+                           ,{vXsrc + vWidth, vYsrc + vHeight}

                                                     };
 
         bSuccess = (::GpiBitBlt( m_hPS
                                                     };
 
         bSuccess = (::GpiBitBlt( m_hPS

-                                ,pSource->GetHPS()
+                                ,pm_impl->GetHPS()

                                 ,4L
                                 ,aPoint
                                 ,lRop
                                 ,4L
                                 ,aPoint
                                 ,lRop


@@ -2806,60 +2653,62 @@ bool wxDC::DoBlit(
     return bSuccess;
 }
 
     return bSuccess;
 }
 

-void wxDC::DoGetSize(
-  int*                              pnWidth
-, int*                              pnHeight
-) const
+void wxPMDCImpl::DoGetSize( int* pnWidth,
+                            int* pnHeight ) const

 {
 {

-    LONG                            lArray[CAPS_HEIGHT];
+    LONG lArray[CAPS_HEIGHT+1];

 
     if(::DevQueryCaps( m_hDC
                       ,CAPS_FAMILY
 
     if(::DevQueryCaps( m_hDC
                       ,CAPS_FAMILY

-                      ,CAPS_HEIGHT
+                      ,CAPS_HEIGHT+1

                       ,lArray
                      ))
     {
                       ,lArray
                      ))
     {

-        *pnWidth  = lArray[CAPS_WIDTH];
-        *pnHeight = lArray[CAPS_HEIGHT];
+        if (pnWidth)
+            *pnWidth  = lArray[CAPS_WIDTH];
+        if (pnHeight)
+            *pnHeight = lArray[CAPS_HEIGHT];

     }
     }

-}; // end of wxDC::DoGetSize(
+}; // end of wxPMDCImpl::DoGetSize(

 
 

-void wxDC::DoGetSizeMM(
-  int*                              pnWidth
-, int*                              pnHeight
-) const
+void wxPMDCImpl::DoGetSizeMM( int* pnWidth,
+                        int* pnHeight ) const

 {
 {

-    LONG                            lArray[CAPS_VERTICAL_RESOLUTION];
+    LONG                            lArray[CAPS_VERTICAL_RESOLUTION+1];

 
     if(::DevQueryCaps( m_hDC
                       ,CAPS_FAMILY
 
     if(::DevQueryCaps( m_hDC
                       ,CAPS_FAMILY

-                      ,CAPS_VERTICAL_RESOLUTION
+                      ,CAPS_VERTICAL_RESOLUTION+1

                       ,lArray
                      ))
     {
                       ,lArray
                      ))
     {

-        int                         nWidth;
-        int                         nHeight;
-        int                         nHorzRes;
-        int                         nVertRes;
+        if(pnWidth)
+        {
+            int nWidth   = lArray[CAPS_WIDTH];
+            int nHorzRes = lArray[CAPS_HORIZONTAL_RESOLUTION]; // returns pel/meter
+            // use fp to avoid returning 0 if nHorzRes < 1000
+            *pnWidth  = (int)((nHorzRes/1000.0) * nWidth);
+        }

 
 

-        nWidth    = lArray[CAPS_WIDTH];
-        nHeight   = lArray[CAPS_HEIGHT];
-        nHorzRes  = lArray[CAPS_HORIZONTAL_RESOLUTION]; // returns pel/meter
-        nVertRes  = lArray[CAPS_VERTICAL_RESOLUTION];   // returns pel/meter
-        nWidth  = (nHorzRes/1000) * nWidth;
-        nHeight = (nVertRes/1000) * nHeight;
+        if(pnHeight)
+        {
+            int nHeight  = lArray[CAPS_HEIGHT];
+            int nVertRes = lArray[CAPS_VERTICAL_RESOLUTION];   // returns pel/meter
+            // use fp to avoid returning 0 if nVertRes < 1000
+            *pnHeight = (int)((nVertRes/1000.0) * nHeight);
+        }

     }
     }

-}; // end of wxDC::DoGetSizeMM
+}; // end of wxPMDCImpl::DoGetSizeMM

 
 

-wxSize wxDC::GetPPI() const
+wxSize wxPMDCImpl::GetPPI() const

 {
 {

-    LONG                            lArray[CAPS_VERTICAL_RESOLUTION];
-    int                             nWidth;
-    int                             nHeight;
+    LONG                            lArray[CAPS_VERTICAL_RESOLUTION+1];
+    int                             nWidth = 0;
+    int                             nHeight = 0;

 
     if(::DevQueryCaps( m_hDC
                       ,CAPS_FAMILY
 
     if(::DevQueryCaps( m_hDC
                       ,CAPS_FAMILY

-                      ,CAPS_VERTICAL_RESOLUTION
+                      ,CAPS_VERTICAL_RESOLUTION+1

                       ,lArray
                      ))
     {
                       ,lArray
                      ))
     {


@@ -2872,20 +2721,15 @@ wxSize wxDC::GetPPI() const
         nPelHeight = lArray[CAPS_HEIGHT];
         nHorzRes   = lArray[CAPS_HORIZONTAL_RESOLUTION]; // returns pel/meter
         nVertRes   = lArray[CAPS_VERTICAL_RESOLUTION];   // returns pel/meter
         nPelHeight = lArray[CAPS_HEIGHT];
         nHorzRes   = lArray[CAPS_HORIZONTAL_RESOLUTION]; // returns pel/meter
         nVertRes   = lArray[CAPS_VERTICAL_RESOLUTION];   // returns pel/meter

-        nWidth   = (nHorzRes/39.3) * nPelWidth;
-        nHeight  = (nVertRes/39.3) * nPelHeight;
+        nWidth   = (int)((nHorzRes/39.3) * nPelWidth);
+        nHeight  = (int)((nVertRes/39.3) * nPelHeight);

     }
     }

-    return (wxSize(nWidth,nHeight));
-} // end of wxDC::GetPPI
+    wxSize ppisize(nWidth, nHeight);
+    return ppisize;
+} // end of wxPMDCImpl::GetPPI

 
 

-void wxDC::SetLogicalScale(
-  double                            dX
-, double                            dY
-)
+void wxPMDCImpl::SetLogicalScale( double dX, double dY )

 {
     m_logicalScaleX = dX;
     m_logicalScaleY = dY;
 {
     m_logicalScaleX = dX;
     m_logicalScaleY = dY;

-}; // end of wxDC::SetLogicalScale
-
-
-
+}; // end of wxPMDCImpl::SetLogicalScale