fixed operator[] in wxUSE_STL build

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

diff --git a/src/mac/carbon/dc.cpp b/src/mac/carbon/dc.cpp
index 3084e7224c6ca38171758d40d6a6449cc8dc5d45..719b77aecc495fda8c757984c4749e7f83c835d6 100644 (file)
--- a/src/mac/carbon/dc.cpp
+++ b/src/mac/carbon/dc.cpp
@@ -1,64 +1,62 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// 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

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

-#ifdef __GNUG__
-#pragma implementation "dc.h"
-#endif
+#include "wx/wxprec.h"

 
 #include "wx/dc.h"
 
 #include "wx/dc.h"

-#include "wx/app.h"
+
+#if !wxMAC_USE_CORE_GRAPHICS
+
+#ifndef WX_PRECOMP
+    #include "wx/log.h"
+    #include "wx/app.h"
+    #include "wx/dcmemory.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/dcprint.h"
 #include "wx/region.h"
 #include "wx/image.h"

-#include "wx/log.h"

 
 

-#if __MSL__ >= 0x6000
-#include "math.h"
-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>

-#include <FixMath.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
 //-----------------------------------------------------------------------------
 

-#define mm2inches        0.0393700787402
-#define inches2mm        25.4
-#define mm2twips        56.6929133859
-#define twips2mm        0.0176388888889
-#define mm2pt            2.83464566929
-#define pt2mm            0.352777777778
-#if !defined( __DARWIN__ ) || defined(__MWERKS__)
-#ifndef M_PI
-const double M_PI = 3.14159265358979 ;
-#endif
-#endif

 const double RAD2DEG  = 180.0 / M_PI;
 const short kEmulatedMode = -1 ;
 const short kUnsupportedMode = -2 ;
 
 extern TECObjectRef s_TECNativeCToUnicode ;
 
 const double RAD2DEG  = 180.0 / M_PI;
 const short kEmulatedMode = -1 ;
 const short kUnsupportedMode = -2 ;
 
 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 )


@@ -67,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) ;


@@ -76,24 +75,26 @@ wxMacPortSetter::~wxMacPortSetter()
 class wxMacFastPortSetter
 {
 public :
 class wxMacFastPortSetter
 {
 public :

-    wxMacFastPortSetter( const wxDC *dc ) 
+    wxMacFastPortSetter( const wxDC *dc )

     {
     {

-           wxASSERT( dc->Ok() ) ;
-           m_swapped = QDSwapPort( (GrafPtr) dc->m_macPort , &m_oldPort ) ;
-           m_clipRgn = NewRgn() ;
-           GetClip( m_clipRgn ) ;
-           m_dc = dc ;
-           dc->MacSetupPort( NULL ) ;
+        wxASSERT( dc->Ok() ) ;
+        m_swapped = QDSwapPort( (GrafPtr) dc->m_macPort , &m_oldPort ) ;
+        m_clipRgn = NewRgn() ;
+        GetClip( m_clipRgn ) ;
+        m_dc = dc ;
+        dc->MacSetupPort( NULL ) ;

     }
     }

+

     ~wxMacFastPortSetter()
     {
         // SetPort( (GrafPtr) m_dc->m_macPort ) ;
         SetClip( m_clipRgn ) ;
         if ( m_swapped )
     ~wxMacFastPortSetter()
     {
         // SetPort( (GrafPtr) m_dc->m_macPort ) ;
         SetClip( m_clipRgn ) ;
         if ( m_swapped )

-               SetPort( m_oldPort ) ;
-           m_dc->MacCleanupPort( NULL ) ;
-           DisposeRgn( m_clipRgn ) ;
+            SetPort( m_oldPort ) ;
+        m_dc->MacCleanupPort( NULL ) ;
+        DisposeRgn( m_clipRgn ) ;

     }
     }

+

 private :
     bool m_swapped ;
     RgnHandle m_clipRgn ;
 private :
     bool m_swapped ;
     RgnHandle m_clipRgn ;


@@ -112,21 +113,26 @@ wxMacWindowClipper::wxMacWindowClipper( const wxWindow* win ) :
     m_formerClip = NewRgn() ;
     m_newClip = NewRgn() ;
     GetClip( m_formerClip ) ;
     m_formerClip = NewRgn() ;
     m_newClip = NewRgn() ;
     GetClip( m_formerClip ) ;

-    
+

     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 ) ;
     }
 }
 

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

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


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

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

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

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

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


@@ -171,85 +177,101 @@ 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 ;

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

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

     m_mm_to_pix_x = mm2pt;
     m_mm_to_pix_y = mm2pt;
     m_internalDeviceOriginX = 0;
     m_mm_to_pix_x = mm2pt;
     m_mm_to_pix_y = mm2pt;
     m_internalDeviceOriginX = 0;


@@ -262,11 +284,10 @@ wxDC::wxDC()
     m_userScaleY = 1.0;
     m_scaleX = 1.0;
     m_scaleY = 1.0;
     m_userScaleY = 1.0;
     m_scaleX = 1.0;
     m_scaleY = 1.0;

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

     m_macPort = NULL ;
     m_macMask = NULL ;
     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 ;


