Updated list of subprojects.

[wxWidgets.git] / src / mac / carbon / dccg.cpp

diff --git a/src/mac/carbon/dccg.cpp b/src/mac/carbon/dccg.cpp
index 5d99b38b95cd3eee1db637c4de17f3d63183ba90..845f0bf88ef711dd567b49ec1970248b6b0ccb1b 100755 (executable)
--- a/src/mac/carbon/dccg.cpp
+++ b/src/mac/carbon/dccg.cpp
@@ -1,42 +1,49 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        dc.cpp
+// Name:        src/mac/carbon/dccg.cpp

 // Purpose:     wxDC class
 // Author:      Stefan Csomor
 // Modified by:
 // Created:     01/02/97
 // RCS-ID:      $Id$
 // Copyright:   (c) Stefan Csomor
 // Purpose:     wxDC class
 // Author:      Stefan Csomor
 // Modified by:
 // Created:     01/02/97
 // RCS-ID:      $Id$
 // Copyright:   (c) Stefan Csomor

-// Licence:       wxWindows licence
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
-#pragma implementation "dc.h"
-#endif
-

 #include "wx/wxprec.h"
 
 #include "wx/dc.h"
 
 #if wxMAC_USE_CORE_GRAPHICS
 
 #include "wx/wxprec.h"
 
 #include "wx/dc.h"
 
 #if wxMAC_USE_CORE_GRAPHICS
 

-#include "wx/app.h"
-#include "wx/mac/uma.h"
-#include "wx/dcmemory.h"
-#include "wx/dcprint.h"
-#include "wx/region.h"
-#include "wx/image.h"
-#include "wx/log.h"
+#ifndef WX_PRECOMP
+    #include "wx/log.h"
+    #include "wx/dcmemory.h"
+    #include "wx/region.h"
+#endif

 
 

+#include "wx/mac/uma.h"

 
 

-#if __MSL__ >= 0x6000
-#include "math.h"
-using namespace std ;
+#ifdef __MSL__
+    #if __MSL__ >= 0x6000
+        #include "math.h"
+        // in case our functions were defined outside std, we make it known all the same
+        namespace std { }
+        using namespace std ;
+    #endif

 #endif
 
 #include "wx/mac/private.h"
 
 IMPLEMENT_ABSTRACT_CLASS(wxDC, wxObject)
 
 #endif
 
 #include "wx/mac/private.h"
 
 IMPLEMENT_ABSTRACT_CLASS(wxDC, wxObject)
 

+#ifndef wxMAC_USE_CORE_GRAPHICS_BLEND_MODES
+#define wxMAC_USE_CORE_GRAPHICS_BLEND_MODES 0
+#endif
+
+#if MAC_OS_X_VERSION_MAX_ALLOWED <= MAC_OS_X_VERSION_10_4
+typedef float CGFloat ;
+#endif
+

 //-----------------------------------------------------------------------------
 // constants
 //-----------------------------------------------------------------------------
 //-----------------------------------------------------------------------------
 // constants
 //-----------------------------------------------------------------------------


@@ -46,30 +53,30 @@ IMPLEMENT_ABSTRACT_CLASS(wxDC, wxObject)
 const double M_PI = 3.14159265358979 ;
 #endif
 #endif
 const double M_PI = 3.14159265358979 ;
 #endif
 #endif

-const double RAD2DEG  = 180.0 / M_PI;
-const short kEmulatedMode = -1 ;
-const short kUnsupportedMode = -2 ;

 
 

-extern TECObjectRef s_TECNativeCToUnicode ;
+static const double RAD2DEG = 180.0 / M_PI;

 
 

-// TODO Update
-// The text ctrl implementation still needs that for the non hiview implementation
+#ifndef __LP64__

 
 

+// TODO: update
+// The textctrl implementation still needs that (needs what?) for the non-HIView implementation
+//

 wxMacWindowClipper::wxMacWindowClipper( const wxWindow* win ) :
 wxMacWindowClipper::wxMacWindowClipper( const wxWindow* win ) :

