Avoid g++ warnings about breaking strict aliasing rules in wxTreeCtrl.

[wxWidgets.git] / src / osx / cocoa / button.mm

diff --git a/src/osx/cocoa/button.mm b/src/osx/cocoa/button.mm
index 8e25f09c48e46210cfa193ca8e53d737a0ade4f4..aa3f632b5f84a6eee050e15936c065bfcb41befe 100644 (file)
--- a/src/osx/cocoa/button.mm
+++ b/src/osx/cocoa/button.mm
@@ -4,7 +4,7 @@
 // Author:      Stefan Csomor
 // Modified by:
 // Created:     1998-01-01
 // Author:      Stefan Csomor
 // Modified by:
 // Created:     1998-01-01

-// RCS-ID:      $Id: button.cpp 54845 2008-07-30 14:52:41Z SC $
+// RCS-ID:      $Id$

 // Copyright:   (c) Stefan Csomor
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 // Copyright:   (c) Stefan Csomor
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////


@@ -23,36 +23,6 @@
 #endif // wxUSE_MARKUP
 
 
 #endif // wxUSE_MARKUP
 
 

-wxSize wxButton::DoGetBestSize() const
-{
-    // We only use help button bezel if we don't have any (non standard) label
-    // to display in the button. Otherwise even wxID_HELP buttons look like
-    // normal push buttons.
-    if ( GetId() == wxID_HELP && GetLabel().empty() )
-        return wxSize( 23 , 23 ) ;
-
-    wxRect r ;
-    m_peer->GetBestRect(&r);
-
-    wxSize sz = r.GetSize();
-    sz.x  = sz.x  + MacGetLeftBorderSize() +
-    MacGetRightBorderSize();
-    sz.y = sz.y + MacGetTopBorderSize() +
-    MacGetBottomBorderSize();
-    
-    const int wBtnStd = GetDefaultSize().x;
-
-    if ( (sz.x < wBtnStd) && !HasFlag(wxBU_EXACTFIT) )
-        sz.x = wBtnStd;
-
-    return sz ;
-}
-
-wxSize wxButton::GetDefaultSize()
-{
-    return wxSize(84, 20);
-}
-

 @implementation wxNSButton
 
 + (void)initialize
 @implementation wxNSButton
 
 + (void)initialize


@@ -130,6 +100,7 @@ public:
         wxWidgetCocoaImpl::SetBitmap(bitmap);
     }
 
         wxWidgetCocoaImpl::SetBitmap(bitmap);
     }
 

+#if wxUSE_MARKUP

     virtual void SetLabelMarkup(const wxString& markup)
     {
         wxMarkupToAttrString toAttr(GetWXPeer(), markup);
     virtual void SetLabelMarkup(const wxString& markup)
     {
         wxMarkupToAttrString toAttr(GetWXPeer(), markup);


@@ -146,6 +117,7 @@ public:
 
         [GetNSButton() setAttributedTitle:attrString];
     }
 
         [GetNSButton() setAttributedTitle:attrString];
     }

+#endif // wxUSE_MARKUP

 
     void SetPressedBitmap( const wxBitmap& bitmap )
     {
 
     void SetPressedBitmap( const wxBitmap& bitmap )
     {


@@ -287,7 +259,7 @@ wxWidgetImplType* wxWidgetImpl::CreateBitmapButton( wxWindowMac* wxpeer,
 
     SetBezelStyleFromBorderFlags(v, style);
 
 
     SetBezelStyleFromBorderFlags(v, style);
 

-    if (bitmap.Ok())
+    if (bitmap.IsOk())

         [v setImage:bitmap.GetNSImage() ];
 
     [v setButtonType:NSMomentaryPushInButton];
         [v setImage:bitmap.GetNSImage() ];
 
     [v setButtonType:NSMomentaryPushInButton];


@@ -377,7 +349,7 @@ wxCFRef<NSImage*> downArray ;
     static wxBitmap trianglebm(disc_triangle_xpm);
     if ( downArray.get() == NULL )
     {
     static wxBitmap trianglebm(disc_triangle_xpm);
     if ( downArray.get() == NULL )
     {

-        downArray.reset( [wxDisclosureNSButton rotateImage:trianglebm.GetNSImage()] );
+        downArray.reset( [[wxDisclosureNSButton rotateImage:trianglebm.GetNSImage()] retain] );

     }
 
     if ( isOpen )
     }
 
     if ( isOpen )


@@ -405,7 +377,7 @@ wxCFRef<NSImage*> downArray ;
         fromRect:NSZeroRect operation:NSCompositeCopy fraction:1.0];
 
     [newImage unlockFocus];
         fromRect:NSZeroRect operation:NSCompositeCopy fraction:1.0];
 
     [newImage unlockFocus];

-    return newImage;
+    return [newImage autorelease];

 }
 
 @end
 }
 
 @end