@@ -278,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__
 #ifdef __WXDEBUG__

-    // needed to debug possible errors with two active drawing methods at the same time on 
+    // needed to debug possible errors with two active drawing methods at the same time on

     // the same DC
     m_macCurrentPortStateHelper = NULL ;
 #endif
     // 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 ;


@@ -301,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 ;


@@ -331,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 ;


@@ -342,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);


@@ -351,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 :


@@ -365,62 +397,55 @@ void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask
                     //m_logicalFunction == wxSRC_OR ? srcOr :
                     //m_logicalFunction == wxSRC_AND ? SRCAND :
                     srcCopy );
                     //m_logicalFunction == wxSRC_OR ? srcOr :
                     //m_logicalFunction == wxSRC_AND ? SRCAND :
                     srcCopy );

-     if ( bmp.GetBitmapType() == kMacBitmapTypePict ) {
-         Rect bitmaprect = { 0 , 0 , hh, ww };
-         ::OffsetRect( &bitmaprect, xx, yy ) ;
-         ::DrawPicture( (PicHandle) bmp.GetPict(), &bitmaprect ) ;
-    }
-     else if ( bmp.GetBitmapType() == kMacBitmapTypeGrafWorld )
+
+     GWorldPtr maskworld = NULL ;
+     GWorldPtr bmapworld = MAC_WXHBITMAP( bmp.GetHBITMAP((WXHBITMAP*)&maskworld) );
+     PixMapHandle bmappixels ;
+
+     // Set foreground and background colours (for bitmaps depth = 1)
+     if (bmp.GetDepth() == 1)

      {
      {

-         GWorldPtr    bmapworld = MAC_WXHBITMAP( bmp.GetHBITMAP() );
-         PixMapHandle bmappixels ;
-         // Set foreground and background colours (for bitmaps depth = 1)
-         if(bmp.GetDepth() == 1)
-        {
-             RGBColor fore = MAC_WXCOLORREF(m_textForegroundColour.GetPixel());
-             RGBColor back = MAC_WXCOLORREF(m_textBackgroundColour.GetPixel());
-             RGBForeColor(&fore);
-             RGBBackColor(&back);
-         }
-         else
-         {
-             RGBColor white = { 0xFFFF, 0xFFFF,0xFFFF} ;
-             RGBColor black = { 0,0,0} ;
-             RGBForeColor( &black ) ;
-             RGBBackColor( &white ) ;
-         }
-         bmappixels = GetGWorldPixMap( bmapworld ) ;
-         wxCHECK_RET(LockPixels(bmappixels),
-                     wxT("DoDrawBitmap:  Unable to lock pixels"));
-         Rect source = { 0, 0, h, w };
-         Rect dest   = { yy, xx, yy + hh, xx + ww };
-         if ( useMask && bmp.GetMask() )
+         RGBColor fore = MAC_WXCOLORREF(m_textForegroundColour.GetPixel());
+         RGBColor back = MAC_WXCOLORREF(m_textBackgroundColour.GetPixel());
+         RGBForeColor(&fore);
+         RGBBackColor(&back);
+     }
+     else
+     {
+         RGBColor white = { 0xFFFF, 0xFFFF, 0xFFFF } ;
+         RGBColor black = { 0, 0, 0 } ;
+         RGBForeColor( &black ) ;
+         RGBBackColor( &white ) ;
+     }
+     bmappixels = GetGWorldPixMap( bmapworld ) ;
+
+     wxCHECK_RET(LockPixels(bmappixels),
+                 wxT("wxDC::DoDrawBitmap - failed to lock pixels"));
+
+     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(bmp.GetMask()->GetMaskBitmap()))))
-             {
-                 CopyDeepMask
-                     (
-                      GetPortBitMapForCopyBits(bmapworld),
-                      GetPortBitMapForCopyBits(MAC_WXHBITMAP(bmp.GetMask()->GetMaskBitmap())),
-                      GetPortBitMapForCopyBits( MAC_WXHBITMAP(m_macPort) ),
-                      &source, &source, &dest, mode, NULL
-                      );
-                 UnlockPixels(GetGWorldPixMap(MAC_WXHBITMAP(bmp.GetMask()->GetMaskBitmap())));
-             }
-         }
-         else {
-             CopyBits( GetPortBitMapForCopyBits( bmapworld ),
-                       GetPortBitMapForCopyBits( MAC_WXHBITMAP(m_macPort) ),
-                       &source, &dest, mode, NULL ) ;
+             CopyDeepMask
+                 (
+                  GetPortBitMapForCopyBits(bmapworld),
+                  GetPortBitMapForCopyBits(MAC_WXHBITMAP(maskworld)),
+                  GetPortBitMapForCopyBits( MAC_WXHBITMAP(m_macPort) ),
+                  &source, &source, &dest, mode, NULL
+                  );
+             UnlockPixels(GetGWorldPixMap(MAC_WXHBITMAP(maskworld)));

          }
          }

