]> git.saurik.com Git - wxWidgets.git/blobdiff - include/wx/dc.h
access TLS cache variable directly and not via a helper function when using compiler...
[wxWidgets.git] / include / wx / dc.h
index 6a86ae856d649a0467df65b22c0bd221b59835cf..10e091b2289f53cbd5c036c06529b27d2eb51257 100644 (file)
@@ -1,7 +1,7 @@
 /////////////////////////////////////////////////////////////////////////////
 // Name:        wx/dc.h
 // Purpose:     wxDC class
-// Author:      Vadim Zeitlin
+// Author:      Vadim Zeitlin 
 // Modified by:
 // Created:     05/25/99
 // RCS-ID:      $Id$
@@ -40,12 +40,13 @@ class WXDLLIMPEXP_FWD_CORE wxWindowDC;
 class WXDLLIMPEXP_FWD_CORE wxScreenDC;
 class WXDLLIMPEXP_FWD_CORE wxMemoryDC;
 class WXDLLIMPEXP_FWD_CORE wxPrinterDC;
+class WXDLLIMPEXP_FWD_CORE wxPrintData;
 
 //-----------------------------------------------------------------------------
 // wxDrawObject helper class
 //-----------------------------------------------------------------------------
 
-class WXDLLEXPORT wxDrawObject
+class WXDLLIMPEXP_CORE wxDrawObject
 {
 public:
 
@@ -114,21 +115,21 @@ class WXDLLIMPEXP_CORE wxDCFactory
 public:
     wxDCFactory() {}
     virtual ~wxDCFactory() {}
-    
-    virtual wxDCImpl* CreateWindowDC( wxWindowDC *owner ) = 0;
+
     virtual wxDCImpl* CreateWindowDC( wxWindowDC *owner, wxWindow *window ) = 0;
-    virtual wxDCImpl* CreateClientDC( wxClientDC *owner ) = 0;
     virtual wxDCImpl* CreateClientDC( wxClientDC *owner, wxWindow *window ) = 0;
-    virtual wxDCImpl* CreatePaintDC( wxPaintDC *owner ) = 0;
     virtual wxDCImpl* CreatePaintDC( wxPaintDC *owner, wxWindow *window ) = 0;
     virtual wxDCImpl* CreateMemoryDC( wxMemoryDC *owner ) = 0;
     virtual wxDCImpl* CreateMemoryDC( wxMemoryDC *owner, wxBitmap &bitmap ) = 0;
     virtual wxDCImpl* CreateMemoryDC( wxMemoryDC *owner, wxDC *dc ) = 0;
     virtual wxDCImpl* CreateScreenDC( wxScreenDC *owner ) = 0;
+#if wxUSE_PRINTING_ARCHITECTURE
     virtual wxDCImpl* CreatePrinterDC( wxPrinterDC *owner, const wxPrintData &data  ) = 0;
-    
-    static void SetDCFactory( wxDCFactory *factory );
-    static wxDCFactory *GetFactory();
+#endif
+
+    static void Set(wxDCFactory *factory);
+    static wxDCFactory *Get();
+
 private:
     static wxDCFactory *m_factory;
 };
@@ -141,18 +142,17 @@ class WXDLLIMPEXP_CORE wxNativeDCFactory: public wxDCFactory
 {
 public:
     wxNativeDCFactory() {}
-    
-    virtual wxDCImpl* CreateWindowDC( wxWindowDC *owner );
+
     virtual wxDCImpl* CreateWindowDC( wxWindowDC *owner, wxWindow *window );
-    virtual wxDCImpl* CreateClientDC( wxClientDC *owner );
     virtual wxDCImpl* CreateClientDC( wxClientDC *owner, wxWindow *window );
-    virtual wxDCImpl* CreatePaintDC( wxPaintDC *owner );
     virtual wxDCImpl* CreatePaintDC( wxPaintDC *owner, wxWindow *window );
     virtual wxDCImpl* CreateMemoryDC( wxMemoryDC *owner );
     virtual wxDCImpl* CreateMemoryDC( wxMemoryDC *owner, wxBitmap &bitmap );
     virtual wxDCImpl* CreateMemoryDC( wxMemoryDC *owner, wxDC *dc );
     virtual wxDCImpl* CreateScreenDC( wxScreenDC *owner );
+#if wxUSE_PRINTING_ARCHITECTURE
     virtual wxDCImpl* CreatePrinterDC( wxPrinterDC *owner, const wxPrintData &data  );
+#endif
 };
 
 //-----------------------------------------------------------------------------
