cleanup

[wxWidgets.git] / src / mac / carbon / dc.cpp
diff --git a/src/mac/carbon/dc.cpp b/src/mac/carbon/dc.cpp

index da8c9f138bdf35f7ec7ca3a87d620ad76218f94a..4b75c0fdbfae9865a62c3aeb96d544a45ffc84a1 100644 (file)
--- a/src/mac/carbon/dc.cpp
+++ b/src/mac/carbon/dc.cpp
@@ -1,44 +1,52 @@
  /////////////////////////////////////////////////////////////////////////////
  /////////////////////////////////////////////////////////////////////////////
-// Name:        dc.cpp
+// Name:        src/mac/carbon/dc.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
  /////////////////////////////////////////////////////////////////////////////
  
  /////////////////////////////////////////////////////////////////////////////
  
-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-#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"
+
+#ifndef WX_PRECOMP
+    #include "wx/log.h"
+    #include "wx/app.h"
+    #include "wx/dcmemory.h"
+    #include "wx/dcprint.h"
+    #include "wx/region.h"
+    #include "wx/image.h"
+#endif
+
  #include "wx/mac/uma.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"
-
-#if __MSL__ >= 0x6000
-namespace std {}
-using namespace std ;
+
+#ifdef __MSL__
+    #if __MSL__ >= 0x6000
+        namespace std {}
+        using namespace std ;
+    #endif
  #endif
  
  #include "wx/mac/private.h"
  #endif
  
  #include "wx/mac/private.h"
+#ifndef __DARWIN__
  #include <ATSUnicode.h>
  #include <TextCommon.h>
  #include <TextEncodingConverter.h>
  #include <ATSUnicode.h>
  #include <TextCommon.h>
  #include <TextEncodingConverter.h>
-#if !USE_SHARED_LIBRARY
-IMPLEMENT_ABSTRACT_CLASS(wxDC, wxObject)
  #endif
  
  #endif
  
+
+// set to 0 if problems arise
+#define wxMAC_EXPERIMENTAL_DC 1
+
+
+IMPLEMENT_ABSTRACT_CLASS(wxDC, wxObject)
+
  //-----------------------------------------------------------------------------
  // constants
  //-----------------------------------------------------------------------------
  //-----------------------------------------------------------------------------
  // constants
  //-----------------------------------------------------------------------------
@@ -49,8 +57,6 @@ const short kUnsupportedMode = -2 ;
  
  extern TECObjectRef s_TECNativeCToUnicode ;
  
  
  extern TECObjectRef s_TECNativeCToUnicode ;
  
-// set to 0 if problems arise
-#define wxMAC_EXPERIMENTAL_DC 1
  
  wxMacPortSetter::wxMacPortSetter( const wxDC* dc ) :
      m_ph( (GrafPtr) dc->m_macPort )
  
  wxMacPortSetter::wxMacPortSetter( const wxDC* dc ) :
      m_ph( (GrafPtr) dc->m_macPort )
@@ -59,6 +65,7 @@ wxMacPortSetter::wxMacPortSetter( const wxDC* dc ) :
      m_dc = dc ;
      dc->MacSetupPort(&m_ph) ;
  }
      m_dc = dc ;
      dc->MacSetupPort(&m_ph) ;
  }
+
  wxMacPortSetter::~wxMacPortSetter()
  {
      m_dc->MacCleanupPort(&m_ph) ;
  wxMacPortSetter::~wxMacPortSetter()
  {
      m_dc->MacCleanupPort(&m_ph) ;
@@ -77,6 +84,7 @@ public :
          m_dc = dc ;
          dc->MacSetupPort( NULL ) ;
      }
          m_dc = dc ;
          dc->MacSetupPort( NULL ) ;
      }
+
      ~wxMacFastPortSetter()
      {
          // SetPort( (GrafPtr) m_dc->m_macPort ) ;
      ~wxMacFastPortSetter()
      {
          // SetPort( (GrafPtr) m_dc->m_macPort ) ;
@@ -86,6 +94,7 @@ public :
          m_dc->MacCleanupPort( NULL ) ;
          DisposeRgn( m_clipRgn ) ;
      }
          m_dc->MacCleanupPort( NULL ) ;
          DisposeRgn( m_clipRgn ) ;
      }
+
  private :
      bool m_swapped ;
      RgnHandle m_clipRgn ;
  private :
      bool m_swapped ;
      RgnHandle m_clipRgn ;
@@ -107,12 +116,17 @@ wxMacWindowClipper::wxMacWindowClipper( const wxWindow* win ) :
  
      if ( win )
      {
  
      if ( win )
      {
-        int x = 0 , y = 0;
-        win->MacWindowToRootWindow( &x,&y ) ;
-        // get area including focus rect
-        CopyRgn( (RgnHandle) ((wxWindow*)win)->MacGetVisibleRegion(true).GetWXHRGN() , m_newClip ) ;
-        if ( !EmptyRgn( m_newClip ) )
-            OffsetRgn( m_newClip , x , y ) ;
+        // guard against half constructed objects, this just leads to a empty clip
+        if ( win->GetPeer() )
+        {
+            int x = 0 , y = 0;
+            win->MacWindowToRootWindow( &x, &y ) ;
+
+            // get area including focus rect
+            CopyRgn( (RgnHandle) ((wxWindow*)win)->MacGetVisibleRegion(true).GetWXHRGN() , m_newClip ) ;
+            if ( !EmptyRgn( m_newClip ) )
+                OffsetRgn( m_newClip , x , y ) ;
+        }
  
          SetClip( m_newClip ) ;
      }
  
          SetClip( m_newClip ) ;
      }
@@ -130,7 +144,7 @@ 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 ) ;
  }
  
@@ -163,78 +177,94 @@ void wxMacCalculateColour( int logical_func , const RGBColor &srcColor , RGBColo
              dstColor.green = dstColor.green & srcColor.green ;
              dstColor.blue = dstColor.blue & srcColor.blue ;
              break ;
              dstColor.green = dstColor.green & srcColor.green ;
              dstColor.blue = dstColor.blue & srcColor.blue ;
              break ;
+
          case wxAND_INVERT: // (NOT src) AND dst
              dstColor.red = dstColor.red & ~srcColor.red ;
              dstColor.green = dstColor.green & ~srcColor.green ;
              dstColor.blue = dstColor.blue & ~srcColor.blue ;
              break ;
          case wxAND_INVERT: // (NOT src) AND dst
              dstColor.red = dstColor.red & ~srcColor.red ;
              dstColor.green = dstColor.green & ~srcColor.green ;
              dstColor.blue = dstColor.blue & ~srcColor.blue ;
              break ;
+
          case wxAND_REVERSE:// src AND (NOT dst)
              dstColor.red = ~dstColor.red & srcColor.red ;
              dstColor.green = ~dstColor.green & srcColor.green ;
              dstColor.blue = ~dstColor.blue & srcColor.blue ;
              break ;
          case wxAND_REVERSE:// src AND (NOT dst)
              dstColor.red = ~dstColor.red & srcColor.red ;
              dstColor.green = ~dstColor.green & srcColor.green ;
              dstColor.blue = ~dstColor.blue & srcColor.blue ;
              break ;
+
          case wxCLEAR:      // 0
              dstColor.red = 0 ;
              dstColor.green = 0 ;
              dstColor.blue = 0 ;
              break ;
          case wxCLEAR:      // 0
              dstColor.red = 0 ;
              dstColor.green = 0 ;
              dstColor.blue = 0 ;
              break ;
+
          case wxCOPY:       // src
              dstColor.red = srcColor.red ;
              dstColor.green = srcColor.green ;
              dstColor.blue = srcColor.blue ;
              break ;
          case wxCOPY:       // src
              dstColor.red = srcColor.red ;
              dstColor.green = srcColor.green ;
              dstColor.blue = srcColor.blue ;
              break ;
+
          case wxEQUIV:      // (NOT src) XOR dst
              dstColor.red = dstColor.red ^ ~srcColor.red ;
              dstColor.green = dstColor.green ^ ~srcColor.green ;
              dstColor.blue = dstColor.blue ^ ~srcColor.blue ;
              break ;
          case wxEQUIV:      // (NOT src) XOR dst
              dstColor.red = dstColor.red ^ ~srcColor.red ;
              dstColor.green = dstColor.green ^ ~srcColor.green ;
              dstColor.blue = dstColor.blue ^ ~srcColor.blue ;
              break ;
+
          case wxINVERT:     // NOT dst
              dstColor.red = ~dstColor.red ;
              dstColor.green = ~dstColor.green ;
              dstColor.blue = ~dstColor.blue ;
              break ;
          case wxINVERT:     // NOT dst
              dstColor.red = ~dstColor.red ;
              dstColor.green = ~dstColor.green ;
              dstColor.blue = ~dstColor.blue ;
              break ;
+
          case wxNAND:       // (NOT src) OR (NOT dst)
              dstColor.red = ~dstColor.red | ~srcColor.red ;
              dstColor.green = ~dstColor.green | ~srcColor.green ;
              dstColor.blue = ~dstColor.blue | ~srcColor.blue ;
              break ;
          case wxNAND:       // (NOT src) OR (NOT dst)
              dstColor.red = ~dstColor.red | ~srcColor.red ;
              dstColor.green = ~dstColor.green | ~srcColor.green ;
              dstColor.blue = ~dstColor.blue | ~srcColor.blue ;
              break ;
+
          case wxNOR:        // (NOT src) AND (NOT dst)
              dstColor.red = ~dstColor.red & ~srcColor.red ;
              dstColor.green = ~dstColor.green & ~srcColor.green ;
              dstColor.blue = ~dstColor.blue & ~srcColor.blue ;
              break ;
          case wxNOR:        // (NOT src) AND (NOT dst)
              dstColor.red = ~dstColor.red & ~srcColor.red ;
              dstColor.green = ~dstColor.green & ~srcColor.green ;
              dstColor.blue = ~dstColor.blue & ~srcColor.blue ;
              break ;
-        case wxNO_OP:      // dst
-            break ;
+
          case wxOR:         // src OR dst
              dstColor.red = dstColor.red | srcColor.red ;
              dstColor.green = dstColor.green | srcColor.green ;
              dstColor.blue = dstColor.blue | srcColor.blue ;
              break ;
          case wxOR:         // src OR dst
              dstColor.red = dstColor.red | srcColor.red ;
              dstColor.green = dstColor.green | srcColor.green ;
              dstColor.blue = dstColor.blue | srcColor.blue ;
              break ;
+
          case wxOR_INVERT:  // (NOT src) OR dst
              dstColor.red = dstColor.red | ~srcColor.red ;
              dstColor.green = dstColor.green | ~srcColor.green ;
              dstColor.blue = dstColor.blue | ~srcColor.blue ;
              break ;
          case wxOR_INVERT:  // (NOT src) OR dst
              dstColor.red = dstColor.red | ~srcColor.red ;
              dstColor.green = dstColor.green | ~srcColor.green ;
              dstColor.blue = dstColor.blue | ~srcColor.blue ;
              break ;
+
          case wxOR_REVERSE: // src OR (NOT dst)
              dstColor.red = ~dstColor.red | srcColor.red ;
              dstColor.green = ~dstColor.green | srcColor.green ;
              dstColor.blue = ~dstColor.blue | srcColor.blue ;
              break ;
          case wxOR_REVERSE: // src OR (NOT dst)
              dstColor.red = ~dstColor.red | srcColor.red ;
              dstColor.green = ~dstColor.green | srcColor.green ;
              dstColor.blue = ~dstColor.blue | srcColor.blue ;
              break ;
+
          case wxSET:        // 1
              dstColor.red = 0xFFFF ;
              dstColor.green = 0xFFFF ;
              dstColor.blue = 0xFFFF ;
              break ;
          case wxSET:        // 1
              dstColor.red = 0xFFFF ;
              dstColor.green = 0xFFFF ;
              dstColor.blue = 0xFFFF ;
              break ;
+
          case wxSRC_INVERT: // (NOT src)
              dstColor.red = ~srcColor.red ;
              dstColor.green = ~srcColor.green ;
              dstColor.blue = ~srcColor.blue ;
              break ;
          case wxSRC_INVERT: // (NOT src)
              dstColor.red = ~srcColor.red ;
              dstColor.green = ~srcColor.green ;
              dstColor.blue = ~srcColor.blue ;
              break ;
+
          case wxXOR:        // src XOR dst
              dstColor.red = dstColor.red ^ srcColor.red ;
              dstColor.green = dstColor.green ^ srcColor.green ;
              dstColor.blue = dstColor.blue ^ srcColor.blue ;
              break ;
          case wxXOR:        // src XOR dst
              dstColor.red = dstColor.red ^ srcColor.red ;
              dstColor.green = dstColor.green ^ srcColor.green ;
              dstColor.blue = dstColor.blue ^ srcColor.blue ;
              break ;
+
+        case wxNO_OP:      // dst
+        default:
+            break ;
      }
  }
  
      }
  }
  
@@ -258,7 +288,6 @@ wxDC::wxDC()
      m_needComputeScaleY = false;
      m_macPort = NULL ;
      m_macMask = NULL ;
      m_needComputeScaleY = false;
      m_macPort = NULL ;
      m_macMask = NULL ;
-    m_ok = false ;
      m_macFontInstalled = false ;
      m_macBrushInstalled = false ;
      m_macPenInstalled = false ;
      m_macFontInstalled = false ;
      m_macBrushInstalled = false ;
      m_macPenInstalled = false ;
@@ -270,11 +299,13 @@ wxDC::wxDC()
      m_pen = *wxBLACK_PEN;
      m_font = *wxNORMAL_FONT;
      m_brush = *wxWHITE_BRUSH;
      m_pen = *wxBLACK_PEN;
      m_font = *wxNORMAL_FONT;
      m_brush = *wxWHITE_BRUSH;
+
  #ifdef __WXDEBUG__
      // needed to debug possible errors with two active drawing methods at the same time on
      // the same DC
      m_macCurrentPortStateHelper = NULL ;
  #endif
  #ifdef __WXDEBUG__
      // needed to debug possible errors with two active drawing methods at the same time on
      // the same DC
      m_macCurrentPortStateHelper = NULL ;
  #endif
+
      m_macATSUIStyle = NULL ;
      m_macAliasWasEnabled = false;
      m_macForegroundPixMap = NULL ;
      m_macATSUIStyle = NULL ;
      m_macAliasWasEnabled = false;
      m_macForegroundPixMap = NULL ;