-         UnlockPixels( bmappixels ) ;

      }
      }

-     else if ( bmp.GetBitmapType() == kMacBitmapTypeIcon )
+     else

      {
      {

-        Rect bitmaprect = { 0 , 0 , bmp.GetHeight(), bmp.GetWidth() } ;
-        OffsetRect( &bitmaprect, xx, yy ) ;
-        PlotCIconHandle( &bitmaprect , atNone , ttNone , MAC_WXHICON(bmp.GetHICON()) ) ;
+         CopyBits( GetPortBitMapForCopyBits( bmapworld ),
+                   GetPortBitMapForCopyBits( MAC_WXHBITMAP(m_macPort) ),
+                   &source, &dest, mode, NULL ) ;

      }
      }

+     UnlockPixels( bmappixels ) ;
+

      m_macPenInstalled = false ;
      m_macBrushInstalled = false ;
      m_macFontInstalled = false ;
      m_macPenInstalled = false ;
      m_macBrushInstalled = false ;
      m_macFontInstalled = false ;


@@ -428,31 +453,44 @@ 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"));
-    DoDrawBitmap( icon , x , y , icon.GetMask() != NULL ) ;
+    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);
+    wxCoord w = icon.GetWidth();
+    wxCoord h = icon.GetHeight();
+    wxCoord ww = XLOG2DEVREL(w);
+    wxCoord hh = YLOG2DEVREL(h);
+
+    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 )
 {
 }
 
 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
     {

-        m_clipping = TRUE;
+        m_clipping = true;

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


@@ -460,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 );


@@ -476,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 )


@@ -486,20 +521,19 @@ 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
         {

-            m_clipping = TRUE;
+            m_clipping = true;

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


@@ -511,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 ;
 }


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

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

         break;
         break;

+

     case wxMM_POINTS:
     case wxMM_POINTS:

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

         break;
         break;

+

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

+

     case wxMM_LOMETRIC:
     case wxMM_LOMETRIC:

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

         break;
         break;

+

     default:
     case wxMM_TEXT:
         SetLogicalScale( 1.0, 1.0 );
         break;
     }
     default:
     case wxMM_TEXT:
         SetLogicalScale( 1.0, 1.0 );
         break;
     }

+

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

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

     }
 }
 
     }
 }
 


@@ -584,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();
 }


@@ -611,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 ;
 }
 


@@ -626,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)
     {


@@ -637,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 ;


@@ -697,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))
         {


@@ -735,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)


@@ -743,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);

     }
 }
 
     }
 }
 


@@ -781,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 ;

-    if ( newAngle < 0 )
+
+    while ( newAngle > 360 )
+        newAngle -= 360 ;
+    while ( newAngle < 0 )

         newAngle += 360 ;
         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;


@@ -816,161 +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));

-    if( (xx1 > xx2) || (yy1 > yy2) ) {
-        alpha2 *= -1;
-    }
+    while ( alpha2 < 0 )
+        alpha2 += 360 ;
+    alpha2 = -alpha2 ;

     Rect r = { yyc - rad, xxc - rad, yyc + rad, xxc + rad };
     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() ;


@@ -978,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);


@@ -990,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() ;


@@ -1017,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() ;


@@ -1052,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 )
     if ( logical_func == wxNO_OP )

-        return TRUE ;
+        return true ;
+

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

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

     }
     }

+

     // 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)  ;


@@ -1083,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()) ) ;


@@ -1158,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 )


@@ -1183,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 )
                     {


@@ -1213,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 ) ;
                                 }
                             }


@@ -1229,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 ) ;
             }
         }


@@ -1243,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 )
                 {


@@ -1255,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 ) ;
                 }


@@ -1272,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 ) ;
                             }
                         }


@@ -1288,74 +1451,63 @@ 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;
-}
-
-#ifndef FixedToInt
-// as macro in FixMath.h for 10.3
-inline Fixed    IntToFixed( int inInt )
-{
-    return (((SInt32) inInt) << 16);
-}

 
 

-inline int    FixedToInt( Fixed inFixed )
-{
-    return (((SInt32) inFixed) >> 16);
+    return true;

 }
 }