@@ -163,12 +163,12 @@ class WXDLLIMPEXP_CORE wxDCImpl: public wxObject
 {
 public:
     wxDCImpl( wxDC *owner );
-    ~wxDCImpl();
-    
+    virtual ~wxDCImpl();
+
     wxDC *GetOwner() const { return m_owner; }
-    
+
     wxWindow* GetWindow() const { return m_window; }
-    
+
     virtual bool IsOk() const { return m_ok; }
 
     // query capabilities
@@ -176,21 +176,40 @@ public:
     virtual bool CanDrawBitmap() const = 0;
     virtual bool CanGetTextExtent() const = 0;
 
+    // get Cairo context
+    virtual void* GetCairoContext() const
+    {
+        return NULL;
+    }
+
     // query dimension, colour deps, resolution
 
     virtual void DoGetSize(int *width, int *height) const = 0;
+    void GetSize(int *width, int *height) const
+    {
+        DoGetSize(width, height);
+        return ;
+    }
+
+    wxSize GetSize() const
+    {
+        int w, h;
+        DoGetSize(&w, &h);
+        return wxSize(w, h);
+    }
+
     virtual void DoGetSizeMM(int* width, int* height) const = 0;
-    
+
     virtual int GetDepth() const = 0;
     virtual wxSize GetPPI() const = 0;
-    
+
     // Right-To-Left (RTL) modes
-    
+
     virtual void SetLayoutDirection(wxLayoutDirection WXUNUSED(dir)) { }
     virtual wxLayoutDirection GetLayoutDirection() const  { return wxLayout_Default; }
-    
-    // page and document 
-    
+
+    // page and document
+
     virtual bool StartDoc(const wxString& WXUNUSED(message)) { return true; }
     virtual void EndDoc() { }
 
@@ -232,38 +251,47 @@ public:
     wxCoord MaxX() const { return m_maxX; }
     wxCoord MinY() const { return m_minY; }
     wxCoord MaxY() const { return m_maxY; }
-    
+
     // setters and getters
 
     virtual void SetFont(const wxFont& font) = 0;
-    virtual const wxFont&   GetFont() const { return m_font; }
-    
+    virtual const wxFont& GetFont() const { return m_font; }
+
     virtual void SetPen(const wxPen& pen) = 0;
-    virtual const wxPen&    GetPen() const { return m_pen; }
-    
+    virtual const wxPen& GetPen() const { return m_pen; }
+
     virtual void SetBrush(const wxBrush& brush) = 0;
-    virtual const wxBrush&  GetBrush() const { return m_brush; }
-    
+    virtual const wxBrush& GetBrush() const { return m_brush; }
+
     virtual void SetBackground(const wxBrush& brush) = 0;
-    virtual const wxBrush&  GetBackground() const { return m_backgroundBrush; }
-    
+    virtual const wxBrush& GetBackground() const { return m_backgroundBrush; }
+
     virtual void SetBackgroundMode(int mode) = 0;
     virtual int GetBackgroundMode() const { return m_backgroundMode; }
 
     virtual void SetTextForeground(const wxColour& colour)
         { m_textForegroundColour = colour; }
-    virtual const wxColour& GetTextForeground() const { return m_textForegroundColour; }
-    
+    virtual const wxColour& GetTextForeground() const
+        { return m_textForegroundColour; }
+
     virtual void SetTextBackground(const wxColour& colour)
         { m_textBackgroundColour = colour; }
-    virtual const wxColour& GetTextBackground() const { return m_textBackgroundColour; }
+    virtual const wxColour& GetTextBackground() const
+        { return m_textBackgroundColour; }
 
 #if wxUSE_PALETTE
     virtual void SetPalette(const wxPalette& palette) = 0;
 #endif // wxUSE_PALETTE
 
+    // inherit the DC attributes (font and colours) from the given window
+    //
+    // this is called automatically when a window, client or paint DC is
+    // created
+    virtual void InheritAttributes(wxWindow *win);
+
+
     // logical functions
-    
+
     virtual void SetLogicalFunction(int function) = 0;
     virtual int GetLogicalFunction() const { return m_logicalFunction; }
 
@@ -282,16 +310,18 @@ public:
                                         wxCoord *heightLine = NULL,
                                         const wxFont *font = NULL) const;
     virtual bool DoGetPartialTextExtents(const wxString& text, wxArrayInt& widths) const;
