fix warnings about unused parameters/variables in release build

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@52690 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/include/wx/mac/carbon/private.h b/include/wx/mac/carbon/private.h
index 69ac83f076a6c82f9d8dc733614023a1285d99de..1964d366fbb7e0d2f77556577974cf420a605610 100644 (file)
--- a/include/wx/mac/carbon/private.h
+++ b/include/wx/mac/carbon/private.h
@@ -269,13 +269,13 @@ private :
     PicHandle    m_backpict ;
     bool        m_shown ;
     long        m_mark ;
     PicHandle    m_backpict ;
     bool        m_shown ;
     long        m_mark ;

-#if wxUSE_TIMER 
+#if wxUSE_TIMER

     wxMacToolTipTimer* m_timer ;
 #endif
     wxCFStringRef m_helpTextRef ;
 } ;
 
     wxMacToolTipTimer* m_timer ;
 #endif
     wxCFStringRef m_helpTextRef ;
 } ;
 

-// Quartz 
+// Quartz

 
 WXDLLIMPEXP_CORE void wxMacCreateBitmapButton( ControlButtonContentInfo*info , const wxBitmap& bitmap , int forceType = 0 );
 WXDLLIMPEXP_CORE void wxMacReleaseBitmapButton( ControlButtonContentInfo*info );
 
 WXDLLIMPEXP_CORE void wxMacCreateBitmapButton( ControlButtonContentInfo*info , const wxBitmap& bitmap , int forceType = 0 );
 WXDLLIMPEXP_CORE void wxMacReleaseBitmapButton( ControlButtonContentInfo*info );


@@ -323,7 +323,7 @@ WXDLLIMPEXP_CORE Rect wxMacGetBoundsForControl( wxWindow* window , const wxPoint
 
 ControlActionUPP GetwxMacLiveScrollbarActionProc();
 
 
 ControlActionUPP GetwxMacLiveScrollbarActionProc();
 

-// additional optional event defines 
+// additional optional event defines

 
 #if MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_X_VERSION_10_5
 enum {
 
 #if MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_X_VERSION_10_5
 enum {


@@ -419,9 +419,12 @@ public :
         T value;
         OSStatus err = GetData<T>( inPartCode , inTag , &value );
 
         T value;
         OSStatus err = GetData<T>( inPartCode , inTag , &value );
 

-        wxASSERT_MSG( err == noErr,
-                      wxString::Format(wxT("GetData Failed for Part [%i] and Tag [%i]"),
-                                       inPartCode, (int)inTag) );
+        if ( err != noErr )
+        {
+            wxFAIL_MSG( wxString::Format(wxT("GetData Failed for Part [%i] and Tag [%i]"),
+                                        inPartCode, (int)inTag) );
+        }
+

         return value;
     }
     template <typename T> OSStatus GetData( ResType inTag , T *data ) const
         return value;
     }
     template <typename T> OSStatus GetData( ResType inTag , T *data ) const

diff --git a/src/mac/carbon/bitmap.cpp b/src/mac/carbon/bitmap.cpp
index 7fd7536644e8a2d081a4fa90009a3fc7f70114ec..19334a0f7c88909ce1efc6fbdc03509a30680821 100644 (file)
--- a/src/mac/carbon/bitmap.cpp
+++ b/src/mac/carbon/bitmap.cpp
@@ -565,8 +565,9 @@ IconRef wxBitmapRefData::GetIconRef()
         HLock((Handle) iconFamily);
         OSStatus err = GetIconRefFromIconFamilyPtr( *iconFamily, GetHandleSize((Handle) iconFamily), &m_iconRef );
         HUnlock((Handle) iconFamily);
         HLock((Handle) iconFamily);
         OSStatus err = GetIconRefFromIconFamilyPtr( *iconFamily, GetHandleSize((Handle) iconFamily), &m_iconRef );
         HUnlock((Handle) iconFamily);

-        wxASSERT_MSG( err == noErr , wxT("Error when constructing icon ref") );

         DisposeHandle( (Handle) iconFamily ) ;
         DisposeHandle( (Handle) iconFamily ) ;

+
+        wxCHECK_MSG( err == noErr, NULL, wxT("Error when constructing icon ref") );

     }
 
     return m_iconRef ;
     }
 
     return m_iconRef ;

diff --git a/src/mac/carbon/fontdlg.cpp b/src/mac/carbon/fontdlg.cpp
index 7c037fc8b42fa5689e23b15fb5ffbc91e9495f28..e7892ad5426ae1b4b81a7f1ccfdd93a3dd298897 100644 (file)
--- a/src/mac/carbon/fontdlg.cpp
+++ b/src/mac/carbon/fontdlg.cpp
@@ -66,7 +66,7 @@ wxMacCarbonFontPanelHandler(EventHandlerCallRef WXUNUSED(nextHandler),
     OSStatus result = eventNotHandledErr ;
     wxFontDialog *fontdialog = (wxFontDialog*) userData ;
     wxFontData& fontdata= fontdialog->GetFontData() ;
     OSStatus result = eventNotHandledErr ;
     wxFontDialog *fontdialog = (wxFontDialog*) userData ;
     wxFontData& fontdata= fontdialog->GetFontData() ;

-    
+

     wxMacCarbonEvent cEvent( event );
     switch(cEvent.GetKind())
     {
     wxMacCarbonEvent cEvent( event );
     switch(cEvent.GetKind())
     {


@@ -92,13 +92,16 @@ wxMacCarbonFontPanelHandler(EventHandlerCallRef WXUNUSED(nextHandler),
             {
                 FMFontStyle fontStyle = cEvent.GetParameter<FMFontStyle>(kEventParamFMFontStyle);
                 FMFontSize fontSize = cEvent.GetParameter<FMFontSize>(kEventParamFMFontSize);
             {
                 FMFontStyle fontStyle = cEvent.GetParameter<FMFontStyle>(kEventParamFMFontStyle);
                 FMFontSize fontSize = cEvent.GetParameter<FMFontSize>(kEventParamFMFontSize);

-                
+

                 CFStringRef cfName = NULL;
 #if 1
                 FMFontFamily fontFamily = cEvent.GetParameter<FMFontFamily>(kEventParamFMFontFamily);
                 ATSFontFamilyRef atsfontfamilyref = FMGetATSFontFamilyRefFromFontFamily( fontFamily ) ;
                 OSStatus err = ATSFontFamilyGetName( atsfontfamilyref , kATSOptionFlagsDefault , &cfName ) ;
                 CFStringRef cfName = NULL;
 #if 1
                 FMFontFamily fontFamily = cEvent.GetParameter<FMFontFamily>(kEventParamFMFontFamily);
                 ATSFontFamilyRef atsfontfamilyref = FMGetATSFontFamilyRefFromFontFamily( fontFamily ) ;
                 OSStatus err = ATSFontFamilyGetName( atsfontfamilyref , kATSOptionFlagsDefault , &cfName ) ;

-                wxASSERT_MSG( err == noErr , wxT("ATSFontFamilyGetName failed") );
+                if ( err == noErr )
+                {
+                    wxFAIL_MSG("ATSFontFamilyGetName failed");
+                }

 #else
                 // we don't use the ATSU naming anymore
                 ByteCount actualLength = 0;
 #else
                 // we don't use the ATSU naming anymore
                 ByteCount actualLength = 0;


@@ -140,7 +143,7 @@ wxMacCarbonFontPanelHandler(EventHandlerCallRef WXUNUSED(nextHandler),
                 }
             }
 #endif // wxMAC_USE_ATSU_TEXT
                 }
             }
 #endif // wxMAC_USE_ATSU_TEXT

-            
+

             // retrieving the color
             RGBColor fontColor ;
             if ( cEvent.GetParameter<RGBColor>(kEventParamFontColor, &fontColor) == noErr )
             // retrieving the color
             RGBColor fontColor ;
             if ( cEvent.GetParameter<RGBColor>(kEventParamFontColor, &fontColor) == noErr )


@@ -181,9 +184,9 @@ wxMacCarbonFontPanelHandler(EventHandlerCallRef WXUNUSED(nextHandler),
                 }
             }
         }
                 }
             }
         }

-            break ;
-    } ;
-    
+        break ;
+    }
+

     return result ;
 }
 
     return result ;
 }
 