-#endif

 
 

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

                               double angle)
 {
                               double angle)
 {

-    wxCHECK_RET( Ok(), wxT("wxDC::DoDrawRotatedText  Invalid window dc") );
-/*
-    if (angle == 0.0 )
-    {
-        DrawText(str, x, y);
-        return;
-    }
+    // TODO: support text background color (only possible by hand, ATSUI does not support it)
+    wxCHECK_RET( Ok(), wxT("wxDC::DoDrawRotatedText - invalid DC") );

 
 

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

         return ;
         return ;

-*/      
+

     wxMacFastPortSetter helper(this) ;
     MacInstallFont() ;
 
     wxMacFastPortSetter helper(this) ;
     MacInstallFont() ;
 

+#if 0

     if ( 0 )
     {
         m_macFormerAliasState = IsAntiAliasedTextEnabled(&m_macFormerAliasSize);
         SetAntiAliasedTextEnabled(true, SInt16(m_scaleY * m_font.MacGetFontSize()));
         m_macAliasWasEnabled = true ;
     }
     if ( 0 )
     {
         m_macFormerAliasState = IsAntiAliasedTextEnabled(&m_macFormerAliasSize);
         SetAntiAliasedTextEnabled(true, SInt16(m_scaleY * m_font.MacGetFontSize()));
         m_macAliasWasEnabled = true ;
     }

+#endif
+

     OSStatus status = noErr ;
     ATSUTextLayout atsuLayout ;
     OSStatus status = noErr ;
     ATSUTextLayout atsuLayout ;

-    UniCharCount chars = str.Length() ;
+    UniCharCount chars = str.length() ;

     UniChar* ubuf = NULL ;
     UniChar* ubuf = NULL ;

+

 #if SIZEOF_WCHAR_T == 4
 #if SIZEOF_WCHAR_T == 4

-       wxMBConvUTF16BE converter ;
+    wxMBConvUTF16 converter ;

 #if wxUSE_UNICODE
 #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 ) ;
+    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 ) ;
 #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 ) ;
+    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
 #endif
     chars = unicharlen / 2 ;
 #else


@@ -1372,19 +1524,21 @@ void  wxDC::DoDrawRotatedText(const wxString& str, wxCoord x, wxCoord y,
         &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 ;
+

     if ( abs(iAngle) > 0 )
     {
         Fixed atsuAngle = IntToFixed( iAngle ) ;
     if ( abs(iAngle) > 0 )
     {
         Fixed atsuAngle = IntToFixed( iAngle ) ;

+

         ATSUAttributeTag atsuTags[] =
         {
             kATSULineRotationTag ,
         ATSUAttributeTag atsuTags[] =
         {
             kATSULineRotationTag ,


@@ -1397,76 +1551,80 @@ void  wxDC::DoDrawRotatedText(const wxString& str, wxCoord x, wxCoord y,
         {
             &atsuAngle ,
         } ;
         {
             &atsuAngle ,
         } ;

+

         status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags)/sizeof(ATSUAttributeTag),
             atsuTags, atsuSizes, atsuValues ) ;
     }
         status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags)/sizeof(ATSUAttributeTag),
             atsuTags, atsuSizes, atsuValues ) ;
     }

+

     status = ::ATSUMeasureText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
         &textBefore , &textAfter, &ascent , &descent );
     status = ::ATSUMeasureText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
         &textBefore , &textAfter, &ascent , &descent );

-    
-    drawX += (int)(sin(angle/RAD2DEG) * FixedToInt(ascent));
-    drawY += (int)(cos(angle/RAD2DEG) * FixedToInt(ascent));
+
+    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
 }
 
 #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"));
+    wxCHECK_RET(Ok(), wxT("wxDC::DoDrawText - invalid DC"));

 
     wxMacFastPortSetter helper(this) ;
     long xx = XLOG2DEVMAC(x);
     long yy = YLOG2DEVMAC(y);
 
     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
 #if TARGET_CARBON
     bool useDrawThemeText = ( DrawThemeTextBox != (void*) kUnresolvedCFragSymbolAddress ) ;
     if ( UMAGetSystemVersion() < 0x1000 || IsKindOf(CLASSINFO( wxPrinterDC ) ) || m_font.GetNoAntiAliasing() )
         useDrawThemeText = false ;
 #endif

+

     MacInstallFont() ;
     MacInstallFont() ;

-    if ( 0 )
-    {
-        m_macFormerAliasState = IsAntiAliasedTextEnabled(&m_macFormerAliasSize);
-        SetAntiAliasedTextEnabled(true, 8);
-        m_macAliasWasEnabled = true ;
-    }
+

     FontInfo fi ;
     ::GetFontInfo( &fi ) ;
     FontInfo fi ;
     ::GetFontInfo( &fi ) ;

+

 #if TARGET_CARBON
     if ( !useDrawThemeText )
 #if TARGET_CARBON
     if ( !useDrawThemeText )

-#endif

         yy += fi.ascent ;
         yy += fi.ascent ;

+#else
+    yy += fi.ascent ;
+#endif
+
+    ::TextMode( (m_backgroundMode == wxTRANSPARENT) ? srcOr : srcCopy ) ;

     ::MoveTo( xx , yy );
     ::MoveTo( xx , yy );