-    wxMacPortSaver( (GrafPtr) GetWindowPort((WindowRef) win->MacGetTopLevelWindowRef()) )
+    wxMacPortSaver( (GrafPtr) GetWindowPort( (WindowRef) win->MacGetTopLevelWindowRef() ) )

 {
 {

-    m_newPort =(GrafPtr) GetWindowPort((WindowRef) win->MacGetTopLevelWindowRef()) ;
+    m_newPort = (GrafPtr) GetWindowPort( (WindowRef) win->MacGetTopLevelWindowRef() ) ;

     m_formerClip = NewRgn() ;
     m_newClip = NewRgn() ;
     GetClip( m_formerClip ) ;
     m_formerClip = NewRgn() ;
     m_newClip = NewRgn() ;
     GetClip( m_formerClip ) ;

-    
+

     if ( win )
     {
         // guard against half constructed objects, this just leads to a empty clip
     if ( win )
     {
         // guard against half constructed objects, this just leads to a empty clip

-        if( win->GetPeer() )
+        if ( win->GetPeer() )

         {
             int x = 0 , y = 0;
         {
             int x = 0 , y = 0;

-            win->MacWindowToRootWindow( &x,&y ) ;
+            win->MacWindowToRootWindow( &x, &y ) ;
+

             // get area including focus rect
             CopyRgn( (RgnHandle) ((wxWindow*)win)->MacGetVisibleRegion(true).GetWXHRGN() , m_newClip ) ;
             if ( !EmptyRgn( m_newClip ) )
             // get area including focus rect
             CopyRgn( (RgnHandle) ((wxWindow*)win)->MacGetVisibleRegion(true).GetWXHRGN() , m_newClip ) ;
             if ( !EmptyRgn( m_newClip ) )


@@ -80,7 +87,7 @@ wxMacWindowClipper::wxMacWindowClipper( const wxWindow* win ) :
     }
 }
 
     }
 }
 

-wxMacWindowClipper::~wxMacWindowClipper() 
+wxMacWindowClipper::~wxMacWindowClipper()

 {
     SetPort( m_newPort ) ;
     SetClip( m_formerClip ) ;
 {
     SetPort( m_newPort ) ;
     SetClip( m_formerClip ) ;


@@ -92,11 +99,11 @@ wxMacWindowStateSaver::wxMacWindowStateSaver( const wxWindow* win ) :
     wxMacWindowClipper( win )
 {
     // the port is already set at this point
     wxMacWindowClipper( win )
 {
     // the port is already set at this point

-    m_newPort =(GrafPtr) GetWindowPort((WindowRef) win->MacGetTopLevelWindowRef()) ;
+    m_newPort = (GrafPtr) GetWindowPort( (WindowRef) win->MacGetTopLevelWindowRef() ) ;

     GetThemeDrawingState( &m_themeDrawingState ) ;
 }
 
     GetThemeDrawingState( &m_themeDrawingState ) ;
 }
 

-wxMacWindowStateSaver::~wxMacWindowStateSaver() 
+wxMacWindowStateSaver::~wxMacWindowStateSaver()

 {
     SetPort( m_newPort ) ;
     SetThemeDrawingState( m_themeDrawingState , true ) ;
 {
     SetPort( m_newPort ) ;
     SetThemeDrawingState( m_themeDrawingState , true ) ;


@@ -109,19 +116,20 @@ wxMacPortSetter::wxMacPortSetter( const wxDC* dc ) :
 {
     wxASSERT( dc->Ok() ) ;
     m_dc = dc ;
 {
     wxASSERT( dc->Ok() ) ;
     m_dc = dc ;

+

 //    dc->MacSetupPort(&m_ph) ;
 }
 //    dc->MacSetupPort(&m_ph) ;
 }

+

 wxMacPortSetter::~wxMacPortSetter()
 {
 //    m_dc->MacCleanupPort(&m_ph) ;
 }
 wxMacPortSetter::~wxMacPortSetter()
 {
 //    m_dc->MacCleanupPort(&m_ph) ;
 }

+#endif

 
 //-----------------------------------------------------------------------------
 // Local functions
 //-----------------------------------------------------------------------------
 
 
 //-----------------------------------------------------------------------------
 // Local functions
 //-----------------------------------------------------------------------------
 

-static inline double dmin(double a, double b) { return a < b ? a : b; }
-static inline double dmax(double a, double b) { return a > b ? a : b; }

 static inline double DegToRad(double deg) { return (deg * M_PI) / 180.0; }
 
 //-----------------------------------------------------------------------------
 static inline double DegToRad(double deg) { return (deg * M_PI) / 180.0; }
 
 //-----------------------------------------------------------------------------


@@ -136,6 +144,10 @@ static inline double DegToRad(double deg) { return (deg * M_PI) / 180.0; }
 // state we were called with, the other one after changing to HI Graphics orientation
 // (this one is used for getting back clippings etc)
 
 // state we were called with, the other one after changing to HI Graphics orientation
 // (this one is used for getting back clippings etc)
 

+//-----------------------------------------------------------------------------
+// wxGraphicPath implementation
+//-----------------------------------------------------------------------------
+

 wxMacCGPath::wxMacCGPath()
 {
     m_path = CGPathCreateMutable() ;
 wxMacCGPath::wxMacCGPath()
 {
     m_path = CGPathCreateMutable() ;


@@ -146,13 +158,13 @@ wxMacCGPath::~wxMacCGPath()
     CGPathRelease( m_path ) ;
 }
 
     CGPathRelease( m_path ) ;
 }
 

-//  Starts a new subpath at
+// opens (starts) a new subpath

 void wxMacCGPath::MoveToPoint( wxCoord x1 , wxCoord y1 )
 {
     CGPathMoveToPoint( m_path , NULL , x1 , y1 ) ;
 }
 
 void wxMacCGPath::MoveToPoint( wxCoord x1 , wxCoord y1 )
 {
     CGPathMoveToPoint( m_path , NULL , x1 , y1 ) ;
 }
 

-void wxMacCGPath::AddLineToPoint( wxCoord x1 , wxCoord y1 ) 
+void wxMacCGPath::AddLineToPoint( wxCoord x1 , wxCoord y1 )

 {
     CGPathAddLineToPoint( m_path , NULL , x1 , y1 ) ;
 }
 {
     CGPathAddLineToPoint( m_path , NULL , x1 , y1 ) ;
 }


@@ -179,21 +191,39 @@ void wxMacCGPath::CloseSubpath()
     CGPathCloseSubpath( m_path ) ;
 }
 
     CGPathCloseSubpath( m_path ) ;
 }
 

-CGPathRef wxMacCGPath::GetPath() const 
+CGPathRef wxMacCGPath::GetPath() const

 {
     return m_path ;
 }
 
 {
     return m_path ;
 }
 

-wxMacCGContext::wxMacCGContext( CGrafPtr port ) 
+void wxMacCGPath::AddArcToPoint( wxCoord x1, wxCoord y1 , wxCoord x2, wxCoord y2, wxCoord r ) 
+{
+    CGPathAddArcToPoint( m_path, NULL , x1, y1, x2, y2, r); 
+}
+
+void wxMacCGPath::AddArc( wxCoord x, wxCoord y, wxCoord r, double startAngle, double endAngle, bool clockwise ) 
+{
+    CGPathAddArc( m_path, NULL , x, y, r, startAngle, endAngle, clockwise); 
+}
+
+//-----------------------------------------------------------------------------
+// wxGraphicContext implementation
+//-----------------------------------------------------------------------------
+
+wxMacCGContext::wxMacCGContext( CGrafPtr port )

 {
     m_qdPort = port ;
     m_cgContext = NULL ;
 {
     m_qdPort = port ;
     m_cgContext = NULL ;

+    m_mode = kCGPathFill;
+    m_macATSUIStyle = NULL ;

 }
 
 wxMacCGContext::wxMacCGContext( CGContextRef cgcontext )
 {
     m_qdPort = NULL ;
     m_cgContext = cgcontext ;
 }
 
 wxMacCGContext::wxMacCGContext( CGContextRef cgcontext )
 {
     m_qdPort = NULL ;
     m_cgContext = cgcontext ;

+    m_mode = kCGPathFill;
+    m_macATSUIStyle = NULL ;

     CGContextSaveGState( m_cgContext ) ;
     CGContextSaveGState( m_cgContext ) ;
 }
     CGContextSaveGState( m_cgContext ) ;
     CGContextSaveGState( m_cgContext ) ;
 }


@@ -202,9 +232,11 @@ wxMacCGContext::wxMacCGContext()
 {
     m_qdPort = NULL ;
     m_cgContext = NULL ;
 {
     m_qdPort = NULL ;
     m_cgContext = NULL ;

+    m_mode = kCGPathFill;
+    m_macATSUIStyle = NULL ;

 }
 
 }
 

-wxMacCGContext::~wxMacCGContext() 
+wxMacCGContext::~wxMacCGContext()

 {
     if ( m_cgContext )
     {
 {
     if ( m_cgContext )
     {


@@ -212,27 +244,45 @@ wxMacCGContext::~wxMacCGContext()
         CGContextRestoreGState( m_cgContext ) ;
         CGContextRestoreGState( m_cgContext ) ;
     }
         CGContextRestoreGState( m_cgContext ) ;
         CGContextRestoreGState( m_cgContext ) ;
     }

+#ifndef __LP64__

     if ( m_qdPort )
     if ( m_qdPort )

-        CGContextRelease( m_cgContext ) ;
+        QDEndCGContext( m_qdPort, &m_cgContext ) ;
+#endif

 }
 
 
 void wxMacCGContext::Clip( const wxRegion &region )
 {
 }
 
 
 void wxMacCGContext::Clip( const wxRegion &region )
 {

-//        ClipCGContextToRegion ( m_cgContext, &bounds , (RgnHandle) dc->m_macCurrentClipRgn ) ;
+//    ClipCGContextToRegion ( m_cgContext, &bounds , (RgnHandle) dc->m_macCurrentClipRgn ) ;

 }
 
 }
 

-void wxMacCGContext::StrokePath( const wxGraphicPath *p ) 
+void wxMacCGContext::StrokePath( const wxGraphicPath *p )

 {
     const wxMacCGPath* path = dynamic_cast< const wxMacCGPath*>( p ) ;
 {
     const wxMacCGPath* path = dynamic_cast< const wxMacCGPath*>( p ) ;

+
+    int width = m_pen.GetWidth();
+    if ( width == 0 )
+        width = 1 ;
+    if ( m_pen.GetStyle() == wxTRANSPARENT )
+        width = 0 ;
+        
+    bool offset = ( width % 2 ) == 1 ; 
+
+    if ( offset )
+        CGContextTranslateCTM( m_cgContext, 0.5, 0.5 );
+

     CGContextAddPath( m_cgContext , path->GetPath() ) ;
     CGContextStrokePath( m_cgContext ) ;
     CGContextAddPath( m_cgContext , path->GetPath() ) ;
     CGContextStrokePath( m_cgContext ) ;

+
+    if ( offset )
+        CGContextTranslateCTM( m_cgContext, -0.5, -0.5 );

 }
 
 }
 

-void wxMacCGContext::DrawPath( const wxGraphicPath *p , int fillStyle ) 
+void wxMacCGContext::DrawPath( const wxGraphicPath *p , int fillStyle )

 {
     const wxMacCGPath* path = dynamic_cast< const wxMacCGPath*>( p ) ;
     CGPathDrawingMode mode = m_mode ;
 {
     const wxMacCGPath* path = dynamic_cast< const wxMacCGPath*>( p ) ;
     CGPathDrawingMode mode = m_mode ;

+

     if ( fillStyle == wxODDEVEN_RULE )
     {
         if ( mode == kCGPathFill )
     if ( fillStyle == wxODDEVEN_RULE )
     {
         if ( mode == kCGPathFill )


@@ -240,529 +290,524 @@ void wxMacCGContext::DrawPath( const wxGraphicPath *p , int fillStyle )
         else if ( mode == kCGPathFillStroke )
             mode = kCGPathEOFillStroke ;
     }
         else if ( mode == kCGPathFillStroke )
             mode = kCGPathEOFillStroke ;
     }

+
+    int width = m_pen.GetWidth();
+    if ( width == 0 )
+        width = 1 ;
+    if ( m_pen.GetStyle() == wxTRANSPARENT )
+        width = 0 ;
+        
+    bool offset = ( width % 2 ) == 1 ; 
+
+    if ( offset )
+        CGContextTranslateCTM( m_cgContext, 0.5, 0.5 );
+

     CGContextAddPath( m_cgContext , path->GetPath() ) ;
     CGContextDrawPath( m_cgContext , mode ) ;
     CGContextAddPath( m_cgContext , path->GetPath() ) ;
     CGContextDrawPath( m_cgContext , mode ) ;

+
+    if ( offset )
+        CGContextTranslateCTM( m_cgContext, -0.5, -0.5 );

 }
 
 }
 

-void wxMacCGContext::FillPath( const wxGraphicPath *p , const wxColor &fillColor , int fillStyle ) 
+void wxMacCGContext::FillPath( const wxGraphicPath *p , const wxColor &fillColor , int fillStyle )

 {
     const wxMacCGPath* path = dynamic_cast< const wxMacCGPath*>( p ) ;
     CGContextSaveGState( m_cgContext ) ;
 {
     const wxMacCGPath* path = dynamic_cast< const wxMacCGPath*>( p ) ;
     CGContextSaveGState( m_cgContext ) ;

-    
+

     RGBColor col = MAC_WXCOLORREF( fillColor.GetPixel() ) ;
     CGContextSetRGBFillColor( m_cgContext , col.red / 65536.0 , col.green / 65536.0 , col.blue / 65536.0 , 1.0 ) ;
     CGPathDrawingMode mode = kCGPathFill ;
 
     if ( fillStyle == wxODDEVEN_RULE )
         mode = kCGPathEOFill ;
     RGBColor col = MAC_WXCOLORREF( fillColor.GetPixel() ) ;
     CGContextSetRGBFillColor( m_cgContext , col.red / 65536.0 , col.green / 65536.0 , col.blue / 65536.0 , 1.0 ) ;
     CGPathDrawingMode mode = kCGPathFill ;
 
     if ( fillStyle == wxODDEVEN_RULE )
         mode = kCGPathEOFill ;

-               
+

     CGContextBeginPath( m_cgContext ) ;
     CGContextAddPath( m_cgContext , path->GetPath() ) ;
     CGContextClosePath( m_cgContext ) ;
     CGContextDrawPath( m_cgContext , mode ) ;
     CGContextBeginPath( m_cgContext ) ;
     CGContextAddPath( m_cgContext , path->GetPath() ) ;
     CGContextClosePath( m_cgContext ) ;
     CGContextDrawPath( m_cgContext , mode ) ;

-    
+

     CGContextRestoreGState( m_cgContext ) ;
 }
     CGContextRestoreGState( m_cgContext ) ;
 }

-    
-wxGraphicPath* wxMacCGContext::CreatePath() 
-{ 
+
+wxGraphicPath* wxMacCGContext::CreatePath()
+{

     // make sure that we now have a real cgref, before doing
     // anything with paths
     CGContextRef cg = GetNativeContext() ;
     cg = NULL ;
     // make sure that we now have a real cgref, before doing
     // anything with paths
     CGContextRef cg = GetNativeContext() ;
     cg = NULL ;

-    return new wxMacCGPath() ; 
+
+    return new wxMacCGPath() ;

 }
 
 // in case we only got a QDPort only create a cgref now
 
 }
 
 // in case we only got a QDPort only create a cgref now
 

-CGContextRef wxMacCGContext::GetNativeContext() 
-{ 
-    if( m_cgContext == NULL )
+CGContextRef wxMacCGContext::GetNativeContext()
+{
+    if ( m_cgContext == NULL )

     {
         Rect bounds ;
     {
         Rect bounds ;

+        OSStatus status = noErr;
+#ifndef __LP64__

         GetPortBounds( (CGrafPtr) m_qdPort , &bounds ) ;
         GetPortBounds( (CGrafPtr) m_qdPort , &bounds ) ;

-        OSStatus status = CreateCGContextForPort((CGrafPtr) m_qdPort , &m_cgContext) ;
+        status = QDBeginCGContext((CGrafPtr) m_qdPort , &m_cgContext) ;
+#endif

         CGContextSaveGState( m_cgContext ) ;
 
         wxASSERT_MSG( status == noErr , wxT("Cannot nest wxDCs on the same window") ) ;
         CGContextSaveGState( m_cgContext ) ;
 
         wxASSERT_MSG( status == noErr , wxT("Cannot nest wxDCs on the same window") ) ;

+

         CGContextTranslateCTM( m_cgContext , 0 , bounds.bottom - bounds.top ) ;
         CGContextScaleCTM( m_cgContext , 1 , -1 ) ;
         CGContextTranslateCTM( m_cgContext , 0 , bounds.bottom - bounds.top ) ;
         CGContextScaleCTM( m_cgContext , 1 , -1 ) ;

-        
+

         CGContextSaveGState( m_cgContext ) ;
         SetPen( m_pen ) ;
         SetBrush( m_brush ) ;
     }
         CGContextSaveGState( m_cgContext ) ;
         SetPen( m_pen ) ;
         SetBrush( m_brush ) ;
     }

-    return m_cgContext ; 
+
+    return m_cgContext ;

 }
 
 }
 

-void wxMacCGContext::SetNativeContext( CGContextRef cg ) 
-{ 
+void wxMacCGContext::SetNativeContext( CGContextRef cg )
+{

     // we allow either setting or clearing but not replacing
     wxASSERT( m_cgContext == NULL || cg == NULL ) ;
     // we allow either setting or clearing but not replacing
     wxASSERT( m_cgContext == NULL || cg == NULL ) ;

+

     if ( cg )
         CGContextSaveGState( cg ) ;
     if ( cg )
         CGContextSaveGState( cg ) ;

-    m_cgContext = cg ; 
+    m_cgContext = cg ;

 }
 
 }
 

-#pragma mark -
+void wxMacCGContext::Translate( wxCoord dx , wxCoord dy ) 
+{
+    CGContextTranslateCTM( m_cgContext, dx, dy );
+}

 
 

-// Experimental support for dashes and patterned brushes
-// uncomment the following lines to enable it
+void wxMacCGContext::Scale( wxCoord xScale , wxCoord yScale )
+{
+    CGContextScaleCTM( m_cgContext , xScale , yScale ) ;
+}
+
+void wxMacCGContext::DrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, wxCoord w, wxCoord h ) 
+{
+    CGImageRef image = (CGImageRef)( bmp.CGImageCreate() ) ;
+    HIRect r = CGRectMake( 0 , 0 , w , h ); 

 
 

-// #define _NEW_GC_DASHES_
-// #define _NEW_GC_SUPPORT_
+    CGContextSaveGState( m_cgContext );
+    CGContextTranslateCTM( m_cgContext, x , y + h );
+    CGContextScaleCTM( m_cgContext, 1, -1 );
+        
+    // in case image is a mask, set the foreground color
+    CGContextSetRGBFillColor( m_cgContext , m_textForegroundColor.Red() / 255.0 , m_textForegroundColor.Green() / 255.0 , 
+        m_textForegroundColor.Blue() / 255.0 , m_textForegroundColor.Alpha() / 255.0 ) ;
+    CGContextDrawImage( m_cgContext, r, image );
+    CGContextRestoreGState( m_cgContext );

 
 

-#if MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_X_VERSION_10_4
-#define kCGColorSpaceGenericRGB   CFSTR("kCGColorSpaceGenericRGB")
-#endif
+    CGImageRelease( image ) ;
+}

 
 

-void EstablishPatternColorSpace(
-       CGContextRef            ctxRef,
-       bool                            useMultibit,
-       bool                            useFill )
+void wxMacCGContext::DrawIcon( const wxIcon &icon, wxCoord x, wxCoord y, wxCoord w, wxCoord h ) 

 {
 {

-       CGColorSpaceRef baseSpace, patternSpace;
-
-       if (ctxRef == NULL)
-               return;
-
-       baseSpace = NULL;
-       patternSpace = NULL;
-
-       if (useMultibit)
-       {
-               patternSpace = CGColorSpaceCreatePattern( NULL );
-
-               if (useFill)
-                       CGContextSetFillColorSpace( ctxRef, patternSpace );
-               else
-                       CGContextSetStrokeColorSpace( ctxRef, patternSpace );
-       }
-       else
-       {
-               baseSpace = CGColorSpaceCreateWithName( kCGColorSpaceGenericRGB );
-               patternSpace = CGColorSpaceCreatePattern( baseSpace );
-
-               if (useFill)
-                       CGContextSetFillColorSpace( ctxRef, patternSpace );
-               else
-                       CGContextSetStrokeColorSpace( ctxRef, patternSpace );
-       }
-
-       // NB: the context owns these now, and this code is finished with them
-       if (patternSpace != NULL)
-               CGColorSpaceRelease( patternSpace );
-       if (baseSpace != NULL)
-               CGColorSpaceRelease( baseSpace );
-}
-
-void ImagePatternRender(
-       void                    *info,
-       CGContextRef    ctxRef )
-{
-       if (ctxRef == NULL)
-               return;
-
-       CGImageRef      imageRef = (CGImageRef)info;
-       if (imageRef != NULL)
-       {
-               CGRect  boundsR = CGRectMake( 0.0, 0.0, (float)CGImageGetWidth( imageRef ), (float)CGImageGetHeight( imageRef ) );
-               CGContextDrawImage( ctxRef, boundsR, imageRef );
-       }
+    CGRect r = CGRectMake( 0 , 0 , w , h ) ;
+    CGContextSaveGState( m_cgContext );
+    CGContextTranslateCTM( m_cgContext, x , y + h );
+    CGContextScaleCTM( m_cgContext, 1, -1 );
+    PlotIconRefInContext( m_cgContext , &r , kAlignNone , kTransformNone ,
+        NULL , kPlotIconRefNormalFlags , MAC_WXHICON( icon.GetHICON() ) ) ;
+    CGContextRestoreGState( m_cgContext ) ;

 }
 
 }
 

-void ImagePatternDispose(
-       void            *info )
+void wxMacCGContext::PushState()

 {
 {

-       CGImageRef      imageRef = (CGImageRef)info;
-       if (imageRef != NULL)
-               CGImageRelease( imageRef );
+    CGContextSaveGState( m_cgContext );

 }
 
 }
 

-// specifies the struct version value and the callback functions for draw and release
-static const CGPatternCallbacks sImagePatternCallback = { 0, &ImagePatternRender, &ImagePatternDispose };
+void wxMacCGContext::PopState() 
+{
+    CGContextRestoreGState( m_cgContext );
+}

 
 

-long CreatePatternFromBitmap(
-       CGPatternRef            *patternRef,
-       const wxBitmap  *rasterInfo,
-       bool                            useMultibit )
+void wxMacCGContext::SetTextColor( const wxColour &col ) 

 {
 {

-       CGRect          boundsR;
-       CGImageRef      imageRef;
-       long                    errorStatus, widthV, heightV, depthV;
+    m_textForegroundColor = col ;
+}

 
 

-       if (patternRef == NULL)
-               return (-1);
+#pragma mark -
+#pragma mark wxMacCGPattern, ImagePattern, HatchPattern classes

 
 

-       *patternRef = NULL;
-       imageRef = NULL;
-       errorStatus = 0;
+// CGPattern wrapper class: always allocate on heap, never call destructor

 
 

-       if ((rasterInfo == NULL) || !rasterInfo->Ok())
-               errorStatus = (-2);
+class wxMacCGPattern
+{
+public :
+    wxMacCGPattern() {}

 
 

-       if (errorStatus == 0)
-       {
-               // build a usable bounding CGRect from the wxBitmap's bounds wxRect
-               widthV = rasterInfo->GetWidth();
-               heightV = rasterInfo->GetHeight();
-               if ((widthV <= 0) || (heightV <= 0))
-                       errorStatus = (-3);
-       }
+    // is guaranteed to be called only with a non-Null CGContextRef
+    virtual void Render( CGContextRef ctxRef ) = 0 ;

 
 

-       if (errorStatus == 0)
-       {
-               depthV = rasterInfo->GetDepth();
-//             isColored = (depthV > 1);
+    operator CGPatternRef() const { return m_patternRef ; }

 
 

-               // FIXME: this is often <= 0 - why???
-//             if (depthV <= 1)
-//                     errorStatus = (-4);
-       }
+protected :
+    virtual ~wxMacCGPattern()
+    {
+        // as this is called only when the m_patternRef is been released;
+        // don't release it again
+    }

 
 

-       if (errorStatus == 0)
-       {
-               imageRef = (CGImageRef)(rasterInfo->CGImageCreate());
-               if (imageRef == NULL)
-                       errorStatus = (-5);
-       }
+    static void _Render( void *info, CGContextRef ctxRef )
+    {
+        wxMacCGPattern* self = (wxMacCGPattern*) info ;
+        if ( self && ctxRef )
+            self->Render( ctxRef ) ;
+    }

 
 

-       if (errorStatus == 0)
-       {
-               // FIXME: switch when this routine belongs to a DC class...
-               boundsR = CGRectMake( 0.0, 0.0, (float)widthV, (float)heightV );
-//             boundsR = CGRectMake( 0.0, 0.0, (float)XLOG2DEVREL( widthV ), (float)XLOG2DEVREL( heightV ) );
+    static void _Dispose( void *info )
+    {
+        wxMacCGPattern* self = (wxMacCGPattern*) info ;
+        delete self ;
+    }

 
 

-               *patternRef = CGPatternCreate(
-                       (void*)imageRef,
-                       boundsR,
-                       CGAffineTransformIdentity,
-                       boundsR.size.width,
-                       boundsR.size.height,
-                       kCGPatternTilingNoDistortion,
-                       (int)useMultibit,
-                       &sImagePatternCallback );
+    CGPatternRef m_patternRef ;

 
 

-               if (*patternRef == (CGPatternRef)NULL)
-                       errorStatus = (-6);
-       }
+    static const CGPatternCallbacks ms_Callbacks ;
+} ;

 
 

-       return errorStatus;
-}
+const CGPatternCallbacks wxMacCGPattern::ms_Callbacks = { 0, &wxMacCGPattern::_Render, &wxMacCGPattern::_Dispose };

 
 

-long CreatePatternFromDashes(
-       CGPatternRef    *patternRef,
-       const wxDash    *sourceDash,
-       int                     count,
-       bool                    useMultibit )
+class ImagePattern : public wxMacCGPattern

 {
 {

-       long            errorStatus;
+public :
+    ImagePattern( const wxBitmap* bmp , const CGAffineTransform& transform )
+    {
+        wxASSERT( bmp && bmp->Ok() ) ;

 
 

-       if (patternRef == NULL)
-               return (-1);
+        Init( (CGImageRef) bmp->CGImageCreate() , transform ) ;
+    }

 
 

-       *patternRef = NULL;
-       if ((sourceDash == NULL) || (count <= 0))
-               return (-2);
+    // ImagePattern takes ownership of CGImageRef passed in
+    ImagePattern( CGImageRef image , const CGAffineTransform& transform )
+    {
+        if ( image )
+            CFRetain( image ) ;

 
 

-       wxBitmap                dashBits( (char*)sourceDash, 8, count, 1 );
-       errorStatus = CreatePatternFromBitmap( patternRef, &dashBits, useMultibit );
+        Init( image , transform ) ;
+    }

 
 

-       return errorStatus;
-}
+    virtual void Render( CGContextRef ctxRef )
+    {
+        if (m_image != NULL)
+            HIViewDrawCGImage( ctxRef, &m_imageBounds, m_image );
+    }
+
+protected :
+    void Init( CGImageRef image, const CGAffineTransform& transform )
+    {
+        m_image = image ;
+        if ( m_image )
+        {
+            m_imageBounds = CGRectMake( 0.0, 0.0, (CGFloat)CGImageGetWidth( m_image ), (CGFloat)CGImageGetHeight( m_image ) ) ;
+            m_patternRef = CGPatternCreate(
+                this , m_imageBounds, transform ,
+                m_imageBounds.size.width, m_imageBounds.size.height,
+                kCGPatternTilingNoDistortion, true , &wxMacCGPattern::ms_Callbacks ) ;
+        }
+    }
+
+    virtual ~ImagePattern()
+    {
+        if ( m_image )
+            CGImageRelease( m_image ) ;
+    }

 
 

-long CreatePatternFromBrush(
-       CGPatternRef    *patternRef,
-       const wxBrush   &sourceBrush,
-       bool                    useMultibit )
+    CGImageRef m_image ;
+    CGRect m_imageBounds ;
+} ;
+
+class HatchPattern : public wxMacCGPattern

 {
 {

-       long            errorStatus;
+public :
+    HatchPattern( int hatchstyle, const CGAffineTransform& transform )
+    {
+        m_hatch = hatchstyle ;
+        m_imageBounds = CGRectMake( 0.0, 0.0, 8.0 , 8.0 ) ;
+        m_patternRef = CGPatternCreate(
+            this , m_imageBounds, transform ,
+            m_imageBounds.size.width, m_imageBounds.size.height,
+            kCGPatternTilingNoDistortion, false , &wxMacCGPattern::ms_Callbacks ) ;
+    }

 
 

-       if (patternRef == NULL)
-               return (-1);
+    void StrokeLineSegments( CGContextRef ctxRef , const CGPoint pts[] , size_t count )
+    {
+#if MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_4
+        if ( UMAGetSystemVersion() >= 0x1040 )
+        {
+            CGContextStrokeLineSegments( ctxRef , pts , count ) ;
+        }
+        else
+#endif
+        {
+            CGContextBeginPath( ctxRef );
+            for (size_t i = 0; i < count; i += 2)
+            {
+                CGContextMoveToPoint(ctxRef, pts[i].x, pts[i].y);
+                CGContextAddLineToPoint(ctxRef, pts[i+1].x, pts[i+1].y);
+            }
+            CGContextStrokePath(ctxRef);
+        }
+    }

 
 

-       *patternRef = NULL;
-       errorStatus = CreatePatternFromBitmap( patternRef, sourceBrush.GetStipple(), useMultibit );
+    virtual void Render( CGContextRef ctxRef )
+    {
+        switch ( m_hatch )
+        {
+            case wxBDIAGONAL_HATCH :
+                {
+                    CGPoint pts[] =
+                    {
+                    { 8.0 , 0.0 } , { 0.0 , 8.0 }
+                    };
+                    StrokeLineSegments( ctxRef , pts , 2 ) ;
+                }
+                break ;

 
 

-       return errorStatus;
-}
+            case wxCROSSDIAG_HATCH :
+                {
+                    CGPoint pts[] =
+                    {
+                        { 0.0 , 0.0 } , { 8.0 , 8.0 } ,
+                        { 8.0 , 0.0 } , { 0.0 , 8.0 }
+                    };
+                    StrokeLineSegments( ctxRef , pts , 4 ) ;
+                }
+                break ;

 
 

-long CreatePatternFromPen(
-       CGPatternRef    *patternRef,
-       const wxPen     &sourcePen,
-       bool                    useMultibit )
-{
-       long            errorStatus;
+            case wxFDIAGONAL_HATCH :
+                {
+                    CGPoint pts[] =
+                    {
+                    { 0.0 , 0.0 } , { 8.0 , 8.0 }
+                    };
+                    StrokeLineSegments( ctxRef , pts , 2 ) ;
+                }
+                break ;

 
 

-       if (patternRef == NULL)
-               return (-1);
+            case wxCROSS_HATCH :
+                {
+                    CGPoint pts[] =
+                    {
+                    { 0.0 , 4.0 } , { 8.0 , 4.0 } ,
+                    { 4.0 , 0.0 } , { 4.0 , 8.0 } ,
+                    };
+                    StrokeLineSegments( ctxRef , pts , 4 ) ;
+                }
+                break ;

 
 

-       *patternRef = NULL;
-       errorStatus = CreatePatternFromBitmap( patternRef, sourcePen.GetStipple(), useMultibit );
+            case wxHORIZONTAL_HATCH :
+                {
+                    CGPoint pts[] =
+                    {
+                    { 0.0 , 4.0 } , { 8.0 , 4.0 } ,
+                    };
+                    StrokeLineSegments( ctxRef , pts , 2 ) ;
+                }
+                break ;

 
 

-       return errorStatus;
-}
+            case wxVERTICAL_HATCH :
+                {
+                    CGPoint pts[] =
+                    {
+                    { 4.0 , 0.0 } , { 4.0 , 8.0 } ,
+                    };
+                    StrokeLineSegments( ctxRef , pts , 2 ) ;
+                }
+                break ;
+
+            default:
+                break;
+        }
+    }
+
+protected :
+    virtual ~HatchPattern() {}
+
+    CGRect      m_imageBounds ;
+    int         m_hatch ;
+};

 
 

-// ------------

 #pragma mark -
 
 #pragma mark -
 

-// FIXME: the NEW_GC_SUPPORT part this routine is unfinished and needs lots of work !!
-//

 void wxMacCGContext::SetPen( const wxPen &pen )
 {
     m_pen = pen ;
     if ( m_cgContext == NULL )
         return ;
 void wxMacCGContext::SetPen( const wxPen &pen )
 {
     m_pen = pen ;
     if ( m_cgContext == NULL )
         return ;

+

     bool fill = m_brush.GetStyle() != wxTRANSPARENT ;
     bool stroke = pen.GetStyle() != wxTRANSPARENT ;
     bool fill = m_brush.GetStyle() != wxTRANSPARENT ;
     bool stroke = pen.GetStyle() != wxTRANSPARENT ;

-    
+

 #if 0
 #if 0

-    // we can benchmark performance, should go into a setting later
+    // we can benchmark performance; should go into a setting eventually

     CGContextSetShouldAntialias( m_cgContext , false ) ;
 #endif
     CGContextSetShouldAntialias( m_cgContext , false ) ;
 #endif

-    if ( fill | stroke )
+
+    if ( fill || stroke )

     {
     {

-        // setup brushes
+        // set up brushes

         m_mode = kCGPathFill ; // just a default
 
         m_mode = kCGPathFill ; // just a default
 

-        if ( fill )
-        {
-            m_mode = kCGPathFill ;
-        }

         if ( stroke )
         {
         if ( stroke )
         {

-#if defined(_NEW_GC_SUPPORT_)
-            // new candidate
-            {
-                CGPatternRef  patternRef;
-                float  alphaArray[1];
-                long  result;
-                bool  hasSetPattern, useMultibit;
-
-                hasSetPattern = false;
-                useMultibit = true;
-                result = CreatePatternFromPen( &patternRef, pen, useMultibit );
-                if (result == 0)
-                {
-                    EstablishPatternColorSpace( m_cgContext, useMultibit, false );
-
-                    alphaArray[0] = 1.0;
-                    CGContextSetStrokePattern( m_cgContext, patternRef, alphaArray );
-                    CGPatternRelease( patternRef );
-
-                    hasSetPattern = true;
-
-//wxLogDebug( wxT("CreatePatternFromPen succeeded!") );
-                }
-
-                // NB: the (-2) result is from wxPen instances that don't have a stipple wxBitmap
-                if (result < (-2))
-                    wxLogDebug( wxT("CreatePatternFromPen failed: result [%ld]"), result );
-
-                if (!hasSetPattern)
-                {
-                    RGBColor col;
+            CGContextSetRGBStrokeColor( m_cgContext , pen.GetColour().Red() / 255.0 , pen.GetColour().Green() / 255.0 , 
+                    pen.GetColour().Blue() / 255.0 , pen.GetColour().Alpha() / 255.0 ) ;

 
 

-#if 1
-                    col = MAC_WXCOLORREF( pen.GetColour().GetPixel() );
-#else
-                    GetThemeBrushAsColor( pen.MacGetTheme(), 32, true, &col );
-#endif
-
-                    CGContextSetRGBStrokeColor(
-                        m_cgContext, (float) col.red / 65536.0,
-                        (float) col.green / 65536.0, (float) col.blue / 65536.0, 1.0 );
-                }
-            }
+            // TODO: * m_dc->m_scaleX
+            CGFloat penWidth = pen.GetWidth();
+            if (penWidth <= 0.0)
+                penWidth = 0.1;
+            CGContextSetLineWidth( m_cgContext , penWidth ) ;

 
 

-#else
-            // original implementation
-            RGBColor col = MAC_WXCOLORREF( pen.GetColour().GetPixel() ) ;
-            CGContextSetRGBStrokeColor( m_cgContext , col.red / 65536.0 , col.green / 65536.0 , col.blue / 65536.0 , 1.0 ) ;
-#endif
-            

             CGLineCap cap ;
             CGLineCap cap ;

-            switch( pen.GetCap() )
+            switch ( pen.GetCap() )

             {
                 case wxCAP_ROUND :
                     cap = kCGLineCapRound ;
                     break ;
             {
                 case wxCAP_ROUND :
                     cap = kCGLineCapRound ;
                     break ;

+

                 case wxCAP_PROJECTING :
                     cap = kCGLineCapSquare ;
                     break ;
                 case wxCAP_PROJECTING :
                     cap = kCGLineCapSquare ;
                     break ;

+

                 case wxCAP_BUTT :
                     cap = kCGLineCapButt ;
                     break ;
                 case wxCAP_BUTT :
                     cap = kCGLineCapButt ;
                     break ;

+

                 default :
                     cap = kCGLineCapButt ;
                     break ;
             }
                 default :
                     cap = kCGLineCapButt ;
                     break ;
             }

-            CGContextSetLineCap( m_cgContext , cap ) ;

 
             CGLineJoin join ;
 
             CGLineJoin join ;

-            switch( pen.GetJoin() )
+            switch ( pen.GetJoin() )

             {
                 case wxJOIN_BEVEL :
                     join = kCGLineJoinBevel ;
                     break ;
             {
                 case wxJOIN_BEVEL :
                     join = kCGLineJoinBevel ;
                     break ;

+

                 case wxJOIN_MITER :
                     join = kCGLineJoinMiter ;
                     break ;
                 case wxJOIN_MITER :
                     join = kCGLineJoinMiter ;
                     break ;

+

                 case wxJOIN_ROUND :
                     join = kCGLineJoinRound ;
                     break ;
                 case wxJOIN_ROUND :
                     join = kCGLineJoinRound ;
                     break ;

+

                 default :
                     join = kCGLineJoinMiter ;
                 default :
                     join = kCGLineJoinMiter ;

-                    break; 
+                    break;

             }
             }

-            CGContextSetLineJoin( m_cgContext , join ) ;
-
-            /* TODO * m_dc->m_scaleX */
-            float penWidth = pen.GetWidth();
-            if (penWidth <= 0.0)
-                penWidth = 0.1;
-            CGContextSetLineWidth( m_cgContext , penWidth ) ; 

 
             m_mode = kCGPathStroke ;
             int count = 0 ;
 
 
             m_mode = kCGPathStroke ;
             int count = 0 ;
 

-#if defined(_NEW_GC_DASHES_)
-            const char *dashData = NULL ;
-            char *userDashData = NULL ;
-            float  alphaArray[1];
-
-            const char dotted[] = { 0xAA, 0x55, 0xAA, 0x55, 0xAA, 0x55, 0xAA, 0x55 };
-            const char dashed[] = { 0xFF, 0xFF, 0x00, 0x00, 0xFF, 0xFF, 0x00, 0x00 };
-            const char short_dashed[] = { 0x00, 0xFF, 0xFF, 0xFF, 0x00, 0x00, 0x00, 0x00 };
-            const char dotted_dashed[] = { 0x00, 0xFF, 0xFF, 0xFF, 0x00, 0x00, 0xFF, 0x00 };
-
-            switch (pen.GetStyle())
-            {
-                case wxSOLID:
-                    // default, undashed pen
-                    break;
-
-                case wxDOT:
-                    dashData = dotted;
-                    count = WXSIZEOF(dotted);
-                    break;
-                case wxLONG_DASH:
-                    dashData = dashed;
-                    count = WXSIZEOF(dashed);
-                    break;
-                case wxSHORT_DASH:
-                    dashData = short_dashed;
-                    count = WXSIZEOF(short_dashed);
-                    break;
-                case wxDOT_DASH:
-                    dashData = dotted_dashed;
-                    count = WXSIZEOF(dotted_dashed);
-                    break;
-                case wxUSER_DASH:
-                    count = pen.GetDashes( (wxDash**)&userDashData );
-                    dashData = userDashData;
-                    break;
-
-                default :
-                    break; 
-            }
+            const CGFloat *lengths = NULL ;
+            CGFloat *userLengths = NULL ;

 
 

-            if ((dashData != NULL) && (count > 0))
-            {
-                CGPatternRef  patternRef;
-                RGBColor col;
-                long  result;
-                bool  useMultibit;
-
-                    useMultibit = true;
-                    result = CreatePatternFromDashes( &patternRef, (const wxDash*)dashData, count, useMultibit );
-                    if (result == 0)
-                    {
-                        col = MAC_WXCOLORREF( pen.GetColour().GetPixel() );
-                        CGContextSetRGBStrokeColor(
-                            m_cgContext, (float) col.red / 65536.0,
-                            (float) col.green / 65536.0, (float) col.blue / 65536.0, 1.0 );
+            const CGFloat dashUnit = penWidth < 1.0 ? 1.0 : penWidth;

 
 

-                        EstablishPatternColorSpace( m_cgContext, useMultibit, false );
+            const CGFloat dotted[] = { dashUnit , dashUnit + 2.0 };
+            const CGFloat short_dashed[] = { 9.0 , 6.0 };
+            const CGFloat dashed[] = { 19.0 , 9.0 };
+            const CGFloat dotted_dashed[] = { 9.0 , 6.0 , 3.0 , 3.0 };

 
 

-                        alphaArray[0] = 1.0;
-                        CGContextSetStrokePattern( m_cgContext, patternRef, alphaArray );
-                        CGPatternRelease( patternRef );
-                    }
- 
-                if (result != 0)
-                    wxLogDebug( wxT("CreatePatternFromDashes failed: result [%ld]"), result );
-           }
-#else
-            const float *lengths = NULL ;
-            float *userLengths = NULL ;
-            
-            const float dotted[] = { 3 , 3 };
-            const float dashed[] = { 19 , 9 };
-            const float short_dashed[] = { 9 , 6 };
-            const float dotted_dashed[] = { 9 , 6 , 3 , 3 };
-            
-            switch( pen.GetStyle() )
+            switch ( pen.GetStyle() )

             {
                 case wxSOLID :
                     break ;
             {
                 case wxSOLID :
                     break ;

+

                 case wxDOT :
                     lengths = dotted ;
                     count = WXSIZEOF(dotted);
                     break ;
                 case wxDOT :
                     lengths = dotted ;
                     count = WXSIZEOF(dotted);
                     break ;

+

                 case wxLONG_DASH :
                     lengths = dashed ;
                     count = WXSIZEOF(dashed) ;
                     break ;
                 case wxLONG_DASH :
                     lengths = dashed ;
                     count = WXSIZEOF(dashed) ;
                     break ;

+

                 case wxSHORT_DASH :
                     lengths = short_dashed ;
                     count = WXSIZEOF(short_dashed) ;
                     break ;
                 case wxSHORT_DASH :
                     lengths = short_dashed ;
                     count = WXSIZEOF(short_dashed) ;
                     break ;

+

                 case wxDOT_DASH :
                     lengths = dotted_dashed ;
                     count = WXSIZEOF(dotted_dashed);
                     break ;
                 case wxDOT_DASH :
                     lengths = dotted_dashed ;
                     count = WXSIZEOF(dotted_dashed);
                     break ;

+

                 case wxUSER_DASH :
                     wxDash *dashes ;
                     count = pen.GetDashes( &dashes ) ;
                     if ((dashes != NULL) && (count > 0))
                     {
                 case wxUSER_DASH :
                     wxDash *dashes ;
                     count = pen.GetDashes( &dashes ) ;
                     if ((dashes != NULL) && (count > 0))
                     {

-                        userLengths = new float[count] ;
-                        for( int i = 0 ; i < count ; ++i )
+                        userLengths = new CGFloat[count] ;
+                        for ( int i = 0 ; i < count ; ++i )

                         {
                         {

-                            userLengths[i] = (float)dashes[i] ;
-                            if (userLengths[i] <= 0.0)
-                            {
-                                userLengths[i] = 1.0;
-//                                wxLogDebug( wxT("wxMacCGContext::SetPen - bad dash length[%d] [%.2f]"), i, (float)dashes[i] );
-                            }
+                            userLengths[i] = dashes[i] * dashUnit ;
+
+                            if ( i % 2 == 1 && userLengths[i] < dashUnit + 2.0 )
+                                userLengths[i] = dashUnit + 2.0 ;
+                            else if ( i % 2 == 0 && userLengths[i] < dashUnit )
+                                userLengths[i] = dashUnit ;

                         }
                     }
                     lengths = userLengths ;
                     break ;
                         }
                     }
                     lengths = userLengths ;
                     break ;

+
+                case wxSTIPPLE :
+                    {
+                        CGFloat  alphaArray[1] = { 1.0 } ;
+                        wxBitmap* bmp = pen.GetStipple() ;
+                        if ( bmp && bmp->Ok() )
+                        {
+                            wxMacCFRefHolder<CGColorSpaceRef> patternSpace( CGColorSpaceCreatePattern( NULL ) ) ;
+                            CGContextSetStrokeColorSpace( m_cgContext , patternSpace ) ;
+                            wxMacCFRefHolder<CGPatternRef> pattern( *( new ImagePattern( bmp , CGContextGetCTM( m_cgContext ) ) ) );
+                            CGContextSetStrokePattern( m_cgContext, pattern , alphaArray ) ;
+                        }
+                    }
+                    break ;
+

                 default :
                 default :

-                    break ; 
+                    {
+                        wxMacCFRefHolder<CGColorSpaceRef> patternSpace( CGColorSpaceCreatePattern( wxMacGetGenericRGBColorSpace() ) ) ;
+                        CGContextSetStrokeColorSpace( m_cgContext , patternSpace ) ;
+                        wxMacCFRefHolder<CGPatternRef> pattern( *( new HatchPattern( pen.GetStyle() , CGContextGetCTM( m_cgContext ) ) ) );
+
+                        CGFloat  colorArray[4] = { pen.GetColour().Red() / 255.0 , pen.GetColour().Green() / 255.0 , 
+                            pen.GetColour().Blue() / 255.0 , pen.GetColour().Alpha() / 255.0 } ;
+
+                        CGContextSetStrokePattern( m_cgContext, pattern , colorArray ) ;
+                    }
+                    break ;

             }
 
             if ((lengths != NULL) && (count > 0))
             {
             }
 
             if ((lengths != NULL) && (count > 0))
             {

-                // we need to change the cap, otherwise everything overlaps
-                // and we get solid lines

                 CGContextSetLineDash( m_cgContext , 0 , lengths , count ) ;
                 CGContextSetLineDash( m_cgContext , 0 , lengths , count ) ;

-                CGContextSetLineCap( m_cgContext , kCGLineCapButt ) ;
+                // force the line cap, otherwise we get artifacts (overlaps) and just solid lines
+                cap = kCGLineCapButt ;

             }
             }

-            else 
+            else

             {
                CGContextSetLineDash( m_cgContext , 0 , NULL , 0 ) ;
             }
 
             {
                CGContextSetLineDash( m_cgContext , 0 , NULL , 0 ) ;
             }
 

+            CGContextSetLineCap( m_cgContext , cap ) ;
+            CGContextSetLineJoin( m_cgContext , join ) ;
+

             delete[] userLengths ;
             delete[] userLengths ;

-#endif

         }
         }

+

         if ( fill && stroke )
         if ( fill && stroke )

-        {

             m_mode = kCGPathFillStroke ;
             m_mode = kCGPathFillStroke ;

-        }

     }
 }
 
     }
 }
 


@@ -780,118 +825,349 @@ void wxMacCGContext::SetBrush( const wxBrush &brush )
     CGContextSetShouldAntialias( m_cgContext , false ) ;
 #endif
 
     CGContextSetShouldAntialias( m_cgContext , false ) ;
 #endif
 

-    if ( fill | stroke )
+    if ( fill || stroke )

     {
         // setup brushes
         m_mode = kCGPathFill ; // just a default
 
         if ( fill )
         {
     {
         // setup brushes
         m_mode = kCGPathFill ; // just a default
 
         if ( fill )
         {

-#if defined(_NEW_GC_SUPPORT_)
-            // new candidate
+            if ( brush.GetStyle() == wxSOLID )

             {
             {

-                CGPatternRef  patternRef;
-                float  alphaArray[1];
-                long  result;
-                bool  hasSetPattern, useMultibit;
-
-                hasSetPattern = false;
-                useMultibit = true;
-                result = CreatePatternFromBrush( &patternRef, brush, useMultibit );
-                if (result == 0)
-                {
-                    EstablishPatternColorSpace( m_cgContext, useMultibit, true );
-
-                    alphaArray[0] = 1.0;
-                    CGContextSetFillPattern( m_cgContext, patternRef, alphaArray );
-                    CGPatternRelease( patternRef );
-
-                    hasSetPattern = true;
-
-//wxLogDebug( wxT("CreatePatternFromBrush succeeded!") );
-                }
+                CGContextSetRGBFillColor( m_cgContext , brush.GetColour().Red() / 255.0 , brush.GetColour().Green() / 255.0 , 
+                    brush.GetColour().Blue() / 255.0 , brush.GetColour().Alpha() / 255.0 ) ;
+            }
+            else if ( brush.IsHatch() )
+            {
+                wxMacCFRefHolder<CGColorSpaceRef> patternSpace( CGColorSpaceCreatePattern( wxMacGetGenericRGBColorSpace() ) ) ;
+                CGContextSetFillColorSpace( m_cgContext , patternSpace ) ;
+                wxMacCFRefHolder<CGPatternRef> pattern( *( new HatchPattern( brush.GetStyle() , CGContextGetCTM( m_cgContext ) ) ) );

 
 

-                // NB: the (-2) result is from wxBrush instances that don't have a stipple wxBitmap
-                if (result < (-2))
-                    wxLogDebug( wxT("CreatePatternFromBrush failed: result [%ld]"), result );
+                CGFloat  colorArray[4] = { brush.GetColour().Red() / 255.0 , brush.GetColour().Green() / 255.0 , 
+                    brush.GetColour().Blue() / 255.0 , brush.GetColour().Alpha() / 255.0 } ;

 
 

-                if (!hasSetPattern)
+                CGContextSetFillPattern( m_cgContext, pattern , colorArray ) ;
+            }
+            else
+            {
+                // now brush is a bitmap
+                CGFloat  alphaArray[1] = { 1.0 } ;
+                wxBitmap* bmp = brush.GetStipple() ;
+                if ( bmp && bmp->Ok() )

                 {
                 {

-                    RGBColor col;
-
-#if 1
-                    col = MAC_WXCOLORREF( brush.GetColour().GetPixel() );
-#else
-                    GetThemeBrushAsColor( brush.MacGetTheme(), 32, true, &col );
-#endif
-
-                    CGContextSetRGBFillColor(
-                        m_cgContext, (float) col.red / 65536.0,
-                        (float) col.green / 65536.0, (float) col.blue / 65536.0, 1.0 );
+                    wxMacCFRefHolder<CGColorSpaceRef> patternSpace( CGColorSpaceCreatePattern( NULL ) ) ;
+                    CGContextSetFillColorSpace( m_cgContext , patternSpace ) ;
+                    wxMacCFRefHolder<CGPatternRef> pattern( *( new ImagePattern( bmp , CGContextGetCTM( m_cgContext ) ) ) );
+                    CGContextSetFillPattern( m_cgContext, pattern , alphaArray ) ;

                 }
             }
 
                 }
             }
 

-#else
-            // original implementation
-            RGBColor col = MAC_WXCOLORREF( brush.GetColour().GetPixel() ) ;
-            CGContextSetRGBFillColor( m_cgContext , col.red / 65536.0 , col.green / 65536.0 , col.blue / 65536.0 , 1.0 ) ;
-#endif
-

             m_mode = kCGPathFill ;
         }
             m_mode = kCGPathFill ;
         }

-        if ( stroke )
-        {
-            m_mode = kCGPathStroke ;
-        }
+

         if ( fill && stroke )
         if ( fill && stroke )

-        {

             m_mode = kCGPathFillStroke ;
             m_mode = kCGPathFillStroke ;

-        }
+        else if ( stroke )
+            m_mode = kCGPathStroke ;

     }
 }
 
     }
 }
 

-void AddEllipticArcToPath(CGContextRef c, CGPoint center, float a, float b, float fromDegree , float toDegree )
+void wxMacCGContext::DrawText( const wxString &str, wxCoord x, wxCoord y, double angle ) 

 {
 {

-    CGContextSaveGState(c);
-    CGContextTranslateCTM(c, center.x, center.y);
-    CGContextScaleCTM(c, a, b);
-    CGContextMoveToPoint(c, 1, 0);
-    CGContextAddArc(c, 0, 0, 1, DegToRad(fromDegree), DegToRad(toDegree), 0);
-    CGContextClosePath(c);
-    CGContextRestoreGState(c);
-} 
-
-void AddRoundedRectToPath(CGContextRef c, CGRect rect, float ovalWidth,
-      float ovalHeight)
+    OSStatus status = noErr ;
+    ATSUTextLayout atsuLayout ;
+    UniCharCount chars = str.length() ;
+    UniChar* ubuf = NULL ;
+
+#if SIZEOF_WCHAR_T == 4
+    wxMBConvUTF16 converter ;
+#if wxUSE_UNICODE
+    size_t unicharlen = converter.WC2MB( NULL , str.wc_str() , 0 ) ;
+    ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
+    converter.WC2MB( (char*) ubuf , str.wc_str(), unicharlen + 2 ) ;
+#else
+    const wxWCharBuffer wchar = str.wc_str( wxConvLocal ) ;
+    size_t unicharlen = converter.WC2MB( NULL , wchar.data() , 0 ) ;
+    ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
+    converter.WC2MB( (char*) ubuf , wchar.data() , unicharlen + 2 ) ;
+#endif
+    chars = unicharlen / 2 ;
+#else
+#if wxUSE_UNICODE
+    ubuf = (UniChar*) str.wc_str() ;
+#else
+    wxWCharBuffer wchar = str.wc_str( wxConvLocal ) ;
+    chars = wxWcslen( wchar.data() ) ;
+    ubuf = (UniChar*) wchar.data() ;
+#endif
+#endif
+
+    status = ::ATSUCreateTextLayoutWithTextPtr( (UniCharArrayPtr) ubuf , 0 , chars , chars , 1 ,
+        &chars , (ATSUStyle*) &m_macATSUIStyle , &atsuLayout ) ;
+
+    wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the rotated text") );
+
+    status = ::ATSUSetTransientFontMatching( atsuLayout , true ) ;
+    wxASSERT_MSG( status == noErr , wxT("couldn't setup transient font matching") );
+
+    int iAngle = int( angle );
+    if ( abs(iAngle) > 0 )
+    {
+        Fixed atsuAngle = IntToFixed( iAngle ) ;
+        ATSUAttributeTag atsuTags[] =
+        {
+            kATSULineRotationTag ,
+        } ;
+        ByteCount atsuSizes[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] =
+        {
+            sizeof( Fixed ) ,
+        } ;
+        ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] =
+        {
+            &atsuAngle ,
+        } ;
+        status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags) / sizeof(ATSUAttributeTag),
+            atsuTags, atsuSizes, atsuValues ) ;
+    }
+
+    {
+        ATSUAttributeTag atsuTags[] =
+        {
+            kATSUCGContextTag ,
+        } ;
+        ByteCount atsuSizes[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] =
+        {
+            sizeof( CGContextRef ) ,
+        } ;
+        ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] =
+        {
+            &m_cgContext ,
+        } ;
+        status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags) / sizeof(ATSUAttributeTag),
+            atsuTags, atsuSizes, atsuValues ) ;
+    }
+
+    ATSUTextMeasurement textBefore, textAfter ;
+    ATSUTextMeasurement ascent, descent ;
+
+    status = ::ATSUGetUnjustifiedBounds( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
+        &textBefore , &textAfter, &ascent , &descent );
+
+    wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );
+
+    Rect rect ;
+/*
+    // TODO
+    if ( m_backgroundMode == wxSOLID )
+    {
+        wxGraphicPath* path = m_graphicContext->CreatePath() ;
+        path->MoveToPoint( drawX , drawY ) ;
+        path->AddLineToPoint(
+            (int) (drawX + sin(angle / RAD2DEG) * FixedToInt(ascent + descent)) ,
+            (int) (drawY + cos(angle / RAD2DEG) * FixedToInt(ascent + descent)) ) ;
+        path->AddLineToPoint(
+            (int) (drawX + sin(angle / RAD2DEG) * FixedToInt(ascent + descent ) + cos(angle / RAD2DEG) * FixedToInt(textAfter)) ,
+            (int) (drawY + cos(angle / RAD2DEG) * FixedToInt(ascent + descent) - sin(angle / RAD2DEG) * FixedToInt(textAfter)) ) ;
+        path->AddLineToPoint(
+            (int) (drawX + cos(angle / RAD2DEG) * FixedToInt(textAfter)) ,
+            (int) (drawY - sin(angle / RAD2DEG) * FixedToInt(textAfter)) ) ;
+
+        m_graphicContext->FillPath( path , m_textBackgroundColour ) ;
+        delete path ;
+    }
+*/
+    x += (int)(sin(angle / RAD2DEG) * FixedToInt(ascent));
+    y += (int)(cos(angle / RAD2DEG) * FixedToInt(ascent));
+
+    status = ::ATSUMeasureTextImage( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
+        IntToFixed(x) , IntToFixed(y) , &rect );
+    wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );
+
+    CGContextSaveGState(m_cgContext);
+    CGContextTranslateCTM(m_cgContext, x, y);
+    CGContextScaleCTM(m_cgContext, 1, -1);
+    status = ::ATSUDrawText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
+        IntToFixed(0) , IntToFixed(0) );
+
+    wxASSERT_MSG( status == noErr , wxT("couldn't draw the rotated text") );
+
+    CGContextRestoreGState(m_cgContext) ;
+
+    ::ATSUDisposeTextLayout(atsuLayout);
+
+#if SIZEOF_WCHAR_T == 4
+    free( ubuf ) ;
+#endif
+}
+    
+void wxMacCGContext::GetTextExtent( const wxString &str, wxCoord *width, wxCoord *height,
+                            wxCoord *descent, wxCoord *externalLeading ) const

 {
 {

-    float fw, fh;
-    if (ovalWidth == 0 || ovalHeight == 0) 
+    wxCHECK_RET( m_macATSUIStyle != NULL, wxT("wxDC(cg)::DoGetTextExtent - no valid font set") ) ;
+
+    OSStatus status = noErr ;
+    
+    ATSUTextLayout atsuLayout ;
+    UniCharCount chars = str.length() ;
+    UniChar* ubuf = NULL ;
+
+#if SIZEOF_WCHAR_T == 4
+    wxMBConvUTF16 converter ;
+#if wxUSE_UNICODE
+    size_t unicharlen = converter.WC2MB( NULL , str.wc_str() , 0 ) ;
+    ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
+    converter.WC2MB( (char*) ubuf , str.wc_str(), unicharlen + 2 ) ;
+#else
+    const wxWCharBuffer wchar = str.wc_str( wxConvLocal ) ;
+    size_t unicharlen = converter.WC2MB( NULL , wchar.data() , 0 ) ;
+    ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
+    converter.WC2MB( (char*) ubuf , wchar.data() , unicharlen + 2 ) ;
+#endif
+    chars = unicharlen / 2 ;
+#else
+#if wxUSE_UNICODE
+    ubuf = (UniChar*) str.wc_str() ;
+#else
+    wxWCharBuffer wchar = str.wc_str( wxConvLocal ) ;
+    chars = wxWcslen( wchar.data() ) ;
+    ubuf = (UniChar*) wchar.data() ;
+#endif
+#endif
+
+    status = ::ATSUCreateTextLayoutWithTextPtr( (UniCharArrayPtr) ubuf , 0 , chars , chars , 1 ,
+        &chars , (ATSUStyle*) &m_macATSUIStyle , &atsuLayout ) ;
+
+    wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the text") );
+
+    ATSUTextMeasurement textBefore, textAfter ;
+    ATSUTextMeasurement textAscent, textDescent ;
+
+    status = ::ATSUGetUnjustifiedBounds( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
+        &textBefore , &textAfter, &textAscent , &textDescent );
+
+    if ( height )
+        *height = FixedToInt(textAscent + textDescent) ;
+    if ( descent )
+        *descent = FixedToInt(textDescent) ;
+    if ( externalLeading )
+        *externalLeading = 0 ;
+    if ( width )
+        *width = FixedToInt(textAfter - textBefore) ;
+
+    ::ATSUDisposeTextLayout(atsuLayout);
+}
+
+void wxMacCGContext::GetPartialTextExtents(const wxString& text, wxArrayInt& widths) const 
+{
+    widths.Empty();
+    widths.Add(0, text.length());
+
+    if (text.empty())
+        return ;
+
+    ATSUTextLayout atsuLayout ;
+    UniCharCount chars = text.length() ;
+    UniChar* ubuf = NULL ;
+
+#if SIZEOF_WCHAR_T == 4
+    wxMBConvUTF16 converter ;
+#if wxUSE_UNICODE
+    size_t unicharlen = converter.WC2MB( NULL , text.wc_str() , 0 ) ;
+    ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
+    converter.WC2MB( (char*) ubuf , text.wc_str(), unicharlen + 2 ) ;
+#else
+    const wxWCharBuffer wchar = text.wc_str( wxConvLocal ) ;
+    size_t unicharlen = converter.WC2MB( NULL , wchar.data() , 0 ) ;
+    ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
+    converter.WC2MB( (char*) ubuf , wchar.data() , unicharlen + 2 ) ;
+#endif
+    chars = unicharlen / 2 ;
+#else
+#if wxUSE_UNICODE
+    ubuf = (UniChar*) text.wc_str() ;
+#else
+    wxWCharBuffer wchar = text.wc_str( wxConvLocal ) ;
+    chars = wxWcslen( wchar.data() ) ;
+    ubuf = (UniChar*) wchar.data() ;
+#endif
+#endif
+
+    ::ATSUCreateTextLayoutWithTextPtr( (UniCharArrayPtr) ubuf , 0 , chars , chars , 1 ,
+        &chars , (ATSUStyle*) &m_macATSUIStyle , &atsuLayout ) ;
+
+    for ( int pos = 0; pos < (int)chars; pos ++ )

     {
     {

-        CGContextAddRect(c, rect);
-        return;
+        unsigned long actualNumberOfBounds = 0;
+        ATSTrapezoid glyphBounds;
+
+        // We get a single bound, since the text should only require one. If it requires more, there is an issue
+        OSStatus result;
+        result = ATSUGetGlyphBounds( atsuLayout, 0, 0, kATSUFromTextBeginning, pos + 1,
+            kATSUseDeviceOrigins, 1, &glyphBounds, &actualNumberOfBounds );
+        if (result != noErr || actualNumberOfBounds != 1 )
+            return ;
+
+        widths[pos] = FixedToInt( glyphBounds.upperRight.x - glyphBounds.upperLeft.x );
+        //unsigned char uch = s[i];

     }
     }

-    CGContextSaveGState(c);
-    CGContextTranslateCTM(c, CGRectGetMinX(rect), CGRectGetMinY(rect));
-    CGContextScaleCTM(c, ovalWidth, ovalHeight);
-    fw = CGRectGetWidth(rect) / ovalWidth;
-    fh = CGRectGetHeight(rect) / ovalHeight;
-    CGContextMoveToPoint(c, fw, fh/2);
-    CGContextAddArcToPoint(c, fw, fh, fw/2, fh, 1);
-    CGContextAddArcToPoint(c, 0, fh, 0, fh/2, 1);
-    CGContextAddArcToPoint(c, 0, 0, fw/2, 0, 1);
-    CGContextAddArcToPoint(c, fw, 0, fw, fh/2, 1);
-    CGContextClosePath(c);
-    CGContextRestoreGState(c);
-} 
+
+    ::ATSUDisposeTextLayout(atsuLayout);
+}
+
+void wxMacCGContext::SetFont( const wxFont &font ) 
+{
+    if ( m_macATSUIStyle )
+    {
+        ::ATSUDisposeStyle((ATSUStyle)m_macATSUIStyle);
+        m_macATSUIStyle = NULL ;
+    }
+
+    if ( font.Ok() )
+    {
+        OSStatus status ;
+
+        status = ATSUCreateAndCopyStyle( (ATSUStyle) font.MacGetATSUStyle() , (ATSUStyle*) &m_macATSUIStyle ) ;
+
+        wxASSERT_MSG( status == noErr, wxT("couldn't create ATSU style") ) ;
+
+        // we need the scale here ...
+
+        Fixed atsuSize = IntToFixed( int( /*m_scaleY*/ 1 * font.MacGetFontSize()) ) ;
+        RGBColor atsuColor = MAC_WXCOLORREF( m_textForegroundColor.GetPixel() ) ;
+        ATSUAttributeTag atsuTags[] =
+        {
+                kATSUSizeTag ,
+                kATSUColorTag ,
+        } ;
+        ByteCount atsuSizes[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] =
+        {
+                sizeof( Fixed ) ,
+                sizeof( RGBColor ) ,
+        } ;
+        ATSUAttributeValuePtr atsuValues[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] =
+        {
+                &atsuSize ,
+                &atsuColor ,
+        } ;
+
+        status = ::ATSUSetAttributes(
+            (ATSUStyle)m_macATSUIStyle, sizeof(atsuTags) / sizeof(ATSUAttributeTag) ,
+            atsuTags, atsuSizes, atsuValues);
+
+        wxASSERT_MSG( status == noErr , wxT("couldn't modify ATSU style") ) ;
+    }
+}
+
+
+#pragma mark -

 
 wxDC::wxDC()
 {
 
 wxDC::wxDC()
 {

-    m_ok = FALSE;
-    m_colour = TRUE;
+    m_ok = false ;
+    m_colour = true;

     m_mm_to_pix_x = mm2pt;
     m_mm_to_pix_y = mm2pt;
     m_mm_to_pix_x = mm2pt;
     m_mm_to_pix_y = mm2pt;

-    m_internalDeviceOriginX = 0;
-    m_internalDeviceOriginY = 0;
+

     m_externalDeviceOriginX = 0;
     m_externalDeviceOriginY = 0;
     m_logicalScaleX = 1.0;
     m_externalDeviceOriginX = 0;
     m_externalDeviceOriginY = 0;
     m_logicalScaleX = 1.0;


@@ -900,37 +1176,37 @@ wxDC::wxDC()
     m_userScaleY = 1.0;
     m_scaleX = 1.0;
     m_scaleY = 1.0;
     m_userScaleY = 1.0;
     m_scaleX = 1.0;
     m_scaleY = 1.0;

-    m_needComputeScaleX = FALSE;
-    m_needComputeScaleY = FALSE;
+    m_needComputeScaleX =
+    m_needComputeScaleY = false;

 
 

-    m_ok = FALSE ;

     m_macPort = 0 ;
     m_macPort = 0 ;

-    m_macLocalOrigin.x = m_macLocalOrigin.y = 0 ;
+    m_macLocalOrigin.x =
+    m_macLocalOrigin.y = 0 ;

 
     m_pen = *wxBLACK_PEN;
     m_font = *wxNORMAL_FONT;
     m_brush = *wxWHITE_BRUSH;
 
     m_macATSUIStyle = NULL ;
 
     m_pen = *wxBLACK_PEN;
     m_font = *wxNORMAL_FONT;
     m_brush = *wxWHITE_BRUSH;
 
     m_macATSUIStyle = NULL ;

-

     m_graphicContext = NULL ;
 }
 
     m_graphicContext = NULL ;
 }
 