-    
+
     // clearing
-    
+
     virtual void Clear() = 0;
 
     // clipping
 
     virtual void DoSetClippingRegion(wxCoord x, wxCoord y,
                                      wxCoord width, wxCoord height) = 0;
-    virtual void DoSetClippingRegionAsRegion(const wxRegion& region) = 0;
+
+    // NB: this function works with device coordinates, not the logical ones!
+    virtual void DoSetDeviceClippingRegion(const wxRegion& region) = 0;
 
     virtual void DoGetClippingBox(wxCoord *x, wxCoord *y,
                                   wxCoord *w, wxCoord *h) const
@@ -350,7 +380,7 @@ public:
         if ( x ) *x = m_deviceOriginX;
         if ( y ) *y = m_deviceOriginY;
     }
-        
+
     virtual void SetDeviceLocalOrigin( wxCoord x, wxCoord y );
 
     virtual void ComputeScaleAndOrigin();
@@ -431,7 +461,7 @@ public:
                              wxCoord xoffset, wxCoord yoffset ) = 0;
     virtual void DrawLines(const wxPointList *list,
                            wxCoord xoffset, wxCoord yoffset );
-                           
+
     virtual void DoDrawPolygon(int n, wxPoint points[],
                                wxCoord xoffset, wxCoord yoffset,
                                int fillStyle = wxODDEVEN_RULE) = 0;
@@ -444,35 +474,39 @@ public:
 
 
 #if wxUSE_SPLINES
-    virtual void DoDrawSpline(wxCoord x1, wxCoord y1, wxCoord x2, wxCoord y2, wxCoord x3, wxCoord y3);
-    virtual void DoDrawSpline(int n, wxPoint points[]);
+    void DrawSpline(wxCoord x1, wxCoord y1,
+                            wxCoord x2, wxCoord y2,
+                            wxCoord x3, wxCoord y3);
+    void DrawSpline(int n, wxPoint points[]);
+    void DrawSpline(const wxPointList *points) { DoDrawSpline(points); }
+
     virtual void DoDrawSpline(const wxPointList *points);
 #endif
-    
+
     // ---------------------------------------------------------
     // wxMemoryDC Impl API
 
     virtual void DoSelect(const wxBitmap& WXUNUSED(bmp))
        { }
-    
+
     virtual const wxBitmap& GetSelectedBitmap() const
         { return wxNullBitmap; }
     virtual wxBitmap& GetSelectedBitmap()
         { return wxNullBitmap; }
-  
+
     // ---------------------------------------------------------
     // wxPrinterDC Impl API
 
     virtual wxRect GetPaperRect()
         { int w = 0; int h = 0; DoGetSize( &w, &h ); return wxRect(0,0,w,h); }
-        
+
     virtual int GetResolution()
         { return -1; }
 
 private:
     wxDC       *m_owner;
 
-protected:    
+protected:
     // unset clipping variables (after clipping region was destroyed)
     void ResetClipping()
     {
@@ -481,9 +515,57 @@ protected:
         m_clipX1 = m_clipX2 = m_clipY1 = m_clipY2 = 0;
     }
 