-    if (  m_backgroundMode == wxTRANSPARENT )
-    {
-        ::TextMode( srcOr) ;
-    }
-    else
-    {
-        ::TextMode( srcCopy ) ;
-    }
+

     int line = 0 ;
     {
         wxString linetext = strtext ;
     int line = 0 ;
     {
         wxString linetext = strtext ;

+

 #if TARGET_CARBON
         if ( useDrawThemeText )
         {
 #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 } ;
+            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 )
             {
             wxMacCFStringHolder mString( linetext , m_font.GetEncoding()) ;
 
             if ( m_backgroundMode != wxTRANSPARENT )
             {

-                Point bounds={0,0} ;
+                Point bounds = {0, 0} ;

                 Rect background = frame ;
                 SInt16 baseline ;
                 ::GetThemeTextDimensions( mString,
                 Rect background = frame ;
                 SInt16 baseline ;
                 ::GetThemeTextDimensions( mString,


@@ -1479,43 +1637,39 @@ void  wxDC::DoDrawText(const wxString& strtext, wxCoord x, wxCoord y)
                 background.bottom = background.top + bounds.v ;
                 ::EraseRect( &background ) ;
             }
                 background.bottom = background.top + bounds.v ;
                 ::EraseRect( &background ) ;
             }

+

             ::DrawThemeTextBox( mString,
                 m_font.MacGetThemeFontID() ,
                 kThemeStateActive,
                 false,
                 &frame,
                 teJustLeft,
             ::DrawThemeTextBox( mString,
                 m_font.MacGetThemeFontID() ,
                 kThemeStateActive,
                 false,
                 &frame,
                 teJustLeft,

-                nil );
+                NULL );

         }
         else
 #endif
         {
         }
         else
 #endif
         {

-            wxCharBuffer text = linetext.mb_str(wxConvLocal) ; 
-            if ( m_backgroundMode != wxTRANSPARENT )
-            {
-                Rect frame = { yy - fi.ascent + line*(fi.descent + fi.ascent + fi.leading)  ,xx , yy - fi.ascent + (line+1)*(fi.descent + fi.ascent + fi.leading) , xx + 10000 } ;
-                short width = ::TextWidth( text , 0 , strlen(text) ) ;
-                frame.right = frame.left + width ;
-
-                ::EraseRect( &frame ) ;
-            }
+            wxCharBuffer text = linetext.mb_str(wxConvLocal) ;

             ::DrawText( text , 0 , strlen(text) ) ;
          }
     }
             ::DrawText( text , 0 , strlen(text) ) ;
          }
     }

+

     ::TextMode( srcOr ) ;
 }
 
     ::TextMode( srcOr ) ;
 }
 

-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 &strtext, 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 )


@@ -1523,14 +1677,17 @@ 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() ;
     FontInfo fi ;
     ::GetFontInfo( &fi ) ;
     MacInstallFont() ;
     FontInfo fi ;
     ::GetFontInfo( &fi ) ;

-#if TARGET_CARBON    
+
+#if TARGET_CARBON

     bool useGetThemeText = ( GetThemeTextDimensions != (void*) kUnresolvedCFragSymbolAddress ) ;
     if ( UMAGetSystemVersion() < 0x1000 || IsKindOf(CLASSINFO( wxPrinterDC ) ) || ((wxFont*)&m_font)->GetNoAntiAliasing() )
         useGetThemeText = false ;
 #endif
     bool useGetThemeText = ( GetThemeTextDimensions != (void*) kUnresolvedCFragSymbolAddress ) ;
     if ( UMAGetSystemVersion() < 0x1000 || IsKindOf(CLASSINFO( wxPrinterDC ) ) || ((wxFont*)&m_font)->GetNoAntiAliasing() )
         useGetThemeText = false ;
 #endif

+

     if ( height )
         *height = YDEV2LOGREL( fi.descent + fi.ascent ) ;
     if ( descent )
     if ( height )
         *height = YDEV2LOGREL( fi.descent + fi.ascent ) ;
     if ( descent )


@@ -1546,7 +1703,7 @@ void  wxDC::DoGetTextExtent( const wxString &strtext, wxCoord *width, wxCoord *h
 
         if ( useGetThemeText )
         {
 
         if ( useGetThemeText )
         {

-            Point bounds={0,0} ;
+            Point bounds = {0, 0} ;

             SInt16 baseline ;
             wxMacCFStringHolder mString( linetext , m_font.GetEncoding() ) ;
             ThemeFontID themeFont = m_font.MacGetThemeFontID() ;
             SInt16 baseline ;
             wxMacCFStringHolder mString( linetext , m_font.GetEncoding() ) ;
             ThemeFontID themeFont = m_font.MacGetThemeFontID() ;


@@ -1560,12 +1717,14 @@ void  wxDC::DoGetTextExtent( const wxString &strtext, wxCoord *width, wxCoord *h
         }
         else
         {
         }
         else
         {

-            wxCharBuffer text = linetext.mb_str(wxConvLocal) ;  
+            wxCharBuffer text = linetext.mb_str(wxConvLocal) ;

             curwidth = ::TextWidth( text , 0 , strlen(text) ) ;
         }
             curwidth = ::TextWidth( text , 0 , strlen(text) ) ;
         }

+

         if ( curwidth > *width )
             *width = XDEV2LOGREL( curwidth ) ;
     }
         if ( curwidth > *width )
             *width = XDEV2LOGREL( curwidth ) ;
     }