-wxDC::~wxDC(void)
+wxDC::~wxDC()

 {
 {

-    if( m_macATSUIStyle )
+    if ( m_macATSUIStyle )

     {
         ::ATSUDisposeStyle((ATSUStyle)m_macATSUIStyle);
         m_macATSUIStyle = NULL ;
     }
 
     {
         ::ATSUDisposeStyle((ATSUStyle)m_macATSUIStyle);
         m_macATSUIStyle = NULL ;
     }
 

-    delete m_graphicContext ;  
+    delete m_graphicContext ;

 }
 
 void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask )
 {
 }
 
 void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask )
 {

-    wxCHECK_RET( Ok(), wxT("invalid window dc") );
-    wxCHECK_RET( bmp.Ok(), wxT("invalid bitmap") );
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawBitmap - invalid DC") );
+    wxCHECK_RET( bmp.Ok(), wxT("wxDC(cg)::DoDrawBitmap - invalid bitmap") );
+

     wxCoord xx = XLOG2DEVMAC(x);
     wxCoord yy = YLOG2DEVMAC(y);
     wxCoord w = bmp.GetWidth();
     wxCoord xx = XLOG2DEVMAC(x);
     wxCoord yy = YLOG2DEVMAC(y);
     wxCoord w = bmp.GetWidth();


@@ -938,17 +1214,24 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
     wxCoord ww = XLOG2DEVREL(w);
     wxCoord hh = YLOG2DEVREL(h);
 
     wxCoord ww = XLOG2DEVREL(w);
     wxCoord hh = YLOG2DEVREL(h);
 

-    CGContextRef cg = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;
-    CGImageRef image = (CGImageRef)( bmp.CGImageCreate() ) ;
-    HIRect r = CGRectMake( xx , yy , ww , hh ) ;
-    HIViewDrawCGImage( cg , &r , image ) ;
-    CGImageRelease( image ) ;
+    if ( bmp.GetDepth()==1 )
+    {
+        wxGraphicPath* path = m_graphicContext->CreatePath() ;
+        path->AddRectangle( xx , yy , ww , hh ) ;
+        m_graphicContext->FillPath( path , m_textBackgroundColour, wxODDEVEN_RULE) ;
+        delete path;
+        m_graphicContext->DrawBitmap( bmp, xx , yy , ww , hh ) ;
+    }
+    else
+    {
+        m_graphicContext->DrawBitmap( bmp, xx , yy , ww , hh ) ;
+    }

 }
 
 void wxDC::DoDrawIcon( const wxIcon &icon, wxCoord x, wxCoord y )
 {
 }
 
 void wxDC::DoDrawIcon( const wxIcon &icon, wxCoord x, wxCoord y )
 {

-    wxCHECK_RET(Ok(), wxT("Invalid dc  wxDC::DoDrawIcon"));
-    wxCHECK_RET(icon.Ok(), wxT("Invalid icon wxDC::DoDrawIcon"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawIcon - invalid DC") );
+    wxCHECK_RET( icon.Ok(), wxT("wxDC(cg)::DoDrawIcon - invalid icon") );

 
     wxCoord xx = XLOG2DEVMAC(x);
     wxCoord yy = YLOG2DEVMAC(y);
 
     wxCoord xx = XLOG2DEVMAC(x);
     wxCoord yy = YLOG2DEVMAC(y);


@@ -957,19 +1240,13 @@ void wxDC::DoDrawIcon( const wxIcon &icon, wxCoord x, wxCoord y )
     wxCoord ww = XLOG2DEVREL(w);
     wxCoord hh = YLOG2DEVREL(h);
 
     wxCoord ww = XLOG2DEVREL(w);
     wxCoord hh = YLOG2DEVREL(h);
 

-    CGContextRef cg = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;
-    CGRect r = CGRectMake( 00 , 00 , ww , hh ) ;
-    CGContextSaveGState(cg);    
-    CGContextTranslateCTM(cg, xx , yy + hh );
-    CGContextScaleCTM(cg, 1, -1);
-    PlotIconRefInContext( cg , &r , kAlignNone , kTransformNone ,
-        NULL , kPlotIconRefNormalFlags , MAC_WXHICON( icon.GetHICON() ) ) ;
-    CGContextRestoreGState( cg ) ;
+    m_graphicContext->DrawIcon( icon , xx, yy, ww, hh ) ;

 }
 
 void wxDC::DoSetClippingRegion( wxCoord x, wxCoord y, wxCoord width, wxCoord height )
 {
 }
 
 void wxDC::DoSetClippingRegion( wxCoord x, wxCoord y, wxCoord width, wxCoord height )
 {

-    wxCHECK_RET(Ok(), wxT("wxDC::DoSetClippingRegion  Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoSetClippingRegion - invalid DC") );
+

     wxCoord xx, yy, ww, hh;
     xx = XLOG2DEVMAC(x);
     yy = YLOG2DEVMAC(y);
     wxCoord xx, yy, ww, hh;
     xx = XLOG2DEVMAC(x);
     yy = YLOG2DEVMAC(y);


@@ -977,38 +1254,43 @@ void wxDC::DoSetClippingRegion( wxCoord x, wxCoord y, wxCoord width, wxCoord hei
     hh = YLOG2DEVREL(height);
 
     CGContextRef cgContext = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;
     hh = YLOG2DEVREL(height);
 
     CGContextRef cgContext = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;

-    CGRect clipRect = CGRectMake( xx ,yy , ww, hh ) ;
+    CGRect clipRect = CGRectMake( xx , yy , ww, hh ) ;

     CGContextClipToRect( cgContext , clipRect ) ;
 
 //    SetRectRgn( (RgnHandle) m_macCurrentClipRgn , xx , yy , xx + ww , yy + hh ) ;
 //    SectRgn( (RgnHandle) m_macCurrentClipRgn , (RgnHandle) m_macBoundaryClipRgn , (RgnHandle) m_macCurrentClipRgn ) ;
     CGContextClipToRect( cgContext , clipRect ) ;
 
 //    SetRectRgn( (RgnHandle) m_macCurrentClipRgn , xx , yy , xx + ww , yy + hh ) ;
 //    SectRgn( (RgnHandle) m_macCurrentClipRgn , (RgnHandle) m_macBoundaryClipRgn , (RgnHandle) m_macCurrentClipRgn ) ;

-    if( m_clipping )
+
+    if ( m_clipping )

     {
     {

-        m_clipX1 = wxMax( m_clipX1 , xx );
-        m_clipY1 = wxMax( m_clipY1 , yy );
-        m_clipX2 = wxMin( m_clipX2, (xx + ww));
-        m_clipY2 = wxMin( m_clipY2, (yy + hh));
+        m_clipX1 = wxMax( m_clipX1, xx );
+        m_clipY1 = wxMax( m_clipY1, yy );
+        m_clipX2 = wxMin( m_clipX2, (xx + ww) );
+        m_clipY2 = wxMin( m_clipY2, (yy + hh) );

     }
     else
     {
     }
     else
     {

-        m_clipping = TRUE;
+        m_clipping = true;
+

         m_clipX1 = xx;
         m_clipY1 = yy;
         m_clipX2 = xx + ww;
         m_clipY2 = yy + hh;
     }
         m_clipX1 = xx;
         m_clipY1 = yy;
         m_clipX2 = xx + ww;
         m_clipY2 = yy + hh;
     }

-    // TODO as soon as we don't reset the context for each operation anymore
+
+    // TODO: as soon as we don't reset the context for each operation anymore

     // we have to update the context as well
 }
 
     // we have to update the context as well
 }
 

-void wxDC::DoSetClippingRegionAsRegion( const wxRegion &region  )
+void wxDC::DoSetClippingRegionAsRegion( const wxRegion &region )

 {
 {

-    wxCHECK_RET( Ok(), wxT("invalid window dc") ) ;
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoSetClippingRegionAsRegion - invalid DC") );
+

     if (region.Empty())
     {
         DestroyClippingRegion();
         return;
     }
     if (region.Empty())
     {
         DestroyClippingRegion();
         return;
     }

+

     wxCoord x, y, w, h;
     region.GetBox( x, y, w, h );
     wxCoord xx, yy, ww, hh;
     wxCoord x, y, w, h;
     region.GetBox( x, y, w, h );
     wxCoord xx, yy, ww, hh;


@@ -1016,6 +1298,7 @@ void wxDC::DoSetClippingRegionAsRegion( const wxRegion &region  )
     yy = YLOG2DEVMAC(y);
     ww = XLOG2DEVREL(w);
     hh = YLOG2DEVREL(h);
     yy = YLOG2DEVMAC(y);
     ww = XLOG2DEVREL(w);
     hh = YLOG2DEVREL(h);

+

     // if we have a scaling that we cannot map onto native regions
     // we must use the box
     if ( ww != w || hh != h )
     // if we have a scaling that we cannot map onto native regions
     // we must use the box
     if ( ww != w || hh != h )


@@ -1024,24 +1307,24 @@ void wxDC::DoSetClippingRegionAsRegion( const wxRegion &region  )
     }
     else
     {
     }
     else
     {

-        /*
+#if 0

         CopyRgn( (RgnHandle) region.GetWXHRGN() , (RgnHandle) m_macCurrentClipRgn ) ;
         if ( xx != x || yy != y )
         CopyRgn( (RgnHandle) region.GetWXHRGN() , (RgnHandle) m_macCurrentClipRgn ) ;
         if ( xx != x || yy != y )

-        {

             OffsetRgn( (RgnHandle) m_macCurrentClipRgn , xx - x , yy - y ) ;
             OffsetRgn( (RgnHandle) m_macCurrentClipRgn , xx - x , yy - y ) ;

-        }
-        SectRgn( (RgnHandle) m_macCurrentClipRgn , (RgnHandle) m_macBoundaryClipRgn , (RgnHandle) m_macCurrentClipRgn ) ;
-        */
-        if( m_clipping )
+        SectRgn( (RgnHandle)m_macCurrentClipRgn , (RgnHandle)m_macBoundaryClipRgn , (RgnHandle)m_macCurrentClipRgn ) ;
+#endif
+
+        if ( m_clipping )

         {
         {

-            m_clipX1 = wxMax( m_clipX1 , xx );
-            m_clipY1 = wxMax( m_clipY1 , yy );
-            m_clipX2 = wxMin( m_clipX2, (xx + ww));
-            m_clipY2 = wxMin( m_clipY2, (yy + hh));
+            m_clipX1 = wxMax( m_clipX1, xx );
+            m_clipY1 = wxMax( m_clipY1, yy );
+            m_clipX2 = wxMin( m_clipX2, (xx + ww) );
+            m_clipY2 = wxMin( m_clipY2, (yy + hh) );

         }
         else
         {
         }
         else
         {

-            m_clipping = TRUE;
+            m_clipping = true;
+

             m_clipX1 = xx;
             m_clipY1 = yy;
             m_clipX2 = xx + ww;
             m_clipX1 = xx;
             m_clipY1 = yy;
             m_clipX2 = xx + ww;


@@ -1053,36 +1336,45 @@ void wxDC::DoSetClippingRegionAsRegion( const wxRegion &region  )
 void wxDC::DestroyClippingRegion()
 {
 //    CopyRgn( (RgnHandle) m_macBoundaryClipRgn , (RgnHandle) m_macCurrentClipRgn ) ;
 void wxDC::DestroyClippingRegion()
 {
 //    CopyRgn( (RgnHandle) m_macBoundaryClipRgn , (RgnHandle) m_macCurrentClipRgn ) ;

+

     CGContextRef cgContext = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;
     CGContextRef cgContext = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;

-    CGContextRestoreGState( cgContext );    
-    CGContextSaveGState( cgContext );    
+    CGContextRestoreGState( cgContext );
+    CGContextSaveGState( cgContext );
+

     m_graphicContext->SetPen( m_pen ) ;
     m_graphicContext->SetBrush( m_brush ) ;
     m_graphicContext->SetPen( m_pen ) ;
     m_graphicContext->SetBrush( m_brush ) ;

-    m_clipping = FALSE;
+
+    m_clipping = false;

 }
 
 void wxDC::DoGetSizeMM( int* width, int* height ) const
 {
 }
 
 void wxDC::DoGetSizeMM( int* width, int* height ) const
 {

-    int w = 0;
-    int h = 0;
+    int w = 0, h = 0;
+

     GetSize( &w, &h );
     GetSize( &w, &h );

-    *width = long( double(w) / (m_scaleX*m_mm_to_pix_x) );
-    *height = long( double(h) / (m_scaleY*m_mm_to_pix_y) );
+    if (width)
+        *width = long( double(w) / (m_scaleX * m_mm_to_pix_x) );
+    if (height)
+        *height = long( double(h) / (m_scaleY * m_mm_to_pix_y) );

 }
 
 void wxDC::SetTextForeground( const wxColour &col )
 {
 }
 
 void wxDC::SetTextForeground( const wxColour &col )
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::SetTextForeground - invalid DC") );
+

     if ( col != m_textForegroundColour )
     {
         m_textForegroundColour = col;
     if ( col != m_textForegroundColour )
     {
         m_textForegroundColour = col;

-        MacInstallFont() ;
+        m_graphicContext->SetTextColor( col );
+        // in the current implementation the font contains the text color
+        m_graphicContext->SetFont(m_font);

     }
 }
 
 void wxDC::SetTextBackground( const wxColour &col )
 {
     }
 }
 
 void wxDC::SetTextBackground( const wxColour &col )
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::SetTextBackground - invalid DC") );
+

     m_textBackgroundColour = col;
 }
 
     m_textBackgroundColour = col;
 }
 


@@ -1091,26 +1383,31 @@ void wxDC::SetMapMode( int mode )
     switch (mode)
     {
     case wxMM_TWIPS:
     switch (mode)
     {
     case wxMM_TWIPS:

-        SetLogicalScale( twips2mm*m_mm_to_pix_x, twips2mm*m_mm_to_pix_y );
+        SetLogicalScale( twips2mm * m_mm_to_pix_x, twips2mm * m_mm_to_pix_y );

         break;
         break;

+

     case wxMM_POINTS:
     case wxMM_POINTS:

-        SetLogicalScale( pt2mm*m_mm_to_pix_x, pt2mm*m_mm_to_pix_y );
+        SetLogicalScale( pt2mm * m_mm_to_pix_x, pt2mm * m_mm_to_pix_y );

         break;
         break;

+

     case wxMM_METRIC:
         SetLogicalScale( m_mm_to_pix_x, m_mm_to_pix_y );
         break;
     case wxMM_METRIC:
         SetLogicalScale( m_mm_to_pix_x, m_mm_to_pix_y );
         break;

+

     case wxMM_LOMETRIC:
     case wxMM_LOMETRIC:

-        SetLogicalScale( m_mm_to_pix_x/10.0, m_mm_to_pix_y/10.0 );
+        SetLogicalScale( m_mm_to_pix_x / 10.0, m_mm_to_pix_y / 10.0 );

         break;
         break;

-    default:
+

     case wxMM_TEXT:
     case wxMM_TEXT:

+    default:

         SetLogicalScale( 1.0, 1.0 );
         break;
     }
         SetLogicalScale( 1.0, 1.0 );
         break;
     }

+

     if (mode != wxMM_TEXT)
     {
     if (mode != wxMM_TEXT)
     {

-        m_needComputeScaleX = TRUE;
-        m_needComputeScaleY = TRUE;
+        m_needComputeScaleX =
+        m_needComputeScaleY = true;

     }
 }
 
     }
 }
 


@@ -1147,7 +1444,7 @@ void wxDC::SetDeviceOrigin( wxCoord x, wxCoord y )
 void wxDC::SetAxisOrientation( bool xLeftRight, bool yBottomUp )
 {
     m_signX = (xLeftRight ?  1 : -1);
 void wxDC::SetAxisOrientation( bool xLeftRight, bool yBottomUp )
 {
     m_signX = (xLeftRight ?  1 : -1);

-    m_signY = (yBottomUp  ? -1 :  1);
+    m_signY = (yBottomUp ? -1 :  1);

     ComputeScaleAndOrigin();
 }
 
     ComputeScaleAndOrigin();
 }
 


@@ -1158,7 +1455,7 @@ wxSize wxDC::GetPPI() const
 
 int wxDC::GetDepth() const
 {
 
 int wxDC::GetDepth() const
 {

-    return 32 ;
+    return 32;

 }
 
 void wxDC::ComputeScaleAndOrigin()
 }
 
 void wxDC::ComputeScaleAndOrigin()


@@ -1168,70 +1465,112 @@ void wxDC::ComputeScaleAndOrigin()
     double origScaleY = m_scaleY;
     m_scaleX = m_logicalScaleX * m_userScaleX;
     m_scaleY = m_logicalScaleY * m_userScaleY;
     double origScaleY = m_scaleY;
     m_scaleX = m_logicalScaleX * m_userScaleX;
     m_scaleY = m_logicalScaleY * m_userScaleY;

-    m_deviceOriginX = m_internalDeviceOriginX + m_externalDeviceOriginX;
-    m_deviceOriginY = m_internalDeviceOriginY + m_externalDeviceOriginY;
+    m_deviceOriginX = m_externalDeviceOriginX;
+    m_deviceOriginY = m_externalDeviceOriginY;
+

     // CMB: if scale has changed call SetPen to recalulate the line width
     if (m_scaleX != origScaleX || m_scaleY != origScaleY)
     {
         // this is a bit artificial, but we need to force wxDC to think
         // the pen has changed
     // CMB: if scale has changed call SetPen to recalulate the line width
     if (m_scaleX != origScaleX || m_scaleY != origScaleY)
     {
         // this is a bit artificial, but we need to force wxDC to think
         // the pen has changed

-        wxPen pen(GetPen());
+        wxPen pen( GetPen() );
+

         m_pen = wxNullPen;
         m_pen = wxNullPen;

-        SetPen(pen);
+        SetPen( pen );

     }
 }
 
     }
 }
 

