compilation warning fix

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

diff --git a/src/mac/carbon/bitmap.cpp b/src/mac/carbon/bitmap.cpp
index ddabfe2384f0cbabafbd3d9cf80bca52f6eef009..cd1c398c99df8882a16750720629cbdd5fe72a72 100644 (file)
--- a/src/mac/carbon/bitmap.cpp
+++ b/src/mac/carbon/bitmap.cpp
@@ -218,20 +218,24 @@ CIconHandle wxMacCreateCIcon(GWorldPtr image , GWorldPtr mask , short dstDepth ,
 
     if ( mask )
     {
 
     if ( mask )
     {

+       Rect r ;
+       GetPortBounds( image , &r ) ;

       LockPixels(GetGWorldPixMap(mask) ) ;
       CopyBits(GetPortBitMapForCopyBits(mask) ,
       LockPixels(GetGWorldPixMap(mask) ) ;
       CopyBits(GetPortBitMapForCopyBits(mask) ,

-          &(**icon).iconBMap , &imageBounds , &imageBounds, srcCopy , nil ) ;
+          &(**icon).iconBMap , &r , &r, srcCopy , nil ) ;

       CopyBits(GetPortBitMapForCopyBits(mask) ,
       CopyBits(GetPortBitMapForCopyBits(mask) ,

-          &(**icon).iconMask , &imageBounds , &imageBounds, srcCopy , nil ) ;
+          &(**icon).iconMask , &r , &r, srcCopy , nil ) ;

       UnlockPixels(GetGWorldPixMap( mask ) ) ;
     }
     else
     {
       UnlockPixels(GetGWorldPixMap( mask ) ) ;
     }
     else
     {

+       Rect r ;
+       GetPortBounds( image , &r ) ;

         LockPixels(GetGWorldPixMap(image));
       CopyBits(GetPortBitMapForCopyBits(image) ,
         LockPixels(GetGWorldPixMap(image));
       CopyBits(GetPortBitMapForCopyBits(image) ,

-          &(**icon).iconBMap , &imageBounds , &imageBounds, srcCopy , nil ) ;
+          &(**icon).iconBMap , &r , &r, srcCopy , nil ) ;

       CopyBits(GetPortBitMapForCopyBits(image) ,
       CopyBits(GetPortBitMapForCopyBits(image) ,

-          &(**icon).iconMask , &imageBounds , &imageBounds, srcCopy , nil ) ;
+          &(**icon).iconMask , &r , &r, srcCopy , nil ) ;

         UnlockPixels(GetGWorldPixMap(image));
     }
     
         UnlockPixels(GetGWorldPixMap(image));
     }
     


@@ -268,14 +272,19 @@ PicHandle wxMacCreatePict(GWorldPtr wp, GWorldPtr mask)
     
   SetGWorld( wp , NULL ) ;
   Rect portRect ;
     
   SetGWorld( wp , NULL ) ;
   Rect portRect ;

-  GetPortBounds( wp , &portRect ) ;
-
+  if ( clipRgn )
+    GetRegionBounds( clipRgn , &portRect ) ;   
+  else
+       GetPortBounds( wp , &portRect ) ;

   pict = OpenPicture(&portRect);   
   if(pict)  
   {
     RGBForeColor( &black ) ;
     RGBBackColor( &white ) ;
   pict = OpenPicture(&portRect);   
   if(pict)  
   {
     RGBForeColor( &black ) ;
     RGBBackColor( &white ) ;

-
+    
+    if ( clipRgn )
+       SetClip( clipRgn ) ;
+    

     LockPixels( GetGWorldPixMap( wp ) ) ;
     CopyBits(GetPortBitMapForCopyBits(wp),          
             GetPortBitMapForCopyBits(wp),       
     LockPixels( GetGWorldPixMap( wp ) ) ;
     CopyBits(GetPortBitMapForCopyBits(wp),          
             GetPortBitMapForCopyBits(wp),       


@@ -286,10 +295,12 @@ PicHandle wxMacCreatePict(GWorldPtr wp, GWorldPtr mask)
     ClosePicture();                 
   }
   SetGWorld( origPort , origDev ) ;
     ClosePicture();                 
   }
   SetGWorld( origPort , origDev ) ;

+  if ( clipRgn )
+       DisposeRgn( clipRgn ) ;

   return pict;                  
 }
 
   return pict;                  
 }
 

-void wxMacCreateBitmapButton( ControlButtonContentInfo*info , const wxBitmap& bitmap ) 
+void wxMacCreateBitmapButton( ControlButtonContentInfo*info , const wxBitmap& bitmap , int forceType ) 

 {
     memset( info , 0 , sizeof(ControlButtonContentInfo) ) ;
     if ( bitmap.Ok() )
 {
     memset( info , 0 , sizeof(ControlButtonContentInfo) ) ;
     if ( bitmap.Ok() )


@@ -305,11 +316,11 @@ void wxMacCreateBitmapButton( ControlButtonContentInfo*info , const wxBitmap& bi
         }
         else if ( bmap->m_bitmapType == kMacBitmapTypeGrafWorld )
         {
         }
         else if ( bmap->m_bitmapType == kMacBitmapTypeGrafWorld )
         {

-            if ( (bmap->m_width == bmap->m_height) && (bmap->m_width & 0x3 == 0) )
+            if ( (forceType == kControlContentCIconHandle || ( bmap->m_width == bmap->m_height && forceType != kControlContentPictHandle ) ) && ((bmap->m_width & 0x3) == 0) )

             {
                 info->contentType = kControlContentCIconHandle ;
                 if ( bitmap.GetMask() )
             {
                 info->contentType = kControlContentCIconHandle ;
                 if ( bitmap.GetMask() )

-                {
+                { 

                     info->u.cIconHandle = wxMacCreateCIcon( MAC_WXHBITMAP(bmap->m_hBitmap) , MAC_WXHBITMAP(bitmap.GetMask()->GetMaskBitmap()) ,
                                                            8 , bmap->m_width ) ;
                 }
                     info->u.cIconHandle = wxMacCreateCIcon( MAC_WXHBITMAP(bmap->m_hBitmap) , MAC_WXHBITMAP(bitmap.GetMask()->GetMaskBitmap()) ,
                                                            8 , bmap->m_width ) ;
                 }


@@ -767,6 +778,8 @@ wxImage wxBitmap::ConvertToImage() const
 
     GWorldPtr origPort;
     GDHandle  origDevice;
 
     GWorldPtr origPort;
     GDHandle  origDevice;

+    RgnHandle maskRgn = NULL ;
+    GWorldPtr tempPort = NULL ;

     int      index;
     RGBColor color;
     // background color set to RGB(16,16,16) in consistent with wxGTK
     int      index;
     RGBColor color;
     // background color set to RGB(16,16,16) in consistent with wxGTK


@@ -775,9 +788,28 @@ wxImage wxBitmap::ConvertToImage() const
     wxMask  *mask = GetMask();
 
     GetGWorld( &origPort, &origDevice );
     wxMask  *mask = GetMask();
 
     GetGWorld( &origPort, &origDevice );

-    LockPixels(GetGWorldPixMap( (GWorldPtr) GetHBITMAP()));
-    SetGWorld(  (GWorldPtr) GetHBITMAP(), NULL);
-
+    if ( GetBitmapType() != kMacBitmapTypeGrafWorld )
+    {
+       tempPort = wxMacCreateGWorld( width , height , -1) ;
+    }
+    else
+    {
+       tempPort =  (GWorldPtr) GetHBITMAP() ;
+       }
+       LockPixels(GetGWorldPixMap(tempPort));
+       SetGWorld( tempPort, NULL);
+       if ( GetBitmapType() == kMacBitmapTypePict || GetBitmapType() == kMacBitmapTypeIcon )
+       {
+        Rect bitmaprect = { 0 , 0 , height, width };
+       if ( GetBitmapType() == kMacBitmapTypeIcon )
+       {
+               ::PlotCIconHandle( &bitmaprect , atNone , ttNone , MAC_WXHICON(GetHICON()) ) ;
+               maskRgn = NewRgn() ;
+               BitMapToRegion( maskRgn , &(**(MAC_WXHICON(GetHICON()))).iconMask ) ; 
+        }
+       else
+               ::DrawPicture( (PicHandle) GetPict(), &bitmaprect ) ;
+       }

     // Copy data into image
     index = 0;
     for (int yy = 0; yy < height; yy++)
     // Copy data into image
     index = 0;
     for (int yy = 0; yy < height; yy++)


@@ -791,27 +823,50 @@ wxImage wxBitmap::ConvertToImage() const
             data[index    ] = r;
             data[index + 1] = g;
             data[index + 2] = b;
             data[index    ] = r;
             data[index + 1] = g;
             data[index + 2] = b;

-            if (mask)
-            {
-                if (mask->PointMasked(xx,yy))
-                {
+            if ( maskRgn )
+            {  
+               Point pt ;
+               pt.h = xx ;
+               pt.v = yy ;
+               if ( !PtInRgn( pt , maskRgn ) )
+               {

                     data[index    ] = mask_r;
                     data[index + 1] = mask_g;
                     data[index + 2] = mask_b;
                     data[index    ] = mask_r;
                     data[index + 1] = mask_g;
                     data[index + 2] = mask_b;

-                }
+               }
+            }
+            else
+            {
+                   if (mask)
+                   {
+                       if (mask->PointMasked(xx,yy))
+                       {
+                           data[index    ] = mask_r;
+                           data[index + 1] = mask_g;
+                           data[index + 2] = mask_b;
+                       }
+                   }

             }
             index += 3;
         }
     }
             }
             index += 3;
         }
     }

-    if (mask)
+    if (mask || maskRgn )

     {
         image.SetMaskColour( mask_r, mask_g, mask_b );
         image.SetMask( true );
     }
 
     // Free resources
     {
         image.SetMaskColour( mask_r, mask_g, mask_b );
         image.SetMask( true );
     }
 
     // Free resources

-    UnlockPixels(GetGWorldPixMap( (GWorldPtr) GetHBITMAP()));
+    UnlockPixels(GetGWorldPixMap( tempPort ));

     SetGWorld(origPort, origDevice);
     SetGWorld(origPort, origDevice);

+    if ( GetBitmapType() != kMacBitmapTypeGrafWorld )
+    {
+       wxMacDestroyGWorld( tempPort ) ;
+    }
+    if ( maskRgn )
+    {
+       DisposeRgn( maskRgn ) ;
+    }

 
     return image;
 }
 
     return image;
 }