+

     if ( theFont )
     {
         // work around the constness
     if ( theFont )
     {
         // work around the constness


@@ -1574,20 +1733,20 @@ 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;
         return false;

-    
+

     wxMacFastPortSetter helper(this) ;
     MacInstallFont() ;
     wxMacFastPortSetter helper(this) ;
     MacInstallFont() ;

-#if TARGET_CARBON    
+
+#if TARGET_CARBON

     bool useGetThemeText = ( GetThemeTextDimensions != (void*) kUnresolvedCFragSymbolAddress ) ;
     if ( UMAGetSystemVersion() < 0x1000 || IsKindOf(CLASSINFO( wxPrinterDC ) ) || ((wxFont*)&m_font)->GetNoAntiAliasing() )
         useGetThemeText = false ;
     bool useGetThemeText = ( GetThemeTextDimensions != (void*) kUnresolvedCFragSymbolAddress ) ;
     if ( UMAGetSystemVersion() < 0x1000 || IsKindOf(CLASSINFO( wxPrinterDC ) ) || ((wxFont*)&m_font)->GetNoAntiAliasing() )
         useGetThemeText = false ;


@@ -1597,14 +1756,15 @@ bool wxDC::DoGetPartialTextExtents(const wxString& text, wxArrayInt& widths) con
         // 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
         // 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
+        // beginning of the string for each succeeding substring, which is much

         // slower than this should be.
         // slower than this should be.

-        for (size_t i=0; i<text.Length(); i++)
+        for (size_t i=0; i<text.length(); i++)

         {
         {

-            wxString str(text.Left(i+1));
-            Point bounds = {0,0};
+            wxString str(text.Left(i + 1));
+            Point bounds = {0, 0};

             SInt16 baseline ;
             wxMacCFStringHolder mString(str, m_font.GetEncoding());
             SInt16 baseline ;
             wxMacCFStringHolder mString(str, m_font.GetEncoding());

+

             ::GetThemeTextDimensions( mString,
                                       m_font.MacGetThemeFontID(),
                                       kThemeStateActive,
             ::GetThemeTextDimensions( mString,
                                       m_font.MacGetThemeFontID(),
                                       kThemeStateActive,


@@ -1614,7 +1774,7 @@ bool wxDC::DoGetPartialTextExtents(const wxString& text, wxArrayInt& widths) con
             widths[i] = XDEV2LOGREL(bounds.h);
         }
     }
             widths[i] = XDEV2LOGREL(bounds.h);
         }
     }

-    else        
+    else

 #endif
     {
         wxCharBuffer buff = text.mb_str(wxConvLocal);
 #endif
     {
         wxCharBuffer buff = text.mb_str(wxConvLocal);


@@ -1624,34 +1784,34 @@ bool wxDC::DoGetPartialTextExtents(const wxString& text, wxArrayInt& widths) con
 
         // Copy to widths, starting at measurements[1]
         // NOTE: this doesn't take into account any multi-byte characters
 
         // 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]);
+        // in buff, it probably should...
+        for (size_t i=0; i<text.length(); i++)
+            widths[i] = XDEV2LOGREL(measurements[i + 1]);

 
 

-        delete [] measurements;        
+        delete [] measurements;

     }
 
     return true;
 }
 
     }
 
     return true;
 }
 

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

 {
 {

-    wxCHECK_MSG(Ok(), 1, wxT("Invalid DC"));
+    wxCHECK_MSG(Ok(), 1, wxT("wxDC::GetCharWidth - invalid DC"));
+

     wxMacFastPortSetter helper(this) ;
     wxMacFastPortSetter helper(this) ;

-    MacInstallFont() ;

     int width = 0 ;
     int width = 0 ;

+    const char text[] = "g" ;
+
+    MacInstallFont() ;
+

 #if TARGET_CARBON
     bool useGetThemeText = ( GetThemeTextDimensions != (void*) kUnresolvedCFragSymbolAddress ) ;
     if ( UMAGetSystemVersion() < 0x1000 || ((wxFont*)&m_font)->GetNoAntiAliasing() )
         useGetThemeText = false ;
 #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 )
     {
     if ( useGetThemeText )
     {

-        Point bounds={0,0} ;
+        Point bounds = {0, 0} ;

         SInt16 baseline ;
         CFStringRef mString = CFStringCreateWithBytes( NULL , (UInt8*) text , 1 , CFStringGetSystemEncoding(), false ) ;
         ::GetThemeTextDimensions( mString,
         SInt16 baseline ;
         CFStringRef mString = CFStringCreateWithBytes( NULL , (UInt8*) text , 1 , CFStringGetSystemEncoding(), false ) ;
         ::GetThemeTextDimensions( mString,


@@ -1668,24 +1828,29 @@ wxCoord   wxDC::GetCharWidth(void) const
     {
         width = ::TextWidth( text , 0 , 1 ) ;
     }
     {
         width = ::TextWidth( text , 0 , 1 ) ;
     }

+

     return YDEV2LOGREL(width) ;
 }
 
     return YDEV2LOGREL(width) ;
 }
 

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

 {
 {

-    wxCHECK_MSG(Ok(), 1, wxT("Invalid DC"));
+    wxCHECK_MSG(Ok(), 1, wxT("wxDC::GetCharHeight - invalid DC"));
+

     wxMacFastPortSetter helper(this) ;
     MacInstallFont() ;
     FontInfo fi ;
     ::GetFontInfo( &fi ) ;
     wxMacFastPortSetter helper(this) ;
     MacInstallFont() ;
     FontInfo fi ;
     ::GetFontInfo( &fi ) ;

+

     return YDEV2LOGREL( fi.descent + fi.ascent );
 }
 
     return YDEV2LOGREL( fi.descent + fi.ascent );
 }
 

-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() ;


@@ -1697,67 +1862,61 @@ 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) ;
     Pattern blackColor ;
     MacSetupBackgroundForCurrentPort(m_backgroundBrush) ;

-    if ( m_font.Ok() )
-    {
-        ::TextFont( m_font.MacGetFontNum() ) ;
-        ::TextSize( (short)(m_scaleY * m_font.MacGetFontSize()) ) ;
-        ::TextFace( m_font.MacGetFontStyle() ) ;
-        m_macFontInstalled = true ;
-        m_macBrushInstalled = false ;
-        m_macPenInstalled = false ;
-        RGBColor forecolor = MAC_WXCOLORREF( m_textForegroundColour.GetPixel());
-        RGBColor backcolor = MAC_WXCOLORREF( m_textBackgroundColour.GetPixel());
-        ::RGBForeColor( &forecolor );
-        ::RGBBackColor( &backcolor );
-    }
-    else
+    if ( m_backgroundMode != wxTRANSPARENT )

     {
     {

-        FontFamilyID fontId ;
-        Str255 fontName ;
-        SInt16 fontSize ;
-        Style fontStyle ;
-        GetThemeFont(kThemeSmallSystemFont , GetApplicationScript() , fontName , &fontSize , &fontStyle ) ;
-        GetFNum( fontName, &fontId );
-        ::TextFont( fontId ) ;
-        ::TextSize( short(m_scaleY * fontSize) ) ;
-        ::TextFace( fontStyle ) ;
-        // todo reset after spacing changes - or store the current spacing somewhere
-        m_macFontInstalled = true ;
-        m_macBrushInstalled = false ;
-        m_macPenInstalled = false ;
-        RGBColor forecolor = MAC_WXCOLORREF( m_textForegroundColour.GetPixel());
-        RGBColor backcolor = MAC_WXCOLORREF( m_textBackgroundColour.GetPixel());
-        ::RGBForeColor( &forecolor );
-        ::RGBBackColor( &backcolor );
+        Pattern whiteColor ;
+        ::BackPat(GetQDGlobalsWhite(&whiteColor));

     }
     }

+
+    wxASSERT( m_font.Ok() ) ;
+
+    ::TextFont( m_font.MacGetFontNum() ) ;
+    ::TextSize( (short)(m_scaleY * m_font.MacGetFontSize()) ) ;
+    ::TextFace( m_font.MacGetFontStyle() ) ;
+    m_macFontInstalled = true ;
+    m_macBrushInstalled = false ;
+    m_macPenInstalled = false ;
+    RGBColor forecolor = MAC_WXCOLORREF( m_textForegroundColour.GetPixel());
+    RGBColor backcolor = MAC_WXCOLORREF( m_textBackgroundColour.GetPixel());
+    ::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


@@ -1771,64 +1930,39 @@ 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 ;
     ::PenMode( mode ) ;
     OSStatus status = noErr ;

+    status = ATSUCreateAndCopyStyle( (ATSUStyle) m_font.MacGetATSUStyle() , (ATSUStyle*) &m_macATSUIStyle ) ;
+    wxASSERT_MSG( status == noErr , wxT("couldn't set create ATSU style") ) ;
+

     Fixed atsuSize = IntToFixed( int(m_scaleY * m_font.MacGetFontSize()) ) ;
     Fixed atsuSize = IntToFixed( int(m_scaleY * m_font.MacGetFontSize()) ) ;

-    Style qdStyle = m_font.MacGetATSUAdditionalQDStyles() ;
-    ATSUFontID    atsuFont = m_font.MacGetATSUFontID() ;
-    status = ::ATSUCreateStyle((ATSUStyle *)&m_macATSUIStyle) ;
-    wxASSERT_MSG( status == noErr , wxT("couldn't create ATSU style") ) ;

     ATSUAttributeTag atsuTags[] =
     {
     ATSUAttributeTag atsuTags[] =
     {

-        kATSUFontTag ,

             kATSUSizeTag ,
             kATSUSizeTag ,

-            //        kATSUColorTag ,
-            //        kATSUBaselineClassTag ,
-            kATSUVerticalCharacterTag,
-            kATSUQDBoldfaceTag ,
-            kATSUQDItalicTag ,
-            kATSUQDUnderlineTag ,
-            kATSUQDCondensedTag ,
-            kATSUQDExtendedTag ,

     } ;
     ByteCount atsuSizes[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
     {
     } ;
     ByteCount atsuSizes[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
     {

-        sizeof( ATSUFontID ) ,

             sizeof( Fixed ) ,
             sizeof( Fixed ) ,

-            //        sizeof( RGBColor ) ,
-            //        sizeof( BslnBaselineClass ) ,
-            sizeof( ATSUVerticalCharacterType),
-            sizeof( Boolean ) ,
-            sizeof( Boolean ) ,
-            sizeof( Boolean ) ,
-            sizeof( Boolean ) ,
-            sizeof( Boolean ) ,

     } ;
     } ;

-    Boolean kTrue = true ;
-    Boolean kFalse = false ;
-    //BslnBaselineClass kBaselineDefault = kBSLNHangingBaseline ;
-    ATSUVerticalCharacterType kHorizontal = kATSUStronglyHorizontal;

     ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
     {
     ATSUAttributeValuePtr    atsuValues[sizeof(atsuTags)/sizeof(ATSUAttributeTag)] =
     {

-        &atsuFont ,

             &atsuSize ,
             &atsuSize ,

-            //        &MAC_WXCOLORREF( m_textForegroundColour.GetPixel() ) ,
-            //        &kBaselineDefault ,
-            &kHorizontal,
-            (qdStyle & bold) ? &kTrue : &kFalse ,
-            (qdStyle & italic) ? &kTrue : &kFalse ,
-            (qdStyle & underline) ? &kTrue : &kFalse ,
-            (qdStyle & condense) ? &kTrue : &kFalse ,
-            (qdStyle & extend) ? &kTrue : &kFalse ,

     } ;
     } ;

+

     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 set create 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 } } ,


@@ -1836,6 +1970,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


@@ -1846,8 +1981,8 @@ 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;
         case wxFDIAGONAL_HATCH:     index = 2; break;
         // hatches
         case wxBDIAGONAL_HATCH:     index = 1; break;
         case wxFDIAGONAL_HATCH:     index = 2; break;


@@ -1855,36 +1990,44 @@ 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];    
+
+    *pattern = gPatterns[index];

 }
 
 void wxDC::MacInstallPen() const
 {
 }
 
 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)
     if (penStyle == wxUSER_DASH)

