Version 0.4 of wxPython for MSW.

[wxWidgets.git] / src / gtk1 / bitmap.cpp

diff --git a/src/gtk1/bitmap.cpp b/src/gtk1/bitmap.cpp
index d2241e591d093489b6b99e7201180fc3a0464b51..07519ea19f6a4158ada558ecc639473b80323d1d 100644 (file)
--- a/src/gtk1/bitmap.cpp
+++ b/src/gtk1/bitmap.cpp
@@ -13,10 +13,16 @@
 #endif
 
 #include "wx/bitmap.h"
 #endif
 
 #include "wx/bitmap.h"

+#include "wx/icon.h"

 #include "gdk/gdkprivate.h"
 
 #include "gdk/gdkprivate.h"
 

-#ifdef USE_GDK_IMLIB
+#ifdef wxUSE_GDK_IMLIB
+

 #include "../gdk_imlib/gdk_imlib.h"
 #include "../gdk_imlib/gdk_imlib.h"

+#include "gdk/gdkx.h"        // GDK_DISPLAY
+#include <X11/Xlib.h>
+#include <X11/Xutil.h>
+

 #endif
 
 //-----------------------------------------------------------------------------
 #endif
 
 //-----------------------------------------------------------------------------


@@ -27,34 +33,34 @@ IMPLEMENT_DYNAMIC_CLASS(wxMask,wxObject)
 
 wxMask::wxMask(void)
 {
 
 wxMask::wxMask(void)
 {

-  m_bitmap = NULL;
-};
+  m_bitmap = (GdkBitmap *) NULL;
+}

 
 wxMask::wxMask( const wxBitmap& WXUNUSED(bitmap), const wxColour& WXUNUSED(colour) )
 {
 
 wxMask::wxMask( const wxBitmap& WXUNUSED(bitmap), const wxColour& WXUNUSED(colour) )
 {

-};
+}

 
 wxMask::wxMask( const wxBitmap& WXUNUSED(bitmap), int WXUNUSED(paletteIndex) )
 {
 
 wxMask::wxMask( const wxBitmap& WXUNUSED(bitmap), int WXUNUSED(paletteIndex) )
 {

-};
+}

 
 wxMask::wxMask( const wxBitmap& WXUNUSED(bitmap) )
 {
 
 wxMask::wxMask( const wxBitmap& WXUNUSED(bitmap) )
 {

-};
+}

 
 wxMask::~wxMask(void)
 {
 
 wxMask::~wxMask(void)
 {

-#ifdef USE_GDK_IMLIB
+#ifdef wxUSE_GDK_IMLIB

   // do not delete the mask, gdk_imlib does it for you
 #else
   if (m_bitmap) gdk_bitmap_unref( m_bitmap );
 #endif  
   // do not delete the mask, gdk_imlib does it for you
 #else
   if (m_bitmap) gdk_bitmap_unref( m_bitmap );
 #endif  

-};
+}

 
 GdkBitmap *wxMask::GetBitmap(void) const
 {
   return m_bitmap;
 
 GdkBitmap *wxMask::GetBitmap(void) const
 {
   return m_bitmap;

-};
+}

   
 //-----------------------------------------------------------------------------
 // wxBitmap
   
 //-----------------------------------------------------------------------------
 // wxBitmap


@@ -74,7 +80,7 @@ class wxBitmapRefData: public wxObjectRefData
     int             m_width;
     int             m_height;
     int             m_bpp;
     int             m_width;
     int             m_height;
     int             m_bpp;

-#ifdef USE_GDK_IMLIB
+#ifdef wxUSE_GDK_IMLIB

     GdkImlibImage  *m_image;
 #endif
     wxPalette      *m_palette;
     GdkImlibImage  *m_image;
 #endif
     wxPalette      *m_palette;


