don't refresh the current cell in SetCellHighlightROPenWidth() if it's invalid (...

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

diff --git a/src/dfb/bitmap.cpp b/src/dfb/bitmap.cpp
index c6ebc605955678f8df4edca489ea8f99b5bc5a1b..91684bdad393c0e1f60892cb54c06938ed91fb79 100644 (file)
--- a/src/dfb/bitmap.cpp
+++ b/src/dfb/bitmap.cpp
@@ -23,6 +23,7 @@
 #include "wx/bitmap.h"
 #include "wx/colour.h"
 #include "wx/image.h"
 #include "wx/bitmap.h"
 #include "wx/colour.h"
 #include "wx/image.h"

+#include "wx/rawbmp.h"

 
 #include "wx/dfb/private.h"
 
 
 #include "wx/dfb/private.h"
 


@@ -30,18 +31,26 @@
 // helpers
 //-----------------------------------------------------------------------------
 
 // helpers
 //-----------------------------------------------------------------------------
 

+// NB: Most of this conversion code is needed because of differences between
+//     wxImage and wxDFB's wxBitmap representations:
+//     (1) wxImage uses RGB order, while DirectFB uses BGR
+//     (2) wxImage has alpha channel in a separate plane, while DirectFB puts
+//         all components into single BGRA plane
+

 // pitch = stride = # of bytes between the start of N-th line and (N+1)-th line
 // pitch = stride = # of bytes between the start of N-th line and (N+1)-th line

+// {Src,Dst}PixSize = # of bytes used to represent one pixel
+template<int SrcPixSize, int DstPixSize>

 static void CopyPixelsAndSwapRGB(unsigned w, unsigned h,
                                  const unsigned char *src,
                                  unsigned src_pitch,
                                  unsigned char *dst,
                                  unsigned dst_pitch)
 {
 static void CopyPixelsAndSwapRGB(unsigned w, unsigned h,
                                  const unsigned char *src,
                                  unsigned src_pitch,
                                  unsigned char *dst,
                                  unsigned dst_pitch)
 {

-    unsigned src_advance = src_pitch - 3 * w;
-    unsigned dst_advance = dst_pitch - 3 * w;
+    unsigned src_advance = src_pitch - SrcPixSize * w;
+    unsigned dst_advance = dst_pitch - DstPixSize * w;

     for ( unsigned y = 0; y < h; y++, src += src_advance, dst += dst_advance )
     {
     for ( unsigned y = 0; y < h; y++, src += src_advance, dst += dst_advance )
     {

-        for ( unsigned x = 0; x < w; x++, src += 3, dst += 3 )
+        for ( unsigned x = 0; x < w; x++, src += SrcPixSize, dst += DstPixSize )

         {
             // copy with RGB -> BGR translation:
             dst[0] = src[2];
         {
             // copy with RGB -> BGR translation:
             dst[0] = src[2];


@@ -52,60 +61,192 @@ static void CopyPixelsAndSwapRGB(unsigned w, unsigned h,
 }
 
 static void CopySurfaceToImage(const wxIDirectFBSurfacePtr& surface,
 }
 
 static void CopySurfaceToImage(const wxIDirectFBSurfacePtr& surface,

-                               const wxImage& image)
+                               wxImage& image)

 {
 {

-    wxCHECK_RET( surface->GetPixelFormat() == DSPF_RGB24,
-                 _T("unexpected pixel format") );
-

     wxIDirectFBSurface::Locked locked(surface, DSLF_READ);
     wxIDirectFBSurface::Locked locked(surface, DSLF_READ);

-    wxCHECK_RET( locked.ptr, _T("failed to lock surface") );
+    wxCHECK_RET( locked.ptr, "failed to lock surface" );
+
+    const unsigned width = image.GetWidth();
+    const unsigned height = image.GetHeight();
+    const DFBSurfacePixelFormat format = surface->GetPixelFormat();
+
+    // copy RGB data from the surface:
+    switch ( format )
+    {
+        case DSPF_RGB24:
+            CopyPixelsAndSwapRGB<3,3>
+            (
+                width, height,
+                (unsigned char*)locked.ptr, locked.pitch,
+                image.GetData(), width * 3
+            );
+            break;
+
+        case DSPF_RGB32:
+        case DSPF_ARGB:
+            CopyPixelsAndSwapRGB<4,3>
+            (
+                width, height,
+                (unsigned char*)locked.ptr, locked.pitch,
+                image.GetData(), width * 3
+            );
+            break;
+
+        default:
+            wxFAIL_MSG( "unexpected pixel format" );
+            return;
+    }

 
 

-    CopyPixelsAndSwapRGB(image.GetWidth(), image.GetHeight(),
-                         (unsigned char*)locked.ptr, locked.pitch,
-                         image.GetData(), image.GetWidth() * 3);
+    // extract alpha channel if the bitmap has it:
+    if ( format == DSPF_ARGB )
+    {
+        // create alpha plane:
+        image.SetAlpha();
+
+        // and copy alpha data to it:
+        const unsigned advance = locked.pitch - 4 * width;
+        unsigned char *alpha = image.GetAlpha();
+        // NB: "+3" is to get pointer to alpha component
+        const unsigned char *src = ((unsigned char*)locked.ptr) + 3;
+
+        for ( unsigned y = 0; y < height; y++, src += advance )
+            for ( unsigned x = 0; x < width; x++, src += 4 )
+                *(alpha++) = *src;
+    }

 }
 
 static void CopyImageToSurface(const wxImage& image,
                                const wxIDirectFBSurfacePtr& surface)
 {
 }
 
 static void CopyImageToSurface(const wxImage& image,
                                const wxIDirectFBSurfacePtr& surface)
 {

-    wxCHECK_RET( surface->GetPixelFormat() == DSPF_RGB24,
-                 _T("unexpected pixel format") );
-

     wxIDirectFBSurface::Locked locked(surface, DSLF_WRITE);
     wxIDirectFBSurface::Locked locked(surface, DSLF_WRITE);

-    wxCHECK_RET( locked.ptr, _T("failed to lock surface") );
+    wxCHECK_RET( locked.ptr, "failed to lock surface" );
+
+    const unsigned width = image.GetWidth();
+    const unsigned height = image.GetHeight();
+    const DFBSurfacePixelFormat format = surface->GetPixelFormat();
+
+    // copy RGB data to the surface:
+    switch ( format )
+    {
+        case DSPF_RGB24:
+            CopyPixelsAndSwapRGB<3,3>
+            (
+               width, height,
+               image.GetData(), width * 3,
+               (unsigned char*)locked.ptr, locked.pitch
+            );
+            break;
+
+        case DSPF_RGB32:
+        case DSPF_ARGB:
+            CopyPixelsAndSwapRGB<3,4>
+            (
+               width, height,
+               image.GetData(), width * 3,
+               (unsigned char*)locked.ptr, locked.pitch
+            );
+            break;
+
+        default:
+            wxFAIL_MSG( "unexpected pixel format" );
+            return;
+    }
+
+    // if the image has alpha channel, merge it in:
+    if ( format == DSPF_ARGB )
+    {
+        wxCHECK_RET( image.HasAlpha(), "logic error - ARGB, but no alpha" );
+
+        const unsigned advance = locked.pitch - 4 * width;
+        const unsigned char *alpha = image.GetAlpha();
+        // NB: "+3" is to get pointer to alpha component
+        unsigned char *dest = ((unsigned char*)locked.ptr) + 3;
+
+        for ( unsigned y = 0; y < height; y++, dest += advance )
+            for ( unsigned x = 0; x < width; x++, dest += 4 )
+                *dest = *(alpha++);
+    }
+}
+
+static wxIDirectFBSurfacePtr
+CreateSurfaceWithFormat(int w, int h, DFBSurfacePixelFormat format)
+{
+    DFBSurfaceDescription desc;
+    desc.flags = (DFBSurfaceDescriptionFlags)
+        (DSDESC_CAPS | DSDESC_WIDTH | DSDESC_HEIGHT);
+    desc.caps = DSCAPS_NONE;
+    desc.width = w;
+    desc.height = h;
+
+    if ( format != DSPF_UNKNOWN )
+    {
+        desc.flags = (DFBSurfaceDescriptionFlags)(
+                            desc.flags | DSDESC_PIXELFORMAT);
+        desc.pixelformat = format;
+    }

 
 

-    CopyPixelsAndSwapRGB(image.GetWidth(), image.GetHeight(),
-                         image.GetData(), image.GetWidth() * 3,
-                         (unsigned char*)locked.ptr, locked.pitch);
+    return wxIDirectFB::Get()->CreateSurface(&desc);

 }
 
 // Creates a surface that will use wxImage's pixel data (RGB only)
 static wxIDirectFBSurfacePtr CreateSurfaceForImage(const wxImage& image)
 {
 }
 
 // Creates a surface that will use wxImage's pixel data (RGB only)
 static wxIDirectFBSurfacePtr CreateSurfaceForImage(const wxImage& image)
 {

-    wxCHECK_MSG( image.Ok(), NULL, _T("invalid image") );
+    wxCHECK_MSG( image.Ok(), NULL, "invalid image" );

     // FIXME_DFB: implement alpha handling by merging alpha buffer with RGB
     //            into a temporary RGBA surface
     // FIXME_DFB: implement alpha handling by merging alpha buffer with RGB
     //            into a temporary RGBA surface

-    wxCHECK_MSG( !image.HasAlpha(), NULL, _T("alpha channel not supported") );
+    wxCHECK_MSG( !image.HasAlpha(), NULL, "alpha channel not supported" );

 
     // NB: wxImage uses RGB order of bytes while DirectFB uses BGR, so we
     //     cannot use preallocated surface that shares data with wxImage, we
     //     have to copy the data to temporary surface instead
 
     // NB: wxImage uses RGB order of bytes while DirectFB uses BGR, so we
     //     cannot use preallocated surface that shares data with wxImage, we
     //     have to copy the data to temporary surface instead

-    DFBSurfaceDescription desc;
-    desc.flags = (DFBSurfaceDescriptionFlags)
-        (DSDESC_CAPS | DSDESC_WIDTH | DSDESC_HEIGHT | DSDESC_PIXELFORMAT);
-    desc.caps = DSCAPS_NONE;
-    desc.width = image.GetWidth();
-    desc.height = image.GetHeight();
-    desc.pixelformat = DSPF_RGB24;
+    return CreateSurfaceWithFormat(image.GetWidth(), image.GetHeight(),
+                                   DSPF_RGB24);
+}

 
 

-    return wxIDirectFB::Get()->CreateSurface(&desc);
+static bool ConvertSurfaceToFormat(wxIDirectFBSurfacePtr& surface,
+                                   DFBSurfacePixelFormat format)
+{
+    if ( surface->GetPixelFormat() == format )
+        return true;
+
+    int w, h;
+    surface->GetSize(&w, &h);
+    wxIDirectFBSurfacePtr s = CreateSurfaceWithFormat(w, h, format);
+    if ( !s )
+        return false;
+
+    if ( !s->SetBlittingFlags(DSBLIT_NOFX) )
+        return false;
+    if ( !s->Blit(surface->GetRaw(), NULL, 0, 0) )
+        return false;
+
+    surface = s;
+    return true;
+}
+
+static DFBSurfacePixelFormat DepthToFormat(int depth)
+{
+    switch ( depth )
+    {
+        case 24:
+            return DSPF_RGB24;
+        case 32:
+            // NB: we treat depth=32 as requesting ARGB for consistency with
+            //     other ports
+            return DSPF_ARGB;
+        default:
+            wxFAIL_MSG( "unsupported depth requested" );
+            // fall through
+        case -1:
+            return DSPF_UNKNOWN;
+    }

 }
 
 //-----------------------------------------------------------------------------
 // wxBitmapRefData
 //-----------------------------------------------------------------------------
 
 }
 
 //-----------------------------------------------------------------------------
 // wxBitmapRefData
 //-----------------------------------------------------------------------------
 

-class wxBitmapRefData: public wxObjectRefData
+class wxBitmapRefData: public wxGDIRefData

 {
 public:
     wxBitmapRefData()
 {
 public:
     wxBitmapRefData()


@@ -134,6 +275,8 @@ public:
 #endif
     }
 
 #endif
     }
 

+    virtual bool IsOk() const { return m_surface; }
+

     wxIDirectFBSurfacePtr m_surface;
     wxMask               *m_mask;
 #if wxUSE_PALETTE
     wxIDirectFBSurfacePtr m_surface;
     wxMask               *m_mask;
 #if wxUSE_PALETTE


@@ -147,7 +290,6 @@ public:
 // wxBitmap
 //-----------------------------------------------------------------------------
 
 // wxBitmap
 //-----------------------------------------------------------------------------
 

-IMPLEMENT_ABSTRACT_CLASS(wxBitmapHandler, wxBitmapHandlerBase)

 IMPLEMENT_DYNAMIC_CLASS(wxBitmap, wxBitmapBase)
 
 wxBitmap::wxBitmap(int width, int height, int depth)
 IMPLEMENT_DYNAMIC_CLASS(wxBitmap, wxBitmapBase)
 
 wxBitmap::wxBitmap(int width, int height, int depth)


@@ -159,7 +301,7 @@ bool wxBitmap::Create(const wxIDirectFBSurfacePtr& surface)
 {
     UnRef();
 
 {
     UnRef();
 

-    wxCHECK_MSG( surface, false, _T("invalid surface") );
+    wxCHECK_MSG( surface, false, "invalid surface" );

 
     m_refData = new wxBitmapRefData();
     M_BITMAP->m_surface = surface;
 
     m_refData = new wxBitmapRefData();
     M_BITMAP->m_surface = surface;


@@ -167,53 +309,65 @@ bool wxBitmap::Create(const wxIDirectFBSurfacePtr& surface)
 }
 
 bool wxBitmap::Create(int width, int height, int depth)
 }
 
 bool wxBitmap::Create(int width, int height, int depth)

+{
+    return CreateWithFormat(width, height, DepthToFormat(depth));
+}
+
+bool wxBitmap::CreateWithFormat(int width, int height, int dfbFormat)

 {
     UnRef();
 
     wxCHECK_MSG( width > 0 && height > 0, false, wxT("invalid bitmap size") );
 {
     UnRef();
 
     wxCHECK_MSG( width > 0 && height > 0, false, wxT("invalid bitmap size") );

-    wxCHECK_MSG( depth == -1, false, wxT("only default depth supported now") );
-
-    DFBSurfaceDescription desc;
-    desc.flags = (DFBSurfaceDescriptionFlags)(
-                        DSDESC_CAPS | DSDESC_WIDTH | DSDESC_HEIGHT);
-    desc.caps = DSCAPS_NONE;
-    desc.width = width;
-    desc.height = height;

 
 

-    return Create(wxIDirectFB::Get()->CreateSurface(&desc));
+    return Create(CreateSurfaceWithFormat(width, height,
+                                          DFBSurfacePixelFormat(dfbFormat)));

 }
 
 #if wxUSE_IMAGE
 }
 
 #if wxUSE_IMAGE

-wxBitmap::wxBitmap(const wxImage& image, int depth)
+wxBitmap::wxBitmap(const wxImage& imageOrig, int depth)

 {
 {

-    wxCHECK_RET( image.Ok(), wxT("invalid image") );
+    wxCHECK_RET( imageOrig.Ok(), wxT("invalid image") );

 
 

-    // create surface in screen's format:
-    if ( !Create(image.GetWidth(), image.GetHeight(), depth) )
+    wxImage image(imageOrig);
+
+    // convert mask to alpha channel, because wxMask isn't implemented yet
+    // FIXME: don't do this, implement proper wxMask support
+    if ( image.HasMask() )
+        image.InitAlpha();
+
+    DFBSurfacePixelFormat format = DepthToFormat(depth);
+    if ( format == DSPF_UNKNOWN && image.HasAlpha() )
+        format = DSPF_ARGB;
+
+    // create surface in screen's format (unless we need alpha channel,
+    // in which case use ARGB):
+    if ( !CreateWithFormat(image.GetWidth(), image.GetHeight(), format) )

         return;
 
     // then copy the image to it:
     wxIDirectFBSurfacePtr dst = M_BITMAP->m_surface;
 
         return;
 
     // then copy the image to it:
     wxIDirectFBSurfacePtr dst = M_BITMAP->m_surface;
 

-    if ( dst->GetPixelFormat() == DSPF_RGB24 )
-    {
-        CopyImageToSurface(image, dst);
-    }
-    else
+    switch ( dst->GetPixelFormat() )

     {
     {

-        // wxBitmap uses different pixel format, so we have to use a temporary
-        // surface and blit to the bitmap via it:
-        wxIDirectFBSurfacePtr src(CreateSurfaceForImage(image));
-        CopyImageToSurface(image, src);
+        case DSPF_RGB24:
+        case DSPF_RGB32:
+        case DSPF_ARGB:
+            CopyImageToSurface(image, dst);
+            break;

 
 

-        if ( !dst->SetBlittingFlags(DSBLIT_NOFX) )
-            return;
-        if ( !dst->Blit(src->GetRaw(), NULL, 0, 0) )
-            return;
+        default:
+        {
+            // wxBitmap uses different pixel format, so we have to use a
+            // temporary surface and blit to the bitmap via it:
+            wxIDirectFBSurfacePtr src(CreateSurfaceForImage(image));
+            CopyImageToSurface(image, src);
+
+            if ( !dst->SetBlittingFlags(DSBLIT_NOFX) )
+                return;
+            if ( !dst->Blit(src->GetRaw(), NULL, 0, 0) )
+                return;
+        }

     }
     }

-
-    // FIXME: implement mask creation from image's mask (or alpha channel?)
-    wxASSERT_MSG( !image.HasMask(), _T("image masks are ignored for now") );

 }
 
 wxImage wxBitmap::ConvertToImage() const
 }
 
 wxImage wxBitmap::ConvertToImage() const


@@ -223,31 +377,80 @@ wxImage wxBitmap::ConvertToImage() const
     wxImage img(GetWidth(), GetHeight());
     wxIDirectFBSurfacePtr src = M_BITMAP->m_surface;
 
     wxImage img(GetWidth(), GetHeight());
     wxIDirectFBSurfacePtr src = M_BITMAP->m_surface;
 

-    if ( src->GetPixelFormat() == DSPF_RGB24 )
+    switch ( src->GetPixelFormat() )

     {
     {

-        CopySurfaceToImage(src, img);
-    }
-    else
-    {
-        // wxBitmap uses different pixel format, so we have to use a temporary
-        // surface and blit to the bitmap via it:
-        wxIDirectFBSurfacePtr dst(CreateSurfaceForImage(img));
+        case DSPF_RGB24:
+        case DSPF_RGB32:
+        case DSPF_ARGB:
+            CopySurfaceToImage(src, img);
+            break;
+        default:
+        {
+            // wxBitmap uses different pixel format, so we have to use a
+            // temporary surface and blit to the bitmap via it:
+            wxIDirectFBSurfacePtr dst(CreateSurfaceForImage(img));

 
 

-        if ( !dst->SetBlittingFlags(DSBLIT_NOFX) )
-            return wxNullImage;
-        if ( !dst->Blit(src->GetRaw(), NULL, 0, 0) )
-            return wxNullImage;
+            if ( !dst->SetBlittingFlags(DSBLIT_NOFX) )
+                return wxNullImage;
+            if ( !dst->Blit(src->GetRaw(), NULL, 0, 0) )
+                return wxNullImage;

 
 

-        CopySurfaceToImage(dst, img);
+            CopySurfaceToImage(dst, img);
+        }

     }
 
     // FIXME: implement mask setting in the image
     }
 
     // FIXME: implement mask setting in the image

-    wxASSERT_MSG( GetMask() == NULL, _T("bitmap masks are ignored for now") );
+    wxASSERT_MSG( GetMask() == NULL, "bitmap masks are ignored for now" );

 
     return img;
 }
 #endif // wxUSE_IMAGE
 
 
     return img;
 }
 #endif // wxUSE_IMAGE
 

+void *wxBitmap::GetRawData(wxPixelDataBase& data, int bpp)
+{
+    wxCHECK_MSG( Ok(), NULL, "invalid bitmap" );
+
+    AllocExclusive();
+
+    DFBSurfacePixelFormat format;
+    if ( bpp == 32 )
+        format = DSPF_ARGB;
+    else
+        format = DSPF_RGB24;
+
+    // convert the bitmap into format compatible with requested raw access;
+    // note that we don't bother converting the bitmap back in UngetRawData(),
+    // as unpacked formats (RGB24, RGB32) are the common case and converting
+    // between them while blitting is fast enough (FIXME?)
+    if ( !ConvertSurfaceToFormat(M_BITMAP->m_surface, format) )
+        return NULL;
+
+    void *bits = NULL;
+    if ( !M_BITMAP->m_surface->Lock
+                               (
+                                 (DFBSurfaceLockFlags)(DSLF_READ | DSLF_WRITE),
+                                 &bits,
+                                 &data.m_stride
+                               ) )
+        return NULL;
+
+    M_BITMAP->m_surface->GetSize(&data.m_width, &data.m_height);
+
+    return bits;
+}
+
+void wxBitmap::UngetRawData(wxPixelDataBase& WXUNUSED(data))
+{
+    M_BITMAP->m_surface->Unlock();
+}
+
+bool wxBitmap::HasAlpha() const
+{
+    wxCHECK_MSG( Ok(), false, "invalid bitmap" );
+
+    return M_BITMAP->m_surface->GetPixelFormat() == DSPF_ARGB;
+}
+

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


@@ -257,12 +460,7 @@ wxBitmap::wxBitmap(const char bits[], int width, int height, int depth)
 {
     wxCHECK_RET( depth == 1, wxT("can only create mono bitmap from XBM data") );
 
 {
     wxCHECK_RET( depth == 1, wxT("can only create mono bitmap from XBM data") );
 

-    wxFAIL_MSG( _T("not implemented") );
-}
-
-bool wxBitmap::IsOk() const
-{
-    return (m_refData != NULL && M_BITMAP->m_surface);
+    wxFAIL_MSG( "not implemented" );

 }
 
 int wxBitmap::GetHeight() const
 }
 
 int wxBitmap::GetHeight() const


@@ -408,21 +606,35 @@ void wxBitmap::SetHeight(int height)
 {
     AllocExclusive();
 
 {
     AllocExclusive();
 

-    wxFAIL_MSG( _T("SetHeight not implemented") );
+    wxFAIL_MSG( "SetHeight not implemented" );

 }
 
 void wxBitmap::SetWidth(int width)
 {
     AllocExclusive();
 
 }
 
 void wxBitmap::SetWidth(int width)
 {
     AllocExclusive();
 

-    wxFAIL_MSG( _T("SetWidth not implemented") );
+    wxFAIL_MSG( "SetWidth not implemented" );

 }
 
 void wxBitmap::SetDepth(int depth)
 {
 }
 
 void wxBitmap::SetDepth(int depth)
 {

+    DFBSurfacePixelFormat format = DepthToFormat(depth);
+    if ( M_BITMAP->m_surface->GetPixelFormat() == format )
+        return;
+

     AllocExclusive();
 
     AllocExclusive();
 

-    wxFAIL_MSG( _T("SetDepth not implemented") );
+    int w, h;
+    M_BITMAP->m_surface->GetSize(&w, &h);
+    wxIDirectFBSurfacePtr s = CreateSurfaceWithFormat(w, h, format);
+    if ( !s )
+        return;
+    if ( !s->SetBlittingFlags(DSBLIT_NOFX) )
+        return;
+    if ( !s->Blit(M_BITMAP->m_surface->GetRaw(), NULL, 0, 0) )
+        return;
+
+    M_BITMAP->m_surface = s;

 }
 
 wxIDirectFBSurfacePtr wxBitmap::GetDirectFBSurface() const
 }
 
 wxIDirectFBSurfacePtr wxBitmap::GetDirectFBSurface() const


@@ -432,12 +644,12 @@ wxIDirectFBSurfacePtr wxBitmap::GetDirectFBSurface() const
     return M_BITMAP->m_surface;
 }
 
     return M_BITMAP->m_surface;
 }
 

-wxObjectRefData *wxBitmap::CreateRefData() const
+wxGDIRefData *wxBitmap::CreateGDIRefData() const

 {
     return new wxBitmapRefData;
 }
 
 {
     return new wxBitmapRefData;
 }
 

-wxObjectRefData *wxBitmap::CloneRefData(const wxObjectRefData *data) const
+wxGDIRefData *wxBitmap::CloneGDIRefData(const wxGDIRefData *data) const

 {
     return new wxBitmapRefData(*(wxBitmapRefData *)data);
 }
 {
     return new wxBitmapRefData(*(wxBitmapRefData *)data);
 }