-void  wxDC::SetPalette( const wxPalette& palette )
+void wxDC::SetPalette( const wxPalette& palette )

 {
 }
 
 {
 }
 

-void  wxDC::SetBackgroundMode( int mode )
+void wxDC::SetBackgroundMode( int mode )

 {
     m_backgroundMode = mode ;
 }
 
 {
     m_backgroundMode = mode ;
 }
 

-void  wxDC::SetFont( const wxFont &font )
+void wxDC::SetFont( const wxFont &font )

 {
     m_font = font;
 {
     m_font = font;

-    MacInstallFont() ;
+    if ( m_graphicContext )
+        m_graphicContext->SetFont( font ) ;

 }
 
 }
 

-void  wxDC::SetPen( const wxPen &pen )
+void wxDC::SetPen( const wxPen &pen )

 {
     if ( m_pen == pen )
         return ;
 {
     if ( m_pen == pen )
         return ;

+    

     m_pen = pen;
     if ( m_graphicContext )
     {
     m_pen = pen;
     if ( m_graphicContext )
     {

-        m_graphicContext->SetPen( m_pen ) ;
+        if ( m_pen.GetStyle() == wxSOLID || m_pen.GetStyle() == wxTRANSPARENT )
+        {
+            m_graphicContext->SetPen( m_pen ) ;
+        }
+        else
+        {
+            // we have to compensate for moved device origins etc. otherwise patterned pens are standing still
+            // eg when using a wxScrollWindow and scrolling around
+            int origX = XLOG2DEVMAC( 0 ) ;
+            int origY = YLOG2DEVMAC( 0 ) ;
+            m_graphicContext->Translate( origX , origY ) ;
+            m_graphicContext->SetPen( m_pen ) ;
+            m_graphicContext->Translate( -origX , -origY ) ;
+        }

     }
 }
 
     }
 }
 