diff --git a/src/mac/carbon/fontdlgosx.mm b/src/mac/carbon/fontdlgosx.mm
index 27f4892485a72da50b032fa3b1dcff9e7d1c46fb..55071430b2d9495313fe57fa0026edf1c1947914 100644 (file)
--- a/src/mac/carbon/fontdlgosx.mm
+++ b/src/mac/carbon/fontdlgosx.mm
@@ -141,8 +141,10 @@ int RunMixedFontDialog(wxFontDialog* WXUNUSED(dialog))
 {
     int retval = wxID_CANCEL ;
 
 {
     int retval = wxID_CANCEL ;
 

-    bool cocoaLoaded = NSApplicationLoad();
-    wxASSERT_MSG(cocoaLoaded,wxT("Couldn't load Cocoa in Carbon Environment")) ;
+    if ( !NSApplicationLoad() )
+    {
+        wxFAIL_MSG("Couldn't load Cocoa in Carbon Environment");
+    }

 
     wxAutoNSAutoreleasePool pool;
 
 
     wxAutoNSAutoreleasePool pool;
 

diff --git a/src/mac/carbon/gauge.cpp b/src/mac/carbon/gauge.cpp
index 3793d8ed604195d7d9853c474c8bab829f0bcf81..b2f36abe9574f69147b0b54f7f9c048ade46d667 100644 (file)
--- a/src/mac/carbon/gauge.cpp
+++ b/src/mac/carbon/gauge.cpp
@@ -11,7 +11,7 @@
 
 #include "wx/wxprec.h"
 
 
 #include "wx/wxprec.h"
 

-#if wxUSE_GAUGE 
+#if wxUSE_GAUGE

 
 #include "wx/gauge.h"
 
 
 #include "wx/gauge.h"
 


@@ -51,40 +51,40 @@ bool wxGauge::Create( wxWindow *parent,
         m_peer->SetData<Boolean>( kControlEntireControl, kControlProgressBarAnimatingTag, (Boolean)false );
 
     MacPostControlCreate( pos, size );
         m_peer->SetData<Boolean>( kControlEntireControl, kControlProgressBarAnimatingTag, (Boolean)false );
 
     MacPostControlCreate( pos, size );

-    
+

     return true;
 }
 
 void wxGauge::SetRange(int r)
 {
     return true;
 }
 
 void wxGauge::SetRange(int r)
 {

-    // we are going via the base class in case there is 
+    // we are going via the base class in case there is

     // some change behind the values by it
     wxGaugeBase::SetRange( r ) ;
     if ( m_peer && m_peer->Ok() ){
     // some change behind the values by it
     wxGaugeBase::SetRange( r ) ;
     if ( m_peer && m_peer->Ok() ){

-        // switch back to determinate mode if not there already 
+        // switch back to determinate mode if not there already

         if ( m_peer->GetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag ) != false )
         {
              m_peer->SetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag, (Boolean)false );
         }
         if ( m_peer->GetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag ) != false )
         {
              m_peer->SetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag, (Boolean)false );
         }

-    
+

         m_peer->SetMaximum( GetRange() ) ;
     }
 }
 
 void wxGauge::SetValue(int pos)
 {
         m_peer->SetMaximum( GetRange() ) ;
     }
 }
 
 void wxGauge::SetValue(int pos)
 {

-    // we are going via the base class in case there is 
+    // we are going via the base class in case there is

     // some change behind the values by it
     wxGaugeBase::SetValue( pos ) ;
 
     if ( m_peer && m_peer->Ok() )
     {
     // some change behind the values by it
     wxGaugeBase::SetValue( pos ) ;
 
     if ( m_peer && m_peer->Ok() )
     {

-        // switch back to determinate mode if not there already 
+        // switch back to determinate mode if not there already

         if ( m_peer->GetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag ) != false )
         {
             m_peer->SetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag, (Boolean)false );
         }
         if ( m_peer->GetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag ) != false )
         {
             m_peer->SetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag, (Boolean)false );
         }

-    
+

         m_peer->SetValue( GetValue() ) ;
 
         // turn off animation in the unnecessary situations as this is consuming a lot of CPU otherwise
         m_peer->SetValue( GetValue() ) ;
 
         // turn off animation in the unnecessary situations as this is consuming a lot of CPU otherwise


@@ -114,9 +114,9 @@ void wxGauge::Pulse()
     {
         if ( m_peer->GetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag ) != true )
         {
     {
         if ( m_peer->GetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag ) != true )
         {

-        m_peer->SetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag, true); 
+        m_peer->SetData<Boolean>( kControlNoPart, kControlProgressBarIndeterminateTag, true);

         }
         }

-        
+

         if ( m_peer->GetData<Boolean>( kControlEntireControl, kControlProgressBarAnimatingTag ) != true )
         {
             m_peer->SetData<Boolean>( kControlEntireControl, kControlProgressBarAnimatingTag, true ) ;
         if ( m_peer->GetData<Boolean>( kControlEntireControl, kControlProgressBarAnimatingTag ) != true )
         {
             m_peer->SetData<Boolean>( kControlEntireControl, kControlProgressBarAnimatingTag, true ) ;

diff --git a/src/mac/carbon/graphics.cpp b/src/mac/carbon/graphics.cpp
index 0b6b699b4abffbd0fe39aa89d12088a390e7e1ba..80e114ea8bb3ab66784caba165e2a7aa291e5219 100644 (file)
--- a/src/mac/carbon/graphics.cpp
+++ b/src/mac/carbon/graphics.cpp
@@ -87,9 +87,9 @@ static const double RAD2DEG = 180.0 / M_PI;
 OSStatus wxMacDrawCGImage(
                   CGContextRef    inContext,
                   const CGRect *  inBounds,
 OSStatus wxMacDrawCGImage(
                   CGContextRef    inContext,
                   const CGRect *  inBounds,

-                  CGImageRef      inImage) 
+                  CGImageRef      inImage)

 {
 {

-#if defined( __LP64__ ) || defined(__WXCOCOA__) 
+#if defined( __LP64__ ) || defined(__WXCOCOA__)

     // todo flip
     CGContextDrawImage(inContext, *inBounds, inImage );
     return noErr;
     // todo flip
     CGContextDrawImage(inContext, *inBounds, inImage );
     return noErr;


@@ -111,7 +111,7 @@ CGColorRef wxMacCreateCGColor( const wxColour& col )
     else
 #endif
     {
     else
 #endif
     {

-        CGFloat components[4] = { col.Red() / 255.0, col.Green() / 255.0, col.Blue()  / 255.0, col.Alpha() / 255.0 } ;    
+        CGFloat components[4] = { col.Red() / 255.0, col.Green() / 255.0, col.Blue()  / 255.0, col.Alpha() / 255.0 } ;

         retval = CGColorCreate( wxMacGetGenericRGBColorSpace() , components ) ;
     }
 
         retval = CGColorCreate( wxMacGetGenericRGBColorSpace() , components ) ;
     }
 


@@ -551,13 +551,13 @@ class wxMacCoreGraphicsColour
         wxMacCoreGraphicsColour();
         wxMacCoreGraphicsColour(const wxBrush &brush);
         ~wxMacCoreGraphicsColour();
         wxMacCoreGraphicsColour();
         wxMacCoreGraphicsColour(const wxBrush &brush);
         ~wxMacCoreGraphicsColour();

-        
+

         void Apply( CGContextRef cgContext );
     protected:
         void Init();
         wxCFRef<CGColorRef> m_color;
         wxCFRef<CGColorSpaceRef> m_colorSpace;
         void Apply( CGContextRef cgContext );
     protected:
         void Init();
         wxCFRef<CGColorRef> m_color;
         wxCFRef<CGColorSpaceRef> m_colorSpace;

-        
+

         bool m_isPattern;
         wxCFRef<CGPatternRef> m_pattern;
         CGFloat* m_patternColorComponents;
         bool m_isPattern;
         wxCFRef<CGPatternRef> m_pattern;
         CGFloat* m_patternColorComponents;


@@ -589,7 +589,7 @@ void wxMacCoreGraphicsColour::Apply( CGContextRef cgContext )
     }
 }
 
     }
 }
 

-wxMacCoreGraphicsColour::wxMacCoreGraphicsColour() 
+wxMacCoreGraphicsColour::wxMacCoreGraphicsColour()

 {
     Init();
 }
 {
     Init();
 }


@@ -766,7 +766,7 @@ public:
     CTFontRef GetCTFont() const { return m_ctFont ; }
 #endif
     wxColour GetColour() const { return m_colour ; }
     CTFontRef GetCTFont() const { return m_ctFont ; }
 #endif
     wxColour GetColour() const { return m_colour ; }

-    
+

     bool GetUnderlined() const { return m_underlined ; }
 private :
     wxColour m_colour;
     bool GetUnderlined() const { return m_underlined ; }
 private :
     wxColour m_colour;


@@ -889,7 +889,7 @@ public :
     // gets the component valuess of the matrix
     virtual void Get(wxDouble* a=NULL, wxDouble* b=NULL,  wxDouble* c=NULL,
                      wxDouble* d=NULL, wxDouble* tx=NULL, wxDouble* ty=NULL) const;
     // gets the component valuess of the matrix
     virtual void Get(wxDouble* a=NULL, wxDouble* b=NULL,  wxDouble* c=NULL,
                      wxDouble* d=NULL, wxDouble* tx=NULL, wxDouble* ty=NULL) const;

-       
+

     // makes this the inverse matrix
     virtual void Invert();
 
     // makes this the inverse matrix
     virtual void Invert();
 


@@ -1253,7 +1253,7 @@ public:
     virtual void GetSize( wxDouble* width, wxDouble* height);
 
     virtual void StartPage( wxDouble width, wxDouble height );
     virtual void GetSize( wxDouble* width, wxDouble* height);
 
     virtual void StartPage( wxDouble width, wxDouble height );

-    
+

     virtual void EndPage();
 
     virtual void Flush();
     virtual void EndPage();
 
     virtual void Flush();