@@ -293,29 +324,35 @@ void wxDC::MacSetupPort(wxMacPortStateHelper* help) const
      wxASSERT( m_macCurrentPortStateHelper == NULL ) ;
      m_macCurrentPortStateHelper = help ;
  #endif
      wxASSERT( m_macCurrentPortStateHelper == NULL ) ;
      m_macCurrentPortStateHelper = help ;
  #endif
+
      SetClip( (RgnHandle) m_macCurrentClipRgn);
      SetClip( (RgnHandle) m_macCurrentClipRgn);
+
  #if ! wxMAC_EXPERIMENTAL_DC
      m_macFontInstalled = false ;
      m_macBrushInstalled = false ;
      m_macPenInstalled = false ;
  #endif
  }
  #if ! wxMAC_EXPERIMENTAL_DC
      m_macFontInstalled = false ;
      m_macBrushInstalled = false ;
      m_macPenInstalled = false ;
  #endif
  }
+
  void wxDC::MacCleanupPort(wxMacPortStateHelper* help) const
  {
  #ifdef __WXDEBUG__
      wxASSERT( m_macCurrentPortStateHelper == help ) ;
      m_macCurrentPortStateHelper = NULL ;
  #endif
  void wxDC::MacCleanupPort(wxMacPortStateHelper* help) const
  {
  #ifdef __WXDEBUG__
      wxASSERT( m_macCurrentPortStateHelper == help ) ;
      m_macCurrentPortStateHelper = NULL ;
  #endif
-    if( m_macATSUIStyle )
+
+    if ( m_macATSUIStyle )
      {
          ::ATSUDisposeStyle((ATSUStyle)m_macATSUIStyle);
          m_macATSUIStyle = NULL ;
      }
      {
          ::ATSUDisposeStyle((ATSUStyle)m_macATSUIStyle);
          m_macATSUIStyle = NULL ;
      }
+
      if ( m_macAliasWasEnabled )
      {
          SetAntiAliasedTextEnabled(m_macFormerAliasState, m_macFormerAliasSize);
          m_macAliasWasEnabled = false ;
      }
      if ( m_macAliasWasEnabled )
      {
          SetAntiAliasedTextEnabled(m_macFormerAliasState, m_macFormerAliasSize);
          m_macAliasWasEnabled = false ;
      }
+
      if ( m_macForegroundPixMap )
      {
          Pattern blackColor ;
      if ( m_macForegroundPixMap )
      {
          Pattern blackColor ;
@@ -323,6 +360,7 @@ void wxDC::MacCleanupPort(wxMacPortStateHelper* help) const
          DisposePixPat( (PixPatHandle) m_macForegroundPixMap ) ;
          m_macForegroundPixMap = NULL ;
      }
          DisposePixPat( (PixPatHandle) m_macForegroundPixMap ) ;
          m_macForegroundPixMap = NULL ;
      }
+
      if ( m_macBackgroundPixMap )
      {
          Pattern whiteColor ;
      if ( m_macBackgroundPixMap )
      {
          Pattern whiteColor ;
@@ -334,8 +372,9 @@ void wxDC::MacCleanupPort(wxMacPortStateHelper* help) const
  
  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::DoDrawBitmap - invalid DC") );
+     wxCHECK_RET( bmp.Ok(), wxT("wxDC::DoDrawBitmap - invalid bitmap") );
+
       wxMacFastPortSetter helper(this) ;
       wxCoord xx = XLOG2DEVMAC(x);
       wxCoord yy = YLOG2DEVMAC(y);
       wxMacFastPortSetter helper(this) ;
       wxCoord xx = XLOG2DEVMAC(x);
       wxCoord yy = YLOG2DEVMAC(y);
@@ -343,6 +382,7 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
       wxCoord h = bmp.GetHeight();
       wxCoord ww = XLOG2DEVREL(w);
       wxCoord hh = YLOG2DEVREL(h);
       wxCoord h = bmp.GetHeight();
       wxCoord ww = XLOG2DEVREL(w);
       wxCoord hh = YLOG2DEVREL(h);
+
       // Set up drawing mode
       short  mode = (m_logicalFunction == wxCOPY ? srcCopy :
                      //m_logicalFunction == wxCLEAR ? WHITENESS :
       // Set up drawing mode
       short  mode = (m_logicalFunction == wxCOPY ? srcCopy :
                      //m_logicalFunction == wxCLEAR ? WHITENESS :
@@ -358,12 +398,13 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
                      //m_logicalFunction == wxSRC_AND ? SRCAND :
                      srcCopy );
  
                      //m_logicalFunction == wxSRC_AND ? SRCAND :
                      srcCopy );
  
-     GWorldPtr    maskworld = NULL ;
-     GWorldPtr    bmapworld = MAC_WXHBITMAP( bmp.GetHBITMAP((WXHBITMAP*)&maskworld) );
+     GWorldPtr maskworld = NULL ;
+     GWorldPtr bmapworld = MAC_WXHBITMAP( bmp.GetHBITMAP((WXHBITMAP*)&maskworld) );
       PixMapHandle bmappixels ;
       PixMapHandle bmappixels ;
+
       // Set foreground and background colours (for bitmaps depth = 1)
       // Set foreground and background colours (for bitmaps depth = 1)
-     if(bmp.GetDepth() == 1)
-    {
+     if (bmp.GetDepth() == 1)
+     {
           RGBColor fore = MAC_WXCOLORREF(m_textForegroundColour.GetPixel());
           RGBColor back = MAC_WXCOLORREF(m_textBackgroundColour.GetPixel());
           RGBForeColor(&fore);
           RGBColor fore = MAC_WXCOLORREF(m_textForegroundColour.GetPixel());
           RGBColor back = MAC_WXCOLORREF(m_textBackgroundColour.GetPixel());
           RGBForeColor(&fore);
@@ -371,19 +412,21 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
       }
       else
       {
       }
       else
       {
-         RGBColor white = { 0xFFFF, 0xFFFF,0xFFFF} ;
-         RGBColor black = { 0,0,0} ;
+         RGBColor white = { 0xFFFF, 0xFFFF, 0xFFFF } ;
+         RGBColor black = { 0, 0, 0 } ;
           RGBForeColor( &black ) ;
           RGBBackColor( &white ) ;
       }
       bmappixels = GetGWorldPixMap( bmapworld ) ;
           RGBForeColor( &black ) ;
           RGBBackColor( &white ) ;
       }
       bmappixels = GetGWorldPixMap( bmapworld ) ;
+
       wxCHECK_RET(LockPixels(bmappixels),
       wxCHECK_RET(LockPixels(bmappixels),
-                 wxT("DoDrawBitmap:  Unable to lock pixels"));
+                 wxT("wxDC::DoDrawBitmap - failed to lock pixels"));
+
       Rect source = { 0, 0, h, w };
       Rect dest   = { yy, xx, yy + hh, xx + ww };
       if ( useMask && maskworld )
       {
       Rect source = { 0, 0, h, w };
       Rect dest   = { yy, xx, yy + hh, xx + ww };
       if ( useMask && maskworld )
       {
-         if( LockPixels(GetGWorldPixMap(MAC_WXHBITMAP(maskworld))))
+         if ( LockPixels(GetGWorldPixMap(MAC_WXHBITMAP(maskworld))))
           {
               CopyDeepMask
                   (
           {
               CopyDeepMask
                   (
@@ -395,7 +438,8 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
               UnlockPixels(GetGWorldPixMap(MAC_WXHBITMAP(maskworld)));
           }
       }
               UnlockPixels(GetGWorldPixMap(MAC_WXHBITMAP(maskworld)));
           }
       }
-     else {
+     else
+     {
           CopyBits( GetPortBitMapForCopyBits( bmapworld ),
                     GetPortBitMapForCopyBits( MAC_WXHBITMAP(m_macPort) ),
                     &source, &dest, mode, NULL ) ;
           CopyBits( GetPortBitMapForCopyBits( bmapworld ),
                     GetPortBitMapForCopyBits( MAC_WXHBITMAP(m_macPort) ),
                     &source, &dest, mode, NULL ) ;
@@ -409,33 +453,40 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
  
  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::DoDrawIcon - invalid DC"));
+    wxCHECK_RET(icon.Ok(), wxT("wxDC::DoDrawIcon - invalid icon"));
+
      wxMacFastPortSetter helper(this) ;
  
      wxCoord xx = XLOG2DEVMAC(x);
      wxCoord yy = YLOG2DEVMAC(y);
      wxMacFastPortSetter helper(this) ;
  
      wxCoord xx = XLOG2DEVMAC(x);
      wxCoord yy = YLOG2DEVMAC(y);
+    wxCoord w = icon.GetWidth();
+    wxCoord h = icon.GetHeight();
+    wxCoord ww = XLOG2DEVREL(w);
+    wxCoord hh = YLOG2DEVREL(h);
  
  
-    Rect r = { yy , xx, yy + 32  , xx + 32 } ;
+    Rect r = { yy , xx, yy + hh, xx + ww } ;
      PlotIconRef( &r , kAlignNone , kTransformNone , kPlotIconRefNormalFlags , MAC_WXHICON( icon.GetHICON() ) ) ;
  }
  
  void wxDC::DoSetClippingRegion( wxCoord x, wxCoord y, wxCoord width, wxCoord height )
  {
      PlotIconRef( &r , kAlignNone , kTransformNone , kPlotIconRefNormalFlags , MAC_WXHICON( icon.GetHICON() ) ) ;
  }
  
  void wxDC::DoSetClippingRegion( wxCoord x, wxCoord y, wxCoord width, wxCoord height )
  {
-    wxCHECK_RET(Ok(), wxT("wxDC::DoSetClippingRegion  Invalid DC"));
+    wxCHECK_RET(Ok(), wxT("wxDC::DoSetClippingRegion - invalid DC"));
+
      wxCoord xx, yy, ww, hh;
      xx = XLOG2DEVMAC(x);
      yy = YLOG2DEVMAC(y);
      ww = XLOG2DEVREL(width);
      hh = YLOG2DEVREL(height);
      wxCoord xx, yy, ww, hh;
      xx = XLOG2DEVMAC(x);
      yy = YLOG2DEVMAC(y);
      ww = XLOG2DEVREL(width);
      hh = YLOG2DEVREL(height);
+
      SetRectRgn( (RgnHandle) m_macCurrentClipRgn , xx , yy , xx + ww , yy + hh ) ;
      SectRgn( (RgnHandle) m_macCurrentClipRgn , (RgnHandle) m_macBoundaryClipRgn , (RgnHandle) m_macCurrentClipRgn ) ;
      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_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_clipX2 = wxMin( m_clipX2, (xx + ww) );
+        m_clipY2 = wxMin( m_clipY2, (yy + hh) );
      }
      else
      {
      }
      else
      {
@@ -447,14 +498,10 @@ void wxDC::DoSetClippingRegion( wxCoord x, wxCoord y, wxCoord width, wxCoord hei
      }
  }
  
      }
  }
  
-void wxDC::DoSetClippingRegionAsRegion( const wxRegion &region  )
+void wxDC::DoSetClippingRegionAsRegion( const wxRegion &region )
  {
  {
-    wxCHECK_RET( Ok(), wxT("invalid window dc") ) ;
-    if (region.Empty())
-    {
-        DestroyClippingRegion();
-        return;
-    }
+    wxCHECK_RET(Ok(), wxT("wxDC::DoSetClippingRegionAsRegion - invalid DC"));
+
      wxMacFastPortSetter helper(this) ;
      wxCoord x, y, w, h;
      region.GetBox( x, y, w, h );
      wxMacFastPortSetter helper(this) ;
      wxCoord x, y, w, h;
      region.GetBox( x, y, w, h );
@@ -463,6 +510,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 )
@@ -473,16 +521,15 @@ void wxDC::DoSetClippingRegionAsRegion( const wxRegion &region  )
      {
          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 ) ;
          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_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_clipX2 = wxMin( m_clipX2, (xx + ww) );
+            m_clipY2 = wxMin( m_clipY2, (yy + hh) );
          }
          else
          {
          }
          else
          {
@@ -498,29 +545,34 @@ void wxDC::DoSetClippingRegionAsRegion( const wxRegion &region  )
  void wxDC::DestroyClippingRegion()
  {
      wxMacFastPortSetter helper(this) ;
  void wxDC::DestroyClippingRegion()
  {
      wxMacFastPortSetter helper(this) ;
+
      CopyRgn( (RgnHandle) m_macBoundaryClipRgn , (RgnHandle) m_macCurrentClipRgn ) ;
      ResetClipping();
  }
  
  void wxDC::DoGetSizeMM( int* width, int* height ) const
  {
      CopyRgn( (RgnHandle) m_macBoundaryClipRgn , (RgnHandle) m_macCurrentClipRgn ) ;
      ResetClipping();
  }
  
  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::SetTextForeground - invalid DC"));
+
      m_textForegroundColour = col;
      m_macFontInstalled = false ;
  }
  
  void wxDC::SetTextBackground( const wxColour &col )
  {
      m_textForegroundColour = col;
      m_macFontInstalled = false ;
  }
  
  void wxDC::SetTextBackground( const wxColour &col )
  {
-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET(Ok(), wxT("wxDC::SetTextBackground - invalid DC"));
+
      m_textBackgroundColour = col;
      m_macFontInstalled = false ;
  }
      m_textBackgroundColour = col;
      m_macFontInstalled = false ;
  }
@@ -530,22 +582,27 @@ 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:
          SetLogicalScale( 1.0, 1.0 );
          break;
      }
      default:
      case wxMM_TEXT:
          SetLogicalScale( 1.0, 1.0 );
          break;
      }
+
      if (mode != wxMM_TEXT)
      {
          m_needComputeScaleX = true;
      if (mode != wxMM_TEXT)
      {
          m_needComputeScaleX = true;
@@ -571,7 +628,8 @@ void wxDC::SetLogicalScale( double x, double y )
  
  void wxDC::SetLogicalOrigin( wxCoord x, wxCoord y )
  {
  
  void wxDC::SetLogicalOrigin( wxCoord x, wxCoord y )
  {
-    m_logicalOriginX = x * m_signX;   // is this still correct ?
+    // is this still correct ?
+    m_logicalOriginX = x * m_signX;
      m_logicalOriginY = y * m_signY;
      ComputeScaleAndOrigin();
  }
      m_logicalOriginY = y * m_signY;
      ComputeScaleAndOrigin();
  }
@@ -598,9 +656,8 @@ wxSize wxDC::GetPPI() const
  int wxDC::GetDepth() const
  {
      if ( IsPortColor( (CGrafPtr) m_macPort ) )
  int wxDC::GetDepth() const
  {
      if ( IsPortColor( (CGrafPtr) m_macPort ) )
-    {
          return ( (**GetPortPixMap( (CGrafPtr) m_macPort)).pixelSize ) ;
          return ( (**GetPortPixMap( (CGrafPtr) m_macPort)).pixelSize ) ;
-    }
+
      return 1 ;
  }
  
      return 1 ;
  }
  
@@ -613,6 +670,7 @@ void wxDC::ComputeScaleAndOrigin()
      m_scaleY = m_logicalScaleY * m_userScaleY;
      m_deviceOriginX = m_internalDeviceOriginX + m_externalDeviceOriginX;
      m_deviceOriginY = m_internalDeviceOriginY + m_externalDeviceOriginY;
      m_scaleY = m_logicalScaleY * m_userScaleY;
      m_deviceOriginX = m_internalDeviceOriginX + m_externalDeviceOriginX;
      m_deviceOriginY = m_internalDeviceOriginY + m_externalDeviceOriginY;
+
      // CMB: if scale has changed call SetPen to recalulate the line width
      if (m_scaleX != origScaleX || m_scaleY != origScaleY)
      {
      // CMB: if scale has changed call SetPen to recalulate the line width
      if (m_scaleX != origScaleX || m_scaleY != origScaleY)
      {
@@ -624,51 +682,54 @@ void wxDC::ComputeScaleAndOrigin()
      }
  }
  
      }
  }
  
-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_macFontInstalled = false ;
  }
  
  {
      m_font = font;
      m_macFontInstalled = false ;
  }
  
-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;
      m_macPenInstalled = false ;
  }
  
      m_pen = pen;
      m_macPenInstalled = false ;
  }
  