+#ifdef __WXWINCE__
+    //! Generic method to draw ellipses, circles and arcs with current pen and brush.
+    /*! \param x Upper left corner of bounding box.
+     *  \param y Upper left corner of bounding box.
+     *  \param w Width of bounding box.
+     *  \param h Height of bounding box.
+     *  \param sa Starting angle of arc
+     *            (counterclockwise, start at 3 o'clock, 360 is full circle).
+     *  \param ea Ending angle of arc.
+     *  \param angle Rotation angle, the Arc will be rotated after
+     *               calculating begin and end.
+     */
+    void DrawEllipticArcRot( wxCoord x, wxCoord y,
+                             wxCoord width, wxCoord height,
+                             double sa = 0, double ea = 0, double angle = 0 )
+    { DoDrawEllipticArcRot( x, y, width, height, sa, ea, angle ); }
+
+    void DrawEllipticArcRot( const wxPoint& pt,
+                             const wxSize& sz,
+                             double sa = 0, double ea = 0, double angle = 0 )
+    { DoDrawEllipticArcRot( pt.x, pt.y, sz.x, sz.y, sa, ea, angle ); }
+
+    void DrawEllipticArcRot( const wxRect& rect,
+                             double sa = 0, double ea = 0, double angle = 0 )
+    { DoDrawEllipticArcRot( rect.x, rect.y, rect.width, rect.height, sa, ea, angle ); }
+
+    virtual void DoDrawEllipticArcRot( wxCoord x, wxCoord y,
+                                       wxCoord w, wxCoord h,
+                                       double sa = 0, double ea = 0, double angle = 0 );
+
+    //! Rotates points around center.
+    /*! This is a quite straight method, it calculates in pixels
+     *  and so it produces rounding errors.
+     *  \param points The points inside will be rotated.
+     *  \param angle Rotating angle (counterclockwise, start at 3 o'clock, 360 is full circle).
+     *  \param center Center of rotation.
+     */
+    void Rotate( wxPointList* points, double angle, wxPoint center = wxPoint(0,0) );
+
+    // used by DrawEllipticArcRot
+    // Careful: wxList gets filled with points you have to delete later.
+    void CalculateEllipticPoints( wxPointList* points,
+                                  wxCoord xStart, wxCoord yStart,
+                                  wxCoord w, wxCoord h,
+                                  double sa, double ea );
+#endif // __WXWINCE__
+
+
     // window on which the DC draws or NULL
     wxWindow   *m_window;
-    
+
     // flags
     bool m_colour:1;
     bool m_ok:1;
@@ -495,7 +577,7 @@ protected:
 
     wxCoord m_logicalOriginX, m_logicalOriginY;
     wxCoord m_deviceOriginX, m_deviceOriginY;           // Usually 0,0, can be change by user
-    
+
     wxCoord m_deviceLocalOriginX, m_deviceLocalOriginY; // non-zero if native top-left corner
                                                         // is not at 0,0. This was the case under
                                                         // Mac's GrafPorts (coordinate system
@@ -507,11 +589,11 @@ protected:
     double m_scaleX, m_scaleY;  // calculated from logical scale and user scale
 
     int m_signX, m_signY;  // Used by SetAxisOrientation() to invert the axes
-    
+
     // what is a mm on a screen you don't know the size of?
     double       m_mm_to_pix_x,
                  m_mm_to_pix_y;
-                 
+
     // bounding and clipping boxes
     wxCoord m_minX, m_minY, m_maxX, m_maxY;
     wxCoord m_clipX1, m_clipY1, m_clipX2, m_clipY2;
@@ -537,41 +619,35 @@ private:
 };
 
 
-class WXDLLIMPEXP_CORE wxDC: public wxObject
+class WXDLLIMPEXP_CORE wxDC : public wxObject
 {
 public:
-    wxDC()   { m_pimpl = NULL; }
-    ~wxDC()  { if (m_pimpl) delete m_pimpl; }
+    virtual ~wxDC() { delete m_pimpl; }
 
     wxDCImpl *GetImpl()
         { return m_pimpl; }
     const wxDCImpl *GetImpl() const
         { return m_pimpl; }
 
-    wxWindow *GetWindow()
+    wxWindow *GetWindow() const
         { return m_pimpl->GetWindow(); }
 
-    bool IsOk() const 
+    bool IsOk() const
         { return m_pimpl && m_pimpl->IsOk(); }
 
     // query capabilities
 
-    bool CanDrawBitmap() const 
+    bool CanDrawBitmap() const
         { return m_pimpl->CanDrawBitmap(); }
     bool CanGetTextExtent() const
         { return m_pimpl->CanGetTextExtent(); }
-        
+
     // query dimension, colour deps, resolution
 
     void GetSize(int *width, int *height) const
         { m_pimpl->DoGetSize(width, height); }
-        
     wxSize GetSize() const
-    {
-        int w, h;
-        m_pimpl->DoGetSize(&w, &h);
-        return wxSize(w, h);
-    }
+        { return m_pimpl->GetSize(); }
 
     void GetSizeMM(int* width, int* height) const
         { m_pimpl->DoGetSizeMM(width, height); }
@@ -581,7 +657,7 @@ public:
         m_pimpl->DoGetSizeMM(&w, &h);
         return wxSize(w, h);
     }
