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;
     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 ) ;
     }
 
         wxMacCoreGraphicsColour();
         wxMacCoreGraphicsColour(const wxBrush &brush);
         ~wxMacCoreGraphicsColour();
-        
+
         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;
     }
 }
 
-wxMacCoreGraphicsColour::wxMacCoreGraphicsColour() 
+wxMacCoreGraphicsColour::wxMacCoreGraphicsColour()
 {
     Init();
 }
     CTFontRef GetCTFont() const { return m_ctFont ; }
 #endif
     wxColour GetColour() const { return m_colour ; }
-    
+
     bool GetUnderlined() const { return m_underlined ; }
 private :
     wxColour m_colour;
     // 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();
 
     virtual void GetSize( wxDouble* width, wxDouble* height);
 
     virtual void StartPage( wxDouble width, wxDouble height );
-    
+
     virtual void EndPage();
 
     virtual void Flush();
     virtual void DrawPath( const wxGraphicsPath &path, int fillStyle = wxODDEVEN_RULE );
 
     virtual bool ShouldOffset() const
-    {     
+    {
         int penwidth = 0 ;
         if ( !m_pen.IsNull() )
         {
         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 );
 }
-    
+
 void wxMacCoreGraphicsContext::EndPage()
 {
     CGContextEndPage(m_cgContext);
 {
     if ( !m_cgContext )
     {
-        OSStatus status = 
+        OSStatus status =
 #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 );
-               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 );
-                       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)
         return true;
 
     EnsureIsValid();
-    
+
     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 )
     {
             case wxXOR :
                 mode = kCGBlendModeXOR;
                 shouldAntiAlias = false;
-                break;  
+                break;
             default :
                 retval = false;
                 break;
             retval = true;
         }
     }
-    
+
     if (retval)
     {
         m_logicalFunction = function;
         // 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);
     }
     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));
 {
     if ( m_font.IsNull() )
         return;
-    
+
     EnsureIsValid();
 #if wxMAC_USE_CORE_TEXT
     if ( UMAGetSystemVersion() >= 0x1050 )
         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 };
-        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) );
-        
+
         y += CTFontGetAscent(font);
-        
+
         CGContextSaveGState(m_cgContext);
         CGContextTranslateCTM(m_cgContext, x, y);
         CGContextScaleCTM(m_cgContext, 1, -1);
         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") );
-        
+
         status = ::ATSUSetTransientFontMatching( atsuLayout , true );
         wxASSERT_MSG( status == noErr , wxT("couldn't setup transient font matching") );
-        
+
         int iAngle = int( angle * RAD2DEG );
         if ( abs(iAngle) > 0 )
         {
             status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags) / sizeof(ATSUAttributeTag),
                                              atsuTags, atsuSizes, atsuValues );
         }
-        
+
         {
             ATSUAttributeTag atsuTags[] =
             {
             status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags) / sizeof(ATSUAttributeTag),
                                              atsuTags, atsuSizes, atsuValues );
         }
-        
+
         ATSUTextMeasurement textBefore, textAfter;
         ATSUTextMeasurement ascent, descent;
-        
+
         status = ::ATSUGetUnjustifiedBounds( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
                                             &textBefore , &textAfter, &ascent , &descent );
-        
+
         wxASSERT_MSG( status == noErr , wxT("couldn't measure the rotated text") );
-        
+
         Rect rect;
         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") );
-        
+
         CGContextSaveGState(m_cgContext);
         CGContextTranslateCTM(m_cgContext, x, y);
         CGContextScaleCTM(m_cgContext, 1, -1);
         status = ::ATSUDrawText( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
                                 IntToFixed(0) , IntToFixed(0) );
-        
+
         wxASSERT_MSG( status == noErr , wxT("couldn't draw the rotated text") );
-        
+
         CGContextRestoreGState(m_cgContext);
-        
+
         ::ATSUDisposeTextLayout(atsuLayout);
-        
+
         return;
     }
 #endif
         *descent = 0;
     if ( externalLeading )
         *externalLeading = 0;
-        
+
     if (str.empty())
         return;
 
         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) );
-            
+
         CGFloat w, a, d, l;
-        
+
         w = CTLineGetTypographicBounds(line, &a, &d, &l) ;
-        
+
         if ( height )
             *height = a+d+l;
         if ( descent )
 #if wxMAC_USE_ATSU_TEXT
     {
         OSStatus status = noErr;
-        
+
         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 );
-        
+
         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 = ::ATSUGetUnjustifiedBounds( atsuLayout, kATSUFromTextBeginning, kATSUToTextEnd,
                                             &textBefore , &textAfter, &textAscent , &textDescent );
-        
+
         if ( height )
             *height = FixedToInt(textAscent + textDescent);
         if ( descent )
             *externalLeading = 0;
         if ( width )
             *width = FixedToInt(textAfter - textBefore);
-        
+
         ::ATSUDisposeTextLayout(atsuLayout);
 
         return;
     {
         wxMacCoreGraphicsFontData* fref = (wxMacCoreGraphicsFontData*)m_font.GetRefData();
         CTFontRef font = fref->GetCTFont();
-        
+
         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) );
-        
+
         int chars = text.length();
         for ( int pos = 0; pos < (int)chars; pos ++ )
         {
             widths[pos] = CTLineGetOffsetForStringIndex( line, pos+1 , NULL )+0.5;
         }
-        
+
         return;
     }
 #endif
         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 );
-                
+
         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") );
 
         {
             unsigned long actualNumberOfBounds = 0;
             ATSTrapezoid glyphBounds;
-            
+
             // We get a single bound, since the text should only require one. If it requires more, there is an issue
             OSStatus result;
             result = ATSUGetGlyphBounds( atsuLayout, 0, 0, kATSUFromTextBeginning, pos + 1,
                                         kATSUseDeviceOrigins, 1, &glyphBounds, &actualNumberOfBounds );
             if (result != noErr || actualNumberOfBounds != 1 )
                 return;
-            
+
             widths[pos] = FixedToInt( glyphBounds.upperRight.x - glyphBounds.upperLeft.x );
             //unsigned char uch = s[i];
         }
-        
+
         ::ATSUDisposeTextLayout(atsuLayout);
     }
 #endif
     virtual wxGraphicsContext * CreateContextFromNativeWindow( void * window );
 
     virtual wxGraphicsContext * CreateContext( wxWindow* window );
-    
+
     virtual wxGraphicsContext * CreateMeasuringContext();
 
     // Path
 
     // 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 :
     {
         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
         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 )
     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* b = new wxMemoryBuffer( buf );
     if ( b->GetDataLen() == 0 )
         return NULL;
-        
+
     return CGDataProviderCreateWithData( b , (const void *) b->GetData() , b->GetDataLen() ,
                                                  wxMacReleaseMemoryBufferProviderCallback );
-}
\ No newline at end of file
+}
 
 
 // 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
         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;
         }
-#endif
+#endif // wxMAC_USE_NATIVE_TOOLBAR
     }
 
     wxSize GetSize() const
                             // 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 )
+                            {
+                                wxFAIL_MSG("Reference count of native tool was illegal before removal");
+
                                 CFRelease( viewRef ) ;
+                            }
                         }
-                           free( object ) ;
-                       result = noErr;
+                        free( object ) ;
+                        result = noErr;
                     }
                     break;
             }
 
                         // 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 ;
             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;
     }
                                 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);
                                 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() )) ;
                                 }
 
                     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() )) ;
                     }
                 }