-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;
      m_macBrushInstalled = false ;
  }
  
      m_brush = brush;
      m_macBrushInstalled = false ;
  }
  
-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;
      m_backgroundBrush = brush;
-    if (!m_backgroundBrush.Ok())
-        return;
-    m_macBrushInstalled = false ;
+    if (m_backgroundBrush.Ok())
+        m_macBrushInstalled = false ;
  }
  
  }
  
-void  wxDC::SetLogicalFunction( int function )
+void wxDC::SetLogicalFunction( int function )
  {
      if (m_logicalFunction == function)
          return;
  {
      if (m_logicalFunction == function)
          return;
+
      m_logicalFunction = function ;
      m_macFontInstalled = false ;
      m_macBrushInstalled = false ;
      m_logicalFunction = function ;
      m_macFontInstalled = false ;
      m_macBrushInstalled = false ;
@@ -684,32 +745,38 @@ 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") );
+
      wxMacFastPortSetter helper(this) ;
      wxMacFastPortSetter helper(this) ;
+
+    // NOTE: Get/SetCPixel are slow!
      RGBColor colour;
      GetCPixel( XLOG2DEVMAC(x), YLOG2DEVMAC(y), &colour );
      RGBColor colour;
      GetCPixel( XLOG2DEVMAC(x), YLOG2DEVMAC(y), &colour );
-    // Convert from Mac colour to wx
-    col->Set( colour.red   >> 8,
-        colour.green >> 8,
-        colour.blue  >> 8);
+
+    // 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::DoDrawLine - invalid DC"));
+
      wxMacFastPortSetter helper(this) ;
      wxMacFastPortSetter helper(this) ;
+
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          MacInstallPen() ;
          wxCoord offset = ( (m_pen.GetWidth() == 0 ? 1 :
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          MacInstallPen() ;
          wxCoord offset = ( (m_pen.GetWidth() == 0 ? 1 :
-        m_pen.GetWidth() ) * (wxCoord)m_scaleX - 1) / 2;
+            m_pen.GetWidth() ) * (wxCoord)m_scaleX - 1) / 2;
          wxCoord xx1 = XLOG2DEVMAC(x1) - offset;
          wxCoord yy1 = YLOG2DEVMAC(y1) - offset;
          wxCoord xx2 = XLOG2DEVMAC(x2) - offset;
          wxCoord yy2 = YLOG2DEVMAC(y2) - offset;
          wxCoord xx1 = XLOG2DEVMAC(x1) - offset;
          wxCoord yy1 = YLOG2DEVMAC(y1) - offset;
          wxCoord xx2 = XLOG2DEVMAC(x2) - offset;
          wxCoord yy2 = YLOG2DEVMAC(y2) - offset;
+
          if ((m_pen.GetCap() == wxCAP_ROUND) &&
              (m_pen.GetWidth() <= 1))
          {
          if ((m_pen.GetCap() == wxCAP_ROUND) &&
              (m_pen.GetWidth() <= 1))
          {
@@ -722,6 +789,7 @@ void  wxDC::DoDrawLine( wxCoord x1, wxCoord y1, wxCoord x2, wxCoord y2 )
                  if (yy1 > yy2)
                      yy2++;
              }
                  if (yy1 > yy2)
                      yy2++;
              }
+
              if (yy1 == yy2)
              {
                  if (xx1 < xx2)
              if (yy1 == yy2)
              {
                  if (xx1 < xx2)
@@ -730,29 +798,33 @@ void  wxDC::DoDrawLine( wxCoord x1, wxCoord y1, wxCoord x2, wxCoord y2 )
                      xx2++;
              }
          }
                      xx2++;
              }
          }
+
          ::MoveTo(xx1, yy1);
          ::LineTo(xx2, yy2);
      }
  }
  
          ::MoveTo(xx1, yy1);
          ::LineTo(xx2, yy2);
      }
  }
  
-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::DoCrossHair - invalid DC"));
+
      wxMacFastPortSetter helper(this) ;
      wxMacFastPortSetter helper(this) ;
+
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
-        int w = 0;
-        int h = 0;
+        int w = 0, h = 0;
+
          GetSize( &w, &h );
          wxCoord xx = XLOG2DEVMAC(x);
          wxCoord yy = YLOG2DEVMAC(y);
          GetSize( &w, &h );
          wxCoord xx = XLOG2DEVMAC(x);
          wxCoord yy = YLOG2DEVMAC(y);
+
          MacInstallPen();
          ::MoveTo( XLOG2DEVMAC(0), yy );
          ::LineTo( XLOG2DEVMAC(w), yy );
          ::MoveTo( xx, YLOG2DEVMAC(0) );
          ::LineTo( xx, YLOG2DEVMAC(h) );
          CalcBoundingBox(x, y);
          MacInstallPen();
          ::MoveTo( XLOG2DEVMAC(0), yy );
          ::LineTo( XLOG2DEVMAC(w), yy );
          ::MoveTo( xx, YLOG2DEVMAC(0) );
          ::LineTo( xx, YLOG2DEVMAC(h) );
          CalcBoundingBox(x, y);
-        CalcBoundingBox(x+w, y+h);
+        CalcBoundingBox(x + w, y + h);
      }
  }
  
      }
  }
  
@@ -768,28 +840,36 @@ void  wxDC::DoCrossHair( wxCoord x, wxCoord y )
  static double wxConvertWXangleToMACangle(double angle)
  {
      double newAngle = 90 - angle ;
  static double wxConvertWXangleToMACangle(double angle)
  {
      double newAngle = 90 - angle ;
-    while ( newAngle > 360 ) newAngle -= 360 ;
-    while ( newAngle < 0 ) newAngle += 360 ;
+
+    while ( newAngle > 360 )
+        newAngle -= 360 ;
+    while ( newAngle < 0 )
+        newAngle += 360 ;
+
      return newAngle ;
  }
  
      return newAngle ;
  }
  
-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::DoDrawArc - invalid DC"));
+
      wxMacFastPortSetter helper(this) ;
      wxMacFastPortSetter helper(this) ;
+
      wxCoord xx1 = XLOG2DEVMAC(x1);
      wxCoord yy1 = YLOG2DEVMAC(y1);
      wxCoord xx2 = XLOG2DEVMAC(x2);
      wxCoord yy2 = YLOG2DEVMAC(y2);
      wxCoord xxc = XLOG2DEVMAC(xc);
      wxCoord yyc = YLOG2DEVMAC(yc);
      wxCoord xx1 = XLOG2DEVMAC(x1);
      wxCoord yy1 = YLOG2DEVMAC(y1);
      wxCoord xx2 = XLOG2DEVMAC(x2);
      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 radius1, radius2;
      double radius1, radius2;
+
      if (xx1 == xx2 && yy1 == yy2)
      {
          radius1 = 0.0;
      if (xx1 == xx2 && yy1 == yy2)
      {
          radius1 = 0.0;
@@ -803,160 +883,207 @@ void  wxDC::DoDrawArc( wxCoord x1, wxCoord y1,
      {
          radius1 = (xx1 - xxc == 0) ?
              (yy1 - yyc < 0) ? 90.0 : -90.0 :
      {
          radius1 = (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;
          radius2 = (xx2 - xxc == 0) ?
              (yy2 - yyc < 0) ? 90.0 : -90.0 :
          radius2 = (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;
      }
      }
+
      wxCoord alpha2 = wxCoord(radius2 - radius1);
      wxCoord alpha1 = wxCoord(wxConvertWXangleToMACangle(radius1));
      wxCoord alpha2 = wxCoord(radius2 - radius1);
      wxCoord alpha1 = wxCoord(wxConvertWXangleToMACangle(radius1));
-    while( alpha2 < 0 ) alpha2 += 360 ;
+    while ( alpha2 < 0 )
+        alpha2 += 360 ;
      alpha2 = -alpha2 ;
      Rect r = { yyc - rad, xxc - rad, yyc + rad, xxc + rad };
      alpha2 = -alpha2 ;
      Rect r = { yyc - rad, xxc - rad, yyc + rad, xxc + rad };
-    if(m_brush.GetStyle() != wxTRANSPARENT) {
+
+    if (m_brush.GetStyle() != wxTRANSPARENT)
+    {
          MacInstallBrush();
          PaintArc(&r, alpha1, alpha2);
      }
          MacInstallBrush();
          PaintArc(&r, alpha1, alpha2);
      }
-    if(m_pen.GetStyle() != wxTRANSPARENT) {
+
+    if (m_pen.GetStyle() != wxTRANSPARENT)
+    {
          MacInstallPen();
          FrameArc(&r, alpha1, alpha2);
      }
  }
  
          MacInstallPen();
          FrameArc(&r, alpha1, alpha2);
      }
  }
  
-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::DoDrawEllepticArc - invalid DC"));
      wxMacFastPortSetter helper(this) ;
      Rect r;
      wxMacFastPortSetter helper(this) ;
      Rect r;
-    double angle = sa - ea;  // Order important Mac in opposite direction to wx
+
+    // Order important Mac in opposite direction to wx
      // we have to make sure that the filling is always counter-clockwise
      // we have to make sure that the filling is always counter-clockwise
+    double angle = sa - ea;
      if ( angle > 0 )
          angle -= 360 ;
      if ( angle > 0 )
          angle -= 360 ;
+
      wxCoord xx = XLOG2DEVMAC(x);
      wxCoord yy = YLOG2DEVMAC(y);
      wxCoord ww = m_signX * XLOG2DEVREL(w);
      wxCoord hh = m_signY * YLOG2DEVREL(h);
      wxCoord xx = XLOG2DEVMAC(x);
      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;
+    }
+
      sa = wxConvertWXangleToMACangle(sa);
      r.top    = yy;
      r.left   = xx;
      r.bottom = yy + hh;
      r.right  = xx + ww;
      sa = wxConvertWXangleToMACangle(sa);
      r.top    = yy;
      r.left   = xx;
      r.bottom = yy + hh;
      r.right  = xx + ww;
-    if(m_brush.GetStyle() != wxTRANSPARENT) {
+
+    if (m_brush.GetStyle() != wxTRANSPARENT)
+    {
          MacInstallBrush();
          PaintArc(&r, (short)sa, (short)angle);
      }
          MacInstallBrush();
          PaintArc(&r, (short)sa, (short)angle);
      }
-    if(m_pen.GetStyle() != wxTRANSPARENT) {
+
+    if (m_pen.GetStyle() != wxTRANSPARENT)
+    {
          MacInstallPen();
          FrameArc(&r, (short)sa, (short)angle);
      }
  }
  
          MacInstallPen();
          FrameArc(&r, (short)sa, (short)angle);
      }
  }
  
-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::DoDrawPoint - invalid DC"));
+
      wxMacFastPortSetter helper(this) ;
      wxMacFastPortSetter helper(this) ;
+
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          wxCoord xx1 = XLOG2DEVMAC(x);
          wxCoord yy1 = YLOG2DEVMAC(y);
          RGBColor pencolor = MAC_WXCOLORREF( m_pen.GetColour().GetPixel());
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          wxCoord xx1 = XLOG2DEVMAC(x);
          wxCoord yy1 = YLOG2DEVMAC(y);
          RGBColor pencolor = MAC_WXCOLORREF( m_pen.GetColour().GetPixel());
-        ::SetCPixel( xx1,yy1,&pencolor) ;
+
+        // NOTE: Get/SetCPixel are slow!
+        ::SetCPixel( xx1, yy1, &pencolor) ;
          CalcBoundingBox(x, y);
      }
  }
  
          CalcBoundingBox(x, y);
      }
  }
  
-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"));
-    wxMacFastPortSetter helper(this) ;
+    wxCHECK_RET(Ok(), wxT("wxDC::DoDrawLines - invalid DC"));
+
      if (m_pen.GetStyle() == wxTRANSPARENT)
          return;
      if (m_pen.GetStyle() == wxTRANSPARENT)
          return;
+
+    wxMacFastPortSetter helper(this) ;
+
      MacInstallPen() ;
      wxCoord offset = ( (m_pen.GetWidth() == 0 ? 1 :
      m_pen.GetWidth() ) * (wxCoord)m_scaleX - 1) / 2 ;
      MacInstallPen() ;
      wxCoord offset = ( (m_pen.GetWidth() == 0 ? 1 :
      m_pen.GetWidth() ) * (wxCoord)m_scaleX - 1) / 2 ;