-    
+
     int GetDepth() const
         { return m_pimpl->GetDepth(); }
     wxSize GetPPI() const
@@ -589,16 +665,16 @@ public:
 
     virtual int GetResolution()
         { return m_pimpl->GetResolution(); }
-        
+
     // Right-To-Left (RTL) modes
-    
+
     void SetLayoutDirection(wxLayoutDirection dir)
         { m_pimpl->SetLayoutDirection( dir ); }
     wxLayoutDirection GetLayoutDirection() const
         { return m_pimpl->GetLayoutDirection(); }
-    
-    // page and document 
-    
+
+    // page and document
+
     bool StartDoc(const wxString& message)
         { return m_pimpl->StartDoc(message); }
     void EndDoc()
@@ -610,12 +686,12 @@ public:
         { m_pimpl->EndPage(); }
 
     // bounding box
-    
+
     void CalcBoundingBox(wxCoord x, wxCoord y)
         { m_pimpl->CalcBoundingBox(x,y); }
     void ResetBoundingBox()
         { m_pimpl->ResetBoundingBox(); }
-        
+
     wxCoord MinX() const
         { return m_pimpl->MinX(); }
     wxCoord MaxX() const
@@ -624,29 +700,29 @@ public:
         { return m_pimpl->MinY(); }
     wxCoord MaxY() const
         { return m_pimpl->MaxY(); }
-    
+
     // setters and getters
 
     void SetFont(const wxFont& font)
         { m_pimpl->SetFont( font ); }
-    const wxFont&   GetFont() const 
+    const wxFont&   GetFont() const
         { return m_pimpl->GetFont(); }
-    
+
     void SetPen(const wxPen& pen)
         { m_pimpl->SetPen( pen ); }
-    const wxPen&    GetPen() const 
+    const wxPen&    GetPen() const
         { return m_pimpl->GetPen(); }
-    
+
     void SetBrush(const wxBrush& brush)
         { m_pimpl->SetBrush( brush ); }
     const wxBrush&  GetBrush() const
         { return m_pimpl->GetBrush(); }
-    
+
     void SetBackground(const wxBrush& brush)
         { m_pimpl->SetBackground( brush ); }
     const wxBrush&  GetBackground() const
         { return m_pimpl->GetBackground(); }
-    
+
     void SetBackgroundMode(int mode)
         { m_pimpl->SetBackgroundMode( mode ); }
     int GetBackgroundMode() const
@@ -656,31 +732,31 @@ public:
         { m_pimpl->SetTextForeground(colour); }
     const wxColour& GetTextForeground() const
         { return m_pimpl->GetTextForeground(); }
-    
+
     void SetTextBackground(const wxColour& colour)
         { m_pimpl->SetTextBackground(colour); }
-    const wxColour& GetTextBackground() const 
+    const wxColour& GetTextBackground() const
         { return m_pimpl->GetTextBackground(); }
 
 #if wxUSE_PALETTE
     void SetPalette(const wxPalette& palette)
         { m_pimpl->SetPalette(palette); }
 #endif // wxUSE_PALETTE
-    
+
     // logical functions
-    
+
     void SetLogicalFunction(int function)
         { m_pimpl->SetLogicalFunction(function); }
     int GetLogicalFunction() const
         { return m_pimpl->GetLogicalFunction(); }
