create stock GDI objects on demand; use const with GDI objects appropriately (patch...

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

diff --git a/src/mac/carbon/icon.cpp b/src/mac/carbon/icon.cpp
index 27cc3149d219eea3446930935f021dd99e32f747..3473bf72baf3c9f993e1185c103a0ee4d4155010 100644 (file)
--- a/src/mac/carbon/icon.cpp
+++ b/src/mac/carbon/icon.cpp
@@ -9,63 +9,54 @@
 // Licence:       wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:       wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-#pragma implementation "icon.h"
-#endif
-

 #include "wx/wxprec.h"
 
 #include "wx/icon.h"
 #include "wx/wxprec.h"
 
 #include "wx/icon.h"

-
-#if !USE_SHARED_LIBRARIES
-IMPLEMENT_DYNAMIC_CLASS(wxIcon, wxBitmap)
-#endif
-

 #include "wx/image.h"
 #include "wx/mac/private.h"
 
 #include "wx/image.h"
 #include "wx/mac/private.h"
 

-#define M_ICONDATA ((wxIconRefData *)m_refData)
+IMPLEMENT_DYNAMIC_CLASS(wxIcon, wxBitmap)

 
 

+#define M_ICONDATA ((wxIconRefData *)m_refData)

 
 

-/*
- * Icons
- */

 
 wxIcon::wxIcon()
 {
 }
 
 
 wxIcon::wxIcon()
 {
 }
 

-wxIcon::wxIcon(const char bits[], int width, int height) 
+wxIcon::wxIcon( const char bits[], int width, int height )

 {
 {

-    wxBitmap bmp(bits,width,height) ;
+    wxBitmap bmp( bits, width, height ) ;

     CopyFromBitmap( bmp ) ;
 }
 
     CopyFromBitmap( bmp ) ;
 }
 

-wxIcon::wxIcon( const char **bits ) 
+wxIcon::wxIcon( const char **bits )

 {
 {

-    wxBitmap bmp(bits) ;
+    wxBitmap bmp( bits ) ;

     CopyFromBitmap( bmp ) ;
 }
 
     CopyFromBitmap( bmp ) ;
 }
 

-wxIcon::wxIcon( char **bits ) 
+wxIcon::wxIcon( char **bits )

 {
 {

-    wxBitmap bmp(bits) ;
+    wxBitmap bmp( bits ) ;

     CopyFromBitmap( bmp ) ;
 }
 
     CopyFromBitmap( bmp ) ;
 }
 

-wxIcon::wxIcon(const wxString& icon_file, int flags,
-    int desiredWidth, int desiredHeight)
+wxIcon::wxIcon(
+    const wxString& icon_file, int flags,
+    int desiredWidth, int desiredHeight )

 {
 {

-    LoadFile(icon_file, (wxBitmapType) flags, desiredWidth, desiredHeight);
+    LoadFile( icon_file, (wxBitmapType) flags, desiredWidth, desiredHeight );

 }
 
 wxIcon::~wxIcon()
 {
 }
 
 }
 
 wxIcon::~wxIcon()
 {
 }
 

-WXHICON wxIcon::GetHICON() const 
+WXHICON wxIcon::GetHICON() const

 {
     wxASSERT( Ok() ) ;
 {
     wxASSERT( Ok() ) ;

+

     return (WXHICON) ((wxIconRefData*)m_refData)->GetHICON() ;
 }
 
     return (WXHICON) ((wxIconRefData*)m_refData)->GetHICON() ;
 }
 


@@ -83,20 +74,21 @@ int wxIcon::GetHeight() const
    return M_ICONDATA->GetHeight();
 }
 
    return M_ICONDATA->GetHeight();
 }
 

-int wxIcon::GetDepth() const{
-       return 32; 
+int wxIcon::GetDepth() const
+{
+    return 32;

 }
 
 }
 

-void wxIcon::SetDepth(int depth){
-       
+void wxIcon::SetDepth( int depth )
+{

 }
 
 }
 

