[Libreoffice-commits] core.git: include/basegfx

[Stephan Bergmann]

 include/basegfx/raster/bpixelraster.hxx |   10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

New commits:
commit 06bf579a007f33a1726197c92125b7475836e3ef
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Fri Jan 19 11:54:51 2018 +0100

    BPixel has a default ctor zero'ing its members
    
    ...so no need to do that again in the body of the BPixelRaster ctor (which had
    needlessly been done ever since the code's introduction with
    6f769fb8aaab1f899b3ca4551f6a8df2953cd938 "INTEGRATION: CWS aw033").  And
    BPixelRaster::reset appears to have otherwise been unused, so can be removed.
    
    Change-Id: Icd8e09bfe225be2c6695d8a137c271dca5a70de7
    Reviewed-on: https://gerrit.libreoffice.org/48184
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/include/basegfx/raster/bpixelraster.hxx b/include/basegfx/raster/bpixelraster.hxx
index 42620c7c01d7..b3f168754a8f 100644
--- a/include/basegfx/raster/bpixelraster.hxx
+++ b/include/basegfx/raster/bpixelraster.hxx
@@ -42,21 +42,13 @@ namespace basegfx
         std::unique_ptr<BPixel[]>   mpContent;
 
     public:
-        // reset
-        void reset()
-        {
-            memset(mpContent.get(), 0, sizeof(BPixel) * mnCount);
-        }
-
         // constructor/destructor
         BPixelRaster(sal_uInt32 nWidth, sal_uInt32 nHeight)
         :   mnWidth(nWidth),
             mnHeight(nHeight),
             mnCount(nWidth * nHeight),
             mpContent(new BPixel[mnCount])
-        {
-            reset();
-        }
+        {}
 
         // coordinate calcs between X/Y and span
         sal_uInt32 getIndexFromXY(sal_uInt32 nX, sal_uInt32 nY) const { return (nX + (nY * mnWidth)); }