-        
+
     // text measurement
 
     wxCoord GetCharHeight() const
         { return m_pimpl->GetCharHeight(); }
     wxCoord GetCharWidth() const
         { return m_pimpl->GetCharWidth(); }
-        
+
     void GetTextExtent(const wxString& string,
                        wxCoord *x, wxCoord *y,
                        wxCoord *descent = NULL,
@@ -694,26 +770,26 @@ public:
         m_pimpl->DoGetTextExtent(string, &w, &h);
         return wxSize(w, h);
     }
-        
+
     void GetMultiLineTextExtent(const wxString& string,
                                         wxCoord *width,
                                         wxCoord *height,
                                         wxCoord *heightLine = NULL,
                                         const wxFont *font = NULL) const
         { m_pimpl->GetMultiLineTextExtent( string, width, height, heightLine, font ); }
-        
+
     wxSize GetMultiLineTextExtent(const wxString& string) const
     {
         wxCoord w, h;
         m_pimpl->GetMultiLineTextExtent(string, &w, &h);
         return wxSize(w, h);
     }
-        
+
     bool GetPartialTextExtents(const wxString& text, wxArrayInt& widths) const
         { return m_pimpl->DoGetPartialTextExtents(text, widths); }
-                                        
+
     // clearing
-    
+
     void Clear()
         { m_pimpl->Clear(); }
 
@@ -725,8 +801,21 @@ public:
         { m_pimpl->DoSetClippingRegion(pt.x, pt.y, sz.x, sz.y); }
     void SetClippingRegion(const wxRect& rect)
         { m_pimpl->DoSetClippingRegion(rect.x, rect.y, rect.width, rect.height); }
-    void SetClippingRegion(const wxRegion& region)
-        { m_pimpl->DoSetClippingRegionAsRegion(region); }
+
+    // unlike the functions above, the coordinates of the region used in this
+    // one are in device coordinates, not the logical ones
+    void SetDeviceClippingRegion(const wxRegion& region)
+        { m_pimpl->DoSetDeviceClippingRegion(region); }
+
+    // this function is deprecated because its name is confusing: you may
+    // expect it to work with logical coordinates but, in fact, it does exactly
+    // the same thing as SetDeviceClippingRegion()
+    //
+    // please review the code using it and either replace it with calls to
+    // SetDeviceClippingRegion() or correct it if it was [wrongly] passing
+    // logical coordinates to this function
+    wxDEPRECATED_INLINE(void SetClippingRegion(const wxRegion& region),
+                        SetDeviceClippingRegion(region); )
 
     void DestroyClippingRegion()
         { m_pimpl->DestroyClippingRegion(); }
@@ -735,7 +824,7 @@ public:
         { m_pimpl->DoGetClippingBox(x, y, w, h); }
     void GetClippingBox(wxRect& rect) const
         { m_pimpl->DoGetClippingBox(&rect.x, &rect.y, &rect.width, &rect.height); }
-        
+
     // coordinates conversions and transforms
 
     wxCoord DeviceToLogicalX(wxCoord x) const
@@ -786,8 +875,8 @@ public:
 
     void SetAxisOrientation(bool xLeftRight, bool yBottomUp)
         { m_pimpl->SetAxisOrientation(xLeftRight, yBottomUp); }
-    
-    // mostly internal        
+
+    // mostly internal
     void SetDeviceLocalOrigin( wxCoord x, wxCoord y )
         { m_pimpl->SetDeviceLocalOrigin( x, y ); }
 
@@ -877,7 +966,7 @@ public:
         { m_pimpl->DoDrawLines(n, points, xoffset, yoffset); }
     void DrawLines(const wxPointList *list,
                    wxCoord xoffset = 0, wxCoord yoffset = 0)
-        { m_pimpl->DrawLines( list, xoffset, yoffset ); }      
+        { m_pimpl->DrawLines( list, xoffset, yoffset ); }
 #if WXWIN_COMPATIBILITY_2_8
     wxDEPRECATED( void DrawLines(const wxList *list,
                                  wxCoord xoffset = 0, wxCoord yoffset = 0) );
@@ -982,12 +1071,12 @@ public:
                       source, srcPt.x, srcPt.y, rop, useMask, srcPtMask.x, srcPtMask.y);
     }
 