+
      wxCoord x1, x2 , y1 , y2 ;
      x1 = XLOG2DEVMAC(points[0].x + xoffset);
      y1 = YLOG2DEVMAC(points[0].y + yoffset);
      wxCoord x1, x2 , y1 , y2 ;
      x1 = XLOG2DEVMAC(points[0].x + xoffset);
      y1 = YLOG2DEVMAC(points[0].y + yoffset);
-    ::MoveTo(x1 - offset, y1 - offset );
+
+    ::MoveTo( x1 - offset, y1 - offset );
      for (int i = 0; i < n-1; i++)
      {
      for (int i = 0; i < n-1; i++)
      {
-        x2 = XLOG2DEVMAC(points[i+1].x + xoffset);
-        y2 = YLOG2DEVMAC(points[i+1].y + yoffset);
+        x2 = XLOG2DEVMAC(points[i + 1].x + xoffset);
+        y2 = YLOG2DEVMAC(points[i + 1].y + yoffset);
          ::LineTo( x2 - offset, y2 - offset );
      }
  }
  
          ::LineTo( x2 - offset, y2 - offset );
      }
  }
  
-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"));
-    wxMacFastPortSetter helper(this) ;
-    wxCoord x1, x2 , y1 , y2 ;
+    wxCHECK_RET(Ok(), wxT("wxDC::DoDrawPolygon - invalid DC"));
+
      if ( m_brush.GetStyle() == wxTRANSPARENT && m_pen.GetStyle() == wxTRANSPARENT )
          return ;
      if ( m_brush.GetStyle() == wxTRANSPARENT && m_pen.GetStyle() == wxTRANSPARENT )
          return ;
+
+    wxMacFastPortSetter helper(this) ;
+
+    wxCoord x1, x2 , y1 , y2 ;
      PolyHandle polygon = OpenPoly();
      x2 = x1 = XLOG2DEVMAC(points[0].x + xoffset);
      y2 = y1 = YLOG2DEVMAC(points[0].y + yoffset);
      PolyHandle polygon = OpenPoly();
      x2 = x1 = XLOG2DEVMAC(points[0].x + xoffset);
      y2 = y1 = YLOG2DEVMAC(points[0].y + yoffset);
-    ::MoveTo(x1,y1);
+
+    ::MoveTo(x1, y1);
      for (int i = 1; i < n; i++)
      {
          x2 = XLOG2DEVMAC(points[i].x + xoffset);
          y2 = YLOG2DEVMAC(points[i].y + yoffset);
          ::LineTo(x2, y2);
      }
      for (int i = 1; i < n; i++)
      {
          x2 = XLOG2DEVMAC(points[i].x + xoffset);
          y2 = YLOG2DEVMAC(points[i].y + yoffset);
          ::LineTo(x2, y2);
      }
+
      // close the polyline if necessary
      if ( x1 != x2 || y1 != y2 )
      // close the polyline if necessary
      if ( x1 != x2 || y1 != y2 )
-    {
-        ::LineTo(x1,y1 ) ;
-    }
+        ::LineTo( x1, y1 ) ;
      ClosePoly();
      ClosePoly();
+
      if (m_brush.GetStyle() != wxTRANSPARENT)
      {
          MacInstallBrush();
          ::PaintPoly( polygon );
      }
      if (m_brush.GetStyle() != wxTRANSPARENT)
      {
          MacInstallBrush();
          ::PaintPoly( polygon );
      }
+
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          MacInstallPen() ;
          ::FramePoly( polygon ) ;
      }
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          MacInstallPen() ;
          ::FramePoly( polygon ) ;
      }
+
      KillPoly( polygon );
  }
  
  void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
  {
      KillPoly( polygon );
  }
  
  void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
  {
-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET(Ok(), wxT("wxDC::DoDrawRectangle - invalid DC"));
+
      wxMacFastPortSetter helper(this) ;
      wxMacFastPortSetter helper(this) ;
+
      wxCoord xx = XLOG2DEVMAC(x);
      wxCoord yy = YLOG2DEVMAC(y);
      wxCoord ww = m_signX * XLOG2DEVREL(width);
      wxCoord hh = m_signY * YLOG2DEVREL(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)
      {
          ww = -ww;
          xx = xx - ww;
      }
      // CMB: handle -ve width and/or height
      if (ww < 0)
      {
          ww = -ww;
          xx = xx - ww;
      }
+
      if (hh < 0)
      {
          hh = -hh;
          yy = yy - hh;
      }
      if (hh < 0)
      {
          hh = -hh;
          yy = yy - hh;
      }
+
      Rect rect = { yy , xx , yy + hh , xx + ww } ;
      Rect rect = { yy , xx , yy + hh , xx + ww } ;
+
      if (m_brush.GetStyle() != wxTRANSPARENT)
      {
          MacInstallBrush() ;
          ::PaintRect( &rect ) ;
      }
      if (m_brush.GetStyle() != wxTRANSPARENT)
      {
          MacInstallBrush() ;
          ::PaintRect( &rect ) ;
      }
+
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          MacInstallPen() ;
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          MacInstallPen() ;
@@ -964,11 +1091,12 @@ void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
      }
  }
  
      }
  }
  
-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::DoDrawRoundedRectangle - invalid DC"));
+
      wxMacFastPortSetter helper(this) ;
      if (radius < 0.0)
          radius = - radius * ((width < height) ? width : height);
      wxMacFastPortSetter helper(this) ;
      if (radius < 0.0)
          radius = - radius * ((width < height) ? width : height);
@@ -976,26 +1104,32 @@ void  wxDC::DoDrawRoundedRectangle(wxCoord x, wxCoord y,
      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)
      {
          ww = -ww;
          xx = xx - ww;
      }
      // CMB: handle -ve width and/or height
      if (ww < 0)
      {
          ww = -ww;
          xx = xx - ww;
      }
+
      if (hh < 0)
      {
          hh = -hh;
          yy = yy - hh;
      }
      if (hh < 0)
      {
          hh = -hh;
          yy = yy - hh;
      }
+
      Rect rect = { yy , xx , yy + hh , xx + ww } ;
      Rect rect = { yy , xx , yy + hh , xx + ww } ;
+
      if (m_brush.GetStyle() != wxTRANSPARENT)
      {
          MacInstallBrush() ;
          ::PaintRoundRect( &rect , int(radius * 2) , int(radius * 2) ) ;
      }
      if (m_brush.GetStyle() != wxTRANSPARENT)
      {
          MacInstallBrush() ;
          ::PaintRoundRect( &rect , int(radius * 2) , int(radius * 2) ) ;
      }
+
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          MacInstallPen() ;
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          MacInstallPen() ;
@@ -1003,34 +1137,42 @@ void  wxDC::DoDrawRoundedRectangle(wxCoord x, wxCoord y,
      }
  }
  
      }
  }
  
-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::DoDrawEllipse - invalid DC"));
+
      wxMacFastPortSetter helper(this) ;
      wxMacFastPortSetter helper(this) ;
+
      wxCoord xx = XLOG2DEVMAC(x);
      wxCoord yy = YLOG2DEVMAC(y);
      wxCoord ww = m_signX * XLOG2DEVREL(width);
      wxCoord hh = m_signY * YLOG2DEVREL(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)
      {
          ww = -ww;
          xx = xx - ww;
      }
      // CMB: handle -ve width and/or height
      if (ww < 0)
      {
          ww = -ww;
          xx = xx - ww;
      }
+
      if (hh < 0)
      {
          hh = -hh;
          yy = yy - hh;
      }
      if (hh < 0)
      {
          hh = -hh;
          yy = yy - hh;
      }
+
      Rect rect = { yy , xx , yy + hh , xx + ww } ;
      Rect rect = { yy , xx , yy + hh , xx + ww } ;
+
      if (m_brush.GetStyle() != wxTRANSPARENT)
      {
          MacInstallBrush() ;
          ::PaintOval( &rect ) ;
      }
      if (m_brush.GetStyle() != wxTRANSPARENT)
      {
          MacInstallBrush() ;
          ::PaintOval( &rect ) ;
      }
+
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          MacInstallPen() ;
      if (m_pen.GetStyle() != wxTRANSPARENT)
      {
          MacInstallPen() ;
@@ -1038,26 +1180,31 @@ void  wxDC::DoDrawEllipse(wxCoord x, wxCoord y, wxCoord width, wxCoord height)
      }
  }
  
      }
  }
  
-bool  wxDC::CanDrawBitmap(void) const
+bool wxDC::CanDrawBitmap(void) const
  {
      return true ;
  }
  
  {
      return true ;
  }
  
-bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
+bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
                     wxDC *source, wxCoord xsrc, wxCoord ysrc, int logical_func , bool useMask,
                     wxDC *source, wxCoord xsrc, wxCoord ysrc, int logical_func , bool useMask,
-                   wxCoord xsrcMask,  wxCoord ysrcMask )
+                   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::DoBlit - invalid DC"));
+    wxCHECK_MSG(source->Ok(), false, wxT("wxDC::DoBlit - invalid source DC"));
+
      if ( logical_func == wxNO_OP )
          return true ;
      if ( logical_func == wxNO_OP )
          return true ;
+
      if (xsrcMask == -1 && ysrcMask == -1)
      {
      if (xsrcMask == -1 && ysrcMask == -1)
      {
-        xsrcMask = xsrc; ysrcMask = ysrc;
+        xsrcMask = xsrc;
+        ysrcMask = ysrc;
      }
      }
+
      // correct the parameter in case this dc does not have a mask at all
      if ( useMask && !source->m_macMask )
          useMask = false ;
      // correct the parameter in case this dc does not have a mask at all
      if ( useMask && !source->m_macMask )
          useMask = false ;
+
      Rect srcrect , dstrect ;
      srcrect.top = source->YLOG2DEVMAC(ysrc) ;
      srcrect.left = source->XLOG2DEVMAC(xsrc)  ;
      Rect srcrect , dstrect ;
      srcrect.top = source->YLOG2DEVMAC(ysrc) ;
      srcrect.left = source->XLOG2DEVMAC(xsrc)  ;
@@ -1069,73 +1216,94 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
      dstrect.right = XLOG2DEVMAC(xdest + width ) ;
      short mode = kUnsupportedMode ;
      bool invertDestinationFirst = false ;
      dstrect.right = XLOG2DEVMAC(xdest + width ) ;
      short mode = kUnsupportedMode ;
      bool invertDestinationFirst = false ;
+
      switch ( logical_func )
      {
      case wxAND:        // src AND dst
          mode = adMin ; // ok
          break ;
      switch ( logical_func )
      {
      case wxAND:        // src AND dst
          mode = adMin ; // ok
          break ;
+
      case wxAND_INVERT: // (NOT src) AND dst
          mode = notSrcOr  ; // ok
          break ;
      case wxAND_INVERT: // (NOT src) AND dst
          mode = notSrcOr  ; // ok
          break ;
+
      case wxAND_REVERSE:// src AND (NOT dst)
          invertDestinationFirst = true ;
          mode = srcOr ;
          break ;
      case wxAND_REVERSE:// src AND (NOT dst)
          invertDestinationFirst = true ;
          mode = srcOr ;
          break ;
+
      case wxCLEAR:      // 0
          mode = kEmulatedMode ;
          break ;
      case wxCLEAR:      // 0
          mode = kEmulatedMode ;
          break ;
+
      case wxCOPY:       // src
          mode = srcCopy ; // ok
          break ;
      case wxCOPY:       // src
          mode = srcCopy ; // ok
          break ;
+
      case wxEQUIV:      // (NOT src) XOR dst
          mode = srcXor ; // ok
          break ;
      case wxEQUIV:      // (NOT src) XOR dst
          mode = srcXor ; // ok
          break ;
+
      case wxINVERT:     // NOT dst
          mode = kEmulatedMode ; //or hilite ;
          break ;
      case wxINVERT:     // NOT dst
          mode = kEmulatedMode ; //or hilite ;
          break ;
+
      case wxNAND:       // (NOT src) OR (NOT dst)
          invertDestinationFirst = true ;
          mode = srcBic ;
          break ;
      case wxNAND:       // (NOT src) OR (NOT dst)
          invertDestinationFirst = true ;
          mode = srcBic ;
          break ;
+
      case wxNOR:        // (NOT src) AND (NOT dst)
          invertDestinationFirst = true ;
          mode = notSrcOr ;
          break ;
      case wxNOR:        // (NOT src) AND (NOT dst)
          invertDestinationFirst = true ;
          mode = notSrcOr ;
          break ;
+
      case wxNO_OP:      // dst
          mode = kEmulatedMode ; // this has already been handled upon entry
          break ;
      case wxNO_OP:      // dst
          mode = kEmulatedMode ; // this has already been handled upon entry
          break ;
+
      case wxOR:         // src OR dst
          mode = notSrcBic ;
          break ;
      case wxOR:         // src OR dst
          mode = notSrcBic ;
          break ;
+
      case wxOR_INVERT:  // (NOT src) OR dst
          mode = srcBic ;
          break ;
      case wxOR_INVERT:  // (NOT src) OR dst
          mode = srcBic ;
          break ;
+
      case wxOR_REVERSE: // src OR (NOT dst)
          invertDestinationFirst = true ;
          mode = notSrcBic ;
          break ;
      case wxOR_REVERSE: // src OR (NOT dst)
          invertDestinationFirst = true ;
          mode = notSrcBic ;
          break ;
+
      case wxSET:        // 1
          mode = kEmulatedMode ;
          break ;
      case wxSET:        // 1
          mode = kEmulatedMode ;
          break ;
+
      case wxSRC_INVERT: // (NOT src)
          mode = notSrcCopy ; // ok
          break ;
      case wxSRC_INVERT: // (NOT src)
          mode = notSrcCopy ; // ok
          break ;
+
      case wxXOR:        // src XOR dst
          mode = notSrcXor ; // ok
          break ;
      case wxXOR:        // src XOR dst
          mode = notSrcXor ; // ok
          break ;
+
      default :
          break ;
      }
      default :
          break ;
      }
+
      if ( mode == kUnsupportedMode )
      {
          wxFAIL_MSG(wxT("unsupported blitting mode" ));
      if ( mode == kUnsupportedMode )
      {
          wxFAIL_MSG(wxT("unsupported blitting mode" ));
+
          return false ;
      }
          return false ;
      }
