every ImplImage has a BitmapEx

Change-Id: I02561cfdff57eab7684d68c98ba2b3d83847c9c5

3 files changed, 16 insertions, 28 deletions

diff --git a/vcl/inc/image.h b/vcl/inc/image.h
index 72a7f3233775..873fc98432dd 100644
--- a/vcl/inc/image.h
+++ b/vcl/inc/image.h
@@ -28,16 +28,10 @@
 struct ImplImage
 {
     BitmapChecksum maBitmapChecksum;
-
-    std::unique_ptr<BitmapEx> mpBitmapEx;
+    BitmapEx maBitmapEx;
     BitmapEx maDisabledBitmapEx;
 
-    ImplImage();
-    ~ImplImage();
-
-private:
-    ImplImage(const ImplImage&) = delete;
-    void operator=(const ImplImage&) = delete;
+    ImplImage(const BitmapEx& rBitmapEx);
 };
 
 #endif // INCLUDED_VCL_INC_IMAGE_H
diff --git a/vcl/source/image/Image.cxx b/vcl/source/image/Image.cxx
index bc663fbbc525..d873d3eeeb86 100644
--- a/vcl/source/image/Image.cxx
+++ b/vcl/source/image/Image.cxx
@@ -82,8 +82,7 @@ void Image::ImplInit(const BitmapEx& rBitmapEx)
 {
     if (!rBitmapEx.IsEmpty())
     {
-        mpImplData.reset(new ImplImage);
-        mpImplData->mpBitmapEx.reset(new BitmapEx(rBitmapEx));
+        mpImplData.reset(new ImplImage(rBitmapEx));
     }
 }
 
@@ -91,9 +90,9 @@ Size Image::GetSizePixel() const
 {
     Size aRet;
 
-    if (mpImplData && mpImplData->mpBitmapEx)
+    if (mpImplData)
     {
-        aRet = mpImplData->mpBitmapEx->GetSizePixel();
+        aRet = mpImplData->maBitmapEx.GetSizePixel();
     }
 
     return aRet;
@@ -103,9 +102,9 @@ BitmapEx Image::GetBitmapEx() const
 {
     BitmapEx aRet;
 
-    if (mpImplData && mpImplData->mpBitmapEx)
+    if (mpImplData)
     {
-        aRet = BitmapEx(*mpImplData->mpBitmapEx);
+        aRet = mpImplData->maBitmapEx;
     }
 
     return aRet;
@@ -127,29 +126,28 @@ bool Image::operator==(const Image& rImage) const
     else if (!rImage.mpImplData || !mpImplData)
         bRet = false;
     else
-        bRet = *rImage.mpImplData->mpBitmapEx == *mpImplData->mpBitmapEx;
+        bRet = rImage.mpImplData->maBitmapEx == mpImplData->maBitmapEx;
 
     return bRet;
 }
 
 void Image::Draw(OutputDevice* pOutDev, const Point& rPos, DrawImageFlags nStyle, const Size* pSize)
 {
-    if (!mpImplData || !mpImplData->mpBitmapEx ||
-        (!pOutDev->IsDeviceOutputNecessary() && pOutDev->GetConnectMetaFile() == nullptr))
+    if (!mpImplData || (!pOutDev->IsDeviceOutputNecessary() && pOutDev->GetConnectMetaFile() == nullptr))
         return;
 
     const Point aSrcPos(0, 0);
-    Size aBitmapSizePixel = mpImplData->mpBitmapEx->GetSizePixel();
+    Size aBitmapSizePixel = mpImplData->maBitmapEx.GetSizePixel();
 
     Size aOutSize = pSize ? *pSize : pOutDev->PixelToLogic(aBitmapSizePixel);
 
     if (nStyle & DrawImageFlags::Disable)
     {
-        BitmapChecksum aChecksum = mpImplData->mpBitmapEx->GetChecksum();
+        BitmapChecksum aChecksum = mpImplData->maBitmapEx.GetChecksum();
         if (mpImplData->maBitmapChecksum != aChecksum)
         {
             mpImplData->maBitmapChecksum = aChecksum;
-            mpImplData->maDisabledBitmapEx = BitmapProcessor::createDisabledImage(*mpImplData->mpBitmapEx);
+            mpImplData->maDisabledBitmapEx = BitmapProcessor::createDisabledImage(mpImplData->maBitmapEx);
         }
         pOutDev->DrawBitmapEx(rPos, aOutSize, aSrcPos, aBitmapSizePixel, mpImplData->maDisabledBitmapEx);
     }
@@ -158,7 +156,7 @@ void Image::Draw(OutputDevice* pOutDev, const Point& rPos, DrawImageFlags nStyle
         if (nStyle & (DrawImageFlags::ColorTransform | DrawImageFlags::Highlight |
                       DrawImageFlags::Deactive | DrawImageFlags::SemiTransparent))
         {
-            BitmapEx aTempBitmapEx(*mpImplData->mpBitmapEx);
+            BitmapEx aTempBitmapEx(mpImplData->maBitmapEx);
 
             if (nStyle & (DrawImageFlags::Highlight | DrawImageFlags::Deactive))
             {
@@ -190,7 +188,7 @@ void Image::Draw(OutputDevice* pOutDev, const Point& rPos, DrawImageFlags nStyle
         }
         else
         {
-            pOutDev->DrawBitmapEx(rPos, aOutSize, aSrcPos, mpImplData->mpBitmapEx->GetSizePixel(), *mpImplData->mpBitmapEx);
+            pOutDev->DrawBitmapEx(rPos, aOutSize, aSrcPos, mpImplData->maBitmapEx.GetSizePixel(), mpImplData->maBitmapEx);
         }
     }
 }
diff --git a/vcl/source/image/ImplImage.cxx b/vcl/source/image/ImplImage.cxx
index ad4312024f92..82a2099e8813 100644
--- a/vcl/source/image/ImplImage.cxx
+++ b/vcl/source/image/ImplImage.cxx
@@ -29,15 +29,11 @@
 #include <image.h>
 #include <memory>
 
-ImplImage::ImplImage()
+ImplImage::ImplImage(const BitmapEx &rBitmapEx)
     : maBitmapChecksum(0)
-    , mpBitmapEx()
+    , maBitmapEx(rBitmapEx)
     , maDisabledBitmapEx()
 {
 }
 
-ImplImage::~ImplImage()
-{
-}
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */