]> git.saurik.com Git - wxWidgets.git/blobdiff - src/x11/bitmap.cpp
projects / wxWidgets.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
UnixWare compilation fix (include sys/filio.h)
[wxWidgets.git] / src / x11 / bitmap.cpp
diff --git a/src/x11/bitmap.cpp b/src/x11/bitmap.cpp
index dc6bb3bb82f27278cd23362a1ec684f503ce6310..b2edff7387452dba52a72d50477475c2a3bce5b7 100644 (file)
--- a/src/x11/bitmap.cpp
+++ b/src/x11/bitmap.cpp
@@ -1,11 +1,11 @@
 /////////////////////////////////////////////////////////////////////////////
 // Name:        bitmap.cpp
 // Purpose:     wxBitmap
 /////////////////////////////////////////////////////////////////////////////
 // Name:        bitmap.cpp
 // Purpose:     wxBitmap
-// Author:      Julian Smart
+// Author:      Julian Smart, Robert Roebling
 // Modified by:
 // Created:     17/09/98
 // RCS-ID:      $Id$
 // Modified by:
 // Created:     17/09/98
 // RCS-ID:      $Id$
-// Copyright:   (c) Julian Smart
+// Copyright:   (c) Julian Smart, Robert Roebling
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
@@ -13,1546 +13,1541 @@
 #pragma implementation "bitmap.h"
 #endif
 
 #pragma implementation "bitmap.h"
 #endif
 
-#ifdef __VMS
-#define XtParent XTPARENT
-#endif
-
-#include "wx/setup.h"
-#include "wx/utils.h"
-#include "wx/palette.h"
 #include "wx/bitmap.h"
 #include "wx/icon.h"
 #include "wx/log.h"
 #include "wx/bitmap.h"
 #include "wx/icon.h"
 #include "wx/log.h"
-#include "wx/control.h"
-#include "wx/dcmemory.h"
 #include "wx/image.h"
 #include "wx/app.h"
 #include "wx/image.h"
 #include "wx/app.h"
-
-#ifdef __VMS__
-#pragma message disable nosimpint
-#endif
-#include <Xm/Xm.h>
-#ifdef __VMS__
-#pragma message enable nosimpint
+#if wxUSE_NANOX
+#include "wx/dcmemory.h"
 #endif
 
 #endif
 
-#include "wx/motif/private.h"
+#include "wx/x11/private.h"
 
 
+/* No point in using libXPM for NanoX */
+#if wxUSE_NANOX
+#undef wxHAVE_LIB_XPM
+#define wxHAVE_LIB_XPM 0
+
+// Copy from the drawable to the wxImage
+bool wxGetImageFromDrawable(GR_DRAW_ID drawable, int srcX, int srcY, int width, int height, wxImage& image);
+#endif
+
+#if wxUSE_XPM
 #if wxHAVE_LIB_XPM
 #if wxHAVE_LIB_XPM
-    #include <X11/xpm.h>
+#include <X11/xpm.h>
+#else
+#include "wx/xpmdecod.h"
+#include "wx/wfstream.h"
+#endif
 #endif
 #include <math.h>
 
 #endif
 #include <math.h>
 
+//-----------------------------------------------------------------------------
+// wxMask
+//-----------------------------------------------------------------------------
 
 
-IMPLEMENT_DYNAMIC_CLASS(wxBitmap, wxGDIObject)
-IMPLEMENT_DYNAMIC_CLASS(wxMask, wxObject)
-
-wxBitmapRefData::wxBitmapRefData()
-{
-    m_ok = FALSE;
-    m_width = 0;
-    m_height = 0;
-    m_depth = 0;
-    m_quality = 0;
-    m_numColors = 0;
-    m_bitmapMask = NULL;
-
-    m_pixmap = (WXPixmap) 0;
-    m_display = (WXDisplay*) 0;
-
-    m_freePixmap = TRUE; //TODO: necessary?
-    m_freeColors = (unsigned long*) 0;
-    m_freeColorsCount = 0;
-
-    // These 5 variables are for wxControl
-    m_insensPixmap = (WXPixmap) 0;
-    m_labelPixmap = (WXPixmap) 0;
-    m_armPixmap = (WXPixmap) 0;
-    m_image = (WXImage*) 0;
-    m_insensImage = (WXImage*) 0;
-}
-
-wxBitmapRefData::~wxBitmapRefData()
-{
-    if (m_labelPixmap)
-        XmDestroyPixmap (DefaultScreenOfDisplay ((Display*) m_display), (Pixmap) m_labelPixmap);
-
-    if (m_armPixmap)
-        XmDestroyPixmap (DefaultScreenOfDisplay ((Display*) m_display), (Pixmap) m_armPixmap);
-
-    if (m_insensPixmap)
-        XmDestroyPixmap (DefaultScreenOfDisplay ((Display*) m_display), (Pixmap) m_insensPixmap);
-
-    if (m_image)
-    {
-        XmUninstallImage ((XImage*) m_image);
-        XtFree ((char *) (XImage*) m_image);
-    }
-
-    if (m_insensImage)
-    {
-        XmUninstallImage ((XImage*) m_insensImage);
-        delete[] ((XImage*) m_insensImage)->data;
-        XtFree ((char *) (XImage*) m_insensImage);
-    }
-    if (m_pixmap && m_freePixmap)
-        XFreePixmap ((Display*) m_display, (Pixmap) m_pixmap);
-
-    if (m_freeColors)
-    {
-        int screen = DefaultScreen((Display*) m_display);
-        Colormap cmp = DefaultColormap((Display*) m_display,screen);
-        long llp;
-        for(llp = 0;llp < m_freeColorsCount;llp++)
-            XFreeColors((Display*) m_display, cmp, &m_freeColors[llp], 1, 0L);
-        delete m_freeColors;
-    };
-
-    if (m_bitmapMask)
-        delete m_bitmapMask;
-    m_bitmapMask = NULL;
-}
-
-wxList wxBitmap::sm_handlers;
-
-#define M_BMPDATA ((wxBitmapRefData *)m_refData)
-
-wxBitmap::wxBitmap()
-{
-    m_refData = NULL;
-}
-
-wxBitmap::~wxBitmap()
-{
-}
-
-wxBitmap::wxBitmap(const char bits[], int width, int height, int depth)
-{
-    m_refData = new wxBitmapRefData;
-
-    (void) Create((void*) bits, wxBITMAP_TYPE_XBM_DATA, width, height, depth);
-}
+IMPLEMENT_DYNAMIC_CLASS(wxMask,wxObject)
 
 
-wxBitmap::wxBitmap(int w, int h, int d)
+wxMask::wxMask()
 {
 {
-    (void)Create(w, h, d);
+    m_bitmap = NULL;
+    m_display = NULL;
 }
 
 }
 
-wxBitmap::wxBitmap(void *data, long type, int width, int height, int depth)
+wxMask::wxMask( const wxBitmap& bitmap, const wxColour& colour )
 {
 {
-    (void) Create(data, type, width, height, depth);
+    m_bitmap = NULL;
+    Create( bitmap, colour );
 }
 
 }
 
-wxBitmap::wxBitmap(const wxString& filename, long type)
+wxMask::wxMask( const wxBitmap& bitmap, int paletteIndex )
 {
 {
-    LoadFile(filename, (int)type);
+    m_bitmap = NULL;
+    Create( bitmap, paletteIndex );
 }
 
 }
 
-// Create from XPM data
-static wxControl* sg_Control = NULL;
-wxBitmap::wxBitmap(char **data, wxControl* control)
+wxMask::wxMask( const wxBitmap& bitmap )
 {
 {
-    // Pass the control to the Create function using a global
-    sg_Control = control;
-
-    (void) Create((void *)data, wxBITMAP_TYPE_XPM_DATA, 0, 0, 0);
-
-    sg_Control = (wxControl*) NULL;
+    m_bitmap = NULL;
+    Create( bitmap );
 }
 
 }
 
-bool wxBitmap::CreateFromXpm(const char **bits)
+wxMask::~wxMask()
 {
 {
-    wxCHECK_MSG( bits, FALSE, _T("NULL pointer in wxBitmap::CreateFromXpm") );
-
-    return Create(bits, wxBITMAP_TYPE_XPM_DATA, 0, 0, 0);
+    if (m_bitmap)
+        XFreePixmap( (Display*) m_display, (Pixmap) m_bitmap );
 }
 
 }
 
-bool wxBitmap::Create(int w, int h, int d)
+bool wxMask::Create( const wxBitmap& bitmap,
+                     const wxColour& colour )
 {
 {
-    UnRef();
-
-    m_refData = new wxBitmapRefData;
-
-    if (d < 1)
-        d = wxDisplayDepth();
-
-    M_BITMAPDATA->m_width = w;
-    M_BITMAPDATA->m_height = h;
-    M_BITMAPDATA->m_depth = d;
-    M_BITMAPDATA->m_freePixmap = TRUE;
+#if !wxUSE_NANOX
+    if (m_bitmap)
+    {
+        XFreePixmap( (Display*) m_display, (Pixmap) m_bitmap );
+        m_bitmap = NULL;
+    }
 
 
-    Display *dpy = (Display*) wxGetDisplay();
+    m_display = bitmap.GetDisplay();
 
 
-    M_BITMAPDATA->m_display = dpy; /* MATTHEW: [4] Remember the display */
+    wxImage image = bitmap.ConvertToImage();
+    if (!image.Ok()) return FALSE;
+    
+    m_display = bitmap.GetDisplay();
+    
+    Display *xdisplay = (Display*) m_display;
+    int xscreen = DefaultScreen( xdisplay );
+    Window xroot = RootWindow( xdisplay, xscreen );
+    
+    m_bitmap = (WXPixmap) XCreatePixmap( xdisplay, xroot, image.GetWidth(), image.GetHeight(), 1 );
+    GC gc = XCreateGC( xdisplay, (Pixmap) m_bitmap, 0, NULL );
 
 
-    M_BITMAPDATA->m_pixmap = (WXPixmap) XCreatePixmap (dpy, RootWindow (dpy, DefaultScreen (dpy)),
-        w, h, d);
+    XSetForeground( xdisplay, gc, WhitePixel(xdisplay,xscreen) );
+    XSetFillStyle( xdisplay, gc, FillSolid );
+    XFillRectangle( xdisplay, (Pixmap) m_bitmap, gc, 0, 0, image.GetWidth(), image.GetHeight() );
 
 
-    M_BITMAPDATA->m_ok = (M_BITMAPDATA->m_pixmap != (WXPixmap) 0) ;
-    return M_BITMAPDATA->m_ok;
-}
+    unsigned char *data = image.GetData();
+    int index = 0;
 
 
-bool wxBitmap::LoadFile(const wxString& filename, long type)
-{
-    UnRef();
+    unsigned char red = colour.Red();
+    unsigned char green = colour.Green();
+    unsigned char blue = colour.Blue();
 
 
-    m_refData = new wxBitmapRefData;
+    int bpp = wxTheApp->GetVisualInfo(m_display)->m_visualDepth;
+    
+    if (bpp == 15)
+    {
+        red = red & 0xf8;
+        green = green & 0xf8;
+        blue = blue & 0xf8;
+    } else
+    if (bpp == 16)
+    {
+        red = red & 0xf8;
+        green = green & 0xfc;
+        blue = blue & 0xf8;
+    } else
+    if (bpp == 12)
+    {
+        red = red & 0xf0;
+        green = green & 0xf0;
+        blue = blue & 0xf0;
+    }
 
 
-    wxBitmapHandler *handler = FindHandler(type);
+    XSetForeground( xdisplay, gc, BlackPixel(xdisplay,xscreen) );
 
 
-    if ( handler == NULL ) {
-        wxImage image;
-        if (!image.LoadFile( filename, type )) return FALSE;
-        if (image.Ok()) 
+    int width = image.GetWidth();
+    int height = image.GetHeight();
+    for (int j = 0; j < height; j++)
+    {
+        int start_x = -1;
+        int i;
+        for (i = 0; i < width; i++)
         {
         {
-            *this = image.ConvertToBitmap();
-            return TRUE;
+            if ((data[index] == red) &&
+                (data[index+1] == green) &&
+                (data[index+2] == blue))
+            {
+                if (start_x == -1)
+                start_x = i;
+            }
+            else
+            {
+                if (start_x != -1)
+                {
+                    XDrawLine( xdisplay, (Pixmap) m_bitmap, gc, start_x, j, i-1, j );
+                    start_x = -1;
+                }
+            }
+            index += 3;
         }
         }
-        else return FALSE;
+        if (start_x != -1)
+            XDrawLine( xdisplay, (Pixmap) m_bitmap, gc, start_x, j, i, j );
     }
     }
+    
+    XFreeGC( xdisplay, gc );
 
 
-    return handler->LoadFile(this, filename, type, -1, -1);
+    return TRUE;
+#else
+    return FALSE;
+#endif
+    // wxUSE_NANOX
 }
 
 }
 
-bool wxBitmap::Create(void *data, long type, int width, int height, int depth)
+bool wxMask::Create( const wxBitmap& bitmap, int paletteIndex )
 {
 {
-    UnRef();
+    unsigned char r,g,b;
+    wxPalette *pal = bitmap.GetPalette();
 
 
-    m_refData = new wxBitmapRefData;
-
-    wxBitmapHandler *handler = FindHandler(type);
+    wxCHECK_MSG( pal, FALSE, wxT("Cannot create mask from bitmap without palette") );
 
 
-    if ( handler == NULL ) {
-        wxLogWarning("no data bitmap handler for type %d defined.", type);
-
-        return FALSE;
-    }
+    pal->GetRGB(paletteIndex, &r, &g, &b);
 
 
-    return handler->Create(this, data, type, width, height, depth);
+    return Create(bitmap, wxColour(r, g, b));
 }
 
 }
 