@@ -1311,7 +1311,7 @@ public:
     virtual void DrawPath( const wxGraphicsPath &path, int fillStyle = wxODDEVEN_RULE );
 
     virtual bool ShouldOffset() const
     virtual void DrawPath( const wxGraphicsPath &path, int fillStyle = wxODDEVEN_RULE );
 
     virtual bool ShouldOffset() const

-    {     
+    {

         int penwidth = 0 ;
         if ( !m_pen.IsNull() )
         {
         int penwidth = 0 ;
         if ( !m_pen.IsNull() )
         {


@@ -1474,12 +1474,12 @@ void wxMacCoreGraphicsContext::StartPage( wxDouble width, wxDouble height )
         r = CGRectMake( 0 , 0 , width  , height );
     else
         r = CGRectMake( 0 , 0 , m_width  , m_height );
         r = CGRectMake( 0 , 0 , width  , height );
     else
         r = CGRectMake( 0 , 0 , m_width  , m_height );

-    
+

     CGContextBeginPage(m_cgContext,  &r );
 //    CGContextTranslateCTM( m_cgContext , 0 ,  height == 0 ? m_height : height );
 //    CGContextScaleCTM( m_cgContext , 1 , -1 );
 }
     CGContextBeginPage(m_cgContext,  &r );
 //    CGContextTranslateCTM( m_cgContext , 0 ,  height == 0 ? m_height : height );
 //    CGContextScaleCTM( m_cgContext , 1 , -1 );
 }

-    
+

 void wxMacCoreGraphicsContext::EndPage()
 {
     CGContextEndPage(m_cgContext);
 void wxMacCoreGraphicsContext::EndPage()
 {
     CGContextEndPage(m_cgContext);


@@ -1494,28 +1494,31 @@ void wxMacCoreGraphicsContext::EnsureIsValid()
 {
     if ( !m_cgContext )
     {
 {
     if ( !m_cgContext )
     {

-        OSStatus status = 
+        OSStatus status =

 #if ! ( defined( __LP64__ ) || defined(__WXCOCOA__) )
             QDBeginCGContext( GetWindowPort( m_windowRef ) , &m_cgContext );
 #else
             paramErr;
 #endif
 #if ! ( defined( __LP64__ ) || defined(__WXCOCOA__) )
             QDBeginCGContext( GetWindowPort( m_windowRef ) , &m_cgContext );
 #else
             paramErr;
 #endif

-        wxASSERT_MSG( status == noErr , wxT("Cannot nest wxDCs on the same window") );
+        if ( status != noErr )
+        {
+            wxFAIL_MSG("Cannot nest wxDCs on the same window");
+        }

 
         CGContextConcatCTM( m_cgContext, m_windowTransform );
 
         CGContextConcatCTM( m_cgContext, m_windowTransform );

-               CGContextSaveGState( m_cgContext );
-               m_releaseContext = true;
-               if ( !HIShapeIsEmpty(m_clipRgn) )
-               {
+        CGContextSaveGState( m_cgContext );
+        m_releaseContext = true;
+        if ( !HIShapeIsEmpty(m_clipRgn) )
+        {

             // the clip region is in device coordinates, so we convert this again to user coordinates
             wxCFRef<HIMutableShapeRef> hishape( HIShapeCreateMutableCopy( m_clipRgn ) );
             CGPoint transformedOrigin = CGPointApplyAffineTransform( CGPointZero,m_windowTransform);
             HIShapeOffset( hishape, -transformedOrigin.x, -transformedOrigin.y );
             // the clip region is in device coordinates, so we convert this again to user coordinates
             wxCFRef<HIMutableShapeRef> hishape( HIShapeCreateMutableCopy( m_clipRgn ) );
             CGPoint transformedOrigin = CGPointApplyAffineTransform( CGPointZero,m_windowTransform);
             HIShapeOffset( hishape, -transformedOrigin.x, -transformedOrigin.y );

-                       HIShapeReplacePathInCGContext( hishape, m_cgContext );
-                       CGContextClip( m_cgContext );
-               }
-               CGContextSaveGState( m_cgContext );
-       }
+            HIShapeReplacePathInCGContext( hishape, m_cgContext );
+            CGContextClip( m_cgContext );
+        }
+        CGContextSaveGState( m_cgContext );
+    }

 }
 
 // TODO test whether the private CGContextSetCompositeOperation works under 10.3 (using NSCompositingModes)
 }
 
 // TODO test whether the private CGContextSetCompositeOperation works under 10.3 (using NSCompositingModes)


@@ -1526,11 +1529,11 @@ bool wxMacCoreGraphicsContext::SetLogicalFunction( int function )
         return true;
 
     EnsureIsValid();
         return true;
 
     EnsureIsValid();

-    
+

     bool retval = false;
     bool shouldAntiAlias = true;
     CGBlendMode mode = kCGBlendModeNormal;
     bool retval = false;
     bool shouldAntiAlias = true;
     CGBlendMode mode = kCGBlendModeNormal;

-    
+

 #if defined(__WXMAC__) && ( MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_5 )
     if ( UMAGetSystemVersion() >= 0x1050 )
     {
 #if defined(__WXMAC__) && ( MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_5 )
     if ( UMAGetSystemVersion() >= 0x1050 )
     {


@@ -1547,7 +1550,7 @@ bool wxMacCoreGraphicsContext::SetLogicalFunction( int function )
             case wxXOR :
                 mode = kCGBlendModeXOR;
                 shouldAntiAlias = false;
             case wxXOR :
                 mode = kCGBlendModeXOR;
                 shouldAntiAlias = false;

-                break;  
+                break;

             default :
                 retval = false;
                 break;
             default :
                 retval = false;
                 break;


@@ -1568,7 +1571,7 @@ bool wxMacCoreGraphicsContext::SetLogicalFunction( int function )
             retval = true;
         }
     }
             retval = true;
         }
     }

-    
+

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


@@ -1591,7 +1594,7 @@ void wxMacCoreGraphicsContext::Clip( const wxRegion &region )
         // this offsetting to device coords is not really correct, but since we cannot apply affine transforms
         // to regions we try at least to have correct translations
         HIMutableShapeRef mutableShape = HIShapeCreateMutableCopy( region.GetWXHRGN() );
         // this offsetting to device coords is not really correct, but since we cannot apply affine transforms
         // to regions we try at least to have correct translations
         HIMutableShapeRef mutableShape = HIShapeCreateMutableCopy( region.GetWXHRGN() );

-        
+

         CGPoint transformedOrigin = CGPointApplyAffineTransform( CGPointZero, m_windowTransform );
         HIShapeOffset( mutableShape, transformedOrigin.x, transformedOrigin.y );
         m_clipRgn.reset(mutableShape);
         CGPoint transformedOrigin = CGPointApplyAffineTransform( CGPointZero, m_windowTransform );
         HIShapeOffset( mutableShape, transformedOrigin.x, transformedOrigin.y );
         m_clipRgn.reset(mutableShape);


@@ -1609,7 +1612,7 @@ void wxMacCoreGraphicsContext::Clip( wxDouble x, wxDouble y, wxDouble w, wxDoubl
     }
     else
     {
     }
     else
     {

-        // the clipping itself must be stored as device coordinates, otherwise 
+        // the clipping itself must be stored as device coordinates, otherwise

         // we cannot apply it back correctly
         r.origin= CGPointApplyAffineTransform( r.origin, m_windowTransform );
         m_clipRgn.reset(HIShapeCreateWithRect(&r));
         // we cannot apply it back correctly
         r.origin= CGPointApplyAffineTransform( r.origin, m_windowTransform );
         m_clipRgn.reset(HIShapeCreateWithRect(&r));


@@ -1869,7 +1872,7 @@ void wxMacCoreGraphicsContext::DrawText( const wxString &str, wxDouble x, wxDoub
 {
     if ( m_font.IsNull() )
         return;
 {
     if ( m_font.IsNull() )
         return;

-    
+

     EnsureIsValid();
 #if wxMAC_USE_CORE_TEXT
     if ( UMAGetSystemVersion() >= 0x1050 )
     EnsureIsValid();
 #if wxMAC_USE_CORE_TEXT
     if ( UMAGetSystemVersion() >= 0x1050 )


@@ -1879,16 +1882,16 @@ void wxMacCoreGraphicsContext::DrawText( const wxString &str, wxDouble x, wxDoub
         CTFontRef font = fref->GetCTFont();
         CGColorRef col = wxMacCreateCGColor( fref->GetColour() );
         CTUnderlineStyle ustyle = fref->GetUnderlined() ? kCTUnderlineStyleSingle : kCTUnderlineStyleNone ;
         CTFontRef font = fref->GetCTFont();
         CGColorRef col = wxMacCreateCGColor( fref->GetColour() );
         CTUnderlineStyle ustyle = fref->GetUnderlined() ? kCTUnderlineStyleSingle : kCTUnderlineStyleNone ;

-        wxCFRef<CFNumberRef> underlined( CFNumberCreate(NULL, kCFNumberSInt32Type, &ustyle) ); 
+        wxCFRef<CFNumberRef> underlined( CFNumberCreate(NULL, kCFNumberSInt32Type, &ustyle) );

          CFStringRef keys[] = { kCTFontAttributeName , kCTForegroundColorAttributeName, kCTUnderlineStyleAttributeName };
         CFTypeRef values[] = { font, col, underlined };
          CFStringRef keys[] = { kCTFontAttributeName , kCTForegroundColorAttributeName, kCTUnderlineStyleAttributeName };
         CFTypeRef values[] = { font, col, underlined };

-        wxCFRef<CFDictionaryRef> attributes( CFDictionaryCreate(kCFAllocatorDefault, (const void**) &keys, (const void**) &values, 
+        wxCFRef<CFDictionaryRef> attributes( CFDictionaryCreate(kCFAllocatorDefault, (const void**) &keys, (const void**) &values,

                                                         WXSIZEOF( keys ), &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks) );
         wxCFRef<CFAttributedStringRef> attrtext( CFAttributedStringCreate(kCFAllocatorDefault, text, attributes) );
         wxCFRef<CTLineRef> line( CTLineCreateWithAttributedString(attrtext) );
                                                         WXSIZEOF( keys ), &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks) );
         wxCFRef<CFAttributedStringRef> attrtext( CFAttributedStringCreate(kCFAllocatorDefault, text, attributes) );
         wxCFRef<CTLineRef> line( CTLineCreateWithAttributedString(attrtext) );

-        
+

         y += CTFontGetAscent(font);
         y += CTFontGetAscent(font);

-        
+

         CGContextSaveGState(m_cgContext);
         CGContextTranslateCTM(m_cgContext, x, y);
         CGContextScaleCTM(m_cgContext, 1, -1);
         CGContextSaveGState(m_cgContext);
         CGContextTranslateCTM(m_cgContext, x, y);
         CGContextScaleCTM(m_cgContext, 1, -1);


@@ -1934,12 +1937,12 @@ void wxMacCoreGraphicsContext::DrawText( const wxString &str, wxDouble x, wxDoub
         ATSUStyle style = (((wxMacCoreGraphicsFontData*)m_font.GetRefData())->GetATSUStyle());
         status = ::ATSUCreateTextLayoutWithTextPtr( unibuf.GetBuffer() , 0 , chars , chars , 1 ,
                                                    &chars , &style , &atsuLayout );
         ATSUStyle style = (((wxMacCoreGraphicsFontData*)m_font.GetRefData())->GetATSUStyle());
         status = ::ATSUCreateTextLayoutWithTextPtr( unibuf.GetBuffer() , 0 , chars , chars , 1 ,
                                                    &chars , &style , &atsuLayout );

-        
+

         wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the rotated text") );
         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") );
         status = ::ATSUSetTransientFontMatching( atsuLayout , true );
         wxASSERT_MSG( status == noErr , wxT("couldn't setup transient font matching") );

-        
+

         int iAngle = int( angle * RAD2DEG );
         if ( abs(iAngle) > 0 )
         {
         int iAngle = int( angle * RAD2DEG );
         if ( abs(iAngle) > 0 )
         {


@@ -1959,7 +1962,7 @@ void wxMacCoreGraphicsContext::DrawText( const wxString &str, wxDouble x, wxDoub
             status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags) / sizeof(ATSUAttributeTag),
                                              atsuTags, atsuSizes, atsuValues );
         }
             status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags) / sizeof(ATSUAttributeTag),
                                              atsuTags, atsuSizes, atsuValues );
         }

-        
+

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


@@ -1976,35 +1979,35 @@ void wxMacCoreGraphicsContext::DrawText( const wxString &str, wxDouble x, wxDoub
             status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags) / sizeof(ATSUAttributeTag),
                                              atsuTags, atsuSizes, atsuValues );
         }
             status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags) / sizeof(ATSUAttributeTag),
                                              atsuTags, atsuSizes, atsuValues );
         }