-    bool StretchBlit(wxCoord dstX, wxCoord dstY, 
+    bool StretchBlit(wxCoord dstX, wxCoord dstY,
                      wxCoord dstWidth, wxCoord dstHeight,
-                     wxDC *source, 
+                     wxDC *source,
                      wxCoord srcX, wxCoord srcY,
                      wxCoord srcWidth, wxCoord srcHeight,
-                     int rop = wxCOPY, bool useMask = false, 
+                     int rop = wxCOPY, bool useMask = false,
                      wxCoord srcMaskX = wxDefaultCoord, wxCoord srcMaskY = wxDefaultCoord)
     {
         return m_pimpl->DoStretchBlit(dstX, dstY, dstWidth, dstHeight,
@@ -1010,11 +1099,11 @@ public:
     void DrawSpline(wxCoord x1, wxCoord y1,
                     wxCoord x2, wxCoord y2,
                     wxCoord x3, wxCoord y3)
-        { m_pimpl->DoDrawSpline(x1,y1,x2,y2,x3,y3); }
+        { m_pimpl->DrawSpline(x1,y1,x2,y2,x3,y3); }
     void DrawSpline(int n, wxPoint points[])
-        { m_pimpl->DoDrawSpline(n,points); }
-    void DrawSpline(const wxPointList *points) 
-        { m_pimpl->DoDrawSpline(points); }
+        { m_pimpl->DrawSpline(n,points); }
+    void DrawSpline(const wxPointList *points)
+        { m_pimpl->DrawSpline(points); }
 #endif // wxUSE_SPLINES
 
 
@@ -1032,10 +1121,14 @@ public:
 
 
 protected:
-    wxDCImpl    *m_pimpl;
+    // ctor takes ownership of the pointer
+    wxDC(wxDCImpl *pimpl) : m_pimpl(pimpl) { }
+
+    wxDCImpl * const m_pimpl;
 
 private:
     DECLARE_ABSTRACT_CLASS(wxDC)
+    DECLARE_NO_COPY_CLASS(wxDC)
 };
 
 // ----------------------------------------------------------------------------
@@ -1043,7 +1136,7 @@ private:
 // restore it automatically when the object goes out of scope
 // ----------------------------------------------------------------------------
 
-class WXDLLEXPORT wxDCTextColourChanger
+class WXDLLIMPEXP_CORE wxDCTextColourChanger
 {
 public:
     wxDCTextColourChanger(wxDC& dc) : m_dc(dc), m_colFgOld() { }
@@ -1079,7 +1172,7 @@ private:
 // restore it automatically when the object goes out of scope
 // ----------------------------------------------------------------------------
 
-class WXDLLEXPORT wxDCPenChanger
+class WXDLLIMPEXP_CORE wxDCPenChanger
 {
 public:
     wxDCPenChanger(wxDC& dc, const wxPen& pen) : m_dc(dc), m_penOld(dc.GetPen())
@@ -1106,7 +1199,7 @@ private:
 // restore it automatically when the object goes out of scope
 // ----------------------------------------------------------------------------
 
-class WXDLLEXPORT wxDCBrushChanger
+class WXDLLIMPEXP_CORE wxDCBrushChanger
 {
 public:
     wxDCBrushChanger(wxDC& dc, const wxBrush& brush) : m_dc(dc), m_brushOld(dc.GetBrush())
@@ -1133,11 +1226,11 @@ private:
 // destroys it in the dtor
 // ----------------------------------------------------------------------------
 
-class WXDLLEXPORT wxDCClipper
+class WXDLLIMPEXP_CORE wxDCClipper
 {
 public:
     wxDCClipper(wxDC& dc, const wxRegion& r) : m_dc(dc)
-        { dc.SetClippingRegion(r); }
+        { dc.SetClippingRegion(r.GetBox()); }
     wxDCClipper(wxDC& dc, const wxRect& r) : m_dc(dc)
         { dc.SetClippingRegion(r.x, r.y, r.width, r.height); }
     wxDCClipper(wxDC& dc, wxCoord x, wxCoord y, wxCoord w, wxCoord h) : m_dc(dc)