-void  wxDC::SetBrush( const wxBrush &brush )
+void wxDC::SetBrush( const wxBrush &brush )

 {
     if (m_brush == brush)
         return;
 {
     if (m_brush == brush)
         return;

+

     m_brush = brush;
     if ( m_graphicContext )
     {
     m_brush = brush;
     if ( m_graphicContext )
     {

-        m_graphicContext->SetBrush( m_brush ) ;
+        if ( brush.GetStyle() == wxSOLID || brush.GetStyle() == wxTRANSPARENT )
+        {
+            m_graphicContext->SetBrush( m_brush ) ;
+        }
+        else
+        {
+            // we have to compensate for moved device origins etc. otherwise patterned brushes are standing still
+            // eg when using a wxScrollWindow and scrolling around
+            int origX = XLOG2DEVMAC(0) ;
+            int origY = YLOG2DEVMAC(0) ;
+            m_graphicContext->Translate( origX , origY ) ;
+            m_graphicContext->SetBrush( m_brush ) ;
+            m_graphicContext->Translate( -origX , -origY ) ;
+        }

     }
 }
 
     }
 }
 

-void  wxDC::SetBackground( const wxBrush &brush )
+void wxDC::SetBackground( const wxBrush &brush )

 {
     if (m_backgroundBrush == brush)
         return;
 {
     if (m_backgroundBrush == brush)
         return;

+

     m_backgroundBrush = brush;
     if (!m_backgroundBrush.Ok())
         return;
 }
 
     m_backgroundBrush = brush;
     if (!m_backgroundBrush.Ok())
         return;
 }
 