+
      CGrafPtr            sourcePort = (CGrafPtr) source->m_macPort ;
      PixMapHandle    bmappixels =  GetGWorldPixMap( sourcePort ) ;
      if ( LockPixels(bmappixels) )
      {
          wxMacFastPortSetter helper(this) ;
      CGrafPtr            sourcePort = (CGrafPtr) source->m_macPort ;
      PixMapHandle    bmappixels =  GetGWorldPixMap( sourcePort ) ;
      if ( LockPixels(bmappixels) )
      {
          wxMacFastPortSetter helper(this) ;
+
          if ( source->GetDepth() == 1 )
          {
              RGBForeColor( &MAC_WXCOLORREF(m_textForegroundColour.GetPixel()) ) ;
          if ( source->GetDepth() == 1 )
          {
              RGBForeColor( &MAC_WXCOLORREF(m_textForegroundColour.GetPixel()) ) ;
@@ -1144,11 +1312,13 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
          else
          {
              // the modes need this, otherwise we'll end up having really nice colors...
          else
          {
              // the modes need this, otherwise we'll end up having really nice colors...
-            RGBColor    white = { 0xFFFF, 0xFFFF,0xFFFF} ;
-            RGBColor    black = { 0,0,0} ;
+            RGBColor white = { 0xFFFF, 0xFFFF, 0xFFFF } ;
+            RGBColor black = { 0, 0, 0 } ;
+
              RGBForeColor( &black ) ;
              RGBBackColor( &white ) ;
          }
              RGBForeColor( &black ) ;
              RGBBackColor( &white ) ;
          }
+
          if ( useMask && source->m_macMask )
          {
              if ( mode == srcCopy )
          if ( useMask && source->m_macMask )
          {
              if ( mode == srcCopy )
@@ -1169,9 +1339,11 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
                  BitMapToRegion( clipRgn , (BitMap*) *GetGWorldPixMap( MAC_WXHBITMAP(source->m_macMask) ) ) ;
                  UnlockPixels( GetGWorldPixMap( MAC_WXHBITMAP(source->m_macMask) ) ) ;
                  OffsetRgn( clipRgn , -srcrect.left + dstrect.left, -srcrect.top +  dstrect.top ) ;
                  BitMapToRegion( clipRgn , (BitMap*) *GetGWorldPixMap( MAC_WXHBITMAP(source->m_macMask) ) ) ;
                  UnlockPixels( GetGWorldPixMap( MAC_WXHBITMAP(source->m_macMask) ) ) ;
                  OffsetRgn( clipRgn , -srcrect.left + dstrect.left, -srcrect.top +  dstrect.top ) ;
+
                  if ( mode == kEmulatedMode )
                  {
                      Pattern pat ;
                  if ( mode == kEmulatedMode )
                  {
                      Pattern pat ;
+
                      ::PenPat(GetQDGlobalsBlack(&pat));
                      if ( logical_func == wxSET )
                      {
                      ::PenPat(GetQDGlobalsBlack(&pat));
                      if ( logical_func == wxSET )
                      {
@@ -1199,13 +1371,14 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
                                  Point srcPoint = { srcrect.top + y , srcrect.left + x } ;
                                  if ( PtInRgn( dstPoint , clipRgn ) )
                                  {
                                  Point srcPoint = { srcrect.top + y , srcrect.left + x } ;
                                  if ( PtInRgn( dstPoint , clipRgn ) )
                                  {
-                                    RGBColor srcColor ;
-                                    RGBColor dstColor ;
+                                    RGBColor srcColor, dstColor ;
+
+                                    // NOTE: Get/SetCPixel are slow!
                                      SetPort( (GrafPtr) sourcePort ) ;
                                      SetPort( (GrafPtr) sourcePort ) ;
-                                    GetCPixel(  srcPoint.h , srcPoint.v , &srcColor) ;
+                                    GetCPixel( srcPoint.h , srcPoint.v , &srcColor ) ;
                                      SetPort( (GrafPtr) m_macPort ) ;
                                      GetCPixel( dstPoint.h , dstPoint.v , &dstColor ) ;
                                      SetPort( (GrafPtr) m_macPort ) ;
                                      GetCPixel( dstPoint.h , dstPoint.v , &dstColor ) ;
-                                    wxMacCalculateColour( logical_func , srcColor ,  dstColor ) ;
+                                    wxMacCalculateColour( logical_func , srcColor , dstColor ) ;
                                      SetCPixel( dstPoint.h , dstPoint.v , &dstColor ) ;
                                  }
                              }
                                      SetCPixel( dstPoint.h , dstPoint.v , &dstColor ) ;
                                  }
                              }
@@ -1215,13 +1388,13 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
                  else
                  {
                      if ( invertDestinationFirst )
                  else
                  {
                      if ( invertDestinationFirst )
-                    {
                          MacInvertRgn( clipRgn ) ;
                          MacInvertRgn( clipRgn ) ;
-                    }
+
                      CopyBits( GetPortBitMapForCopyBits( sourcePort ) ,
                          GetPortBitMapForCopyBits( MAC_WXHBITMAP(m_macPort) ) ,
                          &srcrect, &dstrect, mode, clipRgn ) ;
                  }
                      CopyBits( GetPortBitMapForCopyBits( sourcePort ) ,
                          GetPortBitMapForCopyBits( MAC_WXHBITMAP(m_macPort) ) ,
                          &srcrect, &dstrect, mode, clipRgn ) ;
                  }
+
                  DisposeRgn( clipRgn ) ;
              }
          }
                  DisposeRgn( clipRgn ) ;
              }
          }
@@ -1229,9 +1402,11 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
          {
              RgnHandle clipRgn = NewRgn() ;
              SetRectRgn( clipRgn , dstrect.left , dstrect.top , dstrect.right , dstrect.bottom ) ;
          {
              RgnHandle clipRgn = NewRgn() ;
              SetRectRgn( clipRgn , dstrect.left , dstrect.top , dstrect.right , dstrect.bottom ) ;
+
              if ( mode == kEmulatedMode )
              {
                  Pattern pat ;
              if ( mode == kEmulatedMode )
              {
                  Pattern pat ;
+
                  ::PenPat(GetQDGlobalsBlack(&pat));
                  if ( logical_func == wxSET )
                  {
                  ::PenPat(GetQDGlobalsBlack(&pat));
                  if ( logical_func == wxSET )
                  {
@@ -1241,7 +1416,8 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
                  }
                  else if ( logical_func == wxCLEAR )
                  {
                  }
                  else if ( logical_func == wxCLEAR )
                  {
-                    RGBColor col= { 0x0000, 0x0000, 0x0000 } ;
+                    RGBColor col = { 0x0000, 0x0000, 0x0000 } ;
+
                      ::RGBForeColor( &col  ) ;
                      ::PaintRgn( clipRgn ) ;
                  }
                      ::RGBForeColor( &col  ) ;
                      ::PaintRgn( clipRgn ) ;
                  }
@@ -1258,13 +1434,14 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
                              Point dstPoint = { dstrect.top + y , dstrect.left + x } ;
                              Point srcPoint = { srcrect.top + y , srcrect.left + x } ;
                              {
                              Point dstPoint = { dstrect.top + y , dstrect.left + x } ;
                              Point srcPoint = { srcrect.top + y , srcrect.left + x } ;
                              {
-                                RGBColor srcColor ;
-                                RGBColor dstColor ;
+                                RGBColor srcColor, dstColor ;
+
+                                // NOTE: Get/SetCPixel are slow!
                                  SetPort( (GrafPtr) sourcePort ) ;
                                  SetPort( (GrafPtr) sourcePort ) ;
-                                GetCPixel(  srcPoint.h , srcPoint.v , &srcColor) ;
+                                GetCPixel( srcPoint.h , srcPoint.v , &srcColor) ;
                                  SetPort( (GrafPtr) m_macPort ) ;
                                  GetCPixel( dstPoint.h , dstPoint.v , &dstColor ) ;
                                  SetPort( (GrafPtr) m_macPort ) ;
                                  GetCPixel( dstPoint.h , dstPoint.v , &dstColor ) ;
-                                wxMacCalculateColour( logical_func , srcColor ,  dstColor ) ;
+                                wxMacCalculateColour( logical_func , srcColor , dstColor ) ;
                                  SetCPixel( dstPoint.h , dstPoint.v , &dstColor ) ;
                              }
                          }
                                  SetCPixel( dstPoint.h , dstPoint.v , &dstColor ) ;
                              }
                          }
@@ -1274,203 +1451,156 @@ bool  wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height,
              else
              {
                  if ( invertDestinationFirst )
              else
              {
                  if ( invertDestinationFirst )
-                {
                      MacInvertRgn( clipRgn ) ;
                      MacInvertRgn( clipRgn ) ;
-                }
+
                  CopyBits( GetPortBitMapForCopyBits( sourcePort ) ,
                      GetPortBitMapForCopyBits( MAC_WXHBITMAP(m_macPort) ) ,
                      &srcrect, &dstrect, mode, NULL ) ;
              }
                  CopyBits( GetPortBitMapForCopyBits( sourcePort ) ,
                      GetPortBitMapForCopyBits( MAC_WXHBITMAP(m_macPort) ) ,
                      &srcrect, &dstrect, mode, NULL ) ;
              }
+
              DisposeRgn( clipRgn ) ;
          }
              DisposeRgn( clipRgn ) ;
          }
+
          UnlockPixels( bmappixels ) ;
      }
          UnlockPixels( bmappixels ) ;
      }
+
      m_macPenInstalled = false ;
      m_macBrushInstalled = false ;
      m_macFontInstalled = false ;
      m_macPenInstalled = false ;
      m_macBrushInstalled = false ;
      m_macFontInstalled = 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)
  {
-    // TODO support text background color (only possible by hand, ATSUI does not support it)
-    wxCHECK_RET( Ok(), wxT("wxDC::DoDrawRotatedText  Invalid window dc") );
+    wxCHECK_RET( Ok(), wxT("wxDC::DoDrawRotatedText - invalid DC") );
  
  
-    if ( str.Length() == 0 )
+    if ( str.empty() )
          return ;
  
      wxMacFastPortSetter helper(this) ;
      MacInstallFont() ;
  
          return ;
  
      wxMacFastPortSetter helper(this) ;
      MacInstallFont() ;
  
-    if ( 0 )
-    {
-        m_macFormerAliasState = IsAntiAliasedTextEnabled(&m_macFormerAliasSize);
-        SetAntiAliasedTextEnabled(true, SInt16(m_scaleY * m_font.MacGetFontSize()));
-        m_macAliasWasEnabled = true ;
-    }
      OSStatus status = noErr ;
      ATSUTextLayout atsuLayout ;
      OSStatus status = noErr ;
      ATSUTextLayout atsuLayout ;
-    UniCharCount chars = str.Length() ;
-    UniChar* ubuf = NULL ;
-#if SIZEOF_WCHAR_T == 4
-    wxMBConvUTF16BE 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 ,
+    wxMacUniCharBuffer unibuf( str ) ;
+    UniCharCount chars = unibuf.GetChars() ;
+
+    status = ::ATSUCreateTextLayoutWithTextPtr( unibuf.GetBuffer() , 0 , chars , chars , 1 ,
          &chars , (ATSUStyle*) &m_macATSUIStyle , &atsuLayout ) ;
  
      wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the rotated text") );
          &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 );
      int drawX = XLOG2DEVMAC(x) ;
      int drawY = YLOG2DEVMAC(y) ;
  
      int iAngle = int( angle );
      int drawX = XLOG2DEVMAC(x) ;
      int drawY = YLOG2DEVMAC(y) ;
  
-    ATSUTextMeasurement textBefore ;
-    ATSUTextMeasurement textAfter ;
-    ATSUTextMeasurement ascent ;
-    ATSUTextMeasurement descent ;
+    ATSUTextMeasurement textBefore, textAfter ;
+    ATSUTextMeasurement ascent, descent ;
  
  
+    ATSLineLayoutOptions layoutOptions = kATSLineNoLayoutOptions ;
  
  
-    if ( abs(iAngle) > 0 )
+    if (m_font.GetNoAntiAliasing())
      {
      {
-        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 ) ;
+        layoutOptions |= kATSLineNoAntiAliasing ;
      }
      }
+
+    Fixed atsuAngle = IntToFixed( iAngle ) ;
+
+    ATSUAttributeTag atsuTags[] =
+    {
+        kATSULineLayoutOptionsTag ,
+        kATSULineRotationTag ,
+    } ;
+
+    ByteCount atsuSizes[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
+    {
+        sizeof( ATSLineLayoutOptions ) ,
+        sizeof( Fixed ) ,
+    } ;
+
+    ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
+    {
+        &layoutOptions ,
+        &atsuAngle ,
+    } ;
+
+    status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags)/sizeof(ATSUAttributeTag) - ( abs(iAngle) > 0.001 ? 0 : 1),
+            atsuTags, atsuSizes, atsuValues ) ;
+
      status = ::ATSUMeasureText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
          &textBefore , &textAfter, &ascent , &descent );
      status = ::ATSUMeasureText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
          &textBefore , &textAfter, &ascent , &descent );
+    wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );
+
+    if ( m_backgroundMode == wxSOLID )
+    {
+        // background painting must be done by hand, cannot be done by ATSUI
+        wxCoord x2 , y2 ;
+        PolyHandle polygon = OpenPoly();
+
+        ::MoveTo(drawX, drawY);
+
+        x2 = (int) (drawX + sin(angle / RAD2DEG) * FixedToInt(ascent + descent)) ;
+        y2 = (int) (drawY + cos(angle / RAD2DEG) * FixedToInt(ascent + descent)) ;
+        ::LineTo(x2, y2);
  
  
-    drawX += (int)(sin(angle/RAD2DEG) * FixedToInt(ascent));
-    drawY += (int)(cos(angle/RAD2DEG) * FixedToInt(ascent));
+        x2 = (int) (drawX + sin(angle / RAD2DEG) * FixedToInt(ascent + descent ) + cos(angle / RAD2DEG) * FixedToInt(textAfter)) ;
+        y2 = (int) (drawY + cos(angle / RAD2DEG) * FixedToInt(ascent + descent) - sin(angle / RAD2DEG) * FixedToInt(textAfter)) ;
+        ::LineTo(x2, y2);
+
+        x2 = (int) (drawX + cos(angle / RAD2DEG) * FixedToInt(textAfter)) ;
+        y2 = (int) (drawY - sin(angle / RAD2DEG) * FixedToInt(textAfter)) ;
+        ::LineTo(x2, y2);
+
+        ::LineTo( drawX, drawY) ;
+        ClosePoly();
+
+        ::ErasePoly( polygon );
+        KillPoly( polygon );
+    }
+
+    drawX += (int)(sin(angle / RAD2DEG) * FixedToInt(ascent));
+    drawY += (int)(cos(angle / RAD2DEG) * FixedToInt(ascent));
      status = ::ATSUDrawText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
          IntToFixed(drawX) , IntToFixed(drawY) );
      status = ::ATSUDrawText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
          IntToFixed(drawX) , IntToFixed(drawY) );