-        
+

         ATSUTextMeasurement textBefore, textAfter;
         ATSUTextMeasurement ascent, descent;
         ATSUTextMeasurement textBefore, textAfter;
         ATSUTextMeasurement ascent, descent;

-        
+

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

-        
+

         wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );
         wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );

-        
+

         Rect rect;
         x += (int)(sin(angle) * FixedToInt(ascent));
         y += (int)(cos(angle) * FixedToInt(ascent));
         Rect rect;
         x += (int)(sin(angle) * FixedToInt(ascent));
         y += (int)(cos(angle) * FixedToInt(ascent));

-        
+

         status = ::ATSUMeasureTextImage( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
                                         IntToFixed(x) , IntToFixed(y) , &rect );
         wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );
         status = ::ATSUMeasureTextImage( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
                                         IntToFixed(x) , IntToFixed(y) , &rect );
         wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );

-        
+

         CGContextSaveGState(m_cgContext);
         CGContextTranslateCTM(m_cgContext, x, y);
         CGContextScaleCTM(m_cgContext, 1, -1);
         status = ::ATSUDrawText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
                                 IntToFixed(0) , IntToFixed(0) );
         CGContextSaveGState(m_cgContext);
         CGContextTranslateCTM(m_cgContext, x, y);
         CGContextScaleCTM(m_cgContext, 1, -1);
         status = ::ATSUDrawText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
                                 IntToFixed(0) , IntToFixed(0) );

-        
+

         wxASSERT_MSG( status == noErr , wxT("couldn't draw the rotated text") );
         wxASSERT_MSG( status == noErr , wxT("couldn't draw the rotated text") );

-        
+

         CGContextRestoreGState(m_cgContext);
         CGContextRestoreGState(m_cgContext);

-        
+

         ::ATSUDisposeTextLayout(atsuLayout);
         ::ATSUDisposeTextLayout(atsuLayout);

-        
+

         return;
     }
 #endif
         return;
     }
 #endif