-bool wxBitmap::SaveFile(const wxString& filename, int type, const wxPalette *palette)
+bool wxMask::Create( const wxBitmap& bitmap )
 {
 {
-    wxBitmapHandler *handler = FindHandler(type);
-
-    if ( handler == NULL ) { // try wxImage
-        wxImage image( *this );
-        if (image.Ok()) return image.SaveFile( filename, type );
-        else return FALSE;
+#if !wxUSE_NANOX
+    if (m_bitmap)
+    {
+        XFreePixmap( (Display*) m_display, (Pixmap) m_bitmap );
+        m_bitmap = NULL;
     }
 
     }
 
-    return handler->SaveFile(this, filename, type, palette);
-}
+    if (!bitmap.Ok()) return FALSE;
 
 
-void wxBitmap::SetWidth(int w)
-{
-    if (!M_BITMAPDATA)
-        m_refData = new wxBitmapRefData;
-
-    M_BITMAPDATA->m_width = w;
-}
+    wxCHECK_MSG( bitmap.GetBitmap(), FALSE, wxT("Cannot create mask from colour bitmap") );
+    
+    m_display = bitmap.GetDisplay();
 
 
-void wxBitmap::SetHeight(int h)
-{
-    if (!M_BITMAPDATA)
-        m_refData = new wxBitmapRefData;
+    int xscreen = DefaultScreen( (Display*) m_display );
+    Window xroot = RootWindow( (Display*) m_display, xscreen );
+    
+    m_bitmap = (WXPixmap) XCreatePixmap( (Display*) m_display, xroot, bitmap.GetWidth(), bitmap.GetHeight(), 1 );
 
 
-    M_BITMAPDATA->m_height = h;
-}
+    if (!m_bitmap) return FALSE;
+    
+    GC gc = XCreateGC( (Display*) m_display, (Pixmap) m_bitmap, 0, NULL );
 
 
-void wxBitmap::SetDepth(int d)
-{
-    if (!M_BITMAPDATA)
-        m_refData = new wxBitmapRefData;
+    XCopyPlane( (Display*) m_display, (Pixmap) bitmap.GetBitmap(), (Pixmap) m_bitmap,
+       gc, 0, 0, bitmap.GetWidth(), bitmap.GetHeight(), 0, 0, 1 );
+       
+    XFreeGC( (Display*) m_display, gc );
 
 
-    M_BITMAPDATA->m_depth = d;
+    return TRUE;
+#else
+    return FALSE;
+#endif
+    // wxUSE_NANOX
 }
 
 }
 
-void wxBitmap::SetQuality(int q)
-{
-    if (!M_BITMAPDATA)
-        m_refData = new wxBitmapRefData;
-
-    M_BITMAPDATA->m_quality = q;
-}
+//-----------------------------------------------------------------------------
+// wxBitmap
+//-----------------------------------------------------------------------------
 
 
-void wxBitmap::SetOk(bool isOk)
+class wxBitmapRefData: public wxObjectRefData
 {
 {
-    if (!M_BITMAPDATA)
-        m_refData = new wxBitmapRefData;
+public:
+    wxBitmapRefData();
+    ~wxBitmapRefData();
+
+    WXPixmap        m_pixmap;
+    WXPixmap        m_bitmap;
+    WXDisplay      *m_display;
+    wxMask         *m_mask;
+    int             m_width;
+    int             m_height;
+    int             m_bpp;
+    wxPalette      *m_palette;
+};
 
 
-    M_BITMAPDATA->m_ok = isOk;
+wxBitmapRefData::wxBitmapRefData()
+{
+    m_pixmap = NULL;
+    m_bitmap = NULL;
+    m_display = NULL;
+    m_mask = (wxMask *) NULL;
+    m_width = 0;
+    m_height = 0;
+    m_bpp = 0;
+    m_palette = (wxPalette *) NULL;
 }
 
 }
 
-void wxBitmap::SetPalette(const wxPalette& palette)
+wxBitmapRefData::~wxBitmapRefData()
 {
 {
-    if (!M_BITMAPDATA)
-        m_refData = new wxBitmapRefData;
-
-    M_BITMAPDATA->m_bitmapPalette = palette ;
+    if (m_pixmap) XFreePixmap( (Display*) m_display, (Pixmap) m_pixmap );
+    if (m_bitmap) XFreePixmap( (Display*) m_display, (Pixmap) m_bitmap );
+    if (m_mask) delete m_mask;
+    if (m_palette) delete m_palette;
 }
 
 }
 
-void wxBitmap::SetMask(wxMask *mask)
-{
-    if (!M_BITMAPDATA)
-        m_refData = new wxBitmapRefData;
+//-----------------------------------------------------------------------------
 
 
-    M_BITMAPDATA->m_bitmapMask = mask ;
-}
+// helper function
 
 
-wxBitmap wxBitmap::GetSubBitmap( const wxRect& rect) const
+static WXPixmap wxGetSubPixmap( WXDisplay* xdisplay, WXPixmap xpixmap,
+                                int x, int y, int width, int height,
+                                int depth )
 {
 {
-    wxCHECK_MSG( Ok() &&
-                 (rect.x >= 0) && (rect.y >= 0) &&
-                 (rect.x+rect.width <= M_BMPDATA->m_width) && (rect.y+rect.height <= M_BMPDATA->m_height),
-                 wxNullBitmap, wxT("invalid bitmap or bitmap region") );
-
-    wxBitmap ret( rect.width, rect.height, 0 );
-    wxASSERT_MSG( ret.Ok(), wxT("GetSubBitmap error") );
+    int xscreen = DefaultScreen( (Display*)xdisplay );
+    Window xroot = RootWindow( (Display*)xdisplay, xscreen );
+    Visual* xvisual = DefaultVisual( xdisplay, xscreen );
+
+    XImage* ximage = XCreateImage( (Display*)xdisplay, xvisual, depth,
+                                   ZPixmap, 0, 0, width, height, 32, 0 );
+    ximage->data = (char*)malloc( ximage->bytes_per_line * ximage->height );
+    ximage = XGetSubImage( (Display*)xdisplay, (Pixmap)xpixmap,
+                           x, y, width, height, 
+                           AllPlanes, ZPixmap, ximage, 0, 0 );
+
+    GC gc = XCreateGC( (Display*)xdisplay, (Pixmap)xpixmap, 0, NULL );
+    Pixmap ret = XCreatePixmap( (Display*)xdisplay, xroot,
+                                width, height, depth );
+
+    XPutImage( (Display*)xdisplay, ret, gc, ximage,
+               0, 0, 0, 0, width, height );
+    XDestroyImage( ximage );
+    XFreeGC( (Display*)xdisplay, gc );
 
 
-   // The remaining still TODO
-   return ret;
+    return (WXPixmap)ret;
 }
 
 }
 
-void wxBitmap::AddHandler(wxBitmapHandler *handler)
-{
-    sm_handlers.Append(handler);
-}
+#define M_BMPDATA ((wxBitmapRefData *)m_refData)
 
 
-void wxBitmap::InsertHandler(wxBitmapHandler *handler)
-{
-    sm_handlers.Insert(handler);
-}
+IMPLEMENT_DYNAMIC_CLASS(wxBitmap,wxGDIObject)
 
 
-bool wxBitmap::RemoveHandler(const wxString& name)
+wxBitmap::wxBitmap()
 {
 {
-    wxBitmapHandler *handler = FindHandler(name);
-    if ( handler )
-    {
-        sm_handlers.DeleteObject(handler);
-        return TRUE;
-    }
-    else
-        return FALSE;
 }
 
 }
 
-wxBitmapHandler *wxBitmap::FindHandler(const wxString& name)
+wxBitmap::wxBitmap( int width, int height, int depth )
 {
 {
-    wxNode *node = sm_handlers.First();
-    while ( node )
-    {
-        wxBitmapHandler *handler = (wxBitmapHandler *)node->Data();
-        if ( handler->GetName() == name )
-            return handler;
-        node = node->Next();
-    }
-    return NULL;
+    Create( width, height, depth );
 }
 
 }
 
-wxBitmapHandler *wxBitmap::FindHandler(const wxString& extension, long bitmapType)
+bool wxBitmap::Create( int width, int height, int depth )
 {
 {
-    wxNode *node = sm_handlers.First();
-    while ( node )
-    {
-        wxBitmapHandler *handler = (wxBitmapHandler *)node->Data();
-        if ( handler->GetExtension() == extension &&
-            (bitmapType == -1 || handler->GetType() == bitmapType) )
-            return handler;
-        node = node->Next();
-    }
-    return NULL;
-}
+    UnRef();
 
 
-wxBitmapHandler *wxBitmap::FindHandler(long bitmapType)
-{
-    wxNode *node = sm_handlers.First();
-    while ( node )
-    {
-        wxBitmapHandler *handler = (wxBitmapHandler *)node->Data();
-        if (handler->GetType() == bitmapType)
-            return handler;
-        node = node->Next();
-    }
-    return NULL;
-}
+    wxCHECK_MSG( (width > 0) && (height > 0), FALSE, wxT("invalid bitmap size") )
 
 
-/*
-* wxMask
-*/
+    m_refData = new wxBitmapRefData();
+    
+    M_BMPDATA->m_display = wxGlobalDisplay();
+    
+    wxASSERT_MSG( M_BMPDATA->m_display, wxT("No display") );
+    
+    int xscreen = DefaultScreen( (Display*) M_BMPDATA->m_display );
+    Window xroot = RootWindow( (Display*) M_BMPDATA->m_display, xscreen );
+    
+    int bpp = DefaultDepth( (Display*) M_BMPDATA->m_display, xscreen );
+    if (depth == -1) depth = bpp;
 
 
-wxMask::wxMask()
-{
-    m_pixmap = (WXPixmap) 0;
-}
+    wxCHECK_MSG( (depth == bpp) ||
+                 (depth == 1), FALSE, wxT("invalid bitmap depth") )
 
 
-// Construct a mask from a bitmap and a colour indicating
-// the transparent area
-wxMask::wxMask(const wxBitmap& bitmap, const wxColour& colour)
-{
-    m_pixmap = (WXPixmap) 0;
+    M_BMPDATA->m_mask = (wxMask *) NULL;
+    M_BMPDATA->m_width = width;
+    M_BMPDATA->m_height = height;
 
 
-    Create(bitmap, colour);
+#if wxUSE_NANOX
+    M_BMPDATA->m_pixmap = (WXPixmap) GrNewPixmap(width, height, NULL);
+    M_BMPDATA->m_bpp = bpp;
+
+    wxASSERT_MSG( M_BMPDATA->m_pixmap, wxT("Bitmap creation failed") );
+#else
+    if (depth == 1)
+    {
+        M_BMPDATA->m_bitmap = (WXPixmap) XCreatePixmap( (Display*) M_BMPDATA->m_display, xroot, width, height, 1 );
+        
+        wxASSERT_MSG( M_BMPDATA->m_bitmap, wxT("Bitmap creation failed") );
+        
+        M_BMPDATA->m_bpp = 1;
+    }
+    else
+    {
+        M_BMPDATA->m_pixmap = (WXPixmap) XCreatePixmap( (Display*) M_BMPDATA->m_display, xroot, width, height, depth );
+        
+        wxASSERT_MSG( M_BMPDATA->m_pixmap, wxT("Pixmap creation failed") );
+        
+        M_BMPDATA->m_bpp = depth;
+    }
+#endif
+    return Ok();
 }
 
 }
 
-// Construct a mask from a bitmap and a palette index indicating
-// the transparent area
-wxMask::wxMask(const wxBitmap& bitmap, int paletteIndex)
+bool wxBitmap::Create(void *data, wxBitmapType type,
+                      int width, int height, int depth)
 {
 {
-    m_pixmap = (WXPixmap) 0;
+    UnRef();
 
 
-    Create(bitmap, paletteIndex);
-}
+    wxBitmapHandler *handler = FindHandler(type);
 
 
-// Construct a mask from a mono bitmap (copies the bitmap).
-wxMask::wxMask(const wxBitmap& bitmap)
-{
-    m_pixmap = (WXPixmap) 0;
+    if ( handler == NULL ) {
+        wxLogWarning("no data bitmap handler for type %ld defined.",
+                     (long)type);
 
 
-    Create(bitmap);
-}
+        return FALSE;
+    }
 
 
-wxMask::~wxMask()
-{
-    // TODO: this may be the wrong display
-    if ( m_pixmap )
-        XFreePixmap ((Display*) wxGetDisplay(), (Pixmap) m_pixmap);
+    return handler->Create(this, data, type, width, height, depth);
 }
 
 }
 
-// Create a mask from a mono bitmap (copies the bitmap).
-bool wxMask::Create(const wxBitmap& WXUNUSED(bitmap))
+bool wxBitmap::CreateFromXpm( const char **bits )
 {
 {
-    // TODO
-    return FALSE;
-}
+    wxCHECK_MSG( bits, FALSE, _T("NULL pointer in wxBitmap::CreateFromXpm") );
 
 
-// Create a mask from a bitmap and a palette index indicating
-// the transparent area
-bool wxMask::Create(const wxBitmap& WXUNUSED(bitmap), int WXUNUSED(paletteIndex))
-{
-    // TODO
-    return FALSE;
+    return Create(bits, wxBITMAP_TYPE_XPM_DATA, 0, 0, 0);
 }
 
 }
 