+
      wxASSERT_MSG( status == noErr , wxT("couldn't draw the rotated text") );
      wxASSERT_MSG( status == noErr , wxT("couldn't draw the rotated text") );
+
      Rect rect ;
      status = ::ATSUMeasureTextImage( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
          IntToFixed(drawX) , IntToFixed(drawY) , &rect );
      wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );
      Rect rect ;
      status = ::ATSUMeasureTextImage( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
          IntToFixed(drawX) , IntToFixed(drawY) , &rect );
      wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );
+
      OffsetRect( &rect , -m_macLocalOrigin.x , -m_macLocalOrigin.y ) ;
      CalcBoundingBox(XDEV2LOG(rect.left), YDEV2LOG(rect.top) );
      CalcBoundingBox(XDEV2LOG(rect.right), YDEV2LOG(rect.bottom) );
      ::ATSUDisposeTextLayout(atsuLayout);
      OffsetRect( &rect , -m_macLocalOrigin.x , -m_macLocalOrigin.y ) ;
      CalcBoundingBox(XDEV2LOG(rect.left), YDEV2LOG(rect.top) );
      CalcBoundingBox(XDEV2LOG(rect.right), YDEV2LOG(rect.bottom) );
      ::ATSUDisposeTextLayout(atsuLayout);
-#if SIZEOF_WCHAR_T == 4
-    free( ubuf ) ;
-#endif
  }
  
  }
  
-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"));
-
-    wxMacFastPortSetter helper(this) ;
-    long xx = XLOG2DEVMAC(x);
-    long yy = YLOG2DEVMAC(y);
-#if TARGET_CARBON
-    bool useDrawThemeText = ( DrawThemeTextBox != (void*) kUnresolvedCFragSymbolAddress ) ;
-    if ( UMAGetSystemVersion() < 0x1000 || IsKindOf(CLASSINFO( wxPrinterDC ) ) || m_font.GetNoAntiAliasing() )
-        useDrawThemeText = false ;
-#endif
-    MacInstallFont() ;
-
-    FontInfo fi ;
-    ::GetFontInfo( &fi ) ;
-#if TARGET_CARBON
-    if ( !useDrawThemeText )
-#endif
-        yy += fi.ascent ;
-    ::MoveTo( xx , yy );
-    if (  m_backgroundMode == wxTRANSPARENT )
-    {
-        ::TextMode( srcOr) ;
-    }
-    else
-    {
-        ::TextMode( srcCopy ) ;
-    }
-    int line = 0 ;
-    {
-        wxString linetext = strtext ;
-#if TARGET_CARBON
-        if ( useDrawThemeText )
-        {
-            Rect frame = { yy + line*(fi.descent + fi.ascent + fi.leading)  ,xx , yy + (line+1)*(fi.descent + fi.ascent + fi.leading) , xx + 10000 } ;
-            wxMacCFStringHolder mString( linetext , m_font.GetEncoding()) ;
-
-            if ( m_backgroundMode != wxTRANSPARENT )
-            {
-                Point bounds={0,0} ;
-                Rect background = frame ;
-                SInt16 baseline ;
-                ::GetThemeTextDimensions( mString,
-                    m_font.MacGetThemeFontID() ,
-                    kThemeStateActive,
-                    false,
-                    &bounds,
-                    &baseline );
-                background.right = background.left + bounds.h ;
-                background.bottom = background.top + bounds.v ;
-                ::EraseRect( &background ) ;
-            }
-            ::DrawThemeTextBox( mString,
-                m_font.MacGetThemeFontID() ,
-                kThemeStateActive,
-                false,
-                &frame,
-                teJustLeft,
-                nil );
-        }
-        else
-#endif
-        {
-            wxCharBuffer text = linetext.mb_str(wxConvLocal) ;
-            ::DrawText( text , 0 , strlen(text) ) ;
-         }
-    }
-    ::TextMode( srcOr ) ;
+    DoDrawRotatedText( strtext , x , y , 0) ;
  }
  
  }
  
-bool  wxDC::CanGetTextExtent() const
+bool wxDC::CanGetTextExtent() const
  {
  {
-    wxCHECK_MSG(Ok(), false, wxT("Invalid DC"));
+    wxCHECK_MSG(Ok(), false, wxT("wxDC::CanGetTextExtent - invalid DC"));
+
      return true ;
  }
  
      return true ;
  }
  
-void  wxDC::DoGetTextExtent( const wxString &strtext, 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"));
+    wxCHECK_RET(Ok(), wxT("wxDC::DoGetTextExtent - invalid DC"));
+
      wxMacFastPortSetter helper(this) ;
      wxFont formerFont = m_font ;
      if ( theFont )
      wxMacFastPortSetter helper(this) ;
      wxFont formerFont = m_font ;
      if ( theFont )
@@ -1478,49 +1608,66 @@ void  wxDC::DoGetTextExtent( const wxString &strtext, wxCoord *width, wxCoord *h
          // work around the constness
          *((wxFont*)(&m_font)) = *theFont ;
      }
          // work around the constness
          *((wxFont*)(&m_font)) = *theFont ;
      }
+
      MacInstallFont() ;
      MacInstallFont() ;
-    FontInfo fi ;
-    ::GetFontInfo( &fi ) ;
-#if TARGET_CARBON
-    bool useGetThemeText = ( GetThemeTextDimensions != (void*) kUnresolvedCFragSymbolAddress ) ;
-    if ( UMAGetSystemVersion() < 0x1000 || IsKindOf(CLASSINFO( wxPrinterDC ) ) || ((wxFont*)&m_font)->GetNoAntiAliasing() )
-        useGetThemeText = false ;
-#endif
+
+    OSStatus status = noErr ;
+    ATSUTextLayout atsuLayout ;
+
+    wxMacUniCharBuffer unibuf( str ) ;
+    UniCharCount chars = unibuf.GetChars() ;
+
+    status = ::ATSUCreateTextLayoutWithTextPtr( unibuf.GetBuffer() , 0 , chars , chars , 1 ,
+        &chars , (ATSUStyle*) &m_macATSUIStyle , &atsuLayout ) ;
+
+    wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the text") );
+
+    status = ::ATSUSetTransientFontMatching( atsuLayout , true ) ;
+    wxASSERT_MSG( status == noErr , wxT("couldn't setup transient font matching") );
+
+    ATSLineLayoutOptions layoutOptions = kATSLineNoLayoutOptions ;
+
+    if (m_font.GetNoAntiAliasing())
+    {
+        layoutOptions |= kATSLineNoAntiAliasing ;
+    }
+
+    ATSUAttributeTag atsuTags[] =
+    {
+        kATSULineLayoutOptionsTag ,
+    } ;
+
+    ByteCount atsuSizes[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
+    {
+        sizeof( ATSLineLayoutOptions ) ,
+    } ;
+
+    ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
+    {
+        &layoutOptions ,
+    } ;
+
+    status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags)/sizeof(ATSUAttributeTag) ,
+            atsuTags, atsuSizes, atsuValues ) ;
+
+    ATSUTextMeasurement textBefore, textAfter ;
+    ATSUTextMeasurement textAscent, textDescent ;
+
+    status = ::ATSUGetUnjustifiedBounds( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
+        &textBefore , &textAfter, &textAscent , &textDescent );
+
      if ( height )
      if ( height )
-        *height = YDEV2LOGREL( fi.descent + fi.ascent ) ;
+        *height = YDEV2LOGREL( FixedToInt(textAscent + textDescent) ) ;
      if ( descent )
      if ( descent )
-        *descent =YDEV2LOGREL( fi.descent );
+        *descent =YDEV2LOGREL( FixedToInt(textDescent) );
      if ( externalLeading )
      if ( externalLeading )
-        *externalLeading = YDEV2LOGREL( fi.leading ) ;
-
-    int curwidth = 0 ;
+        *externalLeading = 0 ;
      if ( width )
      if ( width )
-    {
-        *width = 0 ;
-        wxString linetext = strtext ;
+        *width = XDEV2LOGREL( FixedToInt(textAfter - textBefore) ) ;
+
+    ::ATSUDisposeTextLayout(atsuLayout);
+
  
  
-        if ( useGetThemeText )
-        {
-            Point bounds={0,0} ;
-            SInt16 baseline ;
-            wxMacCFStringHolder mString( linetext , m_font.GetEncoding() ) ;
-            ThemeFontID themeFont = m_font.MacGetThemeFontID() ;
-            ::GetThemeTextDimensions( mString,
-                themeFont ,
-                kThemeStateActive,
-                false,
-                &bounds,
-                &baseline );
-            curwidth = bounds.h ;
-        }
-        else
-        {
-            wxCharBuffer text = linetext.mb_str(wxConvLocal) ;
-            curwidth = ::TextWidth( text , 0 , strlen(text) ) ;
-        }
-        if ( curwidth > *width )
-            *width = XDEV2LOGREL( curwidth ) ;
-    }
      if ( theFont )
      {
          // work around the constness
      if ( theFont )
      {
          // work around the constness
@@ -1529,118 +1676,99 @@ void  wxDC::DoGetTextExtent( const wxString &strtext, wxCoord *width, wxCoord *h
      }
  }
  
      }
  }
  
-
  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"));
+    wxCHECK_MSG(Ok(), false, wxT("wxDC::DoGetPartialTextExtents - invalid DC"));
  
      widths.Empty();
  
      widths.Empty();
-    widths.Add(0, text.Length());
+    widths.Add(0, text.length());
  
  
-    if (text.Length() == 0)
+    if (text.length() == 0)
          return false;
  
      wxMacFastPortSetter helper(this) ;
      MacInstallFont() ;
          return false;
  
      wxMacFastPortSetter helper(this) ;
      MacInstallFont() ;
-#if TARGET_CARBON
-    bool useGetThemeText = ( GetThemeTextDimensions != (void*) kUnresolvedCFragSymbolAddress ) ;
-    if ( UMAGetSystemVersion() < 0x1000 || IsKindOf(CLASSINFO( wxPrinterDC ) ) || ((wxFont*)&m_font)->GetNoAntiAliasing() )
-        useGetThemeText = false ;
-
-    if ( useGetThemeText )
-    {
-        // If anybody knows how to do this more efficiently yet still handle
-        // the fractional glyph widths that may be present when using AA
-        // fonts, please change it.  Currently it is measuring from the
-        // begining of the string for each succeding substring, which is much
-        // slower than this should be.
-        for (size_t i=0; i<text.Length(); i++)
-        {
-            wxString str(text.Left(i+1));
-            Point bounds = {0,0};
-            SInt16 baseline ;
-            wxMacCFStringHolder mString(str, m_font.GetEncoding());
-            ::GetThemeTextDimensions( mString,
-                                      m_font.MacGetThemeFontID(),
-                                      kThemeStateActive,
-                                      false,
-                                      &bounds,
-                                      &baseline );
-            widths[i] = XDEV2LOGREL(bounds.h);
-        }
+
+    OSStatus status = noErr ;
+    ATSUTextLayout atsuLayout ;
+
+    wxMacUniCharBuffer unibuf( text ) ;
+    UniCharCount chars = unibuf.GetChars() ;
+
+    status = ::ATSUCreateTextLayoutWithTextPtr( unibuf.GetBuffer() , 0 , chars , chars , 1 ,
+        &chars , (ATSUStyle*) &m_macATSUIStyle , &atsuLayout ) ;
+
+    wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the text") );
+
+    status = ::ATSUSetTransientFontMatching( atsuLayout , true ) ;
+    wxASSERT_MSG( status == noErr , wxT("couldn't setup transient font matching") );
+
+    ATSLineLayoutOptions layoutOptions = kATSLineNoLayoutOptions ;
+
+    if (m_font.GetNoAntiAliasing())
+    {
+        layoutOptions |= kATSLineNoAntiAliasing ;
      }
      }
-    else
-#endif
+
+    ATSUAttributeTag atsuTags[] =
+    {
+        kATSULineLayoutOptionsTag ,
+    } ;
+
+    ByteCount atsuSizes[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
+    {
+        sizeof( ATSLineLayoutOptions ) ,
+    } ;
+
+    ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
      {
      {
-        wxCharBuffer buff = text.mb_str(wxConvLocal);
-        size_t len = strlen(buff);
-        short* measurements = new short[len+1];
-        MeasureText(len, buff.data(), measurements);
+        &layoutOptions ,
+    } ;
  
  
-        // Copy to widths, starting at measurements[1]
-        // NOTE: this doesn't take into account any multi-byte characters
-        // in buff, it probabkly should...
-        for (size_t i=0; i<text.Length(); i++)
-            widths[i] = XDEV2LOGREL(measurements[i+1]);
+    status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags)/sizeof(ATSUAttributeTag) ,
+            atsuTags, atsuSizes, atsuValues ) ;
  
  
-        delete [] measurements;
+    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 ));
      }
  
      }
  
+    ::ATSUDisposeTextLayout(atsuLayout);
+
      return true;
  }
  
      return true;
  }
  
-
-
-wxCoord   wxDC::GetCharWidth(void) const
+wxCoord wxDC::GetCharWidth(void) const
  {
  {
-    wxCHECK_MSG(Ok(), 1, wxT("Invalid DC"));
-    wxMacFastPortSetter helper(this) ;
-    MacInstallFont() ;
-    int width = 0 ;
-#if TARGET_CARBON
-    bool useGetThemeText = ( GetThemeTextDimensions != (void*) kUnresolvedCFragSymbolAddress ) ;
-    if ( UMAGetSystemVersion() < 0x1000 || ((wxFont*)&m_font)->GetNoAntiAliasing() )
-        useGetThemeText = false ;
-#endif
-    char text[] = "g" ;
-#if TARGET_CARBON
-    if ( useGetThemeText )
-    {
-        Point bounds={0,0} ;
-        SInt16 baseline ;
-        CFStringRef mString = CFStringCreateWithBytes( NULL , (UInt8*) text , 1 , CFStringGetSystemEncoding(), false ) ;
-        ::GetThemeTextDimensions( mString,
-            m_font.MacGetThemeFontID(),
-            kThemeStateActive,
-            false,
-            &bounds,
-            &baseline );
-        CFRelease( mString ) ;
-        width = bounds.h ;
-    }
-    else
-#endif
-    {
-        width = ::TextWidth( text , 0 , 1 ) ;
-    }
-    return YDEV2LOGREL(width) ;
+    wxCoord width = 0 ;
+    DoGetTextExtent( wxT("g"), &width , NULL , NULL , NULL , NULL ) ;
+    return width ;
  }
  
  }
  