-void  wxDC::SetLogicalFunction( int function )
+void wxDC::SetLogicalFunction( int function )

 {
     if (m_logicalFunction == function)
         return;
 {
     if (m_logicalFunction == function)
         return;

+

     m_logicalFunction = function ;
     m_logicalFunction = function ;

+#if wxMAC_USE_CORE_GRAPHICS_BLEND_MODES
+    CGContextRef cgContext = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;
+    if ( m_logicalFunction == wxCOPY )
+        CGContextSetBlendMode( cgContext, kCGBlendModeNormal ) ;
+    else if ( m_logicalFunction == wxINVERT )
+        CGContextSetBlendMode( cgContext, kCGBlendModeExclusion ) ;
+    else
+        CGContextSetBlendMode( cgContext, kCGBlendModeNormal ) ;
+#endif

 }
 
 extern bool wxDoFloodFill(wxDC *dc, wxCoord x, wxCoord y,
 }
 
 extern bool wxDoFloodFill(wxDC *dc, wxCoord x, wxCoord y,


@@ -1243,36 +1582,40 @@ bool wxDC::DoFloodFill(wxCoord x, wxCoord y,
     return wxDoFloodFill(this, x, y, col, style);
 }
 
     return wxDoFloodFill(this, x, y, col, style);
 }
 

-bool  wxDC::DoGetPixel( wxCoord x, wxCoord y, wxColour *col ) const
+bool wxDC::DoGetPixel( wxCoord x, wxCoord y, wxColour *col ) const

 {
 {

-    wxCHECK_MSG( Ok(), false, wxT("wxDC::DoGetPixel  Invalid DC") );
-    wxCHECK_MSG( Ok(), false, wxT("wxDC::DoGetPixel  Invalid DC") );
-    wxMacPortSaver helper((CGrafPtr)m_macPort) ;
+    wxCHECK_MSG( Ok(), false, wxT("wxDC(cg)::DoGetPixel - invalid DC") );
+

     RGBColor colour;
     RGBColor colour;

-    GetCPixel( 
-        XLOG2DEVMAC(x) + m_macLocalOriginInPort.x - m_macLocalOrigin.x, 
+#ifndef __LP64__
+    wxMacPortSaver helper((CGrafPtr)m_macPort) ;
+    // NB: GetCPixel is a deprecated QD call, and a slow one at that
+    GetCPixel(
+        XLOG2DEVMAC(x) + m_macLocalOriginInPort.x - m_macLocalOrigin.x,

         YLOG2DEVMAC(y) + m_macLocalOriginInPort.y - m_macLocalOrigin.y, &colour );
         YLOG2DEVMAC(y) + m_macLocalOriginInPort.y - m_macLocalOrigin.y, &colour );

-    // Convert from Mac colour to wx
-    col->Set( colour.red   >> 8,
-        colour.green >> 8,
-        colour.blue  >> 8);
+#endif
+    // convert from Mac colour to wx
+    col->Set( colour.red >> 8, colour.green >> 8, colour.blue >> 8 );
+

     return true ;
 }
 
     return true ;
 }
 

-void  wxDC::DoDrawLine( wxCoord x1, wxCoord y1, wxCoord x2, wxCoord y2 )
+void wxDC::DoDrawLine( wxCoord x1, wxCoord y1, wxCoord x2, wxCoord y2 )

 {
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
-    
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawLine - invalid DC") );
+
+#if !wxMAC_USE_CORE_GRAPHICS_BLEND_MODES

     if ( m_logicalFunction != wxCOPY )
         return ;
     if ( m_logicalFunction != wxCOPY )
         return ;

-    
+#endif
+

     wxCoord xx1 = XLOG2DEVMAC(x1) ;
     wxCoord yy1 = YLOG2DEVMAC(y1) ;
     wxCoord xx2 = XLOG2DEVMAC(x2) ;
     wxCoord yy2 = YLOG2DEVMAC(y2) ;
 
     wxGraphicPath* path = m_graphicContext->CreatePath() ;
     wxCoord xx1 = XLOG2DEVMAC(x1) ;
     wxCoord yy1 = YLOG2DEVMAC(y1) ;
     wxCoord xx2 = XLOG2DEVMAC(x2) ;
     wxCoord yy2 = YLOG2DEVMAC(y2) ;
 
     wxGraphicPath* path = m_graphicContext->CreatePath() ;

-    path->MoveToPoint( xx1 , yy1 ) ;
+    path->MoveToPoint( xx1, yy1 ) ;

     path->AddLineToPoint( xx2 , yy2 ) ;
     path->CloseSubpath() ;
     m_graphicContext->StrokePath( path ) ;
     path->AddLineToPoint( xx2 , yy2 ) ;
     path->CloseSubpath() ;
     m_graphicContext->StrokePath( path ) ;


@@ -1282,21 +1625,21 @@ void  wxDC::DoDrawLine( wxCoord x1, wxCoord y1, wxCoord x2, wxCoord y2 )
     CalcBoundingBox(x2, y2);
 }
 
     CalcBoundingBox(x2, y2);
 }
 

-void  wxDC::DoCrossHair( wxCoord x, wxCoord y )
+void wxDC::DoCrossHair( wxCoord x, wxCoord y )

 {
 {

-    wxCHECK_RET( Ok(), wxT("wxDC::DoCrossHair  Invalid window dc") );
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoCrossHair - invalid DC") );

 
     if ( m_logicalFunction != wxCOPY )
         return ;
 
     if ( m_logicalFunction != wxCOPY )
         return ;

-    
-    int w = 0;
-    int h = 0;
+
+    int w = 0, h = 0;
+

     GetSize( &w, &h );
     wxCoord xx = XLOG2DEVMAC(x);
     wxCoord yy = YLOG2DEVMAC(y);
 
     wxGraphicPath* path = m_graphicContext->CreatePath() ;
     GetSize( &w, &h );
     wxCoord xx = XLOG2DEVMAC(x);
     wxCoord yy = YLOG2DEVMAC(y);
 
     wxGraphicPath* path = m_graphicContext->CreatePath() ;

-    path->MoveToPoint( XLOG2DEVMAC(0), yy  ) ;
+    path->MoveToPoint( XLOG2DEVMAC(0), yy ) ;

     path->AddLineToPoint( XLOG2DEVMAC(w), yy ) ;
     path->CloseSubpath() ;
     path->MoveToPoint( xx, YLOG2DEVMAC(0) ) ;
     path->AddLineToPoint( XLOG2DEVMAC(w), yy ) ;
     path->CloseSubpath() ;
     path->MoveToPoint( xx, YLOG2DEVMAC(0) ) ;


@@ -1306,14 +1649,14 @@ void  wxDC::DoCrossHair( wxCoord x, wxCoord y )
     delete path ;
 
     CalcBoundingBox(x, y);
     delete path ;
 
     CalcBoundingBox(x, y);

-    CalcBoundingBox(x+w, y+h);
+    CalcBoundingBox(x + w, y + h);

 }
 
 }
 

-void  wxDC::DoDrawArc( wxCoord x1, wxCoord y1,
+void wxDC::DoDrawArc( wxCoord x1, wxCoord y1,

                       wxCoord x2, wxCoord y2,
                       wxCoord xc, wxCoord yc )
 {
                       wxCoord x2, wxCoord y2,
                       wxCoord xc, wxCoord yc )
 {

-    wxCHECK_RET(Ok(), wxT("wxDC::DoDrawArc  Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawArc - invalid DC") );

 
     if ( m_logicalFunction != wxCOPY )
         return ;
 
     if ( m_logicalFunction != wxCOPY )
         return ;


@@ -1324,10 +1667,11 @@ void  wxDC::DoDrawArc( wxCoord x1, wxCoord y1,
     wxCoord yy2 = YLOG2DEVMAC(y2);
     wxCoord xxc = XLOG2DEVMAC(xc);
     wxCoord yyc = YLOG2DEVMAC(yc);
     wxCoord yy2 = YLOG2DEVMAC(y2);
     wxCoord xxc = XLOG2DEVMAC(xc);
     wxCoord yyc = YLOG2DEVMAC(yc);

+

     double dx = xx1 - xxc;
     double dy = yy1 - yyc;
     double dx = xx1 - xxc;
     double dy = yy1 - yyc;

-    double radius = sqrt((double)(dx*dx+dy*dy));
-    wxCoord rad   = (wxCoord)radius;
+    double radius = sqrt((double)(dx * dx + dy * dy));
+    wxCoord rad = (wxCoord)radius;

     double sa, ea;
     if (xx1 == xx2 && yy1 == yy2)
     {
     double sa, ea;
     if (xx1 == xx2 && yy1 == yy2)
     {


@@ -1342,31 +1686,32 @@ void  wxDC::DoDrawArc( wxCoord x1, wxCoord y1,
     {
         sa = (xx1 - xxc == 0) ?
             (yy1 - yyc < 0) ? 90.0 : -90.0 :
     {
         sa = (xx1 - xxc == 0) ?
             (yy1 - yyc < 0) ? 90.0 : -90.0 :

-        -atan2(double(yy1-yyc), double(xx1-xxc)) * RAD2DEG;
+        -atan2(double(yy1 - yyc), double(xx1 - xxc)) * RAD2DEG;

         ea = (xx2 - xxc == 0) ?
             (yy2 - yyc < 0) ? 90.0 : -90.0 :
         ea = (xx2 - xxc == 0) ?
             (yy2 - yyc < 0) ? 90.0 : -90.0 :

-        -atan2(double(yy2-yyc), double(xx2-xxc)) * RAD2DEG;
+        -atan2(double(yy2 - yyc), double(xx2 - xxc)) * RAD2DEG;

     }
 
     bool fill = m_brush.GetStyle() != wxTRANSPARENT ;
     }
 
     bool fill = m_brush.GetStyle() != wxTRANSPARENT ;

-    wxMacCGContext* mctx = ((wxMacCGContext*) m_graphicContext) ;
-    CGContextRef ctx = mctx->GetNativeContext() ;
-    CGContextSaveGState( ctx ) ;
-    CGContextTranslateCTM( ctx, xxc , yyc );
-    CGContextScaleCTM( ctx , 1 , -1 ) ;
+    
+    wxGraphicPath* path = m_graphicContext->CreatePath() ;
+    m_graphicContext->PushState() ;
+    m_graphicContext->Translate( xxc, yyc ) ;
+    m_graphicContext->Scale( 1, -1 ) ;

     if ( fill )
     if ( fill )

-        CGContextMoveToPoint( ctx , 0 , 0 ) ;
-    CGContextAddArc( ctx, 0, 0 , rad , DegToRad(sa), DegToRad(ea), 0);
+        path->MoveToPoint( 0, 0 ) ;
+    path->AddArc( 0, 0, rad , DegToRad(sa) , DegToRad(ea), false ) ;

     if ( fill )
     if ( fill )

-        CGContextAddLineToPoint( ctx , 0 , 0 ) ;
-    CGContextRestoreGState( ctx ) ;
-    CGContextDrawPath( ctx , mctx->GetDrawingMode() ) ;
+        path->AddLineToPoint( 0, 0 ) ;
+    m_graphicContext->DrawPath( path ) ;
+    m_graphicContext->PopState() ;
+    delete path ;

 }
 
 }
 

-void  wxDC::DoDrawEllipticArc( wxCoord x, wxCoord y, wxCoord w, wxCoord h,
+void wxDC::DoDrawEllipticArc( wxCoord x, wxCoord y, wxCoord w, wxCoord h,

                               double sa, double ea )
 {
                               double sa, double ea )
 {

-    wxCHECK_RET(Ok(), wxT("wxDC::DoDrawEllepticArc  Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawEllipticArc - invalid DC") );

 
     if ( m_logicalFunction != wxCOPY )
         return ;
 
     if ( m_logicalFunction != wxCOPY )
         return ;


@@ -1375,40 +1720,51 @@ void  wxDC::DoDrawEllipticArc( wxCoord x, wxCoord y, wxCoord w, wxCoord h,
     wxCoord yy = YLOG2DEVMAC(y);
     wxCoord ww = m_signX * XLOG2DEVREL(w);
     wxCoord hh = m_signY * YLOG2DEVREL(h);
     wxCoord yy = YLOG2DEVMAC(y);
     wxCoord ww = m_signX * XLOG2DEVREL(w);
     wxCoord hh = m_signY * YLOG2DEVREL(h);

+

     // handle -ve width and/or height
     // handle -ve width and/or height

-    if (ww < 0) { ww = -ww; xx = xx - ww; }
-    if (hh < 0) { hh = -hh; yy = yy - hh; }
+    if (ww < 0)
+    {
+        ww = -ww;
+        xx = xx - ww;
+    }
+    if (hh < 0)
+    {
+        hh = -hh;
+        yy = yy - hh;
+    }

 
     bool fill = m_brush.GetStyle() != wxTRANSPARENT ;
 
 
     bool fill = m_brush.GetStyle() != wxTRANSPARENT ;
 

-    wxMacCGContext* mctx = ((wxMacCGContext*) m_graphicContext) ;
-    CGContextRef ctx = mctx->GetNativeContext() ;
-
-    CGContextSaveGState( ctx ) ;
-    CGContextTranslateCTM( ctx, xx + ww / 2, yy + hh / 2);
-    CGContextScaleCTM( ctx , 1 * ww / 2 , -1 * hh / 2 ) ;
+    wxGraphicPath* path = m_graphicContext->CreatePath() ;
+    m_graphicContext->PushState() ;
+    m_graphicContext->Translate( xx + ww / 2, yy + hh / 2 ) ;
+    m_graphicContext->Scale( 1 * ww / 2 , -1 * hh / 2 ) ;

     if ( fill )
     if ( fill )

-        CGContextMoveToPoint( ctx , 0 , 0 ) ;
-    CGContextAddArc( ctx, 0, 0, 1, DegToRad(sa), DegToRad(ea), 0);
+        path->MoveToPoint( 0, 0 ) ;
+    path->AddArc( 0, 0, 1 , DegToRad(sa) , DegToRad(ea), false ) ;

     if ( fill )
     if ( fill )

-        CGContextAddLineToPoint( ctx , 0 , 0 ) ;
-    CGContextRestoreGState( ctx ) ;
-    CGContextDrawPath( ctx , mctx->GetDrawingMode() ) ;    
+        path->AddLineToPoint( 0, 0 ) ;
+    m_graphicContext->DrawPath( path ) ;
+    m_graphicContext->PopState() ;
+    delete path ;

 }
 
 }
 

-void  wxDC::DoDrawPoint( wxCoord x, wxCoord y )
+void wxDC::DoDrawPoint( wxCoord x, wxCoord y )

 {
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawPoint - invalid DC") );
+

     DoDrawLine( x , y , x + 1 , y + 1 ) ;
 }
 
     DoDrawLine( x , y , x + 1 , y + 1 ) ;
 }
 

-void  wxDC::DoDrawLines(int n, wxPoint points[],
+void wxDC::DoDrawLines(int n, wxPoint points[],

                         wxCoord xoffset, wxCoord yoffset)
 {
                         wxCoord xoffset, wxCoord yoffset)
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
-    
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawLines - invalid DC") );
+
+#if !wxMAC_USE_CORE_GRAPHICS_BLEND_MODES

     if ( m_logicalFunction != wxCOPY )
         return ;
     if ( m_logicalFunction != wxCOPY )
         return ;

+#endif

 
     wxCoord x1, x2 , y1 , y2 ;
     x1 = XLOG2DEVMAC(points[0].x + xoffset);
 
     wxCoord x1, x2 , y1 , y2 ;
     x1 = XLOG2DEVMAC(points[0].x + xoffset);


@@ -1422,6 +1778,7 @@ void  wxDC::DoDrawLines(int n, wxPoint points[],
 
         path->AddLineToPoint( x2 , y2 ) ;
     }
 
         path->AddLineToPoint( x2 , y2 ) ;
     }

+

     m_graphicContext->StrokePath( path ) ;
     delete path ;
 }
     m_graphicContext->StrokePath( path ) ;
     delete path ;
 }


@@ -1429,34 +1786,34 @@ void  wxDC::DoDrawLines(int n, wxPoint points[],
 #if wxUSE_SPLINES
 void wxDC::DoDrawSpline(wxList *points)
 {
 #if wxUSE_SPLINES
 void wxDC::DoDrawSpline(wxList *points)
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
-    
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawSpline - invalid DC") );
+

     if ( m_logicalFunction != wxCOPY )
         return ;
     if ( m_logicalFunction != wxCOPY )
         return ;

-    
+

     wxGraphicPath* path = m_graphicContext->CreatePath() ;
     wxGraphicPath* path = m_graphicContext->CreatePath() ;

-    
+

     wxList::compatibility_iterator node = points->GetFirst();
     if (node == wxList::compatibility_iterator())
         // empty list
         return;
     wxList::compatibility_iterator node = points->GetFirst();
     if (node == wxList::compatibility_iterator())
         // empty list
         return;

-    
+

     wxPoint *p = (wxPoint *)node->GetData();
     wxPoint *p = (wxPoint *)node->GetData();

-    
+

     wxCoord x1 = p->x;
     wxCoord y1 = p->y;
     wxCoord x1 = p->x;
     wxCoord y1 = p->y;

-    
+

     node = node->GetNext();
     p = (wxPoint *)node->GetData();
     node = node->GetNext();
     p = (wxPoint *)node->GetData();

-    
+

     wxCoord x2 = p->x;
     wxCoord y2 = p->y;
     wxCoord cx1 = ( x1 + x2 ) / 2;
     wxCoord cy1 = ( y1 + y2 ) / 2;
     wxCoord x2 = p->x;
     wxCoord y2 = p->y;
     wxCoord cx1 = ( x1 + x2 ) / 2;
     wxCoord cy1 = ( y1 + y2 ) / 2;

-    
-    path->MoveToPoint( XLOG2DEVMAC( x1 ) , XLOG2DEVMAC( y1 ) ) ;
-    path->AddLineToPoint( XLOG2DEVMAC( cx1 ) , XLOG2DEVMAC( cy1 ) ) ;
-    
+
+    path->MoveToPoint( XLOG2DEVMAC( x1 ) , YLOG2DEVMAC( y1 ) ) ;
+    path->AddLineToPoint( XLOG2DEVMAC( cx1 ) , YLOG2DEVMAC( cy1 ) ) ;
+

 #if !wxUSE_STL
     while ((node = node->GetNext()) != NULL)
 #else
 #if !wxUSE_STL
     while ((node = node->GetNext()) != NULL)
 #else


@@ -1469,37 +1826,38 @@ void wxDC::DoDrawSpline(wxList *points)
         x2 = p->x;
         y2 = p->y;
         wxCoord cx4 = (x1 + x2) / 2;
         x2 = p->x;
         y2 = p->y;
         wxCoord cx4 = (x1 + x2) / 2;

-        wxCoord cy4 = (y1 + y2) / 2;
-        
-        path->AddQuadCurveToPoint( XLOG2DEVMAC( x1 ) , XLOG2DEVMAC( y1 ) , 
-                                   XLOG2DEVMAC( cx4 ) , XLOG2DEVMAC( cy4 ) ) ;
-        
+        wxCoord cy4 = (y1 + y2) / 2;
+
+        path->AddQuadCurveToPoint(
+            XLOG2DEVMAC( x1 ) , YLOG2DEVMAC( y1 ) ,
+            XLOG2DEVMAC( cx4 ) , YLOG2DEVMAC( cy4 ) ) ;
+

         cx1 = cx4;
         cy1 = cy4;
     }
         cx1 = cx4;
         cy1 = cy4;
     }

-            
-    path->AddLineToPoint( XLOG2DEVMAC( x2 ) , XLOG2DEVMAC( y2 ) ) ;
-    
+
+    path->AddLineToPoint( XLOG2DEVMAC( x2 ) , YLOG2DEVMAC( y2 ) ) ;
+

     m_graphicContext->StrokePath( path ) ;
     delete path ;
 }
 #endif
 
     m_graphicContext->StrokePath( path ) ;
     delete path ;
 }
 #endif
 

-void  wxDC::DoDrawPolygon(int n, wxPoint points[],
+void wxDC::DoDrawPolygon( int n, wxPoint points[],

                           wxCoord xoffset, wxCoord yoffset,
                           int fillStyle )
 {
                           wxCoord xoffset, wxCoord yoffset,
                           int fillStyle )
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
-        wxCoord x1, x2 , y1 , y2 ;
-    if ( n== 0 || (m_brush.GetStyle() == wxTRANSPARENT && m_pen.GetStyle() == wxTRANSPARENT ) )
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawPolygon - invalid DC") );
+
+    if ( n <= 0 || (m_brush.GetStyle() == wxTRANSPARENT && m_pen.GetStyle() == wxTRANSPARENT ) )

         return ;
         return ;

-        

     if ( m_logicalFunction != wxCOPY )
         return ;
 
     if ( m_logicalFunction != wxCOPY )
         return ;
 

+    wxCoord x1, x2 , y1 , y2 ;

     x2 = x1 = XLOG2DEVMAC(points[0].x + xoffset);
     y2 = y1 = YLOG2DEVMAC(points[0].y + yoffset);
     x2 = x1 = XLOG2DEVMAC(points[0].x + xoffset);
     y2 = y1 = YLOG2DEVMAC(points[0].y + yoffset);

-    
+

     wxGraphicPath* path = m_graphicContext->CreatePath() ;
     path->MoveToPoint( x1 , y1 ) ;
     for (int i = 1; i < n; i++)
     wxGraphicPath* path = m_graphicContext->CreatePath() ;
     path->MoveToPoint( x1 , y1 ) ;
     for (int i = 1; i < n; i++)


@@ -1509,18 +1867,19 @@ void  wxDC::DoDrawPolygon(int n, wxPoint points[],
 
         path->AddLineToPoint( x2 , y2 ) ;
     }
 
         path->AddLineToPoint( x2 , y2 ) ;
     }