@@ -82,30 +88,30 @@ class wxBitmapRefData: public wxObjectRefData
 
 wxBitmapRefData::wxBitmapRefData(void)
 {
 
 wxBitmapRefData::wxBitmapRefData(void)
 {

-  m_pixmap = NULL;
-  m_bitmap = NULL;
-  m_mask = NULL;
+  m_pixmap = (GdkPixmap *) NULL;
+  m_bitmap = (GdkBitmap *) NULL;
+  m_mask = (wxMask *) NULL;

   m_width = 0;
   m_height = 0;
   m_bpp = 0;
   m_width = 0;
   m_height = 0;
   m_bpp = 0;

-  m_palette = NULL;
-#ifdef USE_GDK_IMLIB
-  m_image = NULL;
+  m_palette = (wxPalette *) NULL;
+#ifdef wxUSE_GDK_IMLIB
+  m_image = (GdkImlibImage *) NULL;

 #endif
 #endif

-};
+}

 
 wxBitmapRefData::~wxBitmapRefData(void)
 {
 
 wxBitmapRefData::~wxBitmapRefData(void)
 {

-#ifdef USE_GDK_IMLIB
+#ifdef wxUSE_GDK_IMLIB

   if (m_pixmap) gdk_imlib_free_pixmap( m_pixmap );
   if (m_pixmap) gdk_imlib_free_pixmap( m_pixmap );

-  if (m_image) gdk_imlib_destroy_image( m_image );
+  if (m_image) gdk_imlib_kill_image( m_image );

 #else
   if (m_pixmap) gdk_pixmap_unref( m_pixmap );
 #endif
   if (m_bitmap) gdk_bitmap_unref( m_bitmap );
   if (m_mask) delete m_mask;
   if (m_palette) delete m_palette;
 #else
   if (m_pixmap) gdk_pixmap_unref( m_pixmap );
 #endif
   if (m_bitmap) gdk_bitmap_unref( m_bitmap );
   if (m_mask) delete m_mask;
   if (m_palette) delete m_palette;

-};
+}

 
 //-----------------------------------------------------------------------------
 
 
 //-----------------------------------------------------------------------------
 


@@ -116,12 +122,12 @@ IMPLEMENT_DYNAMIC_CLASS(wxBitmap,wxGDIObject)
 wxBitmap::wxBitmap(void)
 {
   if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);
 wxBitmap::wxBitmap(void)
 {
   if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);

-};
+}

   
 wxBitmap::wxBitmap( int width, int height, int depth )
 {
   m_refData = new wxBitmapRefData();
   
 wxBitmap::wxBitmap( int width, int height, int depth )
 {
   m_refData = new wxBitmapRefData();

-  M_BMPDATA->m_mask = NULL;
+  M_BMPDATA->m_mask = (wxMask *) NULL;

   M_BMPDATA->m_pixmap = 
     gdk_pixmap_new( (GdkWindow*) &gdk_root_parent, width, height, depth );
   M_BMPDATA->m_width = width;
   M_BMPDATA->m_pixmap = 
     gdk_pixmap_new( (GdkWindow*) &gdk_root_parent, width, height, depth );
   M_BMPDATA->m_width = width;


@@ -129,13 +135,13 @@ wxBitmap::wxBitmap( int width, int height, int depth )
   M_BMPDATA->m_bpp = depth;
    
   if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);
   M_BMPDATA->m_bpp = depth;
    
   if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);

-};
+}

 
 wxBitmap::wxBitmap( char **bits )
 {
   m_refData = new wxBitmapRefData();
 
 
 wxBitmap::wxBitmap( char **bits )
 {
   m_refData = new wxBitmapRefData();
 

-#ifndef USE_GDK_IMLIB
+#ifndef wxUSE_GDK_IMLIB

 
   GdkBitmap *mask = NULL;
   
 
   GdkBitmap *mask = NULL;
   


@@ -146,7 +152,7 @@ wxBitmap::wxBitmap( char **bits )
   {
     M_BMPDATA->m_mask = new wxMask();
     M_BMPDATA->m_mask->m_bitmap = mask;
   {
     M_BMPDATA->m_mask = new wxMask();
     M_BMPDATA->m_mask->m_bitmap = mask;

-  };
+  }

   
   gdk_window_get_size( M_BMPDATA->m_pixmap, &(M_BMPDATA->m_width), &(M_BMPDATA->m_height) );
   
   
   gdk_window_get_size( M_BMPDATA->m_pixmap, &(M_BMPDATA->m_width), &(M_BMPDATA->m_height) );
   


@@ -160,33 +166,35 @@ wxBitmap::wxBitmap( char **bits )
   M_BMPDATA->m_bpp = 24; // ?
    
   if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);
   M_BMPDATA->m_bpp = 24; // ?
    
   if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);