-wxCoord   wxDC::GetCharHeight(void) const
+wxCoord wxDC::GetCharHeight(void) const
  {
  {
-    wxCHECK_MSG(Ok(), 1, wxT("Invalid DC"));
-    wxMacFastPortSetter helper(this) ;
-    MacInstallFont() ;
-    FontInfo fi ;
-    ::GetFontInfo( &fi ) ;
-    return YDEV2LOGREL( fi.descent + fi.ascent );
+    wxCoord height ;
+    DoGetTextExtent( wxT("g") , NULL , &height , NULL , NULL , NULL ) ;
+    return height ;
  }
  
  }
  
-void  wxDC::Clear(void)
+void wxDC::Clear(void)
  {
  {
-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET(Ok(), wxT("wxDC::Clear - invalid DC"));
+
      wxMacFastPortSetter helper(this) ;
      Rect rect = { -31000 , -31000 , 31000 , 31000 } ;
      wxMacFastPortSetter helper(this) ;
      Rect rect = { -31000 , -31000 , 31000 , 31000 } ;
+
      if ( m_backgroundBrush.Ok() && m_backgroundBrush.GetStyle() != wxTRANSPARENT)
      {
          ::PenNormal() ;
      if ( m_backgroundBrush.Ok() && m_backgroundBrush.GetStyle() != wxTRANSPARENT)
      {
          ::PenNormal() ;
@@ -1652,8 +1780,10 @@ void  wxDC::Clear(void)
  void wxDC::MacInstallFont() const
  {
      wxCHECK_RET(Ok(), wxT("Invalid DC"));
  void wxDC::MacInstallFont() const
  {
      wxCHECK_RET(Ok(), wxT("Invalid DC"));
+
      //    if ( m_macFontInstalled )
      //        return ;
      //    if ( m_macFontInstalled )
      //        return ;
+
      Pattern blackColor ;
      MacSetupBackgroundForCurrentPort(m_backgroundBrush) ;
      if ( m_backgroundMode != wxTRANSPARENT )
      Pattern blackColor ;
      MacSetupBackgroundForCurrentPort(m_backgroundBrush) ;
      if ( m_backgroundMode != wxTRANSPARENT )
@@ -1661,9 +1791,8 @@ void wxDC::MacInstallFont() const
          Pattern whiteColor ;
          ::BackPat(GetQDGlobalsWhite(&whiteColor));
      }
          Pattern whiteColor ;
          ::BackPat(GetQDGlobalsWhite(&whiteColor));
      }
-    
+
      wxASSERT( m_font.Ok() ) ;
      wxASSERT( m_font.Ok() ) ;
-    
  
      ::TextFont( m_font.MacGetFontNum() ) ;
      ::TextSize( (short)(m_scaleY * m_font.MacGetFontSize()) ) ;
  
      ::TextFont( m_font.MacGetFontNum() ) ;
      ::TextSize( (short)(m_scaleY * m_font.MacGetFontSize()) ) ;
@@ -1676,30 +1805,36 @@ void wxDC::MacInstallFont() const
      ::RGBForeColor( &forecolor );
      ::RGBBackColor( &backcolor );
  
      ::RGBForeColor( &forecolor );
      ::RGBBackColor( &backcolor );
  
+    // TODO:
      short mode = patCopy ;
      short mode = patCopy ;
-    // todo :
-    switch( m_logicalFunction )
+    switch ( m_logicalFunction )
      {
      case wxCOPY:       // src
          mode = patCopy ;
          break ;
      {
      case wxCOPY:       // src
          mode = patCopy ;
          break ;
+
      case wxINVERT:     // NOT dst
          ::PenPat(GetQDGlobalsBlack(&blackColor));
          mode = patXor ;
          break ;
      case wxINVERT:     // NOT dst
          ::PenPat(GetQDGlobalsBlack(&blackColor));
          mode = patXor ;
          break ;
+
      case wxXOR:        // src XOR dst
          mode = patXor ;
          break ;
      case wxXOR:        // src XOR dst
          mode = patXor ;
          break ;
+
      case wxOR_REVERSE: // src OR (NOT dst)
          mode = notPatOr ;
          break ;
      case wxOR_REVERSE: // src OR (NOT dst)
          mode = notPatOr ;
          break ;
+
      case wxSRC_INVERT: // (NOT src)
          mode = notPatCopy ;
          break ;
      case wxSRC_INVERT: // (NOT src)
          mode = notPatCopy ;
          break ;
+
      case wxAND: // src AND dst
          mode = adMin ;
          break ;
      case wxAND: // src AND dst
          mode = adMin ;
          break ;
-        // unsupported TODO
+
+    // TODO: unsupported
      case wxCLEAR:      // 0
      case wxAND_REVERSE:// src AND (NOT dst)
      case wxAND_INVERT: // (NOT src) AND dst
      case wxCLEAR:      // 0
      case wxAND_REVERSE:// src AND (NOT dst)
      case wxAND_INVERT: // (NOT src) AND dst
@@ -1713,7 +1848,11 @@ void wxDC::MacInstallFont() const
          //        case wxSRC_OR:     // source _bitmap_ OR destination
          //        case wxSRC_AND:     // source _bitmap_ AND destination
          break ;
          //        case wxSRC_OR:     // source _bitmap_ OR destination
          //        case wxSRC_AND:     // source _bitmap_ AND destination
          break ;
+
+    default:
+        break ;
      }
      }
+
      ::PenMode( mode ) ;
      OSStatus status = noErr ;
      status = ATSUCreateAndCopyStyle( (ATSUStyle) m_font.MacGetATSUStyle() , (ATSUStyle*) &m_macATSUIStyle ) ;
      ::PenMode( mode ) ;
      OSStatus status = noErr ;
      status = ATSUCreateAndCopyStyle( (ATSUStyle) m_font.MacGetATSUStyle() , (ATSUStyle*) &m_macATSUIStyle ) ;
@@ -1728,23 +1867,20 @@ void wxDC::MacInstallFont() const
      {
              sizeof( Fixed ) ,
      } ;
      {
              sizeof( Fixed ) ,
      } ;
-//    Boolean kTrue = true ;
-//    Boolean kFalse = false ;
-
-//    ATSUVerticalCharacterType kHorizontal = kATSUStronglyHorizontal;
      ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
      {
              &atsuSize ,
      } ;
      ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
      {
              &atsuSize ,
      } ;
+
      status = ::ATSUSetAttributes((ATSUStyle)m_macATSUIStyle, sizeof(atsuTags)/sizeof(ATSUAttributeTag) ,
          atsuTags, atsuSizes, atsuValues);
  
      status = ::ATSUSetAttributes((ATSUStyle)m_macATSUIStyle, sizeof(atsuTags)/sizeof(ATSUAttributeTag) ,
          atsuTags, atsuSizes, atsuValues);
  
-    wxASSERT_MSG( status == noErr , wxT("couldn't Modify ATSU style") ) ;
-    
+    wxASSERT_MSG( status == noErr , wxT("couldn't modify ATSU style") ) ;
  }
  
  Pattern gPatterns[] =
  }
  
  Pattern gPatterns[] =
-{   // hatch patterns
+{
+    // hatch patterns
      { { 0xFF , 0xFF , 0xFF , 0xFF , 0xFF , 0xFF , 0xFF , 0xFF } } ,
      { { 0x01 , 0x02 , 0x04 , 0x08 , 0x10 , 0x20 , 0x40 , 0x80 } } ,
      { { 0x80 , 0x40 , 0x20 , 0x10 , 0x08 , 0x04 , 0x02 , 0x01 } } ,
      { { 0xFF , 0xFF , 0xFF , 0xFF , 0xFF , 0xFF , 0xFF , 0xFF } } ,
      { { 0x01 , 0x02 , 0x04 , 0x08 , 0x10 , 0x20 , 0x40 , 0x80 } } ,
      { { 0x80 , 0x40 , 0x20 , 0x10 , 0x08 , 0x04 , 0x02 , 0x01 } } ,
@@ -1752,6 +1888,7 @@ Pattern gPatterns[] =
      { { 0x00 , 0x00 , 0x00 , 0xFF , 0x00 , 0x00 , 0x00 , 0x00 } } ,
      { { 0x10 , 0x10 , 0x10 , 0x10 , 0x10 , 0x10 , 0x10 , 0x10 } } ,
      { { 0x81 , 0x42 , 0x24 , 0x18 , 0x18 , 0x24 , 0x42 , 0x81 } } ,
      { { 0x00 , 0x00 , 0x00 , 0xFF , 0x00 , 0x00 , 0x00 , 0x00 } } ,
      { { 0x10 , 0x10 , 0x10 , 0x10 , 0x10 , 0x10 , 0x10 , 0x10 } } ,
      { { 0x81 , 0x42 , 0x24 , 0x18 , 0x18 , 0x24 , 0x42 , 0x81 } } ,
+
      // dash patterns
      { { 0xCC , 0x99 , 0x33 , 0x66 , 0xCC , 0x99 , 0x33 , 0x66 } } , // DOT
      { { 0xFE , 0xFD , 0xFB , 0xF7 , 0xEF , 0xDF , 0xBF , 0x7F } } , // LONG_DASH
      // dash patterns
      { { 0xCC , 0x99 , 0x33 , 0x66 , 0xCC , 0x99 , 0x33 , 0x66 } } , // DOT
      { { 0xFE , 0xFD , 0xFB , 0xF7 , 0xEF , 0xDF , 0xBF , 0x7F } } , // LONG_DASH
@@ -1762,7 +1899,7 @@ Pattern gPatterns[] =
  static void wxMacGetPattern(int penStyle, Pattern *pattern)
  {
      int index = 0;  // solid pattern by default
  static void wxMacGetPattern(int penStyle, Pattern *pattern)
  {
      int index = 0;  // solid pattern by default
-    switch(penStyle)
+    switch (penStyle)
      {
          // hatches
          case wxBDIAGONAL_HATCH:     index = 1; break;
      {
          // hatches
          case wxBDIAGONAL_HATCH:     index = 1; break;
@@ -1771,34 +1908,42 @@ static void wxMacGetPattern(int penStyle, Pattern *pattern)
          case wxHORIZONTAL_HATCH:    index = 4; break;
          case wxVERTICAL_HATCH:      index = 5; break;
          case wxCROSSDIAG_HATCH:     index = 6; break;
          case wxHORIZONTAL_HATCH:    index = 4; break;
          case wxVERTICAL_HATCH:      index = 5; break;
          case wxCROSSDIAG_HATCH:     index = 6; break;
+
          // dashes
          case wxDOT:                 index = 7; break;
          case wxLONG_DASH:           index = 8; break;
          case wxSHORT_DASH:          index = 9; break;
          case wxDOT_DASH:            index = 10; break;
          // dashes
          case wxDOT:                 index = 7; break;
          case wxLONG_DASH:           index = 8; break;
          case wxSHORT_DASH:          index = 9; break;
          case wxDOT_DASH:            index = 10; break;
+
+        default:
+            break;
      }
      }
+
      *pattern = gPatterns[index];
  }
  
  void wxDC::MacInstallPen() const
  {
      *pattern = gPatterns[index];
  }
  
  void wxDC::MacInstallPen() const
  {
-    wxCHECK_RET(Ok(), wxT("Invalid DC"));
+    wxCHECK_RET(Ok(), wxT("wxDC::MacInstallPen - invalid DC"));
+
      //Pattern     blackColor;
      //    if ( m_macPenInstalled )
      //        return ;
      //Pattern     blackColor;
      //    if ( m_macPenInstalled )
      //        return ;
+
      RGBColor forecolor = MAC_WXCOLORREF( m_pen.GetColour().GetPixel());
      RGBColor backcolor = MAC_WXCOLORREF( m_backgroundBrush.GetColour().GetPixel());
      ::RGBForeColor( &forecolor );
      ::RGBBackColor( &backcolor );
      ::PenNormal() ;
      RGBColor forecolor = MAC_WXCOLORREF( m_pen.GetColour().GetPixel());
      RGBColor backcolor = MAC_WXCOLORREF( m_backgroundBrush.GetColour().GetPixel());
      ::RGBForeColor( &forecolor );
      ::RGBBackColor( &backcolor );
      ::PenNormal() ;
-    int penWidth = (int) (m_pen.GetWidth() * m_scaleX) ; ;
+
      // null means only one pixel, at whatever resolution
      // null means only one pixel, at whatever resolution
+    int penWidth = (int) (m_pen.GetWidth() * m_scaleX) ;
      if ( penWidth == 0 )
          penWidth = 1 ;
      ::PenSize(penWidth, penWidth);
  
      if ( penWidth == 0 )
          penWidth = 1 ;
      ::PenSize(penWidth, penWidth);
  
-    int penStyle = m_pen.GetStyle();
      Pattern pat;
      Pattern pat;
+    int penStyle = m_pen.GetStyle();
      if (penStyle == wxUSER_DASH)
      {
          // FIXME: there should be exactly 8 items in the dash
      if (penStyle == wxUSER_DASH)
      {
          // FIXME: there should be exactly 8 items in the dash
@@ -1818,30 +1963,36 @@ void wxDC::MacInstallPen() const
      }
      ::PenPat(&pat);
  
      }
      ::PenPat(&pat);
  
+    // TODO:
      short mode = patCopy ;
      short mode = patCopy ;
-    // todo :
-    switch( m_logicalFunction )
+    switch ( m_logicalFunction )
      {
      case wxCOPY:       // only foreground color, leave background (thus not patCopy)
          mode = patOr ;
          break ;
      {
      case wxCOPY:       // only foreground color, leave background (thus not patCopy)
          mode = patOr ;
          break ;
+
      case wxINVERT:     // NOT dst
          //            ::PenPat(GetQDGlobalsBlack(&blackColor));
          mode = patXor ;
          break ;
      case wxINVERT:     // NOT dst
          //            ::PenPat(GetQDGlobalsBlack(&blackColor));
          mode = patXor ;
          break ;
+
      case wxXOR:        // src XOR dst
          mode = patXor ;
          break ;
      case wxXOR:        // src XOR dst
          mode = patXor ;
          break ;
+
      case wxOR_REVERSE: // src OR (NOT dst)
          mode = notPatOr ;
          break ;
      case wxOR_REVERSE: // src OR (NOT dst)
          mode = notPatOr ;
          break ;
+
      case wxSRC_INVERT: // (NOT src)
          mode = notPatCopy ;
          break ;
      case wxSRC_INVERT: // (NOT src)
          mode = notPatCopy ;
          break ;
+
      case wxAND: // src AND dst
          mode = adMin ;
          break ;
      case wxAND: // src AND dst
          mode = adMin ;
          break ;
-        // unsupported TODO
+
+    // TODO: unsupported
      case wxCLEAR:      // 0
      case wxAND_REVERSE:// src AND (NOT dst)
      case wxAND_INVERT: // (NOT src) AND dst
      case wxCLEAR:      // 0
      case wxAND_REVERSE:// src AND (NOT dst)
      case wxAND_INVERT: // (NOT src) AND dst
@@ -1855,7 +2006,11 @@ void wxDC::MacInstallPen() const
          //        case wxSRC_OR:     // source _bitmap_ OR destination
          //        case wxSRC_AND:     // source _bitmap_ AND destination
          break ;
          //        case wxSRC_OR:     // source _bitmap_ OR destination
          //        case wxSRC_AND:     // source _bitmap_ AND destination
          break ;
+
+    default:
+        break ;
      }
      }
+
      ::PenMode( mode ) ;
      m_macPenInstalled = true ;
      m_macBrushInstalled = false ;
      ::PenMode( mode ) ;
      m_macPenInstalled = true ;
      m_macBrushInstalled = false ;
@@ -1865,22 +2020,23 @@ void wxDC::MacInstallPen() const
  void wxDC::MacSetupBackgroundForCurrentPort(const wxBrush& background )
  {
      Pattern whiteColor ;
  void wxDC::MacSetupBackgroundForCurrentPort(const wxBrush& background )
  {
      Pattern whiteColor ;
+
      if ( background.Ok() )
      {
      if ( background.Ok() )
      {
-        switch( background.MacGetBrushKind() )
+        switch ( background.MacGetBrushKind() )
          {
              case kwxMacBrushTheme :
          {
              case kwxMacBrushTheme :
-            {
                  ::SetThemeBackground( background.MacGetTheme() , wxDisplayDepth() , true ) ;
                  break ;
                  ::SetThemeBackground( background.MacGetTheme() , wxDisplayDepth() , true ) ;
                  break ;
-            }
+
              case kwxMacBrushThemeBackground :
              {
                  Rect extent ;
                  ThemeBackgroundKind bg = background.MacGetThemeBackground( &extent ) ;
              case kwxMacBrushThemeBackground :
              {
                  Rect extent ;
                  ThemeBackgroundKind bg = background.MacGetThemeBackground( &extent ) ;
-                ::ApplyThemeBackground( bg , &extent ,kThemeStateActive , wxDisplayDepth() , true ) ;
-                break ;
+                ::ApplyThemeBackground( bg , &extent, kThemeStateActive , wxDisplayDepth() , true ) ;
              }
              }
+                break ;
+
              case kwxMacBrushColour :
              {
                  ::RGBBackColor( &MAC_WXCOLORREF( background.GetColour().GetPixel()) );
              case kwxMacBrushColour :
              {
                  ::RGBBackColor( &MAC_WXCOLORREF( background.GetColour().GetPixel()) );
@@ -1897,8 +2053,11 @@ void wxDC::MacSetupBackgroundForCurrentPort(const wxBrush& background )
                  {
                      ::BackPat(GetQDGlobalsWhite(&whiteColor));
                  }
                  {
                      ::BackPat(GetQDGlobalsWhite(&whiteColor));
                  }
-                break ;
              }
              }
+                break ;
+
+            default:
+                break ;
          }
      }
  }
          }
      }
  }
@@ -1906,21 +2065,38 @@ void wxDC::MacSetupBackgroundForCurrentPort(const wxBrush& background )
  void wxDC::MacInstallBrush() const
  {
      wxCHECK_RET(Ok(), wxT("Invalid DC"));
  void wxDC::MacInstallBrush() const
  {
      wxCHECK_RET(Ok(), wxT("Invalid DC"));
-    Pattern     blackColor ;
+
      //    if ( m_macBrushInstalled )
      //        return ;
      //    if ( m_macBrushInstalled )
      //        return ;
-    // foreground
+
+    Pattern     blackColor ;
      bool backgroundTransparent = (GetBackgroundMode() == wxTRANSPARENT) ;
      ::RGBForeColor( &MAC_WXCOLORREF( m_brush.GetColour().GetPixel()) );
      ::RGBBackColor( &MAC_WXCOLORREF( m_backgroundBrush.GetColour().GetPixel()) );
      bool backgroundTransparent = (GetBackgroundMode() == wxTRANSPARENT) ;
      ::RGBForeColor( &MAC_WXCOLORREF( m_brush.GetColour().GetPixel()) );
      ::RGBBackColor( &MAC_WXCOLORREF( m_backgroundBrush.GetColour().GetPixel()) );
+
      int brushStyle = m_brush.GetStyle();
      if (brushStyle == wxSOLID)
      {
      int brushStyle = m_brush.GetStyle();
      if (brushStyle == wxSOLID)
      {
-        ::PenPat(GetQDGlobalsBlack(&blackColor));
+        switch ( m_brush.MacGetBrushKind() )
+        {
+            case kwxMacBrushTheme :
+                {
+                    Pattern whiteColor ;
+                    ::BackPat(GetQDGlobalsWhite(&whiteColor));
+                    ::SetThemePen( m_brush.MacGetTheme() , wxDisplayDepth() , true ) ;
+                }
+            break ;
+
+            default :
+                ::PenPat(GetQDGlobalsBlack(&blackColor));
+                break ;
+
+        }
      }
      else if (m_brush.IsHatch())
      {
          Pattern pat ;
      }
      else if (m_brush.IsHatch())
      {
          Pattern pat ;
+
          wxMacGetPattern(brushStyle, &pat);
          ::PenPat(&pat);
      }
          wxMacGetPattern(brushStyle, &pat);
          ::PenPat(&pat);
      }
@@ -1928,15 +2104,18 @@ void wxDC::MacInstallBrush() const
      {
          // we force this in order to be compliant with wxMSW
          backgroundTransparent = false ;
      {
          // we force this in order to be compliant with wxMSW
          backgroundTransparent = false ;
+
          // for these the text fore (and back for MASK_OPAQUE) colors are used
          wxBitmap* bitmap = m_brush.GetStipple() ;
          int width = bitmap->GetWidth() ;
          int height = bitmap->GetHeight() ;
          GWorldPtr gw = NULL ;
          // for these the text fore (and back for MASK_OPAQUE) colors are used
          wxBitmap* bitmap = m_brush.GetStipple() ;
          int width = bitmap->GetWidth() ;
          int height = bitmap->GetHeight() ;
          GWorldPtr gw = NULL ;
+
          if ( m_brush.GetStyle() == wxSTIPPLE )
              gw = MAC_WXHBITMAP(bitmap->GetHBITMAP(NULL))  ;
          else
              gw = MAC_WXHBITMAP(bitmap->GetMask()->GetHBITMAP()) ;
          if ( m_brush.GetStyle() == wxSTIPPLE )
              gw = MAC_WXHBITMAP(bitmap->GetHBITMAP(NULL))  ;
          else
              gw = MAC_WXHBITMAP(bitmap->GetMask()->GetHBITMAP()) ;
+
          PixMapHandle gwpixmaphandle = GetGWorldPixMap( gw ) ;
          LockPixels( gwpixmaphandle ) ;
          bool isMonochrome = !IsPortColor( gw ) ;
          PixMapHandle gwpixmaphandle = GetGWorldPixMap( gw ) ;
          LockPixels( gwpixmaphandle ) ;
          bool isMonochrome = !IsPortColor( gw ) ;
@@ -1945,6 +2124,7 @@ void wxDC::MacInstallBrush() const
              if ( (**gwpixmaphandle).pixelSize == 1 )
                  isMonochrome = true ;
          }
              if ( (**gwpixmaphandle).pixelSize == 1 )
                  isMonochrome = true ;
          }
+
          if ( isMonochrome && width == 8 && height == 8 )
          {
              ::RGBForeColor( &MAC_WXCOLORREF( m_textForegroundColour.GetPixel()) );
          if ( isMonochrome && width == 8 && height == 8 )
          {
              ::RGBForeColor( &MAC_WXCOLORREF( m_textForegroundColour.GetPixel()) );
@@ -1953,10 +2133,12 @@ void wxDC::MacInstallBrush() const
              UInt8 *gwbits = (UInt8*) gwbitmap->baseAddr ;
              int alignment = gwbitmap->rowBytes & 0x7FFF ;
              Pattern pat ;
              UInt8 *gwbits = (UInt8*) gwbitmap->baseAddr ;
              int alignment = gwbitmap->rowBytes & 0x7FFF ;
              Pattern pat ;
+
              for ( int i = 0 ; i < 8 ; ++i )
              {
              for ( int i = 0 ; i < 8 ; ++i )
              {
-                pat.pat[i] = gwbits[i*alignment+0] ;
+                pat.pat[i] = gwbits[i * alignment + 0] ;
              }
              }
+
              UnlockPixels( GetGWorldPixMap( gw ) ) ;
              ::PenPat( &pat ) ;
          }
              UnlockPixels( GetGWorldPixMap( gw ) ) ;
              ::PenPat( &pat ) ;
          }
@@ -1966,6 +2148,7 @@ void wxDC::MacInstallBrush() const
              // caching scheme before putting this into production
              Handle      image;
              long        imageSize;
              // caching scheme before putting this into production
              Handle      image;
              long        imageSize;
+
              PixPatHandle pixpat = NewPixPat() ;
              CopyPixMap(gwpixmaphandle, (**pixpat).patMap);
              imageSize = GetPixRowBytes((**pixpat).patMap) *
              PixPatHandle pixpat = NewPixPat() ;
              CopyPixMap(gwpixmaphandle, (**pixpat).patMap);
              imageSize = GetPixRowBytes((**pixpat).patMap) *
@@ -1973,6 +2156,7 @@ void wxDC::MacInstallBrush() const
                  (**(**pixpat).patMap).bounds.top);
              PtrToHand( (**gwpixmaphandle).baseAddr, &image, imageSize );
              (**pixpat).patData = image;
                  (**(**pixpat).patMap).bounds.top);
              PtrToHand( (**gwpixmaphandle).baseAddr, &image, imageSize );
              (**pixpat).patData = image;
+
              if ( isMonochrome )
              {
                  CTabHandle ctable = ((**((**pixpat).patMap)).pmTable) ;
              if ( isMonochrome )
              {
                  CTabHandle ctable = ((**((**pixpat).patMap)).pmTable) ;
@@ -1989,17 +2173,20 @@ void wxDC::MacInstallBrush() const
                  }
                  ::CTabChanged( ctable ) ;
              }
                  }
                  ::CTabChanged( ctable ) ;
              }
+
              ::PenPixPat(pixpat);
              m_macForegroundPixMap = pixpat ;
          }
              ::PenPixPat(pixpat);
              m_macForegroundPixMap = pixpat ;
          }
+
          UnlockPixels( gwpixmaphandle ) ;
      }
      else
      {
          ::PenPat(GetQDGlobalsBlack(&blackColor));
      }
          UnlockPixels( gwpixmaphandle ) ;
      }
      else
      {
          ::PenPat(GetQDGlobalsBlack(&blackColor));
      }