+

     if ( x1 != x2 || y1 != y2 )
     if ( x1 != x2 || y1 != y2 )

-    {
-        path->AddLineToPoint( x1,y1 ) ;
-    }
+        path->AddLineToPoint( x1, y1 ) ;
+

     path->CloseSubpath() ;
     m_graphicContext->DrawPath( path , fillStyle ) ;
     path->CloseSubpath() ;
     m_graphicContext->DrawPath( path , fillStyle ) ;

+

     delete path ;
 }
 
 void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
 {
     delete path ;
 }
 
 void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawRectangle - invalid DC") );

 
     if ( m_logicalFunction != wxCOPY )
         return ;
 
     if ( m_logicalFunction != wxCOPY )
         return ;


@@ -1529,9 +1888,11 @@ void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
     wxCoord yy = YLOG2DEVMAC(y);
     wxCoord ww = m_signX * XLOG2DEVREL(width);
     wxCoord hh = m_signY * YLOG2DEVREL(height);
     wxCoord yy = YLOG2DEVMAC(y);
     wxCoord ww = m_signX * XLOG2DEVREL(width);
     wxCoord hh = m_signY * YLOG2DEVREL(height);

+

     // CMB: draw nothing if transformed w or h is 0
     if (ww == 0 || hh == 0)
         return;
     // CMB: draw nothing if transformed w or h is 0
     if (ww == 0 || hh == 0)
         return;

+

     // CMB: handle -ve width and/or height
     if (ww < 0)
     {
     // CMB: handle -ve width and/or height
     if (ww < 0)
     {


@@ -1543,31 +1904,46 @@ void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
         hh = -hh;
         yy = yy - hh;
     }
         hh = -hh;
         yy = yy - hh;
     }

+
+    int penwidth = m_pen.GetWidth();
+    if ( penwidth == 0 )
+        penwidth = 1 ;
+    if ( m_pen.GetStyle() == wxTRANSPARENT )
+        penwidth = 0 ;
+        
+    bool offset = ( penwidth % 2 ) == 1 ; 
+

     wxGraphicPath* path = m_graphicContext->CreatePath() ;
     wxGraphicPath* path = m_graphicContext->CreatePath() ;

-    path->AddRectangle(xx ,yy , ww , hh ) ;
+    // if we are offsetting the entire rectangle is moved 0.5, so the border line gets off by 1
+    if ( offset )
+        path->AddRectangle( xx , yy , ww-1 , hh-1 ) ;
+    else
+        path->AddRectangle( xx , yy , ww , hh ) ;
+        

     m_graphicContext->DrawPath( path ) ;
     delete path ;
 }
 
     m_graphicContext->DrawPath( path ) ;
     delete path ;
 }
 

-void  wxDC::DoDrawRoundedRectangle(wxCoord x, wxCoord y,
+void wxDC::DoDrawRoundedRectangle(wxCoord x, wxCoord y,

                                    wxCoord width, wxCoord height,
                                    double radius)
 {
                                    wxCoord width, wxCoord height,
                                    double radius)
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawRoundedRectangle - invalid DC") );

 
     if ( m_logicalFunction != wxCOPY )
         return ;
 
 
     if ( m_logicalFunction != wxCOPY )
         return ;
 

-

     if (radius < 0.0)
         radius = - radius * ((width < height) ? width : height);
     wxCoord xx = XLOG2DEVMAC(x);
     wxCoord yy = YLOG2DEVMAC(y);
     wxCoord ww = m_signX * XLOG2DEVREL(width);
     wxCoord hh = m_signY * YLOG2DEVREL(height);
     if (radius < 0.0)
         radius = - radius * ((width < height) ? width : height);
     wxCoord xx = XLOG2DEVMAC(x);
     wxCoord yy = YLOG2DEVMAC(y);
     wxCoord ww = m_signX * XLOG2DEVREL(width);
     wxCoord hh = m_signY * YLOG2DEVREL(height);

+

     // CMB: draw nothing if transformed w or h is 0
     if (ww == 0 || hh == 0)
         return;
     // CMB: draw nothing if transformed w or h is 0
     if (ww == 0 || hh == 0)
         return;

+

     // CMB: handle -ve width and/or height
     if (ww < 0)
     {
     // CMB: handle -ve width and/or height
     if (ww < 0)
     {


@@ -1579,15 +1955,29 @@ void  wxDC::DoDrawRoundedRectangle(wxCoord x, wxCoord y,
         hh = -hh;
         yy = yy - hh;
     }
         hh = -hh;
         yy = yy - hh;
     }

-    wxMacCGContext* mctx = ((wxMacCGContext*) m_graphicContext) ;
-    CGContextRef ctx = mctx->GetNativeContext() ;
-    AddRoundedRectToPath( ctx  , CGRectMake( xx , yy , ww , hh ) , 16 ,16  ) ;
-    CGContextDrawPath( ctx , mctx->GetDrawingMode() ) ;
+
+    wxGraphicPath* path = m_graphicContext->CreatePath() ;
+    if ( radius == 0)
+    {
+        path->AddRectangle( xx , yy , ww , hh ) ;
+        m_graphicContext->DrawPath( path ) ;
+    }
+    else
+    {
+        path->MoveToPoint( xx + ww, yy + hh / 2);
+        path->AddArcToPoint(xx + ww, yy + hh, xx + ww / 2,yy +  hh, radius);
+        path->AddArcToPoint(xx , yy + hh, xx , yy + hh / 2, radius);
+        path->AddArcToPoint(xx , yy , xx + ww / 2, yy , radius);
+        path->AddArcToPoint(xx + ww, yy , xx + ww, yy + hh / 2, radius);
+        path->CloseSubpath();
+        m_graphicContext->DrawPath( path ) ;
+    }
+    delete path ;

 }
 
 }
 

-void  wxDC::DoDrawEllipse(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
+void wxDC::DoDrawEllipse(wxCoord x, wxCoord y, wxCoord width, wxCoord height)

 {
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawEllipse - invalid DC") );

 
     if ( m_logicalFunction != wxCOPY )
         return ;
 
     if ( m_logicalFunction != wxCOPY )
         return ;


@@ -1596,9 +1986,11 @@ void  wxDC::DoDrawEllipse(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
     wxCoord yy = YLOG2DEVMAC(y);
     wxCoord ww = m_signX * XLOG2DEVREL(width);
     wxCoord hh = m_signY * YLOG2DEVREL(height);
     wxCoord yy = YLOG2DEVMAC(y);
     wxCoord ww = m_signX * XLOG2DEVREL(width);
     wxCoord hh = m_signY * YLOG2DEVREL(height);

+

     // CMB: draw nothing if transformed w or h is 0
     if (ww == 0 || hh == 0)
         return;
     // CMB: draw nothing if transformed w or h is 0
     if (ww == 0 || hh == 0)
         return;

+

     // CMB: handle -ve width and/or height
     if (ww < 0)
     {
     // CMB: handle -ve width and/or height
     if (ww < 0)
     {


@@ -1610,54 +2002,59 @@ void  wxDC::DoDrawEllipse(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
         hh = -hh;
         yy = yy - hh;
     }
         hh = -hh;
         yy = yy - hh;
     }

-
-    wxMacCGContext* mctx = ((wxMacCGContext*) m_graphicContext) ;
-    CGContextRef ctx = mctx->GetNativeContext() ;
-    CGContextSaveGState( ctx ) ;
-    CGContextTranslateCTM( ctx, xx + ww / 2, yy + hh / 2);
-    CGContextScaleCTM( ctx , ww / 2 , hh / 2 ) ;
-    CGContextAddArc( ctx, 0, 0, 1,  0 , 2*M_PI , 0);
-    CGContextRestoreGState( ctx ) ;
-    CGContextDrawPath( ctx , mctx->GetDrawingMode() ) ;
+    
+    wxGraphicPath* path = m_graphicContext->CreatePath() ;
+    m_graphicContext->PushState() ;
+    m_graphicContext->Translate(xx + ww / 2, yy + hh / 2);
+    m_graphicContext->Scale(ww / 2 , hh / 2);
+    path->AddArc( 0, 0, 1, 0 , 2 * M_PI , false ) ;
+    m_graphicContext->DrawPath( path ) ;
+    m_graphicContext->PopState() ;
+    delete path ;

 }
 
 }
 

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

 {
     return true ;
 }
 
 {
     return true ;
 }
 

-bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
-                   wxDC *source, wxCoord xsrc, wxCoord ysrc, int logical_func , bool useMask,
-                   wxCoord xsrcMask,  wxCoord ysrcMask )
+bool wxDC::DoBlit(
+    wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
+    wxDC *source, wxCoord xsrc, wxCoord ysrc, int logical_func , bool useMask,
+    wxCoord xsrcMask, wxCoord ysrcMask )

 {
 {

-    wxCHECK_MSG(Ok(), false, wxT("wxDC::DoBlit Illegal dc"));
-    wxCHECK_MSG(source->Ok(), false, wxT("wxDC::DoBlit  Illegal source DC"));
+    wxCHECK_MSG( Ok(), false, wxT("wxDC(cg)::DoBlit - invalid DC") );
+    wxCHECK_MSG( source->Ok(), false, wxT("wxDC(cg)::DoBlit - invalid source DC") );
+

     if ( logical_func == wxNO_OP )
     if ( logical_func == wxNO_OP )

-        return TRUE ;
+        return true ;
+

     if (xsrcMask == -1 && ysrcMask == -1)
     {
     if (xsrcMask == -1 && ysrcMask == -1)
     {

-        xsrcMask = xsrc; ysrcMask = ysrc;
+        xsrcMask = xsrc;
+        ysrcMask = ysrc;

     }
 
     wxCoord yysrc = source->YLOG2DEVMAC(ysrc) ;
     }
 
     wxCoord yysrc = source->YLOG2DEVMAC(ysrc) ;

-    wxCoord xxsrc = source->XLOG2DEVMAC(xsrc)  ;
-    wxCoord wwsrc = source->XLOG2DEVREL(width ) ;
+    wxCoord xxsrc = source->XLOG2DEVMAC(xsrc) ;
+    wxCoord wwsrc = source->XLOG2DEVREL(width) ;

     wxCoord hhsrc = source->YLOG2DEVREL(height) ;
     wxCoord hhsrc = source->YLOG2DEVREL(height) ;

-    
+

     wxCoord yydest = YLOG2DEVMAC(ydest) ;
     wxCoord xxdest = XLOG2DEVMAC(xdest) ;
     wxCoord yydest = YLOG2DEVMAC(ydest) ;
     wxCoord xxdest = XLOG2DEVMAC(xdest) ;

-    wxCoord wwdest = XLOG2DEVREL(width ) ;
+    wxCoord wwdest = XLOG2DEVREL(width) ;

     wxCoord hhdest = YLOG2DEVREL(height) ;
     wxCoord hhdest = YLOG2DEVREL(height) ;

-    
+

     wxMemoryDC* memdc = dynamic_cast<wxMemoryDC*>(source) ;
     if ( memdc && logical_func == wxCOPY )
     {
         wxBitmap blit = memdc->GetSelectedObject() ;
     wxMemoryDC* memdc = dynamic_cast<wxMemoryDC*>(source) ;
     if ( memdc && logical_func == wxCOPY )
     {
         wxBitmap blit = memdc->GetSelectedObject() ;

+

         wxASSERT_MSG( blit.Ok() , wxT("Invalid bitmap for blitting") ) ;
 
         wxCoord bmpwidth = blit.GetWidth();
         wxCoord bmpheight = blit.GetHeight();
         wxASSERT_MSG( blit.Ok() , wxT("Invalid bitmap for blitting") ) ;
 
         wxCoord bmpwidth = blit.GetWidth();
         wxCoord bmpheight = blit.GetHeight();

-        
+

         if ( xxsrc != 0 || yysrc != 0 || bmpwidth != wwsrc || bmpheight != hhsrc )
         {
             wwsrc = wxMin( wwsrc , bmpwidth - xxsrc ) ;
         if ( xxsrc != 0 || yysrc != 0 || bmpwidth != wwsrc || bmpheight != hhsrc )
         {
             wwsrc = wxMin( wwsrc , bmpwidth - xxsrc ) ;


@@ -1667,6 +2064,7 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
                 if ( xxsrc >= 0 && yysrc >= 0 )
                 {
                     wxRect subrect( xxsrc, yysrc, wwsrc , hhsrc ) ;
                 if ( xxsrc >= 0 && yysrc >= 0 )
                 {
                     wxRect subrect( xxsrc, yysrc, wwsrc , hhsrc ) ;

+                    // TODO we perhaps could add a DrawSubBitmap call to dc for performance reasons

                     blit = blit.GetSubBitmap( subrect ) ;
                 }
                 else
                     blit = blit.GetSubBitmap( subrect ) ;
                 }
                 else


@@ -1681,346 +2079,118 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
                 blit = wxNullBitmap ;
             }
         }
                 blit = wxNullBitmap ;
             }
         }

+

         if ( blit.Ok() )
         {
         if ( blit.Ok() )
         {

-            CGContextRef cg = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;
-            CGImageRef image = (CGImageRef)( blit.CGImageCreate() ) ;
-            HIRect r = CGRectMake( xxdest , yydest , wwdest , hhdest ) ;
-            HIViewDrawCGImage( cg , &r , image ) ;
-            CGImageRelease( image ) ;
+            m_graphicContext->DrawBitmap( blit, xxdest , yydest , wwdest , hhdest ) ;

         }
         }

-           

     }
     else
     {
     }
     else
     {

-    /*
-        CGContextRef cg = (wxMacCGContext*)(source->GetGraphicContext())->GetNativeContext() ;
-        void *data = CGBitmapContextGetData( cg ) ;
-    */
-        return FALSE ; // wxFAIL_MSG( wxT("Blitting is only supported from bitmap contexts") ) ;
+        wxFAIL_MSG( wxT("Blitting is only supported from bitmap contexts, and only with wxCOPY logical operation.") ) ;
+        return false ; 

     }
     }

-    return TRUE;
+
+    return true;

 }
 
 }
 

-void  wxDC::DoDrawRotatedText(const wxString& str, wxCoord x, wxCoord y,
+void wxDC::DoDrawRotatedText(const wxString& str, wxCoord x, wxCoord y,

                               double angle)
 {
                               double angle)
 {

-    wxCHECK_RET( Ok(), wxT("wxDC::DoDrawRotatedText  Invalid window dc") );
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawRotatedText - invalid DC") );
+//    wxCHECK_RET( m_macATSUIStyle != NULL, wxT("wxDC(cg)::DoDrawRotatedText - no valid font set") );

 
 

-    if ( str.Length() == 0 )
+    if ( str.length() == 0 )

         return ;
         return ;

-    

     if ( m_logicalFunction != wxCOPY )
         return ;
 
     if ( m_logicalFunction != wxCOPY )
         return ;
 

-    wxCHECK_RET( m_macATSUIStyle != NULL , wxT("No valid font set") ) ;
-    
-    OSStatus status = noErr ;
-    ATSUTextLayout atsuLayout ;
-    UniCharCount chars = str.Length() ;
-    UniChar* ubuf = NULL ;
-#if SIZEOF_WCHAR_T == 4
-       wxMBConvUTF16 converter ;
-#if wxUSE_UNICODE
-       size_t unicharlen = converter.WC2MB( NULL , str.wc_str() , 0 ) ;
-       ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
-       converter.WC2MB( (char*) ubuf , str.wc_str(), unicharlen + 2 ) ;
-#else
-    const wxWCharBuffer wchar = str.wc_str( wxConvLocal ) ;
-       size_t unicharlen = converter.WC2MB( NULL , wchar.data()  , 0 ) ;
-       ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
-       converter.WC2MB( (char*) ubuf , wchar.data() , unicharlen + 2 ) ;
-#endif
-    chars = unicharlen / 2 ;
-#else
-#if wxUSE_UNICODE
-    ubuf = (UniChar*) str.wc_str() ;
-#else
-    wxWCharBuffer wchar = str.wc_str( wxConvLocal ) ;
-    chars = wxWcslen( wchar.data() ) ;
-    ubuf = (UniChar*) wchar.data() ;
-#endif
-#endif
-

     int drawX = XLOG2DEVMAC(x) ;
     int drawY = YLOG2DEVMAC(y) ;
 
     int drawX = XLOG2DEVMAC(x) ;
     int drawY = YLOG2DEVMAC(y) ;
 

-    status = ::ATSUCreateTextLayoutWithTextPtr( (UniCharArrayPtr) ubuf , 0 , chars , chars , 1 ,
-        &chars , (ATSUStyle*) &m_macATSUIStyle , &atsuLayout ) ;
-
-    wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the rotated text") );
-    
-    status = ::ATSUSetTransientFontMatching( atsuLayout , true ) ;
-    wxASSERT_MSG( status == noErr , wxT("couldn't setup transient font matching") );
-    
-    int iAngle = int( angle );
-    if ( abs(iAngle) > 0 )
-    {
-        Fixed atsuAngle = IntToFixed( iAngle ) ;
-        ATSUAttributeTag atsuTags[] =
-        {
-            kATSULineRotationTag ,
-        } ;
-        ByteCount atsuSizes[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
-        {
-            sizeof( Fixed ) ,
-        } ;
-        ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
-        {
-            &atsuAngle ,
-        } ;
-        status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags)/sizeof(ATSUAttributeTag),
-            atsuTags, atsuSizes, atsuValues ) ;
-    }
-    {
-        CGContextRef cgContext = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;
-        ATSUAttributeTag atsuTags[] =
-        {
-            kATSUCGContextTag ,
-        } ;
-        ByteCount atsuSizes[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
-        {
-            sizeof( CGContextRef ) ,
-        } ;
-        ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
-        {
-            &cgContext ,
-        } ;
-        status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags)/sizeof(ATSUAttributeTag),
-            atsuTags, atsuSizes, atsuValues ) ;
-    }
-
-    ATSUTextMeasurement textBefore ;
-    ATSUTextMeasurement textAfter ;
-    ATSUTextMeasurement ascent ;
-    ATSUTextMeasurement descent ;
-
-    status = ::ATSUGetUnjustifiedBounds( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
-        &textBefore , &textAfter, &ascent , &descent );
-    wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );
-    
-    Rect rect ;
-    
-    if ( m_backgroundMode == wxSOLID )
-    {
-        wxGraphicPath* path = m_graphicContext->CreatePath() ;
-        path->MoveToPoint( 
-            drawX , 
-            drawY ) ;
-        path->AddLineToPoint( 
-            (int) (drawX + sin(angle/RAD2DEG) * FixedToInt(ascent + descent)) , 
-            (int) (drawY + cos(angle/RAD2DEG) * FixedToInt(ascent + descent)) ) ;
-        path->AddLineToPoint( 
-            (int) (drawX + sin(angle/RAD2DEG) * FixedToInt(ascent + descent ) + cos(angle/RAD2DEG) * FixedToInt(textAfter)) ,
-            (int) (drawY + cos(angle/RAD2DEG) * FixedToInt(ascent + descent) - sin(angle/RAD2DEG) * FixedToInt(textAfter)) ) ;
-        path->AddLineToPoint( 
-            (int) (drawX + cos(angle/RAD2DEG) * FixedToInt(textAfter)) , 
-            (int) (drawY - sin(angle/RAD2DEG) * FixedToInt(textAfter)) ) ;
-            
-        m_graphicContext->FillPath( path , m_textBackgroundColour ) ;
-        delete path ;
-    }
-
-    drawX += (int)(sin(angle/RAD2DEG) * FixedToInt(ascent));
-    drawY += (int)(cos(angle/RAD2DEG) * FixedToInt(ascent));
-
-    status = ::ATSUMeasureTextImage( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
-        IntToFixed(drawX) , IntToFixed(drawY) , &rect );
-    wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );
-
-    CGContextSaveGState(((wxMacCGContext*)(m_graphicContext))->GetNativeContext());    
-    CGContextTranslateCTM(((wxMacCGContext*)(m_graphicContext))->GetNativeContext(), drawX, drawY);
-    CGContextScaleCTM(((wxMacCGContext*)(m_graphicContext))->GetNativeContext(), 1, -1);
-    status = ::ATSUDrawText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
-        IntToFixed(0) , IntToFixed(0) );
-    wxASSERT_MSG( status == noErr , wxT("couldn't draw the rotated text") );
-    CGContextRestoreGState( ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ) ;
-
-    CalcBoundingBox(XDEV2LOG(rect.left), YDEV2LOG(rect.top) );
-    CalcBoundingBox(XDEV2LOG(rect.right), YDEV2LOG(rect.bottom) );
-
-    ::ATSUDisposeTextLayout(atsuLayout);
-#if SIZEOF_WCHAR_T == 4
-    free( ubuf ) ;
-#endif
+    m_graphicContext->DrawText( str, drawX ,drawY , angle ) ;

 }
 
 }
 