-// Create a mask from a bitmap and a colour indicating
-// the transparent area
-bool wxMask::Create(const wxBitmap& WXUNUSED(bitmap), const wxColour& WXUNUSED(colour))
+bool wxBitmap::CreateFromImage( const wxImage& image, int depth )
 {
 {
-    // TODO
-    return FALSE;
-}
-
-/*
-* wxBitmapHandler
-*/
+#if wxUSE_NANOX
+    if (!image.Ok())
+    {
+        wxASSERT_MSG(image.Ok(), "Invalid wxImage passed to wxBitmap::CreateFromImage.");
+        return FALSE;
+    }
+    
+    int w = image.GetWidth();
+    int h = image.GetHeight();
+    
+    if (!Create(w, h, depth))
+        return FALSE;
 
 
-IMPLEMENT_DYNAMIC_CLASS(wxBitmapHandler, wxObject)
+    // Unfortunately the mask has to be screen-depth since
+    // 1-bpp bitmaps don't seem to be supported
+    // TODO: implement transparent drawing, presumably
+    // by doing several blits as per the Windows
+    // implementation because Nano-X doesn't support
+    // XSetClipMask.
+    // TODO: could perhaps speed this function up
+    // by making a buffer of pixel values,
+    // and then calling GrArea to write that to the
+    // pixmap. See demos/nxroach.c.
 
 
-bool wxBitmapHandler::Create(wxBitmap *WXUNUSED(bitmap), void *WXUNUSED(data), long WXUNUSED(type),
-                             int WXUNUSED(width), int WXUNUSED(height), int WXUNUSED(depth))
-{
-    return FALSE;
-}
+    bool hasMask = image.HasMask();
+    
+    GC pixmapGC = GrNewGC();
+    Pixmap pixmap = (Pixmap) GetPixmap();
 
 
-bool wxBitmapHandler::LoadFile(wxBitmap *WXUNUSED(bitmap), const wxString& WXUNUSED(name), long WXUNUSED(type),
-                               int WXUNUSED(desiredWidth), int WXUNUSED(desiredHeight))
-{
-    return FALSE;
-}
+    GC maskGC = 0;
+    Pixmap maskPixmap = 0;
 
 
-bool wxBitmapHandler::SaveFile(wxBitmap *WXUNUSED(bitmap), const wxString& WXUNUSED(name), int WXUNUSED(type),
-                               const wxPalette *WXUNUSED(palette))
-{
-    return FALSE;
-}
+    unsigned char maskR = 0;
+    unsigned char maskG = 0;
+    unsigned char maskB = 0;
 
 
-/*
-* Standard handlers
-*/
+    if (hasMask)
+    {
+        maskR = image.GetMaskRed();
+        maskG = image.GetMaskGreen();
+        maskB = image.GetMaskBlue();
+        
+        maskGC = GrNewGC();
+        maskPixmap = GrNewPixmap(w, h, 0);
+        if (!maskPixmap)
+            hasMask = FALSE;
+        else
+        {
+            wxMask* mask = new wxMask;
+            mask->SetBitmap((WXPixmap) maskPixmap);
+            SetMask(mask);
+        }
+    }
 
 
-class WXDLLEXPORT wxXBMFileHandler: public wxBitmapHandler
-{
-    DECLARE_DYNAMIC_CLASS(wxXBMFileHandler)
-public:
-    inline wxXBMFileHandler()
+    GR_COLOR lastPixmapColour = 0;
+    GR_COLOR lastMaskColour = 0;
+    
+    int i, j;
+    for (i = 0; i < w; i++)
     {
     {
-        m_name = "XBM file";
-        m_extension = "xbm";
-        m_type = wxBITMAP_TYPE_XBM;
-    };
+        for (j = 0; j < h; j++)
+        {
+            unsigned char red = image.GetRed(i, j);
+            unsigned char green = image.GetGreen(i, j);
+            unsigned char blue = image.GetBlue(i, j);
 
 
-    virtual bool LoadFile(wxBitmap *bitmap, const wxString& name, long flags,
-        int desiredWidth, int desiredHeight);
-};
-IMPLEMENT_DYNAMIC_CLASS(wxXBMFileHandler, wxBitmapHandler)
+            GR_COLOR colour = GR_RGB(red, green, blue);
 
 
-bool wxXBMFileHandler::LoadFile(wxBitmap *bitmap, const wxString& name, long WXUNUSED(flags),
-                                int WXUNUSED(desiredWidth), int WXUNUSED(desiredHeight))
-{
-    M_BITMAPHANDLERDATA->m_freePixmap = TRUE;
+            // Efficiency measure
+            if (colour != lastPixmapColour || (i == 0 && j == 0))
+            {
+                GrSetGCForeground(pixmapGC, colour);
+                lastPixmapColour = colour;
+            }
+            
+            GrPoint(pixmap, pixmapGC, i, j);
+            
+            if (hasMask)
+            {
+                // scan the bitmap for the transparent colour and set the corresponding
+                // pixels in the mask to BLACK and the rest to WHITE
+                if (maskR == red && maskG == green && maskB == blue)
+                {
+                    colour = GR_RGB(0, 0, 0);
+                }
+                else
+                {
+                    colour = GR_RGB(255, 255, 255);
+                }
+                if (colour != lastMaskColour || (i == 0 && j == 0))
+                {
+                    GrSetGCForeground(maskGC, colour);
+                    lastMaskColour = colour;
+                }
+                GrPoint(maskPixmap, maskGC, i, j);
+            }
+        }
+    }
 
 
-    int hotX, hotY;
-    unsigned int w, h;
-    Pixmap pixmap;
+    GrDestroyGC(pixmapGC);
+    if (hasMask)
+        GrDestroyGC(maskGC);
+    
+    return TRUE;
+#else
+    // !wxUSE_NANOX
+    
+    UnRef();
 
 
-    Display *dpy = (Display*) wxGetDisplay();
-    M_BITMAPDATA->m_display = (WXDisplay*) dpy;
+    wxCHECK_MSG( image.Ok(), FALSE, wxT("invalid image") )
+    wxCHECK_MSG( depth == -1, FALSE, wxT("invalid bitmap depth") )
+
+    m_refData = new wxBitmapRefData();
 
 
-    int value = XReadBitmapFile (dpy, RootWindow (dpy, DefaultScreen (dpy)),
-        (char*) (const char*) name, &w, &h, &pixmap, &hotX, &hotY);
-    M_BITMAPHANDLERDATA->m_width = w;
-    M_BITMAPHANDLERDATA->m_height = h;
-    M_BITMAPHANDLERDATA->m_depth = 1;
-    M_BITMAPHANDLERDATA->m_pixmap = (WXPixmap) pixmap;
+    M_BMPDATA->m_display = wxGlobalDisplay();
+    
+    Display *xdisplay = (Display*) M_BMPDATA->m_display;
+    
+    int xscreen = DefaultScreen( xdisplay );
+    Window xroot = RootWindow( xdisplay, xscreen );
+    Visual* xvisual = DefaultVisual( xdisplay, xscreen );
+    
+    int bpp = wxTheApp->GetVisualInfo(M_BMPDATA->m_display)->m_visualDepth;
+    
+    int width = image.GetWidth();
+    int height = image.GetHeight();
+    M_BMPDATA->m_width = width;
+    M_BMPDATA->m_height = height;
 
 
-    if ((value == BitmapFileInvalid) ||
-        (value == BitmapOpenFailed) ||
-        (value == BitmapNoMemory))
+    if (depth != 1) depth = bpp;
+    M_BMPDATA->m_bpp = depth;
+    
+    if (depth == 1)
     {
     {
-        M_BITMAPHANDLERDATA->m_ok = FALSE;
-        M_BITMAPHANDLERDATA->m_pixmap = (WXPixmap) 0;
+        wxFAIL_MSG( "mono images later" );
     }
     else
     }
     else
-        M_BITMAPHANDLERDATA->m_ok = TRUE;
-
-    return M_BITMAPHANDLERDATA->m_ok ;
-}
-
-class WXDLLEXPORT wxXBMDataHandler: public wxBitmapHandler
-{
-    DECLARE_DYNAMIC_CLASS(wxXBMDataHandler)
-public:
-    inline wxXBMDataHandler()
     {
     {
-        m_name = "XBM data";
-        m_extension = "xbm";
-        m_type = wxBITMAP_TYPE_XBM_DATA;
-    };
-
-    virtual bool Create(wxBitmap *bitmap, void *data, long flags, int width, int height, int depth = 1);
-};
-IMPLEMENT_DYNAMIC_CLASS(wxXBMDataHandler, wxBitmapHandler)
+        // Create image
+    
+        XImage *data_image = XCreateImage( xdisplay, xvisual, bpp, ZPixmap, 0, 0, width, height, 32, 0 );
+        data_image->data = (char*) malloc( data_image->bytes_per_line * data_image->height );
+    
+        if (data_image->data == NULL)
+        {
+            wxLogError( wxT("Out of memory.") );  // TODO clean
+            return FALSE;
+        }
 
 
-bool wxXBMDataHandler::Create( wxBitmap *bitmap, void *data, long WXUNUSED(flags),
-                              int width, int height, int WXUNUSED(depth))
-{
-    M_BITMAPHANDLERDATA->m_width = width;
-    M_BITMAPHANDLERDATA->m_height = height;
-    M_BITMAPHANDLERDATA->m_depth = 1;
-    M_BITMAPHANDLERDATA->m_freePixmap = TRUE;
-
-    Display *dpy = (Display*) wxGetDisplay();
-    M_BITMAPHANDLERDATA->m_display = (WXDisplay*) dpy;
-
-    M_BITMAPHANDLERDATA->m_pixmap = (WXPixmap) XCreateBitmapFromData (dpy, RootWindow (dpy, DefaultScreen (dpy)), (char*) data, width, height);
-    M_BITMAPHANDLERDATA->m_ok = (M_BITMAPHANDLERDATA->m_pixmap != (WXPixmap) 0) ;
-
-    // code for wxControl. TODO: can we avoid doing this until we need it?
-    // E.g. have CreateButtonPixmaps which is called on demand.
-    XImage* image = (XImage *) XtMalloc (sizeof (XImage));
-    image->width = width;
-    image->height = height;
-    image->data = (char*) data;
-    image->depth = 1;
-    image->xoffset = 0;
-    image->format = XYBitmap;
-    image->byte_order = LSBFirst;
-    image->bitmap_unit = 8;
-    image->bitmap_bit_order = LSBFirst;
-    image->bitmap_pad = 8;
-    image->bytes_per_line = (width + 7) >> 3;
-
-    char tmp[128];
-    sprintf (tmp, "Im%x", (unsigned int) image);
-    XmInstallImage (image, tmp);
-
-    // Build our manually stipped pixmap.
-
-    int bpl = (width + 7) / 8;
-    char *data1 = new char[height * bpl];
-    char* bits = (char*) data;
-    int i;
-    for (i = 0; i < height; i++)
-    {
-        int mask = i % 2 ? 0x55 : 0xaa;
-        int j;
-        for (j = 0; j < bpl; j++)
-            data1[i * bpl + j] = bits[i * bpl + j] & mask;
-    }
-    XImage* insensImage = (XImage *) XtMalloc (sizeof (XImage));
-    insensImage->width = width;
-    insensImage->height = height;
-    insensImage->data = data1;
-    insensImage->depth = 1;
-    insensImage->xoffset = 0;
-    insensImage->format = XYBitmap;
-    insensImage->byte_order = LSBFirst;
-    insensImage->bitmap_unit = 8;
-    insensImage->bitmap_bit_order = LSBFirst;
-    insensImage->bitmap_pad = 8;
-    insensImage->bytes_per_line = bpl;
-
-    sprintf (tmp, "Not%x", (unsigned int)insensImage);
-    XmInstallImage (insensImage, tmp);
-
-    M_BITMAPHANDLERDATA->m_image = (WXImage*) image;
-    M_BITMAPHANDLERDATA->m_insensImage = (WXImage*) insensImage;
+        M_BMPDATA->m_pixmap = (WXPixmap) XCreatePixmap( xdisplay, xroot, width, height, depth );
 
 
-    return TRUE;
-}
+        // Create mask
 
 
-#if wxHAVE_LIB_XPM
-class WXDLLEXPORT wxXPMFileHandler: public wxBitmapHandler
-{
-    DECLARE_DYNAMIC_CLASS(wxXPMFileHandler)
-public:
-    inline wxXPMFileHandler()
-    {
-        m_name = "XPM file";
-        m_extension = "xpm";
-        m_type = wxBITMAP_TYPE_XPM;
-    };
+        XImage *mask_image = (XImage*) NULL;
+        if (image.HasMask())
+        { 
+            mask_image = XCreateImage( xdisplay, xvisual, 1, ZPixmap, 0, 0, width, height, 32, 0 );
+            mask_image->data = (char*) malloc( mask_image->bytes_per_line * mask_image->height );
+        
+            if (mask_image->data == NULL)
+            {
+                wxLogError( wxT("Out of memory.") ); // TODO clean
+                return FALSE;
+            }
+            
+            wxMask *mask = new wxMask();
+            mask->SetDisplay( xdisplay );
+            mask->SetBitmap( (WXPixmap) XCreatePixmap( xdisplay, xroot, width, height, 1 ) );
 
 
-    virtual bool LoadFile(wxBitmap *bitmap, const wxString& name, long flags,
-        int desiredWidth, int desiredHeight);
-    virtual bool SaveFile(wxBitmap *bitmap, const wxString& name, int type, const wxPalette *palette = NULL);
-};
+            SetMask( mask );
+        }
 
 
-IMPLEMENT_DYNAMIC_CLASS(wxXPMFileHandler, wxBitmapHandler)
+        if (bpp < 8) bpp = 8;
 
 
-bool wxXPMFileHandler::LoadFile( wxBitmap *bitmap, const wxString& name, long WXUNUSED(flags),
-                                int WXUNUSED(desiredWidth), int WXUNUSED(desiredHeight) )
-{
-    Display *dpy = (Display*) wxGetDisplay();
-    M_BITMAPHANDLERDATA->m_display = (WXDisplay*) dpy;
+        // Render
 
 
-    XpmAttributes xpmAttr;
-    Pixmap pixmap;
-    Pixmap mask = 0;
+        enum byte_order { RGB, RBG, BRG, BGR, GRB, GBR };
+        byte_order b_o = RGB;
 
 
-    M_BITMAPHANDLERDATA->m_ok = FALSE;
-    xpmAttr.valuemask = XpmReturnInfos | XpmCloseness;
-    xpmAttr.closeness = 40000;
-    int errorStatus = XpmReadFileToPixmap(dpy,
-        RootWindow(dpy, DefaultScreen(dpy)), (char*) (const char*) name,
-        &pixmap, &mask, &xpmAttr);
+        wxXVisualInfo* vi = wxTheApp->GetVisualInfo(M_BMPDATA->m_display);
+        unsigned long greenMask = vi->m_visualGreenMask,
+                      redMask   = vi->m_visualRedMask,
+                      blueMask  = vi->m_visualBlueMask;
 
 
-    if (errorStatus == XpmSuccess)
-    {
-        M_BITMAPHANDLERDATA->m_pixmap = (WXPixmap) pixmap;
-        if ( mask )
+        if (bpp > 8)
         {
         {
-            M_BITMAPHANDLERDATA->m_bitmapMask = new wxMask;
-            M_BITMAPHANDLERDATA->m_bitmapMask->SetPixmap((WXPixmap) mask);
+            if ((redMask > greenMask) && (greenMask > blueMask))     b_o = RGB;
+            else if ((redMask > blueMask) && (blueMask > greenMask)) b_o = RBG;
+            else if ((blueMask > redMask) && (redMask > greenMask))  b_o = BRG;
+            else if ((blueMask > greenMask) && (greenMask > redMask))b_o = BGR;
+            else if ((greenMask > redMask) && (redMask > blueMask))  b_o = GRB;
+            else if ((greenMask > blueMask) && (blueMask > redMask)) b_o = GBR;
         }
 
         }
 
-        unsigned int depthRet;
-        int xRet, yRet;
-        unsigned int widthRet, heightRet, borderWidthRet;
-        Window rootWindowRet;
-        XGetGeometry(dpy, pixmap, &rootWindowRet, &xRet, &yRet,
-            &widthRet, &heightRet, &borderWidthRet, &depthRet);
+        int r_mask = image.GetMaskRed();
+        int g_mask = image.GetMaskGreen();
+        int b_mask = image.GetMaskBlue();
 
 
-        M_BITMAPHANDLERDATA->m_width = xpmAttr.width;
-        M_BITMAPHANDLERDATA->m_height = xpmAttr.height;
+        unsigned char* data = image.GetData();
+        wxASSERT_MSG( data, "No image data" );
+        
+        unsigned char *colorCube =
+            wxTheApp->GetVisualInfo(M_BMPDATA->m_display)->m_colorCube;
 
 
-        /*
-        if ( xpmAttr.npixels > 2 )
-        {
-        M_BITMAPHANDLERDATA->m_depth = 8;      // TODO: next time not just a guess :-) ...
-        } else
+        bool hasMask = image.HasMask();
+
+        int index = 0;
+        for (int y = 0; y < height; y++)
         {
         {
-        M_BITMAPHANDLERDATA->m_depth = 1;      // mono
-        }
-        */
+            for (int x = 0; x < width; x++)
+            {
+                int r = data[index];
+                index++;
+                int g = data[index];
+                index++;
+                int b = data[index];
+                index++;
+
+                if (hasMask)
+                {
+                    if ((r == r_mask) && (b == b_mask) && (g == g_mask))
+                        XPutPixel( mask_image, x, y, 0 );
+                    else
+                        XPutPixel( mask_image, x, y, 1 );
+                }
+
+                switch (bpp)
+                {
+                    case 8:
+                    {
+                        int pixel = 0;
+                        pixel = colorCube[ ((r & 0xf8) << 7) + ((g & 0xf8) << 2) + ((b & 0xf8) >> 3) ];
+                        XPutPixel( data_image, x, y, pixel );
+                        break;
+                    }
+                    case 12:  // SGI only
+                    {
+                        int pixel = 0;
+                        switch (b_o)
+                        {
+                            case RGB: pixel = ((r & 0xf0) << 4) | (g & 0xf0) | ((b & 0xf0) >> 4); break;
+                            case RBG: pixel = ((r & 0xf0) << 4) | (b & 0xf0) | ((g & 0xf0) >> 4); break;
+                            case GRB: pixel = ((g & 0xf0) << 4) | (r & 0xf0) | ((b & 0xf0) >> 4); break;
+                            case GBR: pixel = ((g & 0xf0) << 4) | (b & 0xf0) | ((r & 0xf0) >> 4); break;
+                            case BRG: pixel = ((b & 0xf0) << 4) | (r & 0xf0) | ((g & 0xf0) >> 4); break;
+                            case BGR: pixel = ((b & 0xf0) << 4) | (g & 0xf0) | ((r & 0xf0) >> 4); break;
+                        }
+                        XPutPixel( data_image, x, y, pixel );
+                        break;
+                    }
+                    case 15:
+                    {
+                        int pixel = 0;
+                        switch (b_o)
+                        {
+                            case RGB: pixel = ((r & 0xf8) << 7) | ((g & 0xf8) << 2) | ((b & 0xf8) >> 3); break;
+                            case RBG: pixel = ((r & 0xf8) << 7) | ((b & 0xf8) << 2) | ((g & 0xf8) >> 3); break;
+                            case GRB: pixel = ((g & 0xf8) << 7) | ((r & 0xf8) << 2) | ((b & 0xf8) >> 3); break;
+                            case GBR: pixel = ((g & 0xf8) << 7) | ((b & 0xf8) << 2) | ((r & 0xf8) >> 3); break;
+                            case BRG: pixel = ((b & 0xf8) << 7) | ((r & 0xf8) << 2) | ((g & 0xf8) >> 3); break;
+                            case BGR: pixel = ((b & 0xf8) << 7) | ((g & 0xf8) << 2) | ((r & 0xf8) >> 3); break;
+                        }
+                        XPutPixel( data_image, x, y, pixel );
+                        break;
+                    }
+                    case 16:
+                    {
+                        // I actually don't know if for 16-bit displays, it is alway the green
+                        // component or the second component which has 6 bits.
+                        int pixel = 0;
+                        switch (b_o)
+                        {
+                            case RGB: pixel = ((r & 0xf8) << 8) | ((g & 0xfc) << 3) | ((b & 0xf8) >> 3); break;
+                            case RBG: pixel = ((r & 0xf8) << 8) | ((b & 0xfc) << 3) | ((g & 0xf8) >> 3); break;
+                            case GRB: pixel = ((g & 0xf8) << 8) | ((r & 0xfc) << 3) | ((b & 0xf8) >> 3); break;
+                            case GBR: pixel = ((g & 0xf8) << 8) | ((b & 0xfc) << 3) | ((r & 0xf8) >> 3); break;
+                            case BRG: pixel = ((b & 0xf8) << 8) | ((r & 0xfc) << 3) | ((g & 0xf8) >> 3); break;
+                            case BGR: pixel = ((b & 0xf8) << 8) | ((g & 0xfc) << 3) | ((r & 0xf8) >> 3); break;
+                        }
+                        XPutPixel( data_image, x, y, pixel );
+                        break;
+                    }
+                    case 32:
+                    case 24:
+                    {
+                        int pixel = 0;
+                        switch (b_o)
+                        {
+                            case RGB: pixel = (r << 16) | (g << 8) | b; break;
+                            case RBG: pixel = (r << 16) | (b << 8) | g; break;
+                            case BRG: pixel = (b << 16) | (r << 8) | g; break;
+                            case BGR: pixel = (b << 16) | (g << 8) | r; break;
+                            case GRB: pixel = (g << 16) | (r << 8) | b; break;
+                            case GBR: pixel = (g << 16) | (b << 8) | r; break;
+                        }
+                        XPutPixel( data_image, x, y, pixel );
+                    }
+                    default: break;
+                }
+            } // for
+        }  // for
 
 
-        M_BITMAPHANDLERDATA->m_depth = depthRet;
+        // Blit picture
 
 
-        M_BITMAPHANDLERDATA->m_numColors = xpmAttr.npixels;
+        GC gc = XCreateGC( xdisplay, (Pixmap) M_BMPDATA->m_pixmap, 0, NULL );
+        XPutImage( xdisplay, (Pixmap) M_BMPDATA->m_pixmap, gc, data_image, 0, 0, 0, 0, width, height );
+        XDestroyImage( data_image );
+        XFreeGC( xdisplay, gc );
 
 
-        XpmFreeAttributes(&xpmAttr);
+        // Blit mask
+        
+        if (image.HasMask())
+        {
+            GC gc = XCreateGC( xdisplay, (Pixmap) GetMask()->GetBitmap(), 0, NULL );
+            XPutImage( xdisplay, (Pixmap) GetMask()->GetBitmap(), gc, mask_image, 0, 0, 0, 0, width, height );
 
 
-        M_BITMAPHANDLERDATA->m_ok = TRUE;
-        return TRUE;
-    } else
-    {
-        //      XpmDebugError(errorStatus, name);
-        M_BITMAPHANDLERDATA->m_ok = FALSE;
-        return FALSE;
+            XDestroyImage( mask_image );
+            XFreeGC( xdisplay, gc );
+        }
     }
     }
+
+    return TRUE;
+#endif
+    // wxUSE_NANOX
 }
 
 }
 