+
      short mode = patCopy ;
      short mode = patCopy ;
-    switch( m_logicalFunction )
+    switch ( m_logicalFunction )
      {
      case wxCOPY:       // src
          if ( backgroundTransparent )
      {
      case wxCOPY:       // src
          if ( backgroundTransparent )
@@ -2007,26 +2194,30 @@ void wxDC::MacInstallBrush() const
          else
              mode = patCopy ;
          break ;
          else
              mode = patCopy ;
          break ;
+
      case wxINVERT:     // NOT dst
          if ( !backgroundTransparent )
      case wxINVERT:     // NOT dst
          if ( !backgroundTransparent )
-        {
              ::PenPat(GetQDGlobalsBlack(&blackColor));
              ::PenPat(GetQDGlobalsBlack(&blackColor));
-        }
          mode = patXor ;
          break ;
          mode = patXor ;
          break ;
+
      case wxXOR:        // src XOR dst
          mode = patXor ;
          break ;
      case wxXOR:        // src XOR dst
          mode = patXor ;
          break ;
+
      case wxOR_REVERSE: // src OR (NOT dst)
          mode = notPatOr ;
          break ;
      case wxOR_REVERSE: // src OR (NOT dst)
          mode = notPatOr ;
          break ;
+
      case wxSRC_INVERT: // (NOT src)
          mode = notPatCopy ;
          break ;
      case wxSRC_INVERT: // (NOT src)
          mode = notPatCopy ;
          break ;
+
      case wxAND: // src AND dst
          mode = adMin ;
          break ;
      case wxAND: // src AND dst
          mode = adMin ;
          break ;
-        // unsupported TODO
+
+    // TODO: unsupported
      case wxCLEAR:      // 0
      case wxAND_REVERSE:// src AND (NOT dst)
      case wxAND_INVERT: // (NOT src) AND dst
      case wxCLEAR:      // 0
      case wxAND_REVERSE:// src AND (NOT dst)
      case wxAND_INVERT: // (NOT src) AND dst
@@ -2040,7 +2231,11 @@ void wxDC::MacInstallBrush() const
          //        case wxSRC_OR:     // source _bitmap_ OR destination
          //        case wxSRC_AND:     // source _bitmap_ AND destination
          break ;
          //        case wxSRC_OR:     // source _bitmap_ OR destination
          //        case wxSRC_AND:     // source _bitmap_ AND destination
          break ;
+
+    default:
+        break ;
      }
      }
+
      ::PenMode( mode ) ;
      m_macBrushInstalled = true ;
      m_macPenInstalled = false ;
      ::PenMode( mode ) ;
      m_macBrushInstalled = true ;
      m_macPenInstalled = false ;