X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/9e691f46b2ec1b5dbbff2e51131e3c532e688a89..cfbf444a5331dec2fa2cd000210638560f14d980:/src/gtk/bitmap.cpp diff --git a/src/gtk/bitmap.cpp b/src/gtk/bitmap.cpp index 0a27495252..a7e059c40c 100644 --- a/src/gtk/bitmap.cpp +++ b/src/gtk/bitmap.cpp @@ -33,10 +33,7 @@ #ifdef __WXGTK20__ #include #else // GTK+ 1.2 - // VZ: is this still needed? seems to compile fine without it... - #if (GTK_MINOR_VERSION > 0) - #include - #endif + #include #endif // GTK+ 2.0/1.2 extern void gdk_wx_draw_bitmap (GdkDrawable *drawable, @@ -99,7 +96,7 @@ bool wxMask::Create( const wxBitmap& bitmap, m_bitmap = (GdkBitmap*) NULL; } - wxImage image( bitmap ); + wxImage image = bitmap.ConvertToImage(); if (!image.Ok()) return FALSE; m_bitmap = gdk_pixmap_new( wxGetRootWindow()->window, image.GetWidth(), image.GetHeight(), 1 ); @@ -124,20 +121,21 @@ bool wxMask::Create( const wxBitmap& bitmap, GdkVisual *visual = wxTheApp->GetGdkVisual(); int bpp = visual->depth; - if ((bpp == 16) && (visual->red_mask != 0xf800)) bpp = 15; + if ((bpp == 16) && (visual->red_mask != 0xf800)) + bpp = 15; if (bpp == 15) { red = red & 0xf8; green = green & 0xf8; blue = blue & 0xf8; - } else - if (bpp == 16) + } + else if (bpp == 16) { red = red & 0xf8; green = green & 0xfc; blue = blue & 0xf8; - } else - if (bpp == 12) + } + else if (bpp == 12) { red = red & 0xf0; green = green & 0xf0; @@ -332,7 +330,7 @@ bool wxBitmap::CreateFromXpm( const char **bits ) gdk_window_get_size( M_BMPDATA->m_pixmap, &(M_BMPDATA->m_width), &(M_BMPDATA->m_height) ); - M_BMPDATA->m_bpp = visual->depth; // ? + M_BMPDATA->m_bpp = visual->depth; // Can we get a different depth from create_from_xpm_d() ? return TRUE; } @@ -347,7 +345,7 @@ bool wxBitmap::CreateFromImage( const wxImage& image, int depth ) m_refData = new wxBitmapRefData(); // ------ - // convertion to mono bitmap: + // conversion to mono bitmap: // ------ if (depth == 1) { @@ -443,7 +441,7 @@ bool wxBitmap::CreateFromImage( const wxImage& image, int depth ) } // ------ - // convertion to colour bitmap: + // conversion to colour bitmap: // ------ else { @@ -461,10 +459,12 @@ bool wxBitmap::CreateFromImage( const wxImage& image, int depth ) SetDepth( bpp ); - if ((bpp == 16) && (visual->red_mask != 0xf800)) bpp = 15; - if (bpp < 8) bpp = 8; + if ((bpp == 16) && (visual->red_mask != 0xf800)) + bpp = 15; + else if (bpp < 8) + bpp = 8; - // We handle 8-bit bitmaps ourselves using the colour cube, 12-bit + // We handle 8-bit bitmaps ourselves using the colour cube, 12-bit // visuals are not supported by GDK so we do these ourselves, too. // 15-bit and 16-bit should actually work and 24-bit certainly does. #ifdef __sgi @@ -571,7 +571,7 @@ bool wxBitmap::CreateFromImage( const wxImage& image, int depth ) GdkColormap *cmap = gtk_widget_get_default_colormap(); GdkColor *colors = cmap->colors; int max = 3 * (65536); - + for (int i = 0; i < cmap->size; i++) { int rdiff = (r << 8) - colors[i].red; @@ -581,9 +581,9 @@ bool wxBitmap::CreateFromImage( const wxImage& image, int depth ) if (sum < max) { pixel = i; max = sum; } } } - + gdk_image_put_pixel( data_image, x, y, pixel ); - + break; } case 12: // SGI only @@ -680,6 +680,13 @@ bool wxBitmap::CreateFromImage( const wxImage& image, int depth ) wxImage wxBitmap::ConvertToImage() const { + // the colour used as transparent one in wxImage and the one it is replaced + // with when it really occurs in the bitmap + static const int MASK_RED = 1; + static const int MASK_GREEN = 2; + static const int MASK_BLUE = 3; + static const int MASK_BLUE_REPLACEMENT = 2; + wxImage image; wxCHECK_MSG( Ok(), wxNullImage, wxT("invalid bitmap") ); @@ -688,15 +695,16 @@ wxImage wxBitmap::ConvertToImage() const if (GetPixmap()) { gdk_image = gdk_image_get( GetPixmap(), - 0, 0, - GetWidth(), GetHeight() ); - } else - if (GetBitmap()) + 0, 0, + GetWidth(), GetHeight() ); + } + else if (GetBitmap()) { gdk_image = gdk_image_get( GetBitmap(), - 0, 0, - GetWidth(), GetHeight() ); - } else + 0, 0, + GetWidth(), GetHeight() ); + } + else { wxFAIL_MSG( wxT("Ill-formed bitmap") ); } @@ -717,10 +725,10 @@ wxImage wxBitmap::ConvertToImage() const if (GetMask()) { gdk_image_mask = gdk_image_get( GetMask()->GetBitmap(), - 0, 0, - GetWidth(), GetHeight() ); + 0, 0, + GetWidth(), GetHeight() ); - image.SetMaskColour( 16, 16, 16 ); // anything unlikely and dividable + image.SetMaskColour( MASK_RED, MASK_GREEN, MASK_BLUE ); } int bpp = -1; @@ -737,9 +745,10 @@ wxImage wxBitmap::ConvertToImage() const GdkVisual *visual = gdk_window_get_visual( GetPixmap() ); if (visual == NULL) visual = wxTheApp->GetGdkVisual(); - + bpp = visual->depth; - if (bpp == 16) bpp = visual->red_prec + visual->green_prec + visual->blue_prec; + if (bpp == 16) + bpp = visual->red_prec + visual->green_prec + visual->blue_prec; red_shift_right = visual->red_shift; red_shift_left = 8-visual->red_prec; green_shift_right = visual->green_shift; @@ -800,9 +809,15 @@ wxImage wxBitmap::ConvertToImage() const int mask_pixel = gdk_image_get_pixel( gdk_image_mask, i, j ); if (mask_pixel == 0) { - data[pos] = 16; - data[pos+1] = 16; - data[pos+2] = 16; + data[pos] = MASK_RED; + data[pos+1] = MASK_GREEN; + data[pos+2] = MASK_BLUE; + } + else if ( data[pos] == MASK_RED && + data[pos+1] == MASK_GREEN && + data[pos+2] == MASK_BLUE ) + { + data[pos+2] = MASK_BLUE_REPLACEMENT; } } @@ -817,6 +832,7 @@ wxImage wxBitmap::ConvertToImage() const } wxBitmap::wxBitmap( const wxBitmap& bmp ) + : wxGDIObject() { Ref( bmp ); } @@ -954,7 +970,7 @@ bool wxBitmap::SaveFile( const wxString &name, int type, wxPalette *WXUNUSED(pal // Try to save the bitmap via wxImage handlers: { - wxImage image( *this ); + wxImage image = ConvertToImage(); if (image.Ok()) return image.SaveFile( name, type ); } @@ -991,7 +1007,8 @@ bool wxBitmap::LoadFile( const wxString &name, int type ) { wxImage image; if (!image.LoadFile( name, type )) return FALSE; - if (image.Ok()) *this = image.ConvertToBitmap(); + if (image.Ok()) + *this = wxBitmap(image); else return FALSE; }