Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

make pixbuf/alpha handling more coherent in gtk wxBitmap, allow pixbuf without alpha

Browse Source 
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@41661 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Paul Cornett
2006-10-06 16:34:45 +00:00
parent 5a67db4861
commit 7002950658
2 changed files with 84 additions and 79 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
include/wx/rawbmp.h
View File

@@ -164,8 +164,8 @@ typedef wxPixelFormat<unsigned char, 24, 0, 1, 2> wxImagePixelFormat;
#define wxPIXEL_FORMAT_ALPHA 3 #define wxPIXEL_FORMAT_ALPHA 3
#elif defined(__WXGTK__) #elif defined(__WXGTK__)
// Under GTK+ 2.X we use GdkPixbuf, which should be RGBA // Under GTK+ 2.X we use GdkPixbuf, which is standard RGB or RGBA
typedef wxPixelFormat<unsigned char, 32, 0, 1, 2> wxNativePixelFormat; typedef wxPixelFormat<unsigned char, 24, 0, 1, 2> wxNativePixelFormat;
#define wxPIXEL_FORMAT_ALPHA 3 #define wxPIXEL_FORMAT_ALPHA 3
#endif #endif

105
src/gtk/bitmap.cpp
View File

@@ -299,11 +299,13 @@ wxBitmap wxBitmap::Rescale( int clipx, int clipy, int clipwidth, int clipheight,
width = wxMin(width, clipwidth); width = wxMin(width, clipwidth);
height = wxMin(height, clipheight); height = wxMin(height, clipheight);
if (HasPixbuf()) // scale pixbuf if available and it has alpha or there is no mask
if (M_BMPDATA->m_pixbuf != NULL && (
M_BMPDATA->m_mask == NULL || gdk_pixbuf_get_has_alpha(M_BMPDATA->m_pixbuf)))
{ {
bmp.SetDepth(GetDepth()); bmp.SetDepth(GetDepth());
bmp.SetPixbuf(gdk_pixbuf_new(GDK_COLORSPACE_RGB, bmp.SetPixbuf(gdk_pixbuf_new(GDK_COLORSPACE_RGB,
true, //gdk_pixbuf_get_has_alpha(GetPixbuf()), gdk_pixbuf_get_has_alpha(GetPixbuf()),
8, width, height)); 8, width, height));
gdk_pixbuf_scale(GetPixbuf(), bmp.GetPixbuf(), gdk_pixbuf_scale(GetPixbuf(), bmp.GetPixbuf(),
0, 0, width, height, 0, 0, width, height,
@@ -477,12 +479,11 @@ bool wxBitmap::CreateFromImage(const wxImage& image, int depth)
if (image.GetWidth() <= 0 || image.GetHeight() <= 0) if (image.GetWidth() <= 0 || image.GetHeight() <= 0)
return false; return false;
if (depth == 1) // create pixbuf if image has alpha and requested depth is compatible
return CreateFromImageAsPixmap(image, depth); if (image.HasAlpha() && (depth == -1 || depth == 32))
if (image.HasAlpha())
return CreateFromImageAsPixbuf(image); return CreateFromImageAsPixbuf(image);
// otherwise create pixmap, if alpha is present it will be converted to mask
return CreateFromImageAsPixmap(image, depth); return CreateFromImageAsPixmap(image, depth);
} }
@@ -566,21 +567,18 @@ bool wxBitmap::CreateFromImageAsPixmap(const wxImage& image, int depth)
bool wxBitmap::CreateFromImageAsPixbuf(const wxImage& image) bool wxBitmap::CreateFromImageAsPixbuf(const wxImage& image)
{ {
wxASSERT(image.HasAlpha());
int width = image.GetWidth(); int width = image.GetWidth();
int height = image.GetHeight(); int height = image.GetHeight();
GdkPixbuf *pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB, GdkPixbuf *pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB,
true, //image.HasAlpha(), true,
8 /* bits per sample */, 8 /* bits per sample */,
width, height); width, height);
if (!pixbuf) if (!pixbuf)
return false; return false;
wxASSERT( image.HasAlpha() ); // for now
wxASSERT( gdk_pixbuf_get_n_channels(pixbuf) == 4 );
wxASSERT( gdk_pixbuf_get_width(pixbuf) == width );
wxASSERT( gdk_pixbuf_get_height(pixbuf) == height );
SetDepth(32); SetDepth(32);
SetPixbuf(pixbuf); SetPixbuf(pixbuf);
@@ -618,26 +616,30 @@ wxImage wxBitmap::ConvertToImage() const
wxCHECK_MSG(data != NULL, wxNullImage, wxT("couldn't create image") ); wxCHECK_MSG(data != NULL, wxNullImage, wxT("couldn't create image") );
// prefer pixbuf if available, it will preserve alpha and should be quicker
if (HasPixbuf()) if (HasPixbuf())
{ {
GdkPixbuf *pixbuf = GetPixbuf(); GdkPixbuf *pixbuf = GetPixbuf();
wxASSERT( gdk_pixbuf_get_has_alpha(pixbuf) ); unsigned char* alpha = NULL;
if (gdk_pixbuf_get_has_alpha(pixbuf))
{
image.SetAlpha(); image.SetAlpha();
alpha = image.GetAlpha();
unsigned char *alpha = image.GetAlpha(); }
unsigned char *in = gdk_pixbuf_get_pixels(pixbuf); unsigned char *in = gdk_pixbuf_get_pixels(pixbuf);
unsigned char *out = data; unsigned char *out = data;
int rowinc = gdk_pixbuf_get_rowstride(pixbuf) - 4 * w; const int inc = 3 + int(alpha != NULL);
const int rowinc = gdk_pixbuf_get_rowstride(pixbuf) - inc * w;
for (int y = 0; y < h; y++, in += rowinc) for (int y = 0; y < h; y++, in += rowinc)
{ {
for (int x = 0; x < w; x++, in += 4, out += 3, alpha++) for (int x = 0; x < w; x++, in += inc, out += 3)
{ {
out[0] = in[0]; out[0] = in[0];
out[1] = in[1]; out[1] = in[1];
out[2] = in[2]; out[2] = in[2];
*alpha = in[3]; if (alpha != NULL)
*alpha++ = in[3];
} }
} }
} }
@@ -664,8 +666,9 @@ wxImage wxBitmap::ConvertToImage() const
g_object_unref(pixbuf); g_object_unref(pixbuf);
if (pixmap_invert != NULL) if (pixmap_invert != NULL)
g_object_unref(pixmap_invert); g_object_unref(pixmap_invert);
}
if (GetMask()) // convert mask, unless there is already alpha
if (GetMask() && !image.HasAlpha())
{ {
// the colour used as transparent one in wxImage and the one it is // the colour used as transparent one in wxImage and the one it is
// replaced with when it really occurs in the bitmap // replaced with when it really occurs in the bitmap
@@ -695,7 +698,6 @@ wxImage wxBitmap::ConvertToImage() const
} }
g_object_unref(image_mask); g_object_unref(image_mask);
} }
}
return image; return image;
} }
@@ -793,7 +795,7 @@ wxBitmap wxBitmap::GetSubBitmap( const wxRect& rect) const
if (HasPixbuf()) if (HasPixbuf())
{ {
GdkPixbuf *pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB, GdkPixbuf *pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB,
true, //gdk_pixbuf_get_has_alpha(GetPixbuf()), gdk_pixbuf_get_has_alpha(GetPixbuf()),
8, rect.width, rect.height); 8, rect.width, rect.height);
ret.SetPixbuf(pixbuf); ret.SetPixbuf(pixbuf);
ret.SetDepth(M_BMPDATA->m_bpp); ret.SetDepth(M_BMPDATA->m_bpp);
@@ -808,8 +810,8 @@ wxBitmap wxBitmap::GetSubBitmap( const wxRect& rect) const
gdk_draw_drawable( ret.GetPixmap(), gc, GetPixmap(), rect.x, rect.y, 0, 0, rect.width, rect.height ); gdk_draw_drawable( ret.GetPixmap(), gc, GetPixmap(), rect.x, rect.y, 0, 0, rect.width, rect.height );
g_object_unref (gc); g_object_unref (gc);
} }
// make mask, unless there is already alpha
if (GetMask()) if (GetMask() && !HasAlpha())
{ {
wxMask *mask = new wxMask; wxMask *mask = new wxMask;
mask->m_bitmap = gdk_pixmap_new( wxGetRootWindow()->window, rect.width, rect.height, 1 ); mask->m_bitmap = gdk_pixmap_new( wxGetRootWindow()->window, rect.width, rect.height, 1 );
@@ -916,11 +918,17 @@ GdkPixmap *wxBitmap::GetPixmap() const
// create the pixmap on the fly if we use Pixbuf representation: // create the pixmap on the fly if we use Pixbuf representation:
if (M_BMPDATA->m_pixmap == NULL) if (M_BMPDATA->m_pixmap == NULL)
{ {
GdkPixmap** pmask = NULL;
if (gdk_pixbuf_get_has_alpha(M_BMPDATA->m_pixbuf))
{
// make new mask from alpha
delete M_BMPDATA->m_mask; delete M_BMPDATA->m_mask;
M_BMPDATA->m_mask = new wxMask; M_BMPDATA->m_mask = new wxMask;
pmask = &M_BMPDATA->m_mask->m_bitmap;
}
gdk_pixbuf_render_pixmap_and_mask(M_BMPDATA->m_pixbuf, gdk_pixbuf_render_pixmap_and_mask(M_BMPDATA->m_pixbuf,
&M_BMPDATA->m_pixmap, &M_BMPDATA->m_pixmap,
&M_BMPDATA->m_mask->m_bitmap, pmask,
128 /*threshold*/); 128 /*threshold*/);
} }
@@ -943,10 +951,8 @@ GdkPixbuf *wxBitmap::GetPixbuf() const
int width = GetWidth(); int width = GetWidth();
int height = GetHeight(); int height = GetHeight();
// always create the alpha channel so raw bitmap access will work
// correctly
GdkPixbuf *pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB, GdkPixbuf *pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB,
true, // GetMask() != NULL, GetMask() != NULL,
8, width, height); 8, width, height);
M_BMPDATA->m_pixbuf = M_BMPDATA->m_pixbuf =
gdk_pixbuf_get_from_drawable(pixbuf, M_BMPDATA->m_pixmap, NULL, gdk_pixbuf_get_from_drawable(pixbuf, M_BMPDATA->m_pixmap, NULL,
@@ -1020,43 +1026,42 @@ void wxBitmap::PurgeOtherRepresentations(wxBitmap::Representation keep)
void *wxBitmap::GetRawData(wxPixelDataBase& data, int bpp) void *wxBitmap::GetRawData(wxPixelDataBase& data, int bpp)
{ {
if (bpp != 32) void* bits = NULL;
return NULL;
GdkPixbuf *pixbuf = GetPixbuf(); GdkPixbuf *pixbuf = GetPixbuf();
if (!pixbuf) const bool hasAlpha = HasAlpha();
return NULL; // allow access if bpp is valid and matches existence of alpha
if (pixbuf != NULL && (
if (!gdk_pixbuf_get_has_alpha( pixbuf )) bpp == 24 && !hasAlpha ||
return NULL; bpp == 32 && hasAlpha))
{
#if 0
if (gdk_pixbuf_get_has_alpha( pixbuf ))
wxPrintf( wxT("Has alpha, %d channels\n"), gdk_pixbuf_get_n_channels(pixbuf) );
else
wxPrintf( wxT("No alpha, %d channels.\n"), gdk_pixbuf_get_n_channels(pixbuf) );
#endif
data.m_height = gdk_pixbuf_get_height( pixbuf ); data.m_height = gdk_pixbuf_get_height( pixbuf );
data.m_width = gdk_pixbuf_get_width( pixbuf ); data.m_width = gdk_pixbuf_get_width( pixbuf );
data.m_stride = gdk_pixbuf_get_rowstride( pixbuf ); data.m_stride = gdk_pixbuf_get_rowstride( pixbuf );
bits = gdk_pixbuf_get_pixels(pixbuf);
return gdk_pixbuf_get_pixels( pixbuf ); }
return bits;
} }
void wxBitmap::UngetRawData(wxPixelDataBase& WXUNUSED(data)) void wxBitmap::UngetRawData(wxPixelDataBase& WXUNUSED(data))
{ {
} }
bool wxBitmap::HasAlpha() const bool wxBitmap::HasAlpha() const
{ {
return HasPixbuf(); return m_refData != NULL && M_BMPDATA->m_pixbuf != NULL &&
gdk_pixbuf_get_has_alpha(M_BMPDATA->m_pixbuf);
} }
void wxBitmap::UseAlpha() void wxBitmap::UseAlpha()
{ {
GetPixbuf(); GdkPixbuf* pixbuf = GetPixbuf();
// add alpha if necessary
if (!gdk_pixbuf_get_has_alpha(pixbuf))
{
M_BMPDATA->m_pixbuf = NULL;
SetPixbuf(gdk_pixbuf_add_alpha(pixbuf, false, 0, 0, 0));
g_object_unref(pixbuf);
}
} }
//----------------------------------------------------------------------------- //-----------------------------------------------------------------------------