@@ -2027,7 +2030,7 @@ void wxMacCoreGraphicsContext::GetTextExtent( const wxString &str, wxDouble *wid
         *descent = 0;
     if ( externalLeading )
         *externalLeading = 0;
         *descent = 0;
     if ( externalLeading )
         *externalLeading = 0;

-        
+

     if (str.empty())
         return;
 
     if (str.empty())
         return;
 


@@ -2040,15 +2043,15 @@ void wxMacCoreGraphicsContext::GetTextExtent( const wxString &str, wxDouble *wid
         wxCFStringRef text(str, wxLocale::GetSystemEncoding() );
         CFStringRef keys[] = { kCTFontAttributeName  };
         CFTypeRef values[] = { font };
         wxCFStringRef text(str, wxLocale::GetSystemEncoding() );
         CFStringRef keys[] = { kCTFontAttributeName  };
         CFTypeRef values[] = { font };

-        wxCFRef<CFDictionaryRef> attributes( CFDictionaryCreate(kCFAllocatorDefault, (const void**) &keys, (const void**) &values, 
+        wxCFRef<CFDictionaryRef> attributes( CFDictionaryCreate(kCFAllocatorDefault, (const void**) &keys, (const void**) &values,

                                                                 WXSIZEOF( keys ), &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks) );
         wxCFRef<CFAttributedStringRef> attrtext( CFAttributedStringCreate(kCFAllocatorDefault, text, attributes) );
         wxCFRef<CTLineRef> line( CTLineCreateWithAttributedString(attrtext) );
                                                                 WXSIZEOF( keys ), &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks) );
         wxCFRef<CFAttributedStringRef> attrtext( CFAttributedStringCreate(kCFAllocatorDefault, text, attributes) );
         wxCFRef<CTLineRef> line( CTLineCreateWithAttributedString(attrtext) );

-            
+

         CGFloat w, a, d, l;
         CGFloat w, a, d, l;

-        
+

         w = CTLineGetTypographicBounds(line, &a, &d, &l) ;
         w = CTLineGetTypographicBounds(line, &a, &d, &l) ;

-        
+

         if ( height )
             *height = a+d+l;
         if ( descent )
         if ( height )
             *height = a+d+l;
         if ( descent )


@@ -2063,26 +2066,26 @@ void wxMacCoreGraphicsContext::GetTextExtent( const wxString &str, wxDouble *wid
 #if wxMAC_USE_ATSU_TEXT
     {
         OSStatus status = noErr;
 #if wxMAC_USE_ATSU_TEXT
     {
         OSStatus status = noErr;

-        
+

         ATSUTextLayout atsuLayout;
         wxMacUniCharBuffer unibuf( str );
         UniCharCount chars = unibuf.GetChars();
         ATSUTextLayout atsuLayout;
         wxMacUniCharBuffer unibuf( str );
         UniCharCount chars = unibuf.GetChars();

-        
+

         ATSUStyle style = (((wxMacCoreGraphicsFontData*)m_font.GetRefData())->GetATSUStyle());
         status = ::ATSUCreateTextLayoutWithTextPtr( unibuf.GetBuffer() , 0 , chars , chars , 1 ,
                                                    &chars , &style , &atsuLayout );
         ATSUStyle style = (((wxMacCoreGraphicsFontData*)m_font.GetRefData())->GetATSUStyle());
         status = ::ATSUCreateTextLayoutWithTextPtr( unibuf.GetBuffer() , 0 , chars , chars , 1 ,
                                                    &chars , &style , &atsuLayout );

-        
+

         wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the text") );
         wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the text") );

-        
+

         status = ::ATSUSetTransientFontMatching( atsuLayout , true );
         wxASSERT_MSG( status == noErr , wxT("couldn't setup transient font matching") );
 
         ATSUTextMeasurement textBefore, textAfter;
         ATSUTextMeasurement textAscent, textDescent;
         status = ::ATSUSetTransientFontMatching( atsuLayout , true );
         wxASSERT_MSG( status == noErr , wxT("couldn't setup transient font matching") );
 
         ATSUTextMeasurement textBefore, textAfter;
         ATSUTextMeasurement textAscent, textDescent;

-        
+

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

-        
+

         if ( height )
             *height = FixedToInt(textAscent + textDescent);
         if ( descent )
         if ( height )
             *height = FixedToInt(textAscent + textDescent);
         if ( descent )


@@ -2091,7 +2094,7 @@ void wxMacCoreGraphicsContext::GetTextExtent( const wxString &str, wxDouble *wid
             *externalLeading = 0;
         if ( width )
             *width = FixedToInt(textAfter - textBefore);
             *externalLeading = 0;
         if ( width )
             *width = FixedToInt(textAfter - textBefore);

-        
+

         ::ATSUDisposeTextLayout(atsuLayout);
 
         return;
         ::ATSUDisposeTextLayout(atsuLayout);
 
         return;


@@ -2114,21 +2117,21 @@ void wxMacCoreGraphicsContext::GetPartialTextExtents(const wxString& text, wxArr
     {
         wxMacCoreGraphicsFontData* fref = (wxMacCoreGraphicsFontData*)m_font.GetRefData();
         CTFontRef font = fref->GetCTFont();
     {
         wxMacCoreGraphicsFontData* fref = (wxMacCoreGraphicsFontData*)m_font.GetRefData();
         CTFontRef font = fref->GetCTFont();

-        
+

         wxCFStringRef t(text, wxLocale::GetSystemEncoding() );
         CFStringRef keys[] = { kCTFontAttributeName  };
         CFTypeRef values[] = { font };
         wxCFStringRef t(text, wxLocale::GetSystemEncoding() );
         CFStringRef keys[] = { kCTFontAttributeName  };
         CFTypeRef values[] = { font };

-        wxCFRef<CFDictionaryRef> attributes( CFDictionaryCreate(kCFAllocatorDefault, (const void**) &keys, (const void**) &values, 
+        wxCFRef<CFDictionaryRef> attributes( CFDictionaryCreate(kCFAllocatorDefault, (const void**) &keys, (const void**) &values,

                                                                 WXSIZEOF( keys ), &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks) );
         wxCFRef<CFAttributedStringRef> attrtext( CFAttributedStringCreate(kCFAllocatorDefault, t, attributes) );
         wxCFRef<CTLineRef> line( CTLineCreateWithAttributedString(attrtext) );
                                                                 WXSIZEOF( keys ), &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks) );
         wxCFRef<CFAttributedStringRef> attrtext( CFAttributedStringCreate(kCFAllocatorDefault, t, attributes) );
         wxCFRef<CTLineRef> line( CTLineCreateWithAttributedString(attrtext) );

-        
+

         int chars = text.length();
         for ( int pos = 0; pos < (int)chars; pos ++ )
         {
             widths[pos] = CTLineGetOffsetForStringIndex( line, pos+1 , NULL )+0.5;
         }
         int chars = text.length();
         for ( int pos = 0; pos < (int)chars; pos ++ )
         {
             widths[pos] = CTLineGetOffsetForStringIndex( line, pos+1 , NULL )+0.5;
         }

-        
+

         return;
     }
 #endif
         return;
     }
 #endif


@@ -2138,13 +2141,13 @@ void wxMacCoreGraphicsContext::GetPartialTextExtents(const wxString& text, wxArr
         ATSUTextLayout atsuLayout;
         wxMacUniCharBuffer unibuf( text );
         UniCharCount chars = unibuf.GetChars();
         ATSUTextLayout atsuLayout;
         wxMacUniCharBuffer unibuf( text );
         UniCharCount chars = unibuf.GetChars();

-        
+

         ATSUStyle style = (((wxMacCoreGraphicsFontData*)m_font.GetRefData())->GetATSUStyle());
         status = ::ATSUCreateTextLayoutWithTextPtr( unibuf.GetBuffer() , 0 , chars , chars , 1 ,
                                           &chars , &style , &atsuLayout );
         ATSUStyle style = (((wxMacCoreGraphicsFontData*)m_font.GetRefData())->GetATSUStyle());
         status = ::ATSUCreateTextLayoutWithTextPtr( unibuf.GetBuffer() , 0 , chars , chars , 1 ,
                                           &chars , &style , &atsuLayout );

-                
+

         wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the text") );
         wxASSERT_MSG( status == noErr , wxT("couldn't create the layout of the text") );

-        
+

         status = ::ATSUSetTransientFontMatching( atsuLayout , true );
         wxASSERT_MSG( status == noErr , wxT("couldn't setup transient font matching") );
 
         status = ::ATSUSetTransientFontMatching( atsuLayout , true );
         wxASSERT_MSG( status == noErr , wxT("couldn't setup transient font matching") );
 


@@ -2152,18 +2155,18 @@ void wxMacCoreGraphicsContext::GetPartialTextExtents(const wxString& text, wxArr
         {
             unsigned long actualNumberOfBounds = 0;
             ATSTrapezoid glyphBounds;
         {
             unsigned long actualNumberOfBounds = 0;
             ATSTrapezoid glyphBounds;

-            
+

             // We get a single bound, since the text should only require one. If it requires more, there is an issue
             OSStatus result;
             result = ATSUGetGlyphBounds( atsuLayout, 0, 0, kATSUFromTextBeginning, pos + 1,
                                         kATSUseDeviceOrigins, 1, &glyphBounds, &actualNumberOfBounds );
             if (result != noErr || actualNumberOfBounds != 1 )
                 return;
             // We get a single bound, since the text should only require one. If it requires more, there is an issue
             OSStatus result;
             result = ATSUGetGlyphBounds( atsuLayout, 0, 0, kATSUFromTextBeginning, pos + 1,
                                         kATSUseDeviceOrigins, 1, &glyphBounds, &actualNumberOfBounds );
             if (result != noErr || actualNumberOfBounds != 1 )
                 return;

-            
+

             widths[pos] = FixedToInt( glyphBounds.upperRight.x - glyphBounds.upperLeft.x );
             //unsigned char uch = s[i];
         }
             widths[pos] = FixedToInt( glyphBounds.upperRight.x - glyphBounds.upperLeft.x );
             //unsigned char uch = s[i];
         }

-        
+

         ::ATSUDisposeTextLayout(atsuLayout);
     }
 #endif
         ::ATSUDisposeTextLayout(atsuLayout);
     }
 #endif


@@ -2236,7 +2239,7 @@ public :
     virtual wxGraphicsContext * CreateContextFromNativeWindow( void * window );
 
     virtual wxGraphicsContext * CreateContext( wxWindow* window );
     virtual wxGraphicsContext * CreateContextFromNativeWindow( void * window );
 
     virtual wxGraphicsContext * CreateContext( wxWindow* window );

-    
+

     virtual wxGraphicsContext * CreateMeasuringContext();
 
     // Path
     virtual wxGraphicsContext * CreateMeasuringContext();
 
     // Path


@@ -2267,7 +2270,7 @@ public :
 
     // create a native bitmap representation
     virtual wxGraphicsBitmap CreateBitmap( const wxBitmap &bitmap ) ;
 
     // create a native bitmap representation
     virtual wxGraphicsBitmap CreateBitmap( const wxBitmap &bitmap ) ;

-    
+

     // create a native bitmap representation
     virtual wxGraphicsBitmap CreateSubBitmap( const wxGraphicsBitmap &bitmap, wxDouble x, wxDouble y, wxDouble w, wxDouble h  ) ;
 private :
     // create a native bitmap representation
     virtual wxGraphicsBitmap CreateSubBitmap( const wxGraphicsBitmap &bitmap, wxDouble x, wxDouble y, wxDouble w, wxDouble h  ) ;
 private :


@@ -2319,7 +2322,7 @@ wxGraphicsContext * wxMacCoreGraphicsRenderer::CreateContext( const wxMemoryDC&
     {
         int w, h;
         mem_impl->GetSize( &w, &h );
     {
         int w, h;
         mem_impl->GetSize( &w, &h );

-        return new wxMacCoreGraphicsContext( this, 
+        return new wxMacCoreGraphicsContext( this,

             (CGContextRef)(mem_impl->GetGraphicsContext()->GetNativeContext()), (wxDouble) w, (wxDouble) h );
     }
 #endif
             (CGContextRef)(mem_impl->GetGraphicsContext()->GetNativeContext()), (wxDouble) w, (wxDouble) h );
     }
 #endif


@@ -2407,7 +2410,7 @@ wxGraphicsBitmap wxMacCoreGraphicsRenderer::CreateBitmap( const wxBitmap& bmp )
         return wxNullGraphicsBitmap;
 }
 
         return wxNullGraphicsBitmap;
 }
 

-wxGraphicsBitmap wxMacCoreGraphicsRenderer::CreateSubBitmap( const wxGraphicsBitmap &bmp, wxDouble x, wxDouble y, wxDouble w, wxDouble h  ) 
+wxGraphicsBitmap wxMacCoreGraphicsRenderer::CreateSubBitmap( const wxGraphicsBitmap &bmp, wxDouble x, wxDouble y, wxDouble w, wxDouble h  )

 {
     CGImageRef img = static_cast<wxMacCoreGraphicsBitmapData*>(bmp.GetRefData())->GetBitmap();
     if ( img )
 {
     CGImageRef img = static_cast<wxMacCoreGraphicsBitmapData*>(bmp.GetRefData())->GetBitmap();
     if ( img )


@@ -2503,7 +2506,10 @@ CGDataConsumerRef wxMacCGDataConsumerCreateWithCFData( CFMutableDataRef data )
     return CGDataConsumerCreateWithCFData( data );
 }
 
     return CGDataConsumerCreateWithCFData( data );
 }
 

-void wxMacReleaseMemoryBufferProviderCallback(void *info, const void *data, size_t WXUNUSED(size))
+void
+wxMacReleaseMemoryBufferProviderCallback(void *info,
+                                         const void * WXUNUSED_UNLESS_DEBUG(data),
+                                         size_t WXUNUSED(size))

 {
     wxMemoryBuffer* membuf = (wxMemoryBuffer*) info ;
 
 {
     wxMemoryBuffer* membuf = (wxMemoryBuffer*) info ;
 


@@ -2517,7 +2523,7 @@ CGDataProviderRef wxMacCGDataProviderCreateWithMemoryBuffer( const wxMemoryBuffe
     wxMemoryBuffer* b = new wxMemoryBuffer( buf );
     if ( b->GetDataLen() == 0 )
         return NULL;
     wxMemoryBuffer* b = new wxMemoryBuffer( buf );
     if ( b->GetDataLen() == 0 )
         return NULL;

-        
+

     return CGDataProviderCreateWithData( b , (const void *) b->GetData() , b->GetDataLen() ,
                                                  wxMacReleaseMemoryBufferProviderCallback );
     return CGDataProviderCreateWithData( b , (const void *) b->GetData() , b->GetDataLen() ,
                                                  wxMacReleaseMemoryBufferProviderCallback );

-}
\ No newline at end of file
+}

diff --git a/src/mac/carbon/icon.cpp b/src/mac/carbon/icon.cpp
index a19042a3e5a38a28bca2302bba72cabde8e2ad75..c90a0b37c71faa78865d2075c08912042c270c60 100644 (file)
--- a/src/mac/carbon/icon.cpp
+++ b/src/mac/carbon/icon.cpp
@@ -220,7 +220,11 @@ bool wxIcon::LoadFile(
                     HLock((Handle) iconFamily);
                     OSStatus err = GetIconRefFromIconFamilyPtr( *iconFamily, GetHandleSize((Handle) iconFamily), &iconRef );
                     HUnlock((Handle) iconFamily);
                     HLock((Handle) iconFamily);
                     OSStatus err = GetIconRefFromIconFamilyPtr( *iconFamily, GetHandleSize((Handle) iconFamily), &iconRef );
                     HUnlock((Handle) iconFamily);

-                    wxASSERT_MSG( err == noErr , wxT("Error when constructing icon ref") );
+                    if ( err != noErr )
+                    {
+                        wxFAIL_MSG("Error when constructing icon ref");
+                    }
+

                     ReleaseResource( resHandle ) ;
                 }
               }
                     ReleaseResource( resHandle ) ;
                 }
               }

diff --git a/src/mac/carbon/listctrl_mac.cpp b/src/mac/carbon/listctrl_mac.cpp
index e307203d9864a0d46e85e475cc195dafed4f19f8..9bc3cb2247a7a2e8bbbf5268b103b1f49a8e5c7d 100644 (file)
--- a/src/mac/carbon/listctrl_mac.cpp
+++ b/src/mac/carbon/listctrl_mac.cpp
@@ -774,7 +774,8 @@ wxListCtrl::~wxListCtrl()
 }
 
 /*static*/
 }
 
 /*static*/

-wxVisualAttributes wxListCtrl::GetClassDefaultAttributes(wxWindowVariant variant)
+wxVisualAttributes
+wxListCtrl::GetClassDefaultAttributes(wxWindowVariant WXUNUSED(variant))

 {
     wxVisualAttributes attr;
 
 {
     wxVisualAttributes attr;
 

diff --git a/src/mac/carbon/mdi.cpp b/src/mac/carbon/mdi.cpp
index 428a0dc41091279cd096179ff62cfc1cc5fe11e0..3f3131746c1d1c3e46a4811048bed2936caa2514 100644 (file)
--- a/src/mac/carbon/mdi.cpp
+++ b/src/mac/carbon/mdi.cpp
@@ -40,7 +40,7 @@ BEGIN_EVENT_TABLE(wxMDIClientWindow, wxWindow)
     EVT_SCROLL(wxMDIClientWindow::OnScroll)
 END_EVENT_TABLE()
 
     EVT_SCROLL(wxMDIClientWindow::OnScroll)
 END_EVENT_TABLE()
 

-static const wxChar *TRACE_MDI = _T("mdi");
+#define TRACE_MDI "mdi"

 
 static const int IDM_WINDOWTILEHOR  = 4001;
 static const int IDM_WINDOWCASCADE = 4002;
 
 static const int IDM_WINDOWTILEHOR  = 4001;
 static const int IDM_WINDOWCASCADE = 4002;

diff --git a/src/mac/carbon/notebmac.cpp b/src/mac/carbon/notebmac.cpp
index 7f82d490f77565a17199abbec5d3629bef83afea..6d00a7130dfd300c49cd2e47b63aaec4766261cf 100644 (file)
--- a/src/mac/carbon/notebmac.cpp
+++ b/src/mac/carbon/notebmac.cpp
@@ -413,7 +413,10 @@ void wxNotebook::MacSetupTabs()
                 wxMacCreateBitmapButton( &info, bmap ) ;
 
                 OSStatus err = m_peer->SetData<ControlButtonContentInfo>( ii + 1, kControlTabImageContentTag, &info );
                 wxMacCreateBitmapButton( &info, bmap ) ;
 
                 OSStatus err = m_peer->SetData<ControlButtonContentInfo>( ii + 1, kControlTabImageContentTag, &info );

-                wxASSERT_MSG( err == noErr , wxT("Error when setting icon on tab") ) ;
+                if ( err != noErr )
+                {
+                    wxFAIL_MSG("Error when setting icon on tab");
+                }

 
                 wxMacReleaseBitmapButton( &info ) ;
             }
 
                 wxMacReleaseBitmapButton( &info ) ;
             }

diff --git a/src/mac/carbon/overlay.cpp b/src/mac/carbon/overlay.cpp
index 7031ebf5969d7e1af2eff951cc6487aff20cd868..3ae300276a61320deb9512f375785042212fdd86 100644 (file)
--- a/src/mac/carbon/overlay.cpp
+++ b/src/mac/carbon/overlay.cpp
@@ -149,7 +149,7 @@ void wxOverlayImpl::EndDrawing( wxDC* dc)
     wxGCDCImpl *win_impl = wxDynamicCast(impl,wxGCDCImpl);
     if (win_impl)
         win_impl->SetGraphicsContext(NULL);
     wxGCDCImpl *win_impl = wxDynamicCast(impl,wxGCDCImpl);
     if (win_impl)
         win_impl->SetGraphicsContext(NULL);

-        
+

     CGContextFlush( m_overlayContext );
 }
 
     CGContextFlush( m_overlayContext );
 }
 