-    {   
+    {

         // FIXME: there should be exactly 8 items in the dash
         wxDash* dash ;
         int number = m_pen.GetDashes(&dash) ;
         // FIXME: there should be exactly 8 items in the dash
         wxDash* dash ;
         int number = m_pen.GetDashes(&dash) ;


@@ -1902,30 +2045,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


@@ -1939,7 +2088,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 ;


@@ -1949,29 +2102,30 @@ 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()) );
                 int brushStyle = background.GetStyle();
                 if (brushStyle == wxSOLID)
                     ::BackPat(GetQDGlobalsWhite(&whiteColor));
             case kwxMacBrushColour :
             {
                 ::RGBBackColor( &MAC_WXCOLORREF( background.GetColour().GetPixel()) );
                 int brushStyle = background.GetStyle();
                 if (brushStyle == wxSOLID)
                     ::BackPat(GetQDGlobalsWhite(&whiteColor));

-                else if (IS_HATCH(brushStyle))
+                else if (background.IsHatch())

                 {
                     Pattern pat ;
                     wxMacGetPattern(brushStyle, &pat);
                 {
                     Pattern pat ;
                     wxMacGetPattern(brushStyle, &pat);


@@ -1981,8 +2135,11 @@ void wxDC::MacSetupBackgroundForCurrentPort(const wxBrush& background )
                 {
                     ::BackPat(GetQDGlobalsWhite(&whiteColor));
                 }
                 {
                     ::BackPat(GetQDGlobalsWhite(&whiteColor));
                 }

-                break ;

             }
             }

+                break ;
+
+            default:
+                break ;

         }
     }
 }
         }
     }
 }


