check that parent is non-NULL in CreateWindowFromHWND() instead of just crashing...

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

diff --git a/src/msw/bitmap.cpp b/src/msw/bitmap.cpp
index d06251a59966fc50963c7ef6b284b5614de65df2..725d5fd0cbd016443df29e3b43e7c0a1bbfda3d2 100644 (file)
--- a/src/msw/bitmap.cpp
+++ b/src/msw/bitmap.cpp
@@ -40,6 +40,7 @@
 #endif
 
 #include "wx/msw/private.h"
 #endif
 
 #include "wx/msw/private.h"

+#include "wx/msw/dc.h"

 
 #if wxUSE_WXDIB
     #include "wx/msw/dib.h"
 
 #if wxUSE_WXDIB
     #include "wx/msw/dib.h"


@@ -211,7 +212,7 @@ wxBitmapRefData::wxBitmapRefData(const wxBitmapRefData& data)
         m_bitmapMask = new wxMask(*data.m_bitmapMask);
 
     // FIXME: we don't copy m_hBitmap currently but we should, see wxBitmap::
         m_bitmapMask = new wxMask(*data.m_bitmapMask);
 
     // FIXME: we don't copy m_hBitmap currently but we should, see wxBitmap::

-    //        CloneRefData()
+    //        CloneGDIRefData()

 
     wxASSERT_MSG( !data.m_isDIB,
                     _T("can't copy bitmap locked for raw access!") );
 
     wxASSERT_MSG( !data.m_isDIB,
                     _T("can't copy bitmap locked for raw access!") );


@@ -250,7 +251,7 @@ wxGDIImageRefData *wxBitmap::CreateData() const
     return new wxBitmapRefData;
 }
 
     return new wxBitmapRefData;
 }
 

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

 {
     const wxBitmapRefData *
         data = wx_static_cast(const wxBitmapRefData *, dataOrig);
 {
     const wxBitmapRefData *
         data = wx_static_cast(const wxBitmapRefData *, dataOrig);


@@ -277,16 +278,17 @@ wxObjectRefData *wxBitmap::CloneRefData(const wxObjectRefData *dataOrig) const
         self->m_refData = new wxBitmapRefData(*data);
     }
 
         self->m_refData = new wxBitmapRefData(*data);
     }
 

+    wxBitmapRefData * const
+        selfdata = wx_static_cast(wxBitmapRefData *, m_refData);
+

     // copy also the mask
     wxMask * const maskSrc = data->GetMask();
     if ( maskSrc )
     {
     // copy also the mask
     wxMask * const maskSrc = data->GetMask();
     if ( maskSrc )
     {

-        wxBitmapRefData *selfdata = wx_static_cast(wxBitmapRefData *, m_refData);
-

         selfdata->SetMask(new wxMask(*maskSrc));
     }
 
         selfdata->SetMask(new wxMask(*maskSrc));
     }
 

-    return m_refData;
+    return selfdata;

 }
 
 bool wxBitmap::CopyFromIconOrCursor(const wxGDIImage& icon,
 }
 
 bool wxBitmap::CopyFromIconOrCursor(const wxGDIImage& icon,