@@ -166,7 +166,10 @@ void wxOverlayImpl::Reset()
     {
 #ifndef __LP64__
         OSStatus err = QDEndCGContext(GetWindowPort(m_overlayWindow), &m_overlayContext);
     {
 #ifndef __LP64__
         OSStatus err = QDEndCGContext(GetWindowPort(m_overlayWindow), &m_overlayContext);

-        wxASSERT_MSG(  err == noErr , _("Couldn't end the context on the overlay window") );
+        if ( err != noErr )
+        {
+            wxFAIL_MSG("Couldn't end the context on the overlay window");
+        }

 #endif
         m_overlayContext = NULL ;
     }
 #endif
         m_overlayContext = NULL ;
     }

diff --git a/src/mac/carbon/stattext.cpp b/src/mac/carbon/stattext.cpp
index d2f9fd1ceaef0a167d715985ce978da64da5f50d..d0285e6b065092b7e5d050259d7887ea90e20a56 100644 (file)
--- a/src/mac/carbon/stattext.cpp
+++ b/src/mac/carbon/stattext.cpp
@@ -75,7 +75,7 @@ wxSize wxStaticText::DoGetBestSize() const
 {
     Rect bestsize = { 0 , 0 , 0 , 0 } ;
     Point bounds;
 {
     Rect bestsize = { 0 , 0 , 0 , 0 } ;
     Point bounds;

-    
+

     // try the built-in best size if available
     Boolean former = m_peer->GetData<Boolean>( kControlStaticTextIsMultilineTag);
     m_peer->SetData( kControlStaticTextIsMultilineTag, (Boolean)0 );
     // try the built-in best size if available
     Boolean former = m_peer->GetData<Boolean>( kControlStaticTextIsMultilineTag);
     m_peer->SetData( kControlStaticTextIsMultilineTag, (Boolean)0 );

diff --git a/src/mac/carbon/textctrl.cpp b/src/mac/carbon/textctrl.cpp
index 79c591439cd03e4706877389fe88e572a0682f2c..44ec53aa9a5739672493b5325a7fe6e057756e8c 100644 (file)
--- a/src/mac/carbon/textctrl.cpp
+++ b/src/mac/carbon/textctrl.cpp
@@ -189,7 +189,7 @@ wxMacPortSaver( (GrafPtr) GetWindowPort( (WindowRef) win->MacGetTopLevelWindowRe
     m_formerClip = NewRgn() ;
     m_newClip = NewRgn() ;
     GetClip( m_formerClip ) ;
     m_formerClip = NewRgn() ;
     m_newClip = NewRgn() ;
     GetClip( m_formerClip ) ;

-    
+

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


@@ -197,13 +197,13 @@ wxMacPortSaver( (GrafPtr) GetWindowPort( (WindowRef) win->MacGetTopLevelWindowRe
         {
             int x = 0 , y = 0;
             win->MacWindowToRootWindow( &x, &y ) ;
         {
             int x = 0 , y = 0;
             win->MacWindowToRootWindow( &x, &y ) ;

-            
+

             // get area including focus rect
             HIShapeGetAsQDRgn( ((wxWindow*)win)->MacGetVisibleRegion(true).GetWXHRGN() , m_newClip );
             if ( !EmptyRgn( m_newClip ) )
                 OffsetRgn( m_newClip , x , y ) ;
         }
             // get area including focus rect
             HIShapeGetAsQDRgn( ((wxWindow*)win)->MacGetVisibleRegion(true).GetWXHRGN() , m_newClip );
             if ( !EmptyRgn( m_newClip ) )
                 OffsetRgn( m_newClip , x , y ) ;
         }

-        
+

         SetClip( m_newClip ) ;
     }
 }
         SetClip( m_newClip ) ;
     }
 }


@@ -1763,11 +1763,11 @@ void wxMacMLTEControl::AdjustCreationAttributes(const wxColour &background,
             | kTXNSupportFontCommandProcessing
             | kTXNSupportFontCommandUpdating;
 
             | kTXNSupportFontCommandProcessing
             | kTXNSupportFontCommandUpdating;
 

-        // only spell check when not read-only 
+        // only spell check when not read-only

         // use system options for the default
         // use system options for the default

-        bool checkSpelling = false ; 
+        bool checkSpelling = false ;

         if ( !(m_windowStyle & wxTE_READONLY) )
         if ( !(m_windowStyle & wxTE_READONLY) )

-        {   
+        {

 #if wxUSE_SYSTEM_OPTIONS
             if ( wxSystemOptions::HasOption( wxMAC_TEXTCONTROL_USE_SPELL_CHECKER ) && (wxSystemOptions::GetOptionInt( wxMAC_TEXTCONTROL_USE_SPELL_CHECKER ) == 1) )
             {
 #if wxUSE_SYSTEM_OPTIONS
             if ( wxSystemOptions::HasOption( wxMAC_TEXTCONTROL_USE_SPELL_CHECKER ) && (wxSystemOptions::GetOptionInt( wxMAC_TEXTCONTROL_USE_SPELL_CHECKER ) == 1) )
             {


@@ -1775,11 +1775,11 @@ void wxMacMLTEControl::AdjustCreationAttributes(const wxColour &background,
             }
 #endif
         }
             }
 #endif
         }

-        
+

         if ( checkSpelling )
             options |=
                 kTXNSupportSpellCheckCommandProcessing
         if ( checkSpelling )
             options |=
                 kTXNSupportSpellCheckCommandProcessing

-                | kTXNSupportSpellCheckCommandUpdating;              
+                | kTXNSupportSpellCheckCommandUpdating;

 
         TXNSetCommandEventSupport( m_txn , options ) ;
     }
 
         TXNSetCommandEventSupport( m_txn , options ) ;
     }

diff --git a/src/mac/carbon/toolbar.cpp b/src/mac/carbon/toolbar.cpp
index 665abf55183bd2902725f21d0f945a4d93205614..02778a1791c5dc59914f0436f535bbf336e38049 100644 (file)
--- a/src/mac/carbon/toolbar.cpp
+++ b/src/mac/carbon/toolbar.cpp
@@ -52,7 +52,7 @@ END_EVENT_TABLE()
 
 // when embedding native controls in the native toolbar we must make sure the
 // control does not get deleted behind our backs, so the retain count gets increased
 
 // when embedding native controls in the native toolbar we must make sure the
 // control does not get deleted behind our backs, so the retain count gets increased

-// (after creation it is 1), first be the creation of the custom HIToolbarItem wrapper 
+// (after creation it is 1), first be the creation of the custom HIToolbarItem wrapper

 // object, and second by the code 'creating' the custom HIView (which is the same as the
 // already existing native control, therefore we just increase the ref count)
 // when this view is removed from the native toolbar its count gets decremented again
 // object, and second by the code 'creating' the custom HIView (which is the same as the
 // already existing native control, therefore we just increase the ref count)
 // when this view is removed from the native toolbar its count gets decremented again


@@ -119,16 +119,19 @@ public:
         if ( m_toolbarItemRef )
         {
             CFIndex count = CFGetRetainCount( m_toolbarItemRef ) ;
         if ( m_toolbarItemRef )
         {
             CFIndex count = CFGetRetainCount( m_toolbarItemRef ) ;

-                       // different behaviour under Leopard
-                       if ( UMAGetSystemVersion() < 0x1050 )
-                       {
-                wxASSERT_MSG( count == 1 , wxT("Reference Count of native tool was not 1 in wxToolBarTool destructor") );
-                       }
+            // different behaviour under Leopard
+            if ( UMAGetSystemVersion() < 0x1050 )
+            {
+                if ( count != 1 )
+                {
+                    wxFAIL_MSG("Reference count of native tool was not 1 in wxToolBarTool destructor");
+                }
+            }

             wxTheApp->MacAddToAutorelease(m_toolbarItemRef);
             CFRelease(m_toolbarItemRef);
             m_toolbarItemRef = NULL;
         }
             wxTheApp->MacAddToAutorelease(m_toolbarItemRef);
             CFRelease(m_toolbarItemRef);
             m_toolbarItemRef = NULL;
         }

-#endif
+#endif // wxMAC_USE_NATIVE_TOOLBAR

     }
 
     wxSize GetSize() const
     }
 
     wxSize GetSize() const