-void wxIcon::SetWidth(int width){
-       
+void wxIcon::SetWidth( int width )
+{

 }
 
 }
 

-void wxIcon::SetHeight(int height){
-       
+void wxIcon::SetHeight( int height )
+{

 }
 
 bool wxIcon::Ok() const
 }
 
 bool wxIcon::Ok() const


@@ -104,14 +96,16 @@ bool wxIcon::Ok() const
     return m_refData != NULL ;
 }
 
     return m_refData != NULL ;
 }
 

-bool wxIcon::LoadFile(const wxString& filename, wxBitmapType type,
-    int desiredWidth, int desiredHeight)
+bool wxIcon::LoadFile(
+    const wxString& filename, wxBitmapType type,
+    int desiredWidth, int desiredHeight )

 {
     UnRef();
 {
     UnRef();

-    
+

     if ( type == wxBITMAP_TYPE_ICON_RESOURCE )
     {
         OSType theId = 0 ;
     if ( type == wxBITMAP_TYPE_ICON_RESOURCE )
     {
         OSType theId = 0 ;

+

         if ( filename == wxT("wxICON_INFORMATION") )
         {
             theId = kAlertNoteIcon ;
         if ( filename == wxT("wxICON_INFORMATION") )
         {
             theId = kAlertNoteIcon ;


@@ -129,49 +123,56 @@ bool wxIcon::LoadFile(const wxString& filename, wxBitmapType type,
             theId = kAlertStopIcon ;
         }
         else
             theId = kAlertStopIcon ;
         }
         else

-        {/*
+        {
+#if 0

             Str255 theName ;
             OSType theType ;
             wxMacStringToPascal( name , theName ) ;
             Str255 theName ;
             OSType theType ;
             wxMacStringToPascal( name , theName ) ;

-            
+

             Handle resHandle = GetNamedResource( 'cicn' , theName ) ;
             if ( resHandle != 0L )
             {
                 GetResInfo( resHandle , &theId , &theType , theName ) ;
                 ReleaseResource( resHandle ) ;
             }
             Handle resHandle = GetNamedResource( 'cicn' , theName ) ;
             if ( resHandle != 0L )
             {
                 GetResInfo( resHandle , &theId , &theType , theName ) ;
                 ReleaseResource( resHandle ) ;
             }

-        */
+#endif

         }
         }

+

         if ( theId != 0 )
         {
             IconRef iconRef = NULL ;
         if ( theId != 0 )
         {
             IconRef iconRef = NULL ;

-            verify_noerr(GetIconRef(kOnSystemDisk,kSystemIconsCreator,theId, &iconRef)) ;    
+            verify_noerr( GetIconRef( kOnSystemDisk, kSystemIconsCreator, theId, &iconRef ) ) ;

             if ( iconRef )
             {
                 m_refData = new wxIconRefData( (WXHICON) iconRef ) ;
             if ( iconRef )
             {
                 m_refData = new wxIconRefData( (WXHICON) iconRef ) ;

-                return TRUE ;
+
+                return true ;

             }
         }
             }
         }

-        return FALSE ;
+
+        return false ;

     }
     else
     {
     }
     else
     {

-        wxBitmapHandler *handler = wxBitmap::FindHandler(type);
+        wxBitmapHandler *handler = wxBitmap::FindHandler( type );

 
         if ( handler )
         {
             wxBitmap bmp ;
 
         if ( handler )
         {
             wxBitmap bmp ;

-            if ( handler->LoadFile(&bmp , filename, type, desiredWidth, desiredHeight ))
+            if ( handler->LoadFile( &bmp , filename, type, desiredWidth, desiredHeight ))

             {
                 CopyFromBitmap( bmp ) ;
             {
                 CopyFromBitmap( bmp ) ;

+

                 return true ;
             }
                 return true ;
             }

+

             return false ;
         }
         else
         {
             return false ;
         }
         else
         {

-            wxImage loadimage(filename, type);
-            if (loadimage.Ok()) 
+#if wxUSE_IMAGE
+            wxImage loadimage( filename, type );
+            if (loadimage.Ok())

             {
                 if ( desiredWidth == -1 )
                     desiredWidth = loadimage.GetWidth() ;
             {
                 if ( desiredWidth == -1 )
                     desiredWidth = loadimage.GetWidth() ;


@@ -179,20 +180,27 @@ bool wxIcon::LoadFile(const wxString& filename, wxBitmapType type,
                     desiredHeight = loadimage.GetHeight() ;
                 if ( desiredWidth != loadimage.GetWidth() || desiredHeight != loadimage.GetHeight() )
                     loadimage.Rescale( desiredWidth , desiredHeight ) ;
                     desiredHeight = loadimage.GetHeight() ;
                 if ( desiredWidth != loadimage.GetWidth() || desiredHeight != loadimage.GetHeight() )
                     loadimage.Rescale( desiredWidth , desiredHeight ) ;

+    

                 wxBitmap bmp( loadimage );
                 CopyFromBitmap( bmp ) ;
                 wxBitmap bmp( loadimage );
                 CopyFromBitmap( bmp ) ;

+

                 return true;
             }
                 return true;
             }

+#endif

         }
     }
     return true ;
 }
 
         }
     }
     return true ;
 }
 

-void wxIcon::CopyFromBitmap(const wxBitmap& bmp)
+void wxIcon::CopyFromBitmap( const wxBitmap& bmp )

 {
     UnRef() ;
 {
     UnRef() ;

-    
-    m_refData = new wxIconRefData( (WXHICON) wxMacCreateIconRef( bmp ) ) ;
+
+    // as the bitmap owns that ref, we have to acquire it as well
+    IconRef iconRef = bmp.GetBitmapData()->GetIconRef() ;
+    AcquireIconRef( iconRef ) ;
+
+    m_refData = new wxIconRefData( (WXHICON) iconRef ) ;

     M_ICONDATA->SetWidth( bmp.GetWidth() ) ;
     M_ICONDATA->SetHeight( bmp.GetHeight() ) ;
 }
     M_ICONDATA->SetWidth( bmp.GetWidth() ) ;
     M_ICONDATA->SetHeight( bmp.GetHeight() ) ;
 }


@@ -200,12 +208,13 @@ void wxIcon::CopyFromBitmap(const wxBitmap& bmp)
 wxIconRefData::wxIconRefData( WXHICON icon )
 {
     m_iconRef = MAC_WXHICON( icon ) ;
 wxIconRefData::wxIconRefData( WXHICON icon )
 {
     m_iconRef = MAC_WXHICON( icon ) ;

-    // Std sizes
+
+    // Standard sizes

     SetWidth( 32 ) ;
     SetHeight( 32 ) ;
 }
 
     SetWidth( 32 ) ;
     SetHeight( 32 ) ;
 }
 

-void wxIconRefData::Init() 
+void wxIconRefData::Init()

 {
     m_iconRef = NULL ;
 }
 {
     m_iconRef = NULL ;
 }


@@ -221,11 +230,14 @@ void wxIconRefData::Free()
 
 IMPLEMENT_DYNAMIC_CLASS(wxICONResourceHandler, wxBitmapHandler)
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxICONResourceHandler, wxBitmapHandler)
 

-bool  wxICONResourceHandler::LoadFile(wxBitmap *bitmap, const wxString& name, long flags,
-                                      int desiredWidth, int desiredHeight)
+bool  wxICONResourceHandler::LoadFile(
+    wxBitmap *bitmap, const wxString& name, long flags,
+    int desiredWidth, int desiredHeight )

 {
     wxIcon icon ;
     icon.LoadFile( name , wxBITMAP_TYPE_ICON_RESOURCE , desiredWidth , desiredHeight ) ;
     bitmap->CopyFromIcon( icon ) ;
 {
     wxIcon icon ;
     icon.LoadFile( name , wxBITMAP_TYPE_ICON_RESOURCE , desiredWidth , desiredHeight ) ;
     bitmap->CopyFromIcon( icon ) ;

+

     return bitmap->Ok() ;
 }
     return bitmap->Ok() ;
 }

+