-bool wxXPMFileHandler::SaveFile( wxBitmap *bitmap, const wxString& name, int WXUNUSED(type),
-                                const wxPalette *WXUNUSED(palette))
+wxImage wxBitmap::ConvertToImage() const
 {
 {
-    if (M_BITMAPHANDLERDATA->m_ok && M_BITMAPHANDLERDATA->m_pixmap)
+    wxImage image;
+
+    wxCHECK_MSG( Ok(), wxNullImage, wxT("invalid bitmap") );
+
+    Display *xdisplay = (Display*) M_BMPDATA->m_display;
+    wxASSERT_MSG( xdisplay, wxT("No display") );
+
+#if wxUSE_NANOX
+    //int bpp = DefaultDepth(xdisplay, xscreen);
+    wxGetImageFromDrawable((Pixmap) GetPixmap(), 0, 0, GetWidth(), GetHeight(), image);
+    return image;
+#else
+    // !wxUSE_NANOX
+    int bpp = wxTheApp->GetVisualInfo(M_BMPDATA->m_display)->m_visualDepth;
+    XImage *x_image = NULL;
+    if (GetPixmap())
     {
     {
-        Display *dpy =  (Display*) M_BITMAPHANDLERDATA->m_display;
-        int errorStatus = XpmWriteFileFromPixmap(dpy, (char*) (const char*) name,
-            (Pixmap) M_BITMAPHANDLERDATA->m_pixmap,
-            (M_BITMAPHANDLERDATA->m_bitmapMask ? (Pixmap) M_BITMAPHANDLERDATA->m_bitmapMask->GetPixmap() : (Pixmap) 0),
-            (XpmAttributes *) NULL);
-        if (errorStatus == XpmSuccess)
-            return TRUE;
-        else
-            return FALSE;
+        x_image = XGetImage( xdisplay, (Pixmap) GetPixmap(),
+            0, 0,
+            GetWidth(), GetHeight(),
+            AllPlanes, ZPixmap );
+    } else
+    if (GetBitmap())
+    {
+        x_image = XGetImage( xdisplay, (Pixmap) GetBitmap(),
+            0, 0,
+            GetWidth(), GetHeight(),
+            AllPlanes, ZPixmap );
+    } else
+    {
+        wxFAIL_MSG( wxT("Ill-formed bitmap") );
     }
     }
-    else
-        return FALSE;
-}
 
 
-class WXDLLEXPORT wxXPMDataHandler: public wxBitmapHandler
-{
-    DECLARE_DYNAMIC_CLASS(wxXPMDataHandler)
-public:
-    inline wxXPMDataHandler()
-    {
-        m_name = "XPM data";
-        m_extension = "xpm";
-        m_type = wxBITMAP_TYPE_XPM_DATA;
-    };
+    wxCHECK_MSG( x_image, wxNullImage, wxT("couldn't create image") );
 
 
-    virtual bool Create(wxBitmap *bitmap, void *data, long flags, int width, int height, int depth = 1);
-};
-IMPLEMENT_DYNAMIC_CLASS(wxXPMDataHandler, wxBitmapHandler)
+    image.Create( GetWidth(), GetHeight() );
+    char unsigned *data = image.GetData();
 
 
-bool wxXPMDataHandler::Create( wxBitmap *bitmap, void *data, long WXUNUSED(flags),
-                              int width, int height, int WXUNUSED(depth))
-{
-    M_BITMAPHANDLERDATA->m_width = width;
-    M_BITMAPHANDLERDATA->m_height = height;
-    M_BITMAPHANDLERDATA->m_depth = 1;
-    M_BITMAPHANDLERDATA->m_freePixmap = TRUE;
+    if (!data)
+    {
+        XDestroyImage( x_image );
+        wxFAIL_MSG( wxT("couldn't create image") );
+        return wxNullImage;
+    }
 
 
-    Display *dpy = (Display*) wxGetDisplay();
-    M_BITMAPHANDLERDATA->m_display = (WXDisplay*) dpy;
+    XImage *x_image_mask = NULL;
+    if (GetMask())
+    {
+        x_image_mask = XGetImage( xdisplay, (Pixmap) GetMask()->GetBitmap(),
+            0, 0,
+            GetWidth(), GetHeight(), 
+            AllPlanes, ZPixmap );
 
 
-    XpmAttributes xpmAttr;
+        image.SetMaskColour( 16, 16, 16 );  // anything unlikely and dividable
+    }
 
 
-    xpmAttr.valuemask = XpmReturnInfos;    /* nothing yet, but get infos back */
+    int red_shift_right = 0;
+    int green_shift_right = 0;
+    int blue_shift_right = 0;
+    int red_shift_left = 0;
+    int green_shift_left = 0;
+    int blue_shift_left = 0;
+    bool use_shift = FALSE;
 
 
-    XpmColorSymbol symbolicColors[4];
-    if (sg_Control && sg_Control->GetMainWidget())
+    if (GetPixmap())
     {
     {
-        symbolicColors[0].name = "foreground";
-        symbolicColors[0].value = NULL;
-        symbolicColors[1].name = "background";
-        symbolicColors[1].value = NULL;
-        XtVaGetValues((Widget) sg_Control->GetMainWidget(),
-            XmNforeground,  &symbolicColors[0].pixel,
-            XmNbackground,  &symbolicColors[1].pixel,NULL);
-        xpmAttr.numsymbols = 2;
-        xpmAttr.colorsymbols = symbolicColors;
-        xpmAttr.valuemask |= XpmColorSymbols;    // add flag
+        wxXVisualInfo* vi = wxTheApp->GetVisualInfo(M_BMPDATA->m_display);
+
+        red_shift_right = vi->m_visualRedShift;
+        red_shift_left = 8 - vi->m_visualRedPrec;
+        green_shift_right = vi->m_visualGreenShift;
+        green_shift_left = 8 - vi->m_visualGreenPrec;
+        blue_shift_right = vi->m_visualBlueShift;
+        blue_shift_left = 8 - vi->m_visualBluePrec;
+        
+        use_shift = (vi->m_visualType == GrayScale) ||
+                    (vi->m_visualType != PseudoColor);
     }
     }
-
-    Pixmap pixmap;
-    Pixmap mask = 0;
-    int ErrorStatus = XpmCreatePixmapFromData(dpy, RootWindow(dpy, DefaultScreen(dpy)),
-        (char**) data, &pixmap, &mask, &xpmAttr);
-    if (ErrorStatus == XpmSuccess)
+    
+    if (GetBitmap())
     {
     {
-        // Set attributes
-        M_BITMAPHANDLERDATA->m_width = xpmAttr.width;
-        M_BITMAPHANDLERDATA->m_height = xpmAttr.height;
+        bpp = 1;
+    }
 
 
-        unsigned int depthRet;
-        int xRet, yRet;
-        unsigned int widthRet, heightRet, borderWidthRet;
-        Window rootWindowRet;
-        XGetGeometry(dpy, pixmap, &rootWindowRet, &xRet, &yRet,
-            &widthRet, &heightRet, &borderWidthRet, &depthRet);
+    XColor *colors = (XColor*)wxTheApp->
+        GetVisualInfo(M_BMPDATA->m_display)->m_visualColormap;
 
 
-            /*
-            if ( xpmAttr.npixels > 2 )
+    int width = GetWidth();
+    int height = GetHeight();
+    long pos = 0;
+    for (int j = 0; j < height; j++)
+    {
+        for (int i = 0; i < width; i++)
+        {
+            unsigned long pixel = XGetPixel( x_image, i, j );
+            if (bpp == 1)
             {
             {
-            M_BITMAPHANDLERDATA->m_depth = 8;    // next time not just a guess :-) ...
-            } else
+                if (pixel == 0)
+                {
+                    data[pos]   = 0;
+                    data[pos+1] = 0;
+                    data[pos+2] = 0;
+                }
+                else
+                {
+                    data[pos]   = 255;
+                    data[pos+1] = 255;
+                    data[pos+2] = 255;
+                }
+            }
+            else if (use_shift)
+            {
+                data[pos] =   (pixel >> red_shift_right)   << red_shift_left;
+                data[pos+1] = (pixel >> green_shift_right) << green_shift_left;
+                data[pos+2] = (pixel >> blue_shift_right)  << blue_shift_left;
+            }
+            else if (colors)
             {
             {
-            M_BITMAPHANDLERDATA->m_depth = 1;    // mono
+                data[pos] =   colors[pixel].red   >> 8;
+                data[pos+1] = colors[pixel].green >> 8;
+                data[pos+2] = colors[pixel].blue  >> 8;
+            }
+            else
+            {
+                wxFAIL_MSG( wxT("Image conversion failed. Unknown visual type.") );
             }
             }
-        */
 
 
-        M_BITMAPHANDLERDATA->m_depth = depthRet;
+            if (x_image_mask)
+            {
+                int mask_pixel = XGetPixel( x_image_mask, i, j );
+                if (mask_pixel == 0)
+                {
+                    data[pos] = 16;
+                    data[pos+1] = 16;
+                    data[pos+2] = 16;
+                }
+            }
 
 
-        M_BITMAPHANDLERDATA->m_numColors = xpmAttr.npixels;
-        XpmFreeAttributes(&xpmAttr);
-        M_BITMAPHANDLERDATA->m_ok = TRUE;
-        M_BITMAPHANDLERDATA->m_pixmap = (WXPixmap) pixmap;
-        if ( mask )
-        {
-            M_BITMAPHANDLERDATA->m_bitmapMask = new wxMask;
-            M_BITMAPHANDLERDATA->m_bitmapMask->SetPixmap((WXPixmap) mask);
+            pos += 3;
         }
     }
         }
     }