@@ -628,12 +631,15 @@ static pascal OSStatus ControlToolbarItemHandler( EventHandlerCallRef inCallRef,
                             // depending whether the wxControl corresponding to this HIView has already been destroyed or
                             // not, ref counts differ, so we cannot assert a special value
                             CFIndex count =  CFGetRetainCount( viewRef ) ;
                             // depending whether the wxControl corresponding to this HIView has already been destroyed or
                             // not, ref counts differ, so we cannot assert a special value
                             CFIndex count =  CFGetRetainCount( viewRef ) ;

-                            wxASSERT_MSG( count >=1 , wxT("Reference Count of native tool was illegal before removal") );

                             if ( count >= 1 )
                             if ( count >= 1 )

+                            {
+                                wxFAIL_MSG("Reference count of native tool was illegal before removal");
+

                                 CFRelease( viewRef ) ;
                                 CFRelease( viewRef ) ;

+                            }

                         }
                         }

-                           free( object ) ;
-                       result = noErr;
+                        free( object ) ;
+                        result = noErr;

                     }
                     break;
             }
                     }
                     break;
             }


@@ -680,7 +686,7 @@ static pascal OSStatus ControlToolbarItemHandler( EventHandlerCallRef inCallRef,
 
                         // Extra width to avoid edge of combobox being cut off
                         sz.x += 3;
 
                         // Extra width to avoid edge of combobox being cut off
                         sz.x += 3;

-                        
+

                         HISize min, max;
                         min.width = max.width = sz.x ;
                         min.height = max.height = sz.y ;
                         HISize min, max;
                         min.width = max.width = sz.x ;
                         min.height = max.height = sz.y ;


@@ -868,11 +874,14 @@ wxToolBar::~wxToolBar()
             MacInstallNativeToolbar( false );
 
         CFIndex count = CFGetRetainCount( m_macHIToolbarRef ) ;
             MacInstallNativeToolbar( false );
 
         CFIndex count = CFGetRetainCount( m_macHIToolbarRef ) ;

-               // Leopard seems to have one refcount more, so we cannot check reliably at the moment
-               if ( UMAGetSystemVersion() < 0x1050 )
-               {
-            wxASSERT_MSG( count == 1 , wxT("Reference Count of native control was not 1 in wxToolBar destructor") );
-               }
+        // Leopard seems to have one refcount more, so we cannot check reliably at the moment
+        if ( UMAGetSystemVersion() < 0x1050 )
+        {
+            if ( count != 1 )
+            {
+                wxFAIL_MSG("Reference count of native control was not 1 in wxToolBar destructor");
+            }
+        }

         CFRelease( (HIToolbarRef)m_macHIToolbarRef );
         m_macHIToolbarRef = NULL;
     }
         CFRelease( (HIToolbarRef)m_macHIToolbarRef );
         m_macHIToolbarRef = NULL;
     }