@@ -1990,21 +2147,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 (IS_HATCH(brushStyle))
+    else if (m_brush.IsHatch())

     {
         Pattern pat ;
     {
         Pattern pat ;

+

         wxMacGetPattern(brushStyle, &pat);
         ::PenPat(&pat);
     }
         wxMacGetPattern(brushStyle, &pat);
         ::PenPat(&pat);
     }


@@ -2012,15 +2186,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 )
         if ( m_brush.GetStyle() == wxSTIPPLE )

-            gw = MAC_WXHBITMAP(bitmap->GetHBITMAP())  ;
+            gw = MAC_WXHBITMAP(bitmap->GetHBITMAP(NULL))  ;

         else
         else

-            gw = MAC_WXHBITMAP(bitmap->GetMask()->GetMaskBitmap()) ;
+            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 ) ;


@@ -2029,6 +2206,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()) );


@@ -2037,10 +2215,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 ) ;
         }


@@ -2050,6 +2230,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) *


@@ -2057,6 +2238,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) ;


@@ -2073,17 +2255,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 )


@@ -2091,26 +2276,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


@@ -2124,7 +2313,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 ;


@@ -2174,3 +2367,5 @@ wxCoord wxDCBase::LogicalToDeviceYRel(wxCoord y) const
 {
     return ((wxDC *)this)->YLOG2DEVREL(y);
 }
 {
     return ((wxDC *)this)->YLOG2DEVREL(y);
 }

+
+#endif // !wxMAC_USE_CORE_GRAPHICS