-    else
-    {
-        //      XpmDebugError(ErrorStatus, NULL);
-        M_BITMAPHANDLERDATA->m_ok = FALSE;
-    }
-    return M_BITMAPHANDLERDATA->m_ok ;
-}
 
 
-#endif // wxHAVE_LIB_XPM
+    XDestroyImage( x_image );
+    if (x_image_mask) XDestroyImage( x_image_mask );
+    return image;
+#endif
+    // wxUSE_NANOX
+}
 
 
-void wxBitmap::CleanUpHandlers()
+wxBitmap::wxBitmap( const wxBitmap& bmp )
 {
 {
-    wxNode *node = sm_handlers.First();
-    while ( node )
-    {
-        wxBitmapHandler *handler = (wxBitmapHandler *)node->Data();
-        wxNode *next = node->Next();
-        delete handler;
-        delete node;
-        node = next;
-    }
+    Ref( bmp );
 }
 
 }
 
-void wxBitmap::InitStandardHandlers()
+wxBitmap::wxBitmap( const wxString &filename, wxBitmapType type )
 {
 {
-    // Initialize all standard bitmap or derived class handlers here.
-    AddHandler(new wxXBMFileHandler);
-    AddHandler(new wxXBMDataHandler);
-
-    // XPM is considered standard for Motif, although it can be omitted if
-    // libXpm is not installed
-#if wxHAVE_LIB_XPM
-    AddHandler(new wxXPMFileHandler);
-    AddHandler(new wxXPMDataHandler);
-#endif // wxHAVE_LIB_XPM
+    LoadFile( filename, type );
 }
 
 }
 
-WXPixmap wxBitmap::GetLabelPixmap (WXWidget w)
+wxBitmap::wxBitmap( const char bits[], int width, int height, int depth )
 {
 {
-    if (M_BITMAPDATA->m_image == (WXPixmap) 0)
-        return M_BITMAPDATA->m_pixmap;
-
-    Display *dpy = (Display*) M_BITMAPDATA->m_display;
-
-#ifdef FOO
-    /*
-    If we do:
-    if (labelPixmap) return labelPixmap;
-    things can be wrong, because colors can have been changed.
-
-      If we do:
-      if (labelPixmap)
-      XmDestroyPixmap(DefaultScreenOfDisplay(dpy),labelPixmap) ;
-      we got BadDrawable if the pixmap is referenced by multiples widgets
-
-        this is a catch22!!
-
-      So, before doing thing really clean, I just do nothing; if the pixmap is
-      referenced by many widgets, Motif performs caching functions.
-      And if pixmap is referenced with multiples colors, we just have some
-      memory leaks... I hope we can deal with them...
-    */
-    // Must be destroyed, because colours can have been changed!
-    if (M_BITMAPDATA->m_labelPixmap)
-        XmDestroyPixmap (DefaultScreenOfDisplay (dpy), M_BITMAPDATA->m_labelPixmap);
-#endif
-
-    char tmp[128];
-    sprintf (tmp, "Im%x", (unsigned int) M_BITMAPDATA->m_image);
-
-    Pixel fg, bg;
-    Widget widget = (Widget) w;
-
-    while (XmIsGadget ( widget ))
-        widget = XtParent (widget);
-    XtVaGetValues (widget, XmNbackground, &bg, XmNforeground, &fg, NULL);
-
-    M_BITMAPDATA->m_labelPixmap = (WXPixmap) XmGetPixmap (DefaultScreenOfDisplay (dpy), tmp, fg, bg);
+    m_refData = new wxBitmapRefData;
 
 
-    return M_BITMAPDATA->m_labelPixmap;
+    (void) Create((void*) bits, wxBITMAP_TYPE_XBM_DATA, width, height, depth);
 }
 
 }
 
-WXPixmap wxBitmap::GetArmPixmap (WXWidget w)
+wxBitmap::~wxBitmap()
 {
 {
-    if (M_BITMAPDATA->m_image == (WXPixmap) 0)
-        return M_BITMAPDATA->m_pixmap;
-
-    Display *dpy = (Display*) M_BITMAPDATA->m_display;
-#ifdef FOO
-    // See GetLabelPixmap () comment
-
-    // Must be destroyed, because colours can have been changed!
-    if (M_BITMAPDATA->m_armPixmap)
-        XmDestroyPixmap (DefaultScreenOfDisplay (dpy), M_BITMAPDATA->m_armPixmap);
-#endif
-
-    char tmp[128];
-    sprintf (tmp, "Im%x", (unsigned int) M_BITMAPDATA->m_image);
-
-    Pixel fg, bg;
-    Widget widget = (Widget) w;
-
-    XtVaGetValues (widget, XmNarmColor, &bg, NULL);
-    while (XmIsGadget (widget))
-        widget = XtParent (widget);
-    XtVaGetValues (widget, XmNforeground, &fg, NULL);
+}
 
 
-    M_BITMAPDATA->m_armPixmap = (WXPixmap) XmGetPixmap (DefaultScreenOfDisplay (dpy), tmp, fg, bg);
+wxBitmap& wxBitmap::operator = ( const wxBitmap& bmp )
+{
+    if ( m_refData != bmp.m_refData )
+        Ref( bmp );
 
 
-    return M_BITMAPDATA->m_armPixmap;
+    return *this;
 }
 
 }
 
-WXPixmap wxBitmap::GetInsensPixmap (WXWidget w)
+bool wxBitmap::operator == ( const wxBitmap& bmp ) const
 {
 {
-    Display *dpy = (Display*) M_BITMAPDATA->m_display;
-
-    if (M_BITMAPDATA->m_insensPixmap)
-        return M_BITMAPDATA->m_insensPixmap;
+    return m_refData == bmp.m_refData;
+}
 
 
-    if (!w)
-    {
-        M_BITMAPDATA->m_insensPixmap = (WXPixmap) XCreateInsensitivePixmap(dpy, (Pixmap) M_BITMAPDATA->m_pixmap);
-        if (M_BITMAPDATA->m_insensPixmap)
-            return M_BITMAPDATA->m_insensPixmap;
-        else
-            return M_BITMAPDATA->m_pixmap;
-    }
+bool wxBitmap::operator != ( const wxBitmap& bmp ) const
+{
+    return m_refData != bmp.m_refData;
+}
 
 
-    if (M_BITMAPDATA->m_insensImage == (WXPixmap) 0)
-        return M_BITMAPDATA->m_pixmap;
+bool wxBitmap::Ok() const
+{
+    return (m_refData != NULL);
+}
 
 
-#ifdef FOO
-    See GetLabelPixmap () comment
-        // Must be destroyed, because colours can have been changed!
-        if (M_BITMAPDATA->m_insensPixmap)
-            XmDestroyPixmap (DefaultScreenOfDisplay (dpy), (Pixmap) M_BITMAPDATA->m_insensPixmap);
-#endif
+int wxBitmap::GetHeight() const
+{
+    wxCHECK_MSG( Ok(), -1, wxT("invalid bitmap") );
 
 
-    char tmp[128];
-    sprintf (tmp, "Not%x", (unsigned int) M_BITMAPDATA->m_insensImage);
+    return M_BMPDATA->m_height;
+}
 
 
-    Pixel fg, bg;
-    Widget widget = (Widget) w;
+int wxBitmap::GetWidth() const
+{
+    wxCHECK_MSG( Ok(), -1, wxT("invalid bitmap") );
 
 
-    while (XmIsGadget (widget))
-        widget = XtParent (widget);
-    XtVaGetValues (widget, XmNbackground, &bg, XmNforeground, &fg, NULL);
+    return M_BMPDATA->m_width;
+}
 
 
-    M_BITMAPDATA->m_insensPixmap = (WXPixmap) XmGetPixmap (DefaultScreenOfDisplay (dpy), tmp, fg, bg);
+int wxBitmap::GetDepth() const
+{
+    wxCHECK_MSG( Ok(), -1, wxT("invalid bitmap") );
 
 
-    return M_BITMAPDATA->m_insensPixmap;
+    return M_BMPDATA->m_bpp;
 }
 
 }
 