-void  wxDC::DoDrawText(const wxString& strtext, wxCoord x, wxCoord y)
+void wxDC::DoDrawText(const wxString& strtext, wxCoord x, wxCoord y)

 {
 {

-    wxCHECK_RET(Ok(), wxT("wxDC::DoDrawText  Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoDrawText - invalid DC") );
+

     DoDrawRotatedText( strtext , x , y , 0.0 ) ;
 }
 
     DoDrawRotatedText( strtext , x , y , 0.0 ) ;
 }
 

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

 {
 {

-    wxCHECK_MSG(Ok(), false, wxT("Invalid DC"));
+    wxCHECK_MSG( Ok(), false, wxT("wxDC(cg)::CanGetTextExtent - invalid DC") );
+

     return true ;
 }
 
     return true ;
 }
 

-void  wxDC::DoGetTextExtent( const wxString &str, wxCoord *width, wxCoord *height,
+void wxDC::DoGetTextExtent( const wxString &str, wxCoord *width, wxCoord *height,

                             wxCoord *descent, wxCoord *externalLeading ,
                             wxFont *theFont ) const
 {
                             wxCoord *descent, wxCoord *externalLeading ,
                             wxFont *theFont ) const
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
-    wxFont formerFont = m_font ;
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::DoGetTextExtent - invalid DC") );
+

     if ( theFont )
     {
     if ( theFont )
     {

-        // work around the constness
-        *((wxFont*)(&m_font)) = *theFont ;
-        MacInstallFont() ;   
+        m_graphicContext->SetFont( *theFont ) ;

     }
 
     }
 

-    if ( str.Length() == 0 )
-        return ;
-    
-    wxCHECK_RET( m_macATSUIStyle != NULL , wxT("No valid font set") ) ;
-    
-    OSStatus status = noErr ;
-    ATSUTextLayout atsuLayout ;
-    UniCharCount chars = str.Length() ;
-    UniChar* ubuf = NULL ;
-#if SIZEOF_WCHAR_T == 4
-       wxMBConvUTF16 converter ;
-#if wxUSE_UNICODE
-       size_t unicharlen = converter.WC2MB( NULL , str.wc_str() , 0 ) ;
-       ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
-       converter.WC2MB( (char*) ubuf , str.wc_str(), unicharlen + 2 ) ;
-#else
-    const wxWCharBuffer wchar = str.wc_str( wxConvLocal ) ;
-       size_t unicharlen = converter.WC2MB( NULL , wchar.data()  , 0 ) ;
-       ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
-       converter.WC2MB( (char*) ubuf , wchar.data() , unicharlen + 2 ) ;
-#endif
-    chars = unicharlen / 2 ;
-#else
-#if wxUSE_UNICODE
-    ubuf = (UniChar*) str.wc_str() ;
-#else
-    wxWCharBuffer wchar = str.wc_str( wxConvLocal ) ;
-    chars = wxWcslen( wchar.data() ) ;
-    ubuf = (UniChar*) wchar.data() ;
-#endif
-#endif
-
-
-    status = ::ATSUCreateTextLayoutWithTextPtr( (UniCharArrayPtr) ubuf , 0 , chars , chars , 1 ,
-        &chars , (ATSUStyle*) &m_macATSUIStyle , &atsuLayout ) ;
-
-    wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the text") );
+    wxCoord h , d , e , w ;

     
     

-    ATSUTextMeasurement textBefore ;
-    ATSUTextMeasurement textAfter ;
-    ATSUTextMeasurement textAscent ;
-    ATSUTextMeasurement textDescent ;
+    m_graphicContext->GetTextExtent( str, &w, &h, &d, &e ) ;

 
 

-    status = ::ATSUGetUnjustifiedBounds( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
-        &textBefore , &textAfter, &textAscent , &textDescent );
-        

     if ( height )
     if ( height )

-        *height = YDEV2LOGREL( FixedToInt(textAscent + textDescent) ) ;
+        *height = YDEV2LOGREL( h ) ;

     if ( descent )
     if ( descent )

-        *descent =YDEV2LOGREL( FixedToInt(textDescent) );
+        *descent =YDEV2LOGREL( d);

     if ( externalLeading )
     if ( externalLeading )

-        *externalLeading = 0 ;
+        *externalLeading = YDEV2LOGREL( e);

     if ( width )
     if ( width )

-        *width = XDEV2LOGREL( FixedToInt(textAfter - textBefore) ) ;
-    
-    ::ATSUDisposeTextLayout(atsuLayout);
-#if SIZEOF_WCHAR_T == 4
-    free( ubuf ) ;
-#endif
+        *width = XDEV2LOGREL( w ) ;
+

     if ( theFont )
     {
     if ( theFont )
     {

-        // work around the constness
-        *((wxFont*)(&m_font)) = formerFont ;
-        MacInstallFont() ;
+        m_graphicContext->SetFont( m_font ) ;

     }
 }
 
     }
 }
 

-

 bool wxDC::DoGetPartialTextExtents(const wxString& text, wxArrayInt& widths) const
 {
 bool wxDC::DoGetPartialTextExtents(const wxString& text, wxArrayInt& widths) const
 {

-    wxCHECK_MSG(Ok(), false, wxT("Invalid DC"));
-
-    widths.Empty();
-    widths.Add(0, text.Length());
+    wxCHECK_MSG( Ok(), false, wxT("wxDC(cg)::DoGetPartialTextExtents - invalid DC") );

 
 

-    if (text.Length() == 0)
-        return false;
-    
-    ATSUTextLayout atsuLayout ;
-    UniCharCount chars = text.Length() ;
-    UniChar* ubuf = NULL ;
-#if SIZEOF_WCHAR_T == 4
-    wxMBConvUTF16 converter ;
-#if wxUSE_UNICODE
-    size_t unicharlen = converter.WC2MB( NULL , text.wc_str() , 0 ) ;
-    ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
-    converter.WC2MB( (char*) ubuf , text.wc_str(), unicharlen + 2 ) ;
-#else
-    const wxWCharBuffer wchar = text.wc_str( wxConvLocal ) ;
-    size_t unicharlen = converter.WC2MB( NULL , wchar.data()  , 0 ) ;
-    ubuf = (UniChar*) malloc( unicharlen + 2 ) ;
-    converter.WC2MB( (char*) ubuf , wchar.data() , unicharlen + 2 ) ;
-#endif
-    chars = unicharlen / 2 ;
-#else
-#if wxUSE_UNICODE
-    ubuf = (UniChar*) text.wc_str() ;
-#else
-    wxWCharBuffer wchar = text.wc_str( wxConvLocal ) ;
-    chars = wxWcslen( wchar.data() ) ;
-    ubuf = (UniChar*) wchar.data() ;
-#endif
-#endif
+    m_graphicContext->GetPartialTextExtents( text, widths ) ;
+    for ( size_t i = 0 ; i < widths.GetCount() ; ++i )
+        widths[i] = XDEV2LOGREL( widths[i] );

 
 

-       OSStatus status;
-    status = ::ATSUCreateTextLayoutWithTextPtr( (UniCharArrayPtr) ubuf , 0 , chars , chars , 1 ,
-        &chars , (ATSUStyle*) &m_macATSUIStyle , &atsuLayout ) ;
-        
-       for ( int pos = 0; pos < (int)chars; pos ++ ) {
-                       unsigned long actualNumberOfBounds = 0;
-                       ATSTrapezoid glyphBounds;
-
-                       // We get a single bound, since the text should only require one. If it requires more, there is an issue
-                       OSStatus result; 
-                       result = ATSUGetGlyphBounds( atsuLayout, 0, 0, kATSUFromTextBeginning, pos + 1, kATSUseDeviceOrigins, 1, &glyphBounds, &actualNumberOfBounds );
-                       if (result != noErr || actualNumberOfBounds != 1 )
-                       {
-                               return false;
-                       }
-
-                       widths[pos] = XDEV2LOGREL(FixedToInt( glyphBounds.upperRight.x - glyphBounds.upperLeft.x ));
-                       //unsigned char uch = s[i];
-                       
-       }
-    ::ATSUDisposeTextLayout(atsuLayout);

     return true;
 }
 
     return true;
 }
 

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

 {
     wxCoord width ;
 {
     wxCoord width ;

-    DoGetTextExtent(wxT("g") , &width , NULL , NULL , NULL , NULL ) ;
+    DoGetTextExtent( wxT("g") , &width , NULL , NULL , NULL , NULL ) ;
+

     return width ;
 }
 
     return width ;
 }
 

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

 {
     wxCoord height ;
 {
     wxCoord height ;

-    DoGetTextExtent(wxT("g") , NULL , &height , NULL , NULL , NULL ) ;
+    DoGetTextExtent( wxT("g") , NULL , &height , NULL , NULL , NULL ) ;
+

     return height ;
 }
 
     return height ;
 }
 

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

 {
 {

-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET( Ok(), wxT("wxDC(cg)::Clear - invalid DC") );

 
 

-    if ( m_backgroundBrush.Ok() && m_backgroundBrush.GetStyle() != wxTRANSPARENT)
-    {      
+    if (m_backgroundBrush.Ok() && m_backgroundBrush.GetStyle() != wxTRANSPARENT)
+    {

         HIRect rect = CGRectMake( -10000 , -10000 , 20000 , 20000 ) ;
         CGContextRef cg = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;
         HIRect rect = CGRectMake( -10000 , -10000 , 20000 , 20000 ) ;
         CGContextRef cg = ((wxMacCGContext*)(m_graphicContext))->GetNativeContext() ;

-        switch( m_backgroundBrush.MacGetBrushKind() )
+        switch ( m_backgroundBrush.MacGetBrushKind() )

         {
             case kwxMacBrushTheme :
                 {
         {
             case kwxMacBrushTheme :
                 {


@@ -2034,99 +2204,64 @@ void  wxDC::Clear(void)
                     else
 #endif
                     {
                     else
 #endif
                     {

-                                       RGBColor        color;
+                        RGBColor color;

                         GetThemeBrushAsColor( m_backgroundBrush.MacGetTheme(), 32, true, &color );
                         GetThemeBrushAsColor( m_backgroundBrush.MacGetTheme(), 32, true, &color );

-                        CGContextSetRGBFillColor( cg, (float) color.red / 65536,
-                                                       (float) color.green / 65536, (float) color.blue / 65536, 1 );
-                                       CGContextFillRect( cg, rect );
+                        CGContextSetRGBFillColor( cg, (CGFloat) color.red / 65536,
+                            (CGFloat) color.green / 65536, (CGFloat) color.blue / 65536, 1 );
+                            CGContextFillRect( cg, rect );

                     }
 
                     // reset to normal value
                     RGBColor col = MAC_WXCOLORREF( GetBrush().GetColour().GetPixel() ) ;
                     CGContextSetRGBFillColor( cg, col.red / 65536.0, col.green / 65536.0, col.blue / 65536.0, 1.0 );
                 }
                     }
 
                     // reset to normal value
                     RGBColor col = MAC_WXCOLORREF( GetBrush().GetColour().GetPixel() ) ;
                     CGContextSetRGBFillColor( cg, col.red / 65536.0, col.green / 65536.0, col.blue / 65536.0, 1.0 );
                 }

-            break ;
+                break ;
+

             case kwxMacBrushThemeBackground :
                 {
                     wxFAIL_MSG( wxT("There shouldn't be theme backgrounds under Quartz") ) ;
             case kwxMacBrushThemeBackground :
                 {
                     wxFAIL_MSG( wxT("There shouldn't be theme backgrounds under Quartz") ) ;

+

 #if MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_3
                     if ( UMAGetSystemVersion() >= 0x1030 )
                     {
                         HIThemeBackgroundDrawInfo drawInfo ;
                         drawInfo.version = 0 ;
                         drawInfo.state = kThemeStateActive ;
 #if MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_3
                     if ( UMAGetSystemVersion() >= 0x1030 )
                     {
                         HIThemeBackgroundDrawInfo drawInfo ;
                         drawInfo.version = 0 ;
                         drawInfo.state = kThemeStateActive ;

-                        drawInfo.kind = m_backgroundBrush.MacGetThemeBackground(NULL) ;
+                        drawInfo.kind = m_backgroundBrush.MacGetThemeBackground( NULL ) ;

                         if ( drawInfo.kind == kThemeBackgroundMetal )
                         if ( drawInfo.kind == kThemeBackgroundMetal )

-                            HIThemeDrawBackground( &rect , &drawInfo, cg ,
-                                kHIThemeOrientationNormal) ;
-                            HIThemeApplyBackground( &rect , &drawInfo, cg ,
-                                kHIThemeOrientationNormal) ;
+                        {
+                            HIThemeDrawBackground( &rect, &drawInfo, cg, kHIThemeOrientationNormal ) ;
+                            HIThemeApplyBackground( &rect, &drawInfo, cg, kHIThemeOrientationNormal ) ;
+                        }

                     }
                     }

-                    else

 #endif
 #endif

-                    {
-                    }

                 }
                 }

-            break ;
-            case kwxMacBrushColour :
-            {
-                RGBColor col = MAC_WXCOLORREF( m_backgroundBrush.GetColour().GetPixel()) ;
-                CGContextSetRGBFillColor( cg , col.red / 65536.0 , col.green / 65536.0 , col.blue / 65536.0 , 1.0 ) ;
-                CGContextFillRect(cg, rect);
-
-                // reset to normal value
-                col = MAC_WXCOLORREF( GetBrush().GetColour().GetPixel() ) ;
-                CGContextSetRGBFillColor( cg , col.red / 65536.0 , col.green / 65536.0 , col.blue / 65536.0 , 1.0 ) ;
-            }
-            break ;
-        }
-    }
-}
-
-void wxDC::MacInstallFont() const
-{
-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
-
-    if( m_macATSUIStyle )
-    {
-        ::ATSUDisposeStyle((ATSUStyle)m_macATSUIStyle);
-        m_macATSUIStyle = NULL ;
-    }
+                break ;

 
 

-    if ( m_font.Ok() )
-    {
-        OSStatus status = noErr ;
-        status = ATSUCreateAndCopyStyle( (ATSUStyle) m_font.MacGetATSUStyle() , (ATSUStyle*) &m_macATSUIStyle ) ;
-        wxASSERT_MSG( status == noErr , wxT("couldn't set create ATSU style") ) ;
+            case kwxMacBrushColour :
+                {
+                    // FIXME: doesn't correctly render stippled brushes !!
+                    // FIXME: should this be replaced by ::SetBrush() ??

 
 

-        Fixed atsuSize = IntToFixed( int(m_scaleY * m_font.MacGetFontSize()) ) ;
-        RGBColor atsuColor = MAC_WXCOLORREF( m_textForegroundColour.GetPixel() ) ;
-        ATSUAttributeTag atsuTags[] =
-        {
-                kATSUSizeTag ,
-                kATSUColorTag ,
-        } ;
-        ByteCount atsuSizes[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
-        {
-                sizeof( Fixed ) ,
-                sizeof( RGBColor ) ,
-        } ;
-    //    Boolean kTrue = true ;
-    //    Boolean kFalse = false ;
+                    RGBColor col = MAC_WXCOLORREF( m_backgroundBrush.GetColour().GetPixel()) ;
+                    CGContextSetRGBFillColor( cg , col.red / 65536.0 , col.green / 65536.0 , col.blue / 65536.0 , 1.0 ) ;
+                    CGContextFillRect(cg, rect);

 
 

-    //    ATSUVerticalCharacterType kHorizontal = kATSUStronglyHorizontal;
-        ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
-        {
-                &atsuSize ,
-                &atsuColor ,
-        } ;
-        status = ::ATSUSetAttributes((ATSUStyle)m_macATSUIStyle, sizeof(atsuTags)/sizeof(ATSUAttributeTag) ,
-            atsuTags, atsuSizes, atsuValues);
+                    // reset to normal value
+                    col = MAC_WXCOLORREF( GetBrush().GetColour().GetPixel() ) ;
+                    CGContextSetRGBFillColor( cg , col.red / 65536.0 , col.green / 65536.0 , col.blue / 65536.0 , 1.0 ) ;
+                }
+                break ;

 
 

-        wxASSERT_MSG( status == noErr , wxT("couldn't Modify ATSU style") ) ;
+            default :
+                wxFAIL_MSG( wxT("unknown brush kind") ) ;
+                break ;
+        }

     }
 }
 
     }
 }
 

+#pragma mark -
+

 // ---------------------------------------------------------------------------
 // coordinates transformations
 // ---------------------------------------------------------------------------
 // ---------------------------------------------------------------------------
 // coordinates transformations
 // ---------------------------------------------------------------------------


@@ -2172,4 +2307,3 @@ wxCoord wxDCBase::LogicalToDeviceYRel(wxCoord y) const
 }
 
 #endif // wxMAC_USE_CORE_GRAPHICS
 }
 
 #endif // wxMAC_USE_CORE_GRAPHICS

-