-};
+}

   
 wxBitmap::wxBitmap( const wxBitmap& bmp )
 {
   Ref( bmp );
    
   if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);
   
 wxBitmap::wxBitmap( const wxBitmap& bmp )
 {
   Ref( bmp );
    
   if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);

-};
+}

   
 wxBitmap::wxBitmap( const wxBitmap* bmp )
 {
   if (bmp) Ref( *bmp );
    
   if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);
   
 wxBitmap::wxBitmap( const wxBitmap* bmp )
 {
   if (bmp) Ref( *bmp );
    
   if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);

-};
+}

 
 wxBitmap::wxBitmap( const wxString &filename, int type )
 {
   LoadFile( filename, type );
 
 wxBitmap::wxBitmap( const wxString &filename, int type )
 {
   LoadFile( filename, type );

-};
+  
+  if (wxTheBitmapList) wxTheBitmapList->AddBitmap(this);
+}

 
 // CMB 15/5/98: add constructor for xbm bitmaps
 wxBitmap::wxBitmap( const char bits[], int width, int height, int WXUNUSED(depth))
 {
   m_refData = new wxBitmapRefData();
 
 
 // CMB 15/5/98: add constructor for xbm bitmaps
 wxBitmap::wxBitmap( const char bits[], int width, int height, int WXUNUSED(depth))
 {
   m_refData = new wxBitmapRefData();
 

-  M_BMPDATA->m_mask = NULL;
+  M_BMPDATA->m_mask = (wxMask *) NULL;

   M_BMPDATA->m_bitmap = 
     gdk_bitmap_create_from_data( (GdkWindow*) &gdk_root_parent, (gchar *) bits, width, height );
   M_BMPDATA->m_width = width;
   M_BMPDATA->m_bitmap = 
     gdk_bitmap_create_from_data( (GdkWindow*) &gdk_root_parent, (gchar *) bits, width, height );
   M_BMPDATA->m_width = width;


@@ -199,88 +207,129 @@ wxBitmap::wxBitmap( const char bits[], int width, int height, int WXUNUSED(depth
 wxBitmap::~wxBitmap(void)
 {
   if (wxTheBitmapList) wxTheBitmapList->DeleteObject(this);
 wxBitmap::~wxBitmap(void)
 {
   if (wxTheBitmapList) wxTheBitmapList->DeleteObject(this);

-};
+}

   
 wxBitmap& wxBitmap::operator = ( const wxBitmap& bmp )
 {
   if (*this == bmp) return (*this); 
   Ref( bmp ); 
   return *this; 
   
 wxBitmap& wxBitmap::operator = ( const wxBitmap& bmp )
 {
   if (*this == bmp) return (*this); 
   Ref( bmp ); 
   return *this; 

-};
+}

   
 bool wxBitmap::operator == ( const wxBitmap& bmp )
 {
   return m_refData == bmp.m_refData; 
   
 bool wxBitmap::operator == ( const wxBitmap& bmp )
 {
   return m_refData == bmp.m_refData; 

-};
+}

   
 bool wxBitmap::operator != ( const wxBitmap& bmp )
 {
   return m_refData != bmp.m_refData; 
   
 bool wxBitmap::operator != ( const wxBitmap& bmp )
 {
   return m_refData != bmp.m_refData; 

-};
+}

   
 bool wxBitmap::Ok(void) const
 {
   
 bool wxBitmap::Ok(void) const
 {

-  return m_refData != NULL;
-};
+  return (m_refData != NULL);
+}

   
 int wxBitmap::GetHeight(void) const
 {
   
 int wxBitmap::GetHeight(void) const
 {

-  if (!Ok()) return 0;
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return -1;
+  }
+

   return M_BMPDATA->m_height;
   return M_BMPDATA->m_height;

-};
+}

 
 int wxBitmap::GetWidth(void) const
 {
 
 int wxBitmap::GetWidth(void) const
 {

-  if (!Ok()) return 0;
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return -1;
+  }
+  

   return M_BMPDATA->m_width;
   return M_BMPDATA->m_width;

-};
+}

 
 int wxBitmap::GetDepth(void) const
 {
 
 int wxBitmap::GetDepth(void) const
 {

-  if (!Ok()) return 0;
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return -1;
+  }
+  

   return M_BMPDATA->m_bpp;
   return M_BMPDATA->m_bpp;

-};
+}

 
 void wxBitmap::SetHeight( int height )
 {
   if (!Ok()) return;
 
 void wxBitmap::SetHeight( int height )
 {
   if (!Ok()) return;

+  
+  wxFAIL_MSG( "wxBitmap::SetHeight not implemented" );
+  

   M_BMPDATA->m_height = height;
   M_BMPDATA->m_height = height;

-};
+}

 
 void wxBitmap::SetWidth( int width )
 {
   if (!Ok()) return;
 
 void wxBitmap::SetWidth( int width )
 {
   if (!Ok()) return;

+  
+  wxFAIL_MSG( "wxBitmap::SetWidth not implemented" );
+  

   M_BMPDATA->m_width = width;
   M_BMPDATA->m_width = width;

-};
+}

 
 void wxBitmap::SetDepth( int depth )
 {
   if (!Ok()) return;
 
 void wxBitmap::SetDepth( int depth )
 {
   if (!Ok()) return;

+  
+  wxFAIL_MSG( "wxBitmap::SetDepth not implemented" );
+  

   M_BMPDATA->m_bpp = depth;
   M_BMPDATA->m_bpp = depth;

-};
+}

 
 wxMask *wxBitmap::GetMask(void) const
 {
 
 wxMask *wxBitmap::GetMask(void) const
 {

-  if (!Ok()) return NULL;
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return (wxMask *) NULL;
+  }

   
   return M_BMPDATA->m_mask;
   
   return M_BMPDATA->m_mask;

-};
+}

 
 void wxBitmap::SetMask( wxMask *mask )
 {
 
 void wxBitmap::SetMask( wxMask *mask )
 {

-  if (!Ok()) return;
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return;
+  }

   
   if (M_BMPDATA->m_mask) delete M_BMPDATA->m_mask;
   
   if (M_BMPDATA->m_mask) delete M_BMPDATA->m_mask;

+  

   M_BMPDATA->m_mask = mask;
   M_BMPDATA->m_mask = mask;

-};
+}

 
 void wxBitmap::Resize( int height, int width )
 {
 
 void wxBitmap::Resize( int height, int width )
 {

-  if (!Ok()) return;
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return;
+  }

   
   

-#ifdef USE_GDK_IMLIB
+#ifdef wxUSE_GDK_IMLIB

   
   

-  if (M_BMPDATA->m_bitmap) return;  // not supported for bitmaps
+  if (M_BMPDATA->m_bitmap)
+  {
+    wxFAIL_MSG( "wxBitmap::Resize not supported for mono-bitmaps" );
+    return;
+  }

   
   if (!M_BMPDATA->m_image) RecreateImage();
   
   
   if (!M_BMPDATA->m_image) RecreateImage();
   


@@ -295,28 +344,46 @@ void wxBitmap::Resize( int height, int width )
   
   Render();
   
   
   Render();
   

+#else
+  
+  wxFAIL_MSG( "wxBitmap::Resize not implemented without GdkImlib" );
+  

 #endif
 #endif

-};
+}

 
 bool wxBitmap::SaveFile( const wxString &name, int WXUNUSED(type), 
   wxPalette *WXUNUSED(palette) )
 {
 
 bool wxBitmap::SaveFile( const wxString &name, int WXUNUSED(type), 
   wxPalette *WXUNUSED(palette) )
 {

-#ifdef USE_GDK_IMLIB
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return FALSE;
+  }
+  
+#ifdef wxUSE_GDK_IMLIB

 
 

-  if (!Ok()) return FALSE;
+  if (M_BMPDATA->m_bitmap)
+  {
+    wxFAIL_MSG( "wxBitmap::SaveFile not supported for mono-bitmaps" );
+    return FALSE;
+  }

   
   if (!M_BMPDATA->m_image) RecreateImage();
   
   
   if (!M_BMPDATA->m_image) RecreateImage();
   

-  return gdk_imlib_save_image( M_BMPDATA->m_image, WXSTRINGCAST name, NULL );
+  return gdk_imlib_save_image( M_BMPDATA->m_image, WXSTRINGCAST name, (GdkImlibSaveInfo *) NULL );

 
 

+#else
+  
+  wxFAIL_MSG( "wxBitmap::SaveFile not implemented without GdkImlib" );
+  

 #endif
 
   return FALSE;
 #endif
 
   return FALSE;

-};
+}

 
 bool wxBitmap::LoadFile( const wxString &name, int WXUNUSED(type) )
 {
 
 bool wxBitmap::LoadFile( const wxString &name, int WXUNUSED(type) )
 {

-#ifdef USE_GDK_IMLIB
+#ifdef wxUSE_GDK_IMLIB

 
   UnRef();
   m_refData = new wxBitmapRefData();
 
   UnRef();
   m_refData = new wxBitmapRefData();


@@ -327,7 +394,7 @@ bool wxBitmap::LoadFile( const wxString &name, int WXUNUSED(type) )
   {
     UnRef();
     return FALSE;
   {
     UnRef();
     return FALSE;

-  };
+  }

 
   Render();
                                  
 
   Render();
                                  


@@ -335,63 +402,158 @@ bool wxBitmap::LoadFile( const wxString &name, int WXUNUSED(type) )
   M_BMPDATA->m_bpp = 24; // ?
   
   return TRUE;
   M_BMPDATA->m_bpp = 24; // ?
   
   return TRUE;

+  
+#else
+  
+  wxFAIL_MSG( "wxBitmap::LoadFile not implemented without GdkImlib" );
+  

 #endif
 
   return FALSE;
 #endif
 
   return FALSE;

-};
+}

         
 wxPalette *wxBitmap::GetPalette(void) const
 {
         
 wxPalette *wxBitmap::GetPalette(void) const
 {

-  if (!Ok()) return NULL;
+  if (!Ok()) return (wxPalette *) NULL;

   return M_BMPDATA->m_palette;
   return M_BMPDATA->m_palette;

-};
+}

 
 GdkPixmap *wxBitmap::GetPixmap(void) const
 {
 
 GdkPixmap *wxBitmap::GetPixmap(void) const
 {

-  if (!Ok()) return NULL;
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return (GdkPixmap *) NULL;
+  }
+  
+//  if (!M_BMPDATA->m_image) RecreateImage();
+  

   return M_BMPDATA->m_pixmap;
   return M_BMPDATA->m_pixmap;

-};
+}

   
 GdkBitmap *wxBitmap::GetBitmap(void) const
 {
   
 GdkBitmap *wxBitmap::GetBitmap(void) const
 {

-  if (!Ok()) return NULL;
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return (GdkBitmap *) NULL;
+  }

   
   return M_BMPDATA->m_bitmap;
   
   return M_BMPDATA->m_bitmap;

-};
+}

   
 void wxBitmap::DestroyImage(void)
 {
   
 void wxBitmap::DestroyImage(void)
 {

-  if (!Ok()) return;
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return;
+  }

   
   if (M_BMPDATA->m_image)
   {
     gdk_imlib_destroy_image( M_BMPDATA->m_image );
   
   if (M_BMPDATA->m_image)
   {
     gdk_imlib_destroy_image( M_BMPDATA->m_image );

-    M_BMPDATA->m_image = NULL;
-  };
-};
+    M_BMPDATA->m_image = (GdkImlibImage *) NULL;
+  }
+}

 
 void wxBitmap::RecreateImage(void)
 {
 
 void wxBitmap::RecreateImage(void)
 {

-};
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return;
+  }
+  
+#ifdef wxUSE_GDK_IMLIB
+
+  DestroyImage();
+  
+  wxCHECK_RET( M_BMPDATA->m_pixmap != NULL, "invalid bitmap" );
+  
+  long size = (long)(M_BMPDATA->m_width)*(long)(M_BMPDATA->m_height)*(long)3;
+  unsigned char *data = new unsigned char[size];
+  for (long i = 0; i < size; i++) data[i] = 100;
+  
+  GdkImage *image = gdk_image_get( M_BMPDATA->m_pixmap, 0, 0, M_BMPDATA->m_width, M_BMPDATA->m_height );
+  
+  long pos = 0;
+  for (int j = 0; j < M_BMPDATA->m_height; j++)
+  {
+    for (int i = 0; i < M_BMPDATA->m_width; i++)
+    {
+      XColor xcol;
+      xcol.pixel = gdk_image_get_pixel( image, i, j );
+      Colormap cm = ((GdkColormapPrivate*)gdk_imlib_get_colormap())->xcolormap;
+      XQueryColor( gdk_display, cm, &xcol );
+      
+      data[pos] = xcol.red;
+      data[pos+1] = xcol.green;
+      data[pos+2] = xcol.blue;
+      pos += 3;
+    }
+  }
+  
+  wxCHECK_RET( M_BMPDATA->m_pixmap != NULL, "invalid bitmap" );
+  
+  M_BMPDATA->m_image = gdk_imlib_create_image_from_data( 
+     data, (unsigned char*)NULL, M_BMPDATA->m_width, M_BMPDATA->m_height );
+  
+  delete[] data;
+  
+  gdk_image_destroy( image );
+  
+  Render();
+  
+#else
+  
+  wxFAIL_MSG( "wxBitmap::RecreateImage not implemented without GdkImlib" );
+  
+#endif
+}

 
 void wxBitmap::Render(void)
 {
 
 void wxBitmap::Render(void)
 {

-  if (!Ok()) return;
+  if (!Ok())
+  {
+    wxFAIL_MSG( "invalid bitmap" );
+    return;
+  }

   
   

-#ifdef USE_GDK_IMLIB
+#ifdef wxUSE_GDK_IMLIB

 
 

+  if (!M_BMPDATA->m_image) RecreateImage();
+  
+  if (M_BMPDATA->m_pixmap)
+  { 
+    gdk_imlib_free_pixmap( M_BMPDATA->m_pixmap );
+    M_BMPDATA->m_pixmap = (GdkPixmap*) NULL;
+  }
+  if (M_BMPDATA->m_mask)
+  {
+    delete M_BMPDATA->m_mask;
+    M_BMPDATA->m_mask = (wxMask*) NULL;
+  }
+  

   gdk_imlib_render( M_BMPDATA->m_image, M_BMPDATA->m_image->rgb_width, M_BMPDATA->m_image->rgb_height );
   M_BMPDATA->m_width = M_BMPDATA->m_image->rgb_width;
   M_BMPDATA->m_height = M_BMPDATA->m_image->rgb_height;
   M_BMPDATA->m_pixmap = gdk_imlib_move_image( M_BMPDATA->m_image );
   gdk_imlib_render( M_BMPDATA->m_image, M_BMPDATA->m_image->rgb_width, M_BMPDATA->m_image->rgb_height );
   M_BMPDATA->m_width = M_BMPDATA->m_image->rgb_width;
   M_BMPDATA->m_height = M_BMPDATA->m_image->rgb_height;
   M_BMPDATA->m_pixmap = gdk_imlib_move_image( M_BMPDATA->m_image );

+  
+  wxCHECK_RET( M_BMPDATA->m_pixmap != NULL, "pixmap rendering failed" ) 
+  

   GdkBitmap *mask = gdk_imlib_move_mask( M_BMPDATA->m_image );
   if (mask)
   {
     M_BMPDATA->m_mask = new wxMask();
     M_BMPDATA->m_mask->m_bitmap = mask;
   GdkBitmap *mask = gdk_imlib_move_mask( M_BMPDATA->m_image );
   if (mask)
   {
     M_BMPDATA->m_mask = new wxMask();
     M_BMPDATA->m_mask->m_bitmap = mask;

-  };
+  }
+  
+#else
+  
+  wxFAIL_MSG( "wxBitmap::Render not implemented without GdkImlib" );

   
 #endif
   
 #endif

-};
+}