-// We may need this sometime...
-
-/****************************************************************************
+wxMask *wxBitmap::GetMask() const
+{
+    wxCHECK_MSG( Ok(), (wxMask *) NULL, wxT("invalid bitmap") );
 
 
-  NAME
-  XCreateInsensitivePixmap - create a grayed-out copy of a pixmap
+    return M_BMPDATA->m_mask;
+}
 
 
-  SYNOPSIS
-  Pixmap XCreateInsensitivePixmap( Display *display, Pixmap pixmap )
+void wxBitmap::SetMask( wxMask *mask )
+{
+    wxCHECK_RET( Ok(), wxT("invalid bitmap") );
 
 
-  DESCRIPTION
-  This function creates a grayed-out copy of the argument pixmap, suitable
-  for use as a XmLabel's XmNlabelInsensitivePixmap resource.
+    if (M_BMPDATA->m_mask) delete M_BMPDATA->m_mask;
 
 
-  RETURN VALUES
-  The return value is the new Pixmap id or zero on error.  Errors include
-  a NULL display argument or an invalid Pixmap argument.
+    M_BMPDATA->m_mask = mask;
+}
 
 
-  ERRORS
-  If one of the XLib functions fail, it will produce a X error.  The
-  default X error handler prints a diagnostic and calls exit().
+bool wxBitmap::CopyFromIcon(const wxIcon& icon)
+{
+    *this = icon;
+    return TRUE;
+}
 
 
-  SEE ALSO
-  XCopyArea(3), XCreateBitmapFromData(3), XCreateGC(3), XCreatePixmap(3),
-  XFillRectangle(3), exit(2)
+wxBitmap wxBitmap::GetSubBitmap( const wxRect& rect) const
+{
+    wxCHECK_MSG( Ok() &&
+                 (rect.x >= 0) && (rect.y >= 0) &&
+                 (rect.x+rect.width  <= M_BMPDATA->m_width ) &&
+                 (rect.y+rect.height <= M_BMPDATA->m_height),
+                 wxNullBitmap, wxT("invalid bitmap or bitmap region") );
 
 
-  AUTHOR
-  John R Veregge - john@puente.jpl.nasa.gov
-  Advanced Engineering and Prototyping Group (AEG)
-  Information Systems Technology Section (395)
-  Jet Propulsion Lab - Calif Institute of Technology
+    wxBitmap ret( rect.width, rect.height, M_BMPDATA->m_bpp );
+    wxASSERT_MSG( ret.Ok(), wxT("GetSubBitmap error") );
 
 
-*****************************************************************************/
+    if( GetMask() )
+    {
+        wxMask* mask = new wxMask();
+        mask->SetDisplay( GetMask()->GetDisplay() );
+        mask->SetBitmap( wxGetSubPixmap( GetMask()->GetDisplay(),
+                                         GetMask()->GetBitmap(),
+                                         rect.x, rect.y,
+                                         rect.width, rect.height,
+                                         1 ) );
+
+        ret.SetMask( mask );
+    }
 
 
-Pixmap
-XCreateInsensitivePixmap( Display *display, Pixmap pixmap )
+    if( GetPixmap() )
+    {
+        ret.SetPixmap( wxGetSubPixmap( GetDisplay(),
+                                       GetPixmap(),
+                                       rect.x, rect.y,
+                                       rect.width, rect.height,
+                                       M_BMPDATA->m_bpp ) );
+    }
 
 
-{
-    static char stipple_data[] =
-        {
-            0x55, 0x55, 0xAA, 0xAA, 0x55, 0x55, 0xAA, 0xAA,
-            0x55, 0x55, 0xAA, 0xAA, 0x55, 0x55, 0xAA, 0xAA,
-            0x55, 0x55, 0xAA, 0xAA, 0x55, 0x55, 0xAA, 0xAA,
-            0x55, 0x55, 0xAA, 0xAA, 0x55, 0x55, 0xAA, 0xAA
-        };
-    GC        gc;
-    Pixmap    ipixmap, stipple;
-    unsigned    width, height, depth;
-
-    Window    window;    /* These return values */
-    unsigned    border;    /* from XGetGeometry() */
-    int        x, y;    /* are not needed.     */
-
-    ipixmap = 0;
-
-    if ( NULL == display || 0 == pixmap )
-        return ipixmap;
-
-    if ( 0 == XGetGeometry( display, pixmap, &window, &x, &y,
-                &width, &height, &border, &depth )
-       )
-        return ipixmap; /* BadDrawable: probably an invalid pixmap */
-
-    /* Get the stipple pixmap to be used to 'gray-out' the argument pixmap.
-     */
-    stipple = XCreateBitmapFromData( display, pixmap, stipple_data, 16, 16 );
-    if ( 0 != stipple )
+    if( GetBitmap() )
     {
     {
-        gc = XCreateGC( display, pixmap, (XtGCMask)0, (XGCValues*)NULL );
-        if ( NULL != gc )
-        {
-            /* Create an identical copy of the argument pixmap.
-             */
-            ipixmap = XCreatePixmap( display, pixmap, width, height, depth );
-            if ( 0 != ipixmap )
-            {
-                /* Copy the argument pixmap into the new pixmap.
-                 */
-                XCopyArea( display, pixmap, ipixmap,
-                        gc, 0, 0, width, height, 0, 0 );
-
-                /* Refill the new pixmap using the stipple algorithm/pixmap.
-                 */
-                XSetStipple( display, gc, stipple );
-                XSetFillStyle( display, gc, FillStippled );
-                XFillRectangle( display, ipixmap, gc, 0, 0, width, height );
-            }
-            XFreeGC( display, gc );
-        }
-        XFreePixmap( display, stipple );
+        ret.SetBitmap( wxGetSubPixmap( GetDisplay(),
+                                       GetBitmap(),
+                                       rect.x, rect.y,
+                                       rect.width, rect.height,
+                                       1 ) );
     }
     }
-    return ipixmap;
-}
 
 
-// Creates a bitmap with transparent areas drawn in
-// the given colour.
-wxBitmap wxCreateMaskedBitmap(const wxBitmap& bitmap, wxColour& colour)
-{
-    wxBitmap newBitmap(bitmap.GetWidth(),
-        bitmap.GetHeight(),
-        bitmap.GetDepth());
-    wxMemoryDC destDC;
-    wxMemoryDC srcDC;
-    srcDC.SelectObject(bitmap);
-    destDC.SelectObject(newBitmap);
-
-    wxBrush brush(colour, wxSOLID);
-    destDC.SetOptimization(FALSE);
-    destDC.SetBackground(brush);
-    destDC.Clear();
-    destDC.Blit(0, 0, bitmap.GetWidth(), bitmap.GetHeight(), & srcDC, 0, 0, wxCOPY, TRUE);
-
-    return newBitmap;
+    return ret;
 }
 
 }
 
+bool wxBitmap::SaveFile( const wxString &name, wxBitmapType type,
+                         const wxPalette *palette ) const
+{
+    wxCHECK_MSG( Ok(), FALSE, wxT("invalid bitmap") );
 
 
+    wxBitmapHandler *handler = FindHandler(type);
 
 
+    // Try to save the bitmap via wxImage handlers:
+    if (handler == NULL)
+    { 
+        wxImage image(this->ConvertToImage());
+        if (image.Ok()) return image.SaveFile( name, type );
 
 
-//-----------------------------------------------------------------------------
-// wxImage conversion routines
-//-----------------------------------------------------------------------------
+        return FALSE;
+    }
 
 
-/*
+    return handler->SaveFile(this, name, type, palette);
+}
 
 
-Date: Wed, 05 Jan 2000 11:45:40 +0100
-From: Frits Boel <boel@niob.knaw.nl>
-To: julian.smart@ukonline.co.uk
-Subject: Patch for Motif ConvertToBitmap
+bool wxBitmap::LoadFile( const wxString &name, wxBitmapType type )
+{
+    UnRef();
 
 
-Hi Julian,
+    if (!wxFileExists(name)) return FALSE;
 
 
-I've been working on a wxWin application for image processing. From the
-beginning, I was surprised by the (lack of) speed of ConvertToBitmap,
-till I looked in the source code of image.cpp. I saw that converting a
-wxImage to a bitmap with 8-bit pixels is done with comparing every pixel
-to the 256 colors of the palet. A very time-consuming piece of code!
+    wxBitmapHandler *handler = FindHandler(type);
 
 
-Because I wanted a faster application, I've made a 'patch' for this. In
-short: every pixel of the image is compared to a sorted list with
-colors. If the color is found in the list, the palette entry is
-returned; if the color is not found, the color palette is searched and
-then the palette entry is returned and the color added to the sorted
-list.
+    if (handler == NULL)
+    {
+        wxImage image;
+        if (!image.LoadFile( name, type ))
+            return FALSE;
 
 
-Maybe there is another method for this, namely changing the palette
-itself (if the colors are known, as is the case with tiffs with a
-colormap). I did not look at this, maybe someone else did?
+        if (image.Ok()) 
+        {
+            *this = wxBitmap(image);
+            return TRUE;
+        }
+        else return FALSE;
+    }
 
 
-The code of the patch is attached, have a look on it, and maybe you will
-ship it with the next release of wxMotif?
+    return handler->LoadFile(this, name, type, -1, -1);
+}
 
 
-Regards,
+void wxBitmap::SetPalette(const wxPalette& palette)
+{
+    wxCHECK_RET(Ok(), wxT("invalid bitmap"));
+    wxCHECK_RET(GetDepth() > 1 && GetDepth() <= 8,
+                wxT("cannot set palette for bitmap of this depth"));
 
 
-Frits Boel
-Software engineer at Hubrecht Laboratory, The Netherlands.
+    delete M_BMPDATA->m_palette;
+    M_BMPDATA->m_palette = NULL;
 
 
-*/
+    if (!palette.Ok()) return;
+   
+    M_BMPDATA->m_palette = new wxPalette(palette);
+}
 
 
-class wxSearchColor
+wxPalette *wxBitmap::GetPalette() const
 {
 {
-public:
-  wxSearchColor( void );
-  wxSearchColor( int size, XColor *colors );
-  ~wxSearchColor( void );
+    if (!Ok()) return (wxPalette *) NULL;
 
 
-  int SearchColor( int r, int g, int b );
-private:
-  int AddColor( unsigned int value, int pos );
+    return M_BMPDATA->m_palette;
+}
 
 
-  int          size;
-  XColor       *colors;
-  unsigned int *color;
-  int          *entry;
+void wxBitmap::SetHeight( int height )
+{
+    if (!m_refData) m_refData = new wxBitmapRefData();
 
 
-  int bottom;
-  int top;
-};
+    M_BMPDATA->m_height = height;
+}
 
 
-wxSearchColor::wxSearchColor( void )
+void wxBitmap::SetWidth( int width )
 {
 {
-  size   = 0;
-  colors = (XColor*) NULL;
-  color  = (unsigned int *) NULL;
-  entry  = (int*) NULL;
+    if (!m_refData) m_refData = new wxBitmapRefData();
 
 
-  bottom = 0;
-  top    = 0;
+    M_BMPDATA->m_width = width;
 }
 
 }
 
-wxSearchColor::wxSearchColor( int size_, XColor *colors_ )
+void wxBitmap::SetDepth( int depth )
 {
 {
-    int i;
-    size   = size_;
-    colors = colors_;
-    color  = new unsigned int[size];
-    entry  = new int         [size];
-
-    for (i = 0; i < size; i++ ) {
-        entry[i] = -1;
-    }
+    if (!m_refData) m_refData = new wxBitmapRefData();
 
 
-    bottom = top = ( size >> 1 );
+    M_BMPDATA->m_bpp = depth;
 }
 
 }
 
-wxSearchColor::~wxSearchColor( void )
+void wxBitmap::SetPixmap( WXPixmap pixmap )
 {
 {
-  if ( color ) delete color;
-  if ( entry ) delete entry;
+    if (!m_refData) m_refData = new wxBitmapRefData();
+
+    M_BMPDATA->m_pixmap = pixmap;
 }
 
 }
 
-int wxSearchColor::SearchColor( int r, int g, int b )
+void wxBitmap::SetBitmap( WXPixmap bitmap )
 {
 {
-  unsigned int value = ( ( ( r * 256 ) + g ) * 256 ) + b;
-  int          begin = bottom;
-  int          end   = top;
-  int          middle = 0;
+    if (!m_refData) m_refData = new wxBitmapRefData();
 
 
-  while ( begin <= end ) {
+    M_BMPDATA->m_bitmap = bitmap;
+}
 
 
-    middle = ( begin + end ) >> 1;
+WXPixmap wxBitmap::GetPixmap() const
+{
+    wxCHECK_MSG( Ok(), (WXPixmap) NULL, wxT("invalid bitmap") );
 
 
-    if ( value == color[middle] ) {
-      return( entry[middle] );
-    } else if ( value < color[middle] ) {
-      end = middle - 1;
-    } else {
-      begin = middle + 1;
-    }
+    return M_BMPDATA->m_pixmap;
+}
 
 
-  }
+WXPixmap wxBitmap::GetBitmap() const
+{
+    wxCHECK_MSG( Ok(), (WXPixmap) NULL, wxT("invalid bitmap") );
 
 
-  return AddColor( value, middle );
+    return M_BMPDATA->m_bitmap;
 }
 
 }
 
-int wxSearchColor::AddColor( unsigned int value, int pos )
+WXPixmap wxBitmap::GetDrawable() const
 {
 {
-  int i;
-  int pixel = -1;
-  int max = 3 * (65536);
-  for ( i = 0; i < 256; i++ ) {
-    int rdiff = ((value >> 8) & 0xFF00 ) - colors[i].red;
-    int gdiff = ((value     ) & 0xFF00 ) - colors[i].green;
-    int bdiff = ((value << 8) & 0xFF00 ) - colors[i].blue;
-    int sum = abs (rdiff) + abs (gdiff) + abs (bdiff);
-    if (sum < max) { pixel = i; max = sum; }
-  }
-
-  if ( entry[pos] < 0 ) {
-    color[pos] = value;
-    entry[pos] = pixel;
-  } else if ( value < color[pos] ) {
-
-    if ( bottom > 0 ) {
-      for ( i = bottom; i < pos; i++ ) {
-        color[i-1] = color[i];
-        entry[i-1] = entry[i];
-      }
-      bottom--;
-      color[pos-1] = value;
-      entry[pos-1] = pixel;
-    } else if ( top < size-1 ) {
-      for ( i = top; i >= pos; i-- ) {
-        color[i+1] = color[i];
-        entry[i+1] = entry[i];
-      }
-      top++;
-      color[pos] = value;
-      entry[pos] = pixel;
-    }
+    wxCHECK_MSG( Ok(), (WXPixmap) NULL, wxT("invalid bitmap") );
 
 
-  } else {
-
-    if ( top < size-1 ) {
-      for ( i = top; i > pos; i-- ) {
-        color[i+1] = color[i];
-        entry[i+1] = entry[i];
-      }
-      top++;
-      color[pos+1] = value;
-      entry[pos+1] = pixel;
-    } else if ( bottom > 0 ) {
-      for ( i = bottom; i < pos; i++ ) {
-        color[i-1] = color[i];
-        entry[i-1] = entry[i];
-      }
-      bottom--;
-      color[pos] = value;
-      entry[pos] = pixel;
-    }
+    return M_BMPDATA->m_bpp == 1 ? M_BMPDATA->m_bitmap : M_BMPDATA->m_pixmap;
+}
 
 
-  }
+WXDisplay *wxBitmap::GetDisplay() const
+{
+    wxCHECK_MSG( Ok(), (WXDisplay*) NULL, wxT("invalid bitmap") );
 
 
-  return( pixel );
+    return M_BMPDATA->m_display;
 }
 
 }
 