@@ -1183,7 +1192,11 @@ bool wxToolBar::Realize()
                                 if ( tool2->IsControl() )
                                 {
                                     CFIndex count = CFGetRetainCount( tool2->GetControl()->GetPeer()->GetControlRef() ) ;
                                 if ( tool2->IsControl() )
                                 {
                                     CFIndex count = CFGetRetainCount( tool2->GetControl()->GetPeer()->GetControlRef() ) ;

-                                    wxASSERT_MSG( count == 3 || count == 2 , wxT("Reference Count of native tool was illegal before removal") );
+                                    if ( count != 3 && count != 2 )
+                                    {
+                                        wxFAIL_MSG("Reference count of native tool was illegal before removal");
+                                    }
+

                                     wxASSERT( IsValidControlHandle(tool2->GetControl()->GetPeer()->GetControlRef() )) ;
                                 }
                                 err = HIToolbarRemoveItemAtIndex(refTB, idx);
                                     wxASSERT( IsValidControlHandle(tool2->GetControl()->GetPeer()->GetControlRef() )) ;
                                 }
                                 err = HIToolbarRemoveItemAtIndex(refTB, idx);


@@ -1195,7 +1208,11 @@ bool wxToolBar::Realize()
                                 if ( tool2->IsControl() )
                                 {
                                     CFIndex count = CFGetRetainCount( tool2->GetControl()->GetPeer()->GetControlRef() ) ;
                                 if ( tool2->IsControl() )
                                 {
                                     CFIndex count = CFGetRetainCount( tool2->GetControl()->GetPeer()->GetControlRef() ) ;

-                                    wxASSERT_MSG( count == 2 , wxT("Reference Count of native tool was not 2 after removal") );
+                                    if ( count != 2 )
+                                    {
+                                        wxFAIL_MSG("Reference count of native tool was not 2 after removal");
+                                    }
+

                                     wxASSERT( IsValidControlHandle(tool2->GetControl()->GetPeer()->GetControlRef() )) ;
                                 }
 
                                     wxASSERT( IsValidControlHandle(tool2->GetControl()->GetPeer()->GetControlRef() )) ;
                                 }
 


@@ -1214,7 +1231,10 @@ bool wxToolBar::Realize()
                     if ( tool->IsControl() )
                     {
                         CFIndex count = CFGetRetainCount( tool->GetControl()->GetPeer()->GetControlRef() ) ;
                     if ( tool->IsControl() )
                     {
                         CFIndex count = CFGetRetainCount( tool->GetControl()->GetPeer()->GetControlRef() ) ;

-                        wxASSERT_MSG( count == 3 || count == 2, wxT("Reference Count of native tool was illegal after insertion") );
+                        if ( count != 3 && count != 2 )
+                        {
+                            wxFAIL_MSG("Reference count of native tool was illegal before removal");
+                        }

                         wxASSERT( IsValidControlHandle(tool->GetControl()->GetPeer()->GetControlRef() )) ;
                     }
                 }
                         wxASSERT( IsValidControlHandle(tool->GetControl()->GetPeer()->GetControlRef() )) ;
                     }
                 }

diff --git a/src/mac/carbon/toplevel.cpp b/src/mac/carbon/toplevel.cpp
index 12e9408f9b678213a283682557317f37b4a1b9b5..b26e92f1ce18d50563b2ddf01502cc470cddad67 100644 (file)
--- a/src/mac/carbon/toplevel.cpp
+++ b/src/mac/carbon/toplevel.cpp
@@ -61,7 +61,7 @@
 #define kWindowUnifiedTitleAndToolbarAttribute (1 << 7)
 
 // trace mask for activation tracing messages
 #define kWindowUnifiedTitleAndToolbarAttribute (1 << 7)
 
 // trace mask for activation tracing messages

-static const wxChar *TRACE_ACTIVATE = _T("activation");
+#define TRACE_ACTIVATE "activation"

 
 // ----------------------------------------------------------------------------
 // globals
 
 // ----------------------------------------------------------------------------
 // globals

diff --git a/src/mac/carbon/utils.cpp b/src/mac/carbon/utils.cpp
index e302a68cdd15bad134a14efb658bcda4e00ef076..02f46c0bc5f69b170aa8b6e25a56d72e6556ee0f 100644 (file)
--- a/src/mac/carbon/utils.cpp
+++ b/src/mac/carbon/utils.cpp
@@ -314,13 +314,12 @@ wxWindow* wxFindWindowAtPoint(const wxPoint& pt)
 
 wxString wxGetOsDescription()
 {
 
 wxString wxGetOsDescription()
 {

-    char data[128];

     struct utsname name;
     uname(&name);
     struct utsname name;
     uname(&name);

-       return wxString::Format(_T("Mac OS X (%s %s %s)"),
-                       wxString::FromAscii(name.sysname).c_str(),
-                       wxString::FromAscii(name.release).c_str(),
-                       wxString::FromAscii(name.machine).c_str());
+    return wxString::Format(_T("Mac OS X (%s %s %s)"),
+            wxString::FromAscii(name.sysname).c_str(),
+            wxString::FromAscii(name.release).c_str(),
+            wxString::FromAscii(name.machine).c_str());

 }
 
 #ifndef __DARWIN__
 }
 
 #ifndef __DARWIN__


@@ -1561,7 +1560,7 @@ unsigned int wxMacDataItemBrowserControl::GetLineFromItem(const wxMacDataItem* i
 {
     DataBrowserTableViewRowIndex row;
     OSStatus err = GetItemRow( (DataBrowserItemID) item , &row);
 {
     DataBrowserTableViewRowIndex row;
     OSStatus err = GetItemRow( (DataBrowserItemID) item , &row);

-    wxASSERT( err == noErr);
+    wxCHECK( err == noErr, (unsigned)-1 );

     return row;
 }
 
     return row;
 }
 


@@ -1569,7 +1568,7 @@ wxMacDataItem*  wxMacDataItemBrowserControl::GetItemFromLine(unsigned int n) con
 {
     DataBrowserItemID id;
     OSStatus err =  GetItemID( (DataBrowserTableViewRowIndex) n , &id);
 {
     DataBrowserItemID id;
     OSStatus err =  GetItemID( (DataBrowserTableViewRowIndex) n , &id);

-    wxASSERT( err == noErr);
+    wxCHECK( err == noErr, NULL );

     return (wxMacDataItem*) id;
 }
 
     return (wxMacDataItem*) id;
 }
 

diff --git a/src/mac/carbon/window.cpp b/src/mac/carbon/window.cpp
index 11a7167c4a43b26fc679d5251abe2dbe34bda7f3..acb4a6b6cbb97ee3573a82936da9374a3cd498e7 100644 (file)
--- a/src/mac/carbon/window.cpp
+++ b/src/mac/carbon/window.cpp
@@ -232,7 +232,11 @@ static pascal OSStatus wxMacWindowControlEventHandler( EventHandlerCallRef handl
                     bool created = false ;
                     CGContextRef cgContext = NULL ;
                     OSStatus err = cEvent.GetParameter<CGContextRef>(kEventParamCGContextRef, &cgContext) ;
                     bool created = false ;
                     CGContextRef cgContext = NULL ;
                     OSStatus err = cEvent.GetParameter<CGContextRef>(kEventParamCGContextRef, &cgContext) ;

-                    wxASSERT_MSG( err == noErr , wxT("Unable to retrieve CGContextRef") ) ;
+                    if ( err != noErr )
+                    {
+                        wxFAIL_MSG("Unable to retrieve CGContextRef");
+                    }
+

                     thisWindow->MacSetCGContextRef( cgContext ) ;
 
                     {
                     thisWindow->MacSetCGContextRef( cgContext ) ;
 
                     {


@@ -3216,13 +3220,13 @@ bool wxWindowMac::IsShownOnScreen() const
             // CS : put a breakpoint here to investigate differences
             // between native an wx visibilities
             // the only place where I've encountered them until now
             // CS : put a breakpoint here to investigate differences
             // between native an wx visibilities
             // the only place where I've encountered them until now

-            // are the hiding/showing sequences where the vis-changed event is 
+            // are the hiding/showing sequences where the vis-changed event is

             // first sent to the innermost control, while wx does things
             // from the outmost control
             wxVis = wxWindowBase::IsShownOnScreen();
             return wxVis;
         }
             // first sent to the innermost control, while wx does things
             // from the outmost control
             wxVis = wxWindowBase::IsShownOnScreen();
             return wxVis;
         }

-        
+

         return m_peer->IsVisible();
     }
 #endif
         return m_peer->IsVisible();
     }
 #endif