-
-bool wxBitmap::CreateFromImage( const wxImage& image, int depth )
+#if wxUSE_NANOX
+// Copy from the drawable to the wxImage
+bool wxGetImageFromDrawable(GR_DRAW_ID drawable, int srcX, int srcY, int width, int height, wxImage& image)
 {
 {
-    wxCHECK_MSG( image.Ok(), FALSE, wxT("invalid image") )
-    wxCHECK_MSG( depth == -1, FALSE, wxT("invalid bitmap depth") )
+    GR_SCREEN_INFO sinfo;
+    int x, y;
+    GR_PIXELVAL *pixels;
+    GR_PALETTE* palette = NULL;
+    unsigned char rgb[3], *pp;
+
+    GrGetScreenInfo(&sinfo);
+
+    if (sinfo.pixtype == MWPF_PALETTE) {
+               if(!(palette = (GR_PALETTE*) malloc(sizeof(GR_PALETTE)))) {
+                       return FALSE;
+               }
+               GrGetSystemPalette(palette);
+       }
+
+    if(!(pixels = (GR_PIXELVAL*) malloc(sizeof(GR_PIXELVAL) * width * height)))
+    {
+        return FALSE;
+    }
 
 
-    m_refData = new wxBitmapRefData();
+    image.Create(width, height);
+
+       GrReadArea(drawable, srcX, srcY, width, height,
+                                               pixels);
+
+
+               for(x = 0; x < sinfo.cols; x++) {
+
+                       pp = (unsigned char *)pixels +
+                               ((x + (y * sinfo.cols)) *
+                               sizeof(GR_PIXELVAL));
+
+                       switch(sinfo.pixtype) {
+                       /* FIXME: These may need modifying on big endian. */
+                               case MWPF_TRUECOLOR0888:
+                               case MWPF_TRUECOLOR888:
+                                       rgb[0] = pp[2];
+                                       rgb[1] = pp[1];
+                                       rgb[2] = pp[0];
+                                       break;
+                               case MWPF_PALETTE:
+                                       rgb[0] = palette->palette[pp[0]].r;
+                                       rgb[1] = palette->palette[pp[0]].g;
+                                       rgb[2] = palette->palette[pp[0]].b;
+                                       break;
+                               case MWPF_TRUECOLOR565:
+                                       rgb[0] = pp[1] & 0xf8;
+                                       rgb[1] = ((pp[1] & 0x07) << 5) |
+                                               ((pp[0] & 0xe0) >> 3);
+                                       rgb[2] = (pp[0] & 0x1f) << 3;
+                                       break;
+                               case MWPF_TRUECOLOR555:
+                                       rgb[0] = (pp[1] & 0x7c) << 1;
+                                       rgb[1] = ((pp[1] & 0x03) << 6) |
+                                               ((pp[0] & 0xe0) >> 2);
+                                       rgb[2] = (pp[0] & 0x1f) << 3;
+                                       break;
+                               case MWPF_TRUECOLOR332:
+                                       rgb[0] = pp[0] & 0xe0;
+                                       rgb[1] = (pp[0] & 0x1c) << 3;
+                                       rgb[2] = (pp[0] & 0x03) << 6;
+                                       break;
+                               default:
+                                       fprintf(stderr, "Unsupported pixel "
+                                                       "format\n");
+                                       return 1;
+                       }
+
+            image.SetRGB(x, y, rgb[0], rgb[1], rgb[2]);
+
+       }
+
+    free(pixels);
+    if(palette) free(palette);
 
 
-    int width = image.GetWidth();
-    int height = image.GetHeight();
+    return TRUE;
+}
 
 
-    SetHeight( height );
-    SetWidth( width );
+#if 0
+int GrGetPixelColor(GR_SCREEN_INFO* sinfo, GR_PALETTE* palette, GR_PIXELVAL pixel,
+    unsigned char* red, unsigned char* green, unsigned char* blue)
+{
+       unsigned char rgb[3], *pp;
 
 
-    Display *dpy = (Display*) wxGetDisplay();
-    Visual* vis = DefaultVisual( dpy, DefaultScreen( dpy ) );
-    int bpp = DefaultDepth( dpy, DefaultScreen( dpy ) );
+    pp = (unsigned char*) & pixel ;
 
 
-    // Create image
+    switch (sinfo.pixtype)
+    {
+                       /* FIXME: These may need modifying on big endian. */
+                               case MWPF_TRUECOLOR0888:
+                               case MWPF_TRUECOLOR888:
+                                       rgb[0] = pp[2];
+                                       rgb[1] = pp[1];
+                                       rgb[2] = pp[0];
+                                       break;
+                               case MWPF_PALETTE:
+                                       rgb[0] = palette->palette[pp[0]].r;
+                                       rgb[1] = palette->palette[pp[0]].g;
+                                       rgb[2] = palette->palette[pp[0]].b;
+                                       break;
+                               case MWPF_TRUECOLOR565:
+                                       rgb[0] = pp[1] & 0xf8;
+                                       rgb[1] = ((pp[1] & 0x07) << 5) |
+                                               ((pp[0] & 0xe0) >> 3);
+                                       rgb[2] = (pp[0] & 0x1f) << 3;
+                                       break;
+                               case MWPF_TRUECOLOR555:
+                                       rgb[0] = (pp[1] & 0x7c) << 1;
+                                       rgb[1] = ((pp[1] & 0x03) << 6) |
+                                               ((pp[0] & 0xe0) >> 2);
+                                       rgb[2] = (pp[0] & 0x1f) << 3;
+                                       break;
+                               case MWPF_TRUECOLOR332:
+                                       rgb[0] = pp[0] & 0xe0;
+                                       rgb[1] = (pp[0] & 0x1c) << 3;
+                                       rgb[2] = (pp[0] & 0x03) << 6;
+                                       break;
+                               default:
+                                       fprintf(stderr, "Unsupported pixel format\n");
+                                       return 0;
+       }
+
+
+    *(red) = rgb[0];
+    *(green) = rgb[1];
+    *(blue) = rgb[2];
+    return 1;
+}
+#endif
 
 
-    XImage *data_image = XCreateImage( dpy, vis, bpp, ZPixmap, 0, 0, width, height, 32, 0 );
-    data_image->data = (char*) malloc( data_image->bytes_per_line * data_image->height );
+#endif
+  // wxUSE_NANOX
 
 
-    Create( width, height, bpp );
+// ============================================================================
+// Bitmap handlers
+// ============================================================================
 
 
-    // Create mask
+IMPLEMENT_ABSTRACT_CLASS(wxBitmapHandler, wxBitmapHandlerBase);
 
 
-    XImage *mask_image = (XImage*) NULL;
-    if (image.HasMask())
-    {
-        mask_image = XCreateImage( dpy, vis, 1, ZPixmap, 0, 0, width, height, 32, 0 );
-        mask_image->data = (char*) malloc( mask_image->bytes_per_line * mask_image->height );
-    }
+#define M_BMPHANDLERDATA ((wxBitmapRefData *)bitmap->GetRefData())
 
 
-    // Retrieve depth info
+#if wxUSE_XPM
 
 
-    XVisualInfo vinfo_template;
-    XVisualInfo *vi;
+#if wxHAVE_LIB_XPM || wxUSE_STREAMS
 
 
-    vinfo_template.visual = vis;
-    vinfo_template.visualid = XVisualIDFromVisual( vis );
-    vinfo_template.depth = bpp;
-    int nitem = 0;
+// ----------------------------------------------------------------------------
+// wxXPMFileHandler
+// ----------------------------------------------------------------------------
 
 
-    vi = XGetVisualInfo( dpy, VisualIDMask|VisualDepthMask, &vinfo_template, &nitem );
+class wxXPMFileHandler : public wxBitmapHandler
+{
+    DECLARE_DYNAMIC_CLASS(wxXPMFileHandler);
+public:
+    wxXPMFileHandler()
+    {
+        m_name = "XPM file";
+        m_extension = "xpm";
+        m_type = wxBITMAP_TYPE_XPM;
+    };
 
 
-    wxCHECK_MSG( vi, FALSE, wxT("no visual") );
+    virtual bool LoadFile(wxBitmap *bitmap, const wxString& name, long flags,
+                          int desiredWidth, int desiredHeight);
 
 
-    XFree( vi );
+    virtual bool SaveFile(const wxBitmap *bitmap, const wxString& name,
+                          int type, const wxPalette *palette = NULL);
 
 
-    if ((bpp == 16) && (vi->red_mask != 0xf800)) bpp = 15;
-    if (bpp < 8) bpp = 8;
+    virtual bool Create(wxBitmap *bitmap, void *data, long flags,
+                        int width, int height, int depth = 1)
+        { return FALSE; }
+};
 
 
-    // Render
+IMPLEMENT_DYNAMIC_CLASS(wxXPMFileHandler, wxBitmapHandler);
 
 
-    enum byte_order { RGB, RBG, BRG, BGR, GRB, GBR };
-    byte_order b_o = RGB;
+bool wxXPMFileHandler::LoadFile(wxBitmap *bitmap, const wxString& name,
+                                long flags, int desiredWidth,
+                                int desiredHeight)
+{
+#if wxHAVE_LIB_XPM
+    if (!bitmap->GetRefData())
+        bitmap->SetRefData( new wxBitmapRefData() );
 
 
-    if (bpp >= 24)
-    {
-        if ((vi->red_mask > vi->green_mask) && (vi->green_mask > vi->blue_mask))      b_o = RGB;
-        else if ((vi->red_mask > vi->blue_mask) && (vi->blue_mask > vi->green_mask))  b_o = RGB;
-        else if ((vi->blue_mask > vi->red_mask) && (vi->red_mask > vi->green_mask))   b_o = BRG;
-        else if ((vi->blue_mask > vi->green_mask) && (vi->green_mask > vi->red_mask)) b_o = BGR;
-        else if ((vi->green_mask > vi->red_mask) && (vi->red_mask > vi->blue_mask))   b_o = GRB;
-        else if ((vi->green_mask > vi->blue_mask) && (vi->blue_mask > vi->red_mask))  b_o = GBR;
-    }
+    M_BMPHANDLERDATA->m_display = wxGlobalDisplay();
+    
+    Display *xdisplay = (Display*) M_BMPHANDLERDATA->m_display;
+    
+    int xscreen = DefaultScreen( xdisplay );
+    Window xroot = RootWindow( xdisplay, xscreen );
+    
+    int bpp = DefaultDepth( xdisplay, xscreen );
 
 
-    int r_mask = image.GetMaskRed();
-    int g_mask = image.GetMaskGreen();
-    int b_mask = image.GetMaskBlue();
+    XpmAttributes xpmAttr;
+    xpmAttr.valuemask = XpmReturnInfos;    // nothing yet, but get infos back
 
 
-    XColor colors[256];
-    if (bpp == 8)
+    Pixmap pixmap;
+    Pixmap mask = 0;
+    
+    int ErrorStatus = XpmReadFileToPixmap( xdisplay, xroot,
+                                           (char*) name.c_str(),
+                                           &pixmap, &mask, &xpmAttr);
+    
+    if (ErrorStatus == XpmSuccess)
     {
     {
-        Colormap cmap = (Colormap) wxTheApp->GetMainColormap( dpy );
-
-        for (int i = 0; i < 256; i++) colors[i].pixel = i;
-        XQueryColors( dpy, cmap, colors, 256 );
-    }
+        M_BMPHANDLERDATA->m_width = xpmAttr.width;
+        M_BMPHANDLERDATA->m_height = xpmAttr.height;
 
 
-    wxSearchColor scolor( 256, colors );
-    unsigned char* data = image.GetData();
+        M_BMPHANDLERDATA->m_bpp = bpp;  // mono as well?
 
 
-    bool hasMask = image.HasMask();
+        XpmFreeAttributes(&xpmAttr);
 
 
-    int index = 0;
-    for (int y = 0; y < height; y++)
-    {
-        for (int x = 0; x < width; x++)
+        M_BMPHANDLERDATA->m_bitmap = (WXPixmap) pixmap;
+        
+        if (mask)
         {
         {
-            int r = data[index];
-            index++;
-            int g = data[index];
-            index++;
-            int b = data[index];
-            index++;
-
-            if (hasMask)
-            {
-              if ((r == r_mask) && (b == b_mask) && (g == g_mask))
-                XPutPixel( mask_image, x, y, 0 );
-              else
-                XPutPixel( mask_image, x, y, 1 );
-            }
+            M_BMPHANDLERDATA->m_mask = new wxMask;
+            M_BMPHANDLERDATA->m_mask->SetBitmap( (WXPixmap) mask );
+            M_BMPHANDLERDATA->m_mask->SetDisplay( xdisplay );
+        }
+    }
+    else
+    {
+        UnRef();
+    
+        return FALSE;
+    }
 
 
-            switch (bpp)
-            {
-            case 8:
-                {
-#if 0 // Old, slower code
-                    int pixel = -1;
-                    /*
-                    if (wxTheApp->m_colorCube)
-                    {
-                    pixel = wxTheApp->m_colorCube
-                    [ ((r & 0xf8) << 7) + ((g & 0xf8) << 2) + ((b & 0xf8) >> 3) ];
-                    }
-                    else
-                    {
-                    */
-                    int max = 3 * (65536);
-                    for (int i = 0; i < 256; i++)
-                    {
-                        int rdiff = (r << 8) - colors[i].red;
-                        int gdiff = (g << 8) - colors[i].green;
-                        int bdiff = (b << 8) - colors[i].blue;
-                        int sum = abs (rdiff) + abs (gdiff) + abs (bdiff);
-                        if (sum < max) { pixel = i; max = sum; }
-                    }
-                    /*
-                    }
-                    */
-#endif
+    return TRUE;
+#elif wxUSE_STREAMS
+    wxXPMDecoder decoder;
+    wxFileInputStream stream(name);
+    if (stream.Ok())
+    {
+        wxImage image(decoder.ReadFile(stream));
+        if (image.Ok())
+            return CreateFromImage(image);
+        else
+            return FALSE;
+    }
+    else
+        return FALSE;
+#endif // wxHAVE_LIB_XPM / wxUSE_STREAMS
+}
 
 
-                    // And this is all to get the 'right' color...
-                    int pixel = scolor.SearchColor( r, g, b );
-                    XPutPixel( data_image, x, y, pixel );
-                    break;
-                }
-            case 15:
-                {
-                    int pixel = ((r & 0xf8) << 7) | ((g & 0xf8) << 2) | ((b & 0xf8) >> 3);
-                    XPutPixel( data_image, x, y, pixel );
-                    break;
-                }
-            case 16:
-                {
-                    int pixel = ((r & 0xf8) << 8) | ((g & 0xfc) << 3) | ((b & 0xf8) >> 3);
-                    XPutPixel( data_image, x, y, pixel );
-                    break;
-                }
-            case 32:
-            case 24:
-                {
-                    int pixel = 0;
-                    switch (b_o)
-                    {
-                    case RGB: pixel = (r << 16) | (g << 8) | b; break;
-                    case RBG: pixel = (r << 16) | (b << 8) | g; break;
-                    case BRG: pixel = (b << 16) | (r << 8) | g; break;
-                    case BGR: pixel = (b << 16) | (g << 8) | r; break;
-                    case GRB: pixel = (g << 16) | (r << 8) | b; break;
-                    case GBR: pixel = (g << 16) | (b << 8) | r; break;
-                    }
-                    XPutPixel( data_image, x, y, pixel );
-                }
-            default: break;
-            }
-        } // for
-    }  // for
+bool wxXPMFileHandler::SaveFile(const wxBitmap *bitmap, const wxString& name,
+                                int type,
+                                const wxPalette *WXUNUSED(palette))
+{
+    wxImage image(bitmap->ConvertToImage());
+    if (image.Ok()) return image.SaveFile( name, (wxBitmapType)type );
 
 
-    // Blit picture
+    return FALSE;
+}
 
 
-    XGCValues gcvalues;
-    gcvalues.foreground = BlackPixel( dpy, DefaultScreen( dpy ) );
-    GC gc = XCreateGC( dpy, RootWindow ( dpy, DefaultScreen(dpy) ), GCForeground, &gcvalues );
-    XPutImage( dpy, (Drawable)GetPixmap(), gc, data_image, 0, 0, 0, 0, width, height );
+#endif // wxHAVE_LIB_XPM || wxUSE_STREAMS
 
 
-    XDestroyImage( data_image );
-    XFreeGC( dpy, gc );
+// ----------------------------------------------------------------------------
+// wxXPMDataHandler
+// ----------------------------------------------------------------------------
 
 
-    // Blit mask
-    if (image.HasMask())
+class wxXPMDataHandler : public wxBitmapHandler
+{
+    DECLARE_DYNAMIC_CLASS(wxXPMDataHandler);
+public:
+    wxXPMDataHandler()
     {
     {
-        wxBitmap maskBitmap(width, height, 1);
-
-        GC gcMask = XCreateGC( dpy, (Pixmap) maskBitmap.GetPixmap(), (XtGCMask) 0, (XGCValues*)NULL );
-        XPutImage( dpy, (Drawable)maskBitmap.GetPixmap(), gcMask, mask_image, 0, 0, 0, 0, width, height );
-
-        XDestroyImage( mask_image );
-        XFreeGC( dpy, gcMask );
-
-        wxMask* mask = new wxMask;
-        mask->SetPixmap(maskBitmap.GetPixmap());
+        m_name = "XPM data";
+        m_extension = "xpm";
+        m_type = wxBITMAP_TYPE_XPM_DATA;
+    };
 
 
-        SetMask(mask);
+    virtual bool LoadFile(wxBitmap *bitmap, const wxString& name, long flags,
+                          int desiredWidth, int desiredHeight)
+        { return FALSE; }
 
 
-        maskBitmap.SetPixmapNull();
-    }
+    virtual bool SaveFile(const wxBitmap *bitmap, const wxString& name,
+                          int type, const wxPalette *palette = NULL)
+        { return FALSE; }
 
 
+    virtual bool Create(wxBitmap *bitmap, void *data, long flags,
+                        int width, int height, int depth = 1);
+};
 
 
-    return TRUE;
-}
+IMPLEMENT_DYNAMIC_CLASS(wxXPMDataHandler, wxBitmapHandler);
 
 
-wxImage wxBitmap::ConvertToImage() const
+bool wxXPMDataHandler::Create(wxBitmap *bitmap, void *bits,
+                              long WXUNUSED(flags),
+                              int width, int height, int WXUNUSED(depth))
 {
 {
-    wxImage image;
-    
-    wxCHECK_MSG( Ok(), wxNullImage, wxT("invalid bitmap") );
-
-    Display *dpy = (Display*) wxGetDisplay();
-    Visual* vis = DefaultVisual( dpy, DefaultScreen( dpy ) );
-    int bpp = DefaultDepth( dpy, DefaultScreen( dpy ) );
+#if wxHAVE_LIB_XPM
+    wxCHECK_MSG( bits != NULL, FALSE, wxT("invalid bitmap data") )
 
 
-    XImage *ximage = XGetImage( dpy,
-        (Drawable)GetPixmap(),
-        0, 0,
-        GetWidth(), GetHeight(),
-        AllPlanes, ZPixmap );
+    if (!bitmap->GetRefData())
+        bitmap->SetRefData( new wxBitmapRefData() );
 
 
-    wxCHECK_MSG( ximage, wxNullImage, wxT("couldn't create image") );
+    M_BMPHANDLERDATA->m_display = wxGlobalDisplay();
+    
+    Display *xdisplay = (Display*) M_BMPHANDLERDATA->m_display;
+    
+    int xscreen = DefaultScreen( xdisplay );
+    Window xroot = RootWindow( xdisplay, xscreen );
+    
+    int bpp = DefaultDepth( xdisplay, xscreen );
 
 
-    image.Create( GetWidth(), GetHeight() );
-    char unsigned *data = image.GetData();
+    XpmAttributes xpmAttr;
+    xpmAttr.valuemask = XpmReturnInfos;    // nothing yet, but get infos back
 
 
-    if (!data)
+    Pixmap pixmap = 0;
+    Pixmap mask = 0;
+    
+    int ErrorStatus = XpmCreatePixmapFromData( xdisplay, xroot, (char**) bits,
+                                               &pixmap, &mask, &xpmAttr );
+    
+    if (ErrorStatus == XpmSuccess)
     {
     {
-        XDestroyImage( ximage );
-        wxFAIL_MSG( wxT("couldn't create image") );
-        return wxNullImage;
-    }
+        M_BMPHANDLERDATA->m_width = xpmAttr.width;
+        M_BMPHANDLERDATA->m_height = xpmAttr.height;
 
 
-    /*
-    GdkImage *gdk_image_mask = (GdkImage*) NULL;
-    if (GetMask())
-    {
-    gdk_image_mask = gdk_image_get( GetMask()->GetBitmap(),
-    0, 0,
-    GetWidth(), GetHeight() );
+        M_BMPHANDLERDATA->m_bpp = bpp;  // mono as well?
 
 
-      image.SetMaskColour( 16, 16, 16 );  // anything unlikely and dividable
-      }
-    */
+#if __WXDEBUG__
+        unsigned int depthRet;
+        int xRet, yRet;
+        unsigned int widthRet, heightRet, borderWidthRet;
+        XGetGeometry( xdisplay, pixmap, &xroot, &xRet, &yRet,
+            &widthRet, &heightRet, &borderWidthRet, &depthRet);
 
 
-    // Retrieve depth info
+        wxASSERT_MSG( bpp == (int)depthRet, wxT("colour depth mismatch") );
+#endif
 
 
-    XVisualInfo vinfo_template;
-    XVisualInfo *vi;
+        XpmFreeAttributes(&xpmAttr);
+        
+        M_BMPHANDLERDATA->m_pixmap = (WXPixmap) pixmap;
+        
+        if (mask)
+        {
+            M_BMPHANDLERDATA->m_mask = new wxMask;
+            M_BMPHANDLERDATA->m_mask->SetBitmap( (WXPixmap) mask );
+            M_BMPHANDLERDATA->m_mask->SetDisplay( xdisplay );
+        }
+       return TRUE;
+    }
+    else
+    {
+        bitmap->UnRef();
+        
+        return FALSE;
+    }
+#else
+    wxXPMDecoder decoder;
+    wxImage image(decoder.ReadData(bits));
+    if (image.Ok())
+       return bitmap->CreateFromImage(image);
+    else
+       return FALSE;
+#endif
+}
 
 
-    vinfo_template.visual = vis;
-    vinfo_template.visualid = XVisualIDFromVisual( vis );
-    vinfo_template.depth = bpp;
-    int nitem = 0;
+#endif // wxUSE_XPM
 
 
-    vi = XGetVisualInfo( dpy, VisualIDMask|VisualDepthMask, &vinfo_template, &nitem );
+// ----------------------------------------------------------------------------
+// wxXBMDataHandler
+// ----------------------------------------------------------------------------
 
 
-    wxCHECK_MSG( vi, wxNullImage, wxT("no visual") );
+class WXDLLEXPORT wxXBMDataHandler: public wxBitmapHandler
+{
+    DECLARE_DYNAMIC_CLASS(wxXBMDataHandler)
+public:
+    inline wxXBMDataHandler()
+    {
+        m_name = "XBM data";
+        m_extension = "xbm";
+        m_type = wxBITMAP_TYPE_XBM_DATA;
+    };
 
 
-    if ((bpp == 16) && (vi->red_mask != 0xf800)) bpp = 15;
+    virtual bool LoadFile(wxBitmap *bitmap, const wxString& name, long flags,
+                          int desiredWidth, int desiredHeight)
+        { return FALSE; }
 
 
-    XFree( vi );
+    virtual bool SaveFile(const wxBitmap *bitmap, const wxString& name,
+                          int type, const wxPalette *palette = NULL)
+        { return FALSE; }
 
 
-    XColor colors[256];
-    if (bpp == 8)
-    {
-        Colormap cmap = (Colormap)wxTheApp->GetMainColormap( dpy );
+    virtual bool Create(wxBitmap *bitmap, void *data, long flags,
+                        int width, int height, int depth = 1);
+};
 
 
-        for (int i = 0; i < 256; i++) colors[i].pixel = i;
-        XQueryColors( dpy, cmap, colors, 256 );
-    }
+IMPLEMENT_DYNAMIC_CLASS(wxXBMDataHandler, wxBitmapHandler);
 
 
-    long pos = 0;
-    for (int j = 0; j < GetHeight(); j++)
-    {
-        for (int i = 0; i < GetWidth(); i++)
-        {
-            int pixel = XGetPixel( ximage, i, j );
-            if (bpp <= 8)
-            {
-                data[pos] = colors[pixel].red >> 8;
-                data[pos+1] = colors[pixel].green >> 8;
-                data[pos+2] = colors[pixel].blue >> 8;
-            } else if (bpp == 15)
-            {
-                data[pos] = (pixel >> 7) & 0xf8;
-                data[pos+1] = (pixel >> 2) & 0xf8;
-                data[pos+2] = (pixel << 3) & 0xf8;
-            } else if (bpp == 16)
-            {
-                data[pos] = (pixel >> 8) & 0xf8;
-                data[pos+1] = (pixel >> 3) & 0xfc;
-                data[pos+2] = (pixel << 3) & 0xf8;
-            } else
-            {
-                data[pos] = (pixel >> 16) & 0xff;
-                data[pos+1] = (pixel >> 8) & 0xff;
-                data[pos+2] = pixel & 0xff;
-            }
+bool wxXBMDataHandler::Create( wxBitmap *bitmap, void *bits,
+                               long WXUNUSED(flags),
+                               int width, int height, int depth)
+{
+#if !wxUSE_NANOX
+    if (!bitmap->GetRefData())
+        bitmap->SetRefData( new wxBitmapRefData() );
 
 
-            /*
-            if (gdk_image_mask)
-            {
-            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;
-            }
-            }
-            */
+    M_BMPHANDLERDATA->m_display = wxGlobalDisplay();
 
 
-            pos += 3;
-        }
-    }
+    Display *xdisplay = (Display*) M_BMPHANDLERDATA->m_display;
+    
+    int xscreen = DefaultScreen( xdisplay );
+    Window xroot = RootWindow( xdisplay, xscreen );
+    
+    M_BMPHANDLERDATA->m_mask = (wxMask *) NULL;
+    M_BMPHANDLERDATA->m_bitmap =
+        (WXPixmap) XCreateBitmapFromData( xdisplay, xroot,
+                                          (char *) bits, width, height );
+    M_BMPHANDLERDATA->m_width = width;
+    M_BMPHANDLERDATA->m_height = height;
+    M_BMPHANDLERDATA->m_bpp = 1;
 
 
-    XDestroyImage( ximage );
-    /*
-    if (gdk_image_mask) gdk_image_destroy( gdk_image_mask );
-    */
+    return TRUE;
+#endif
+    wxCHECK_MSG( M_BMPHANDLERDATA->m_bitmap, FALSE,
+                 wxT("couldn't create bitmap") );
+}
 
 
-    return image;
+void wxBitmap::InitStandardHandlers()
+{
+    AddHandler(new wxXBMDataHandler);
+#if wxUSE_XPM
+#if wxHAVE_LIB_XPM || wxUSE_STREAMS
+    AddHandler(new wxXPMFileHandler);
+#endif
+    AddHandler(new wxXPMDataHandler);
+#endif
 }
 }
wxWidgets (Impactor)