[Libreoffice-commits] core.git: Branch 'libreoffice-5-0' - vcl/source

Caolán McNamara caolanm at redhat.com
Mon Sep 21 09:09:37 PDT 2015


 vcl/source/outdev/transparent.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 79a3a1ee610297ba6982e482cd471ee743e6ce8f
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Fri Sep 18 13:05:31 2015 +0100

    valgrind: memory leak in scoped_ptr/unique_ptr -> VclPtr convert
    
    regression from
    
    commit 820576af4fd6441a752742b43d804e9837839925
    Author: Noel Grandin <noel at peralex.com>
    Date:   Thu Mar 19 13:54:12 2015 +0200
    
        start wrapping OutputDevice in VclPtr
    
     9,800 (568 direct, 9,232 indirect) bytes in 1 blocks are definitely lost in loss record 12,696 of 12,898
        by 0xC602E72: ScopedVclPtrInstance<VirtualDevice>::ScopedVclPtrInstance<>() (vclptr.hxx:375)
    
    (cherry picked from commit 23d4d2f6ac5aa2ca94109e07421a1ed8330b85bd)
    
    Change-Id: I356f39c339fd28a9e19a00d61b6f1bee492f2638
    Reviewed-on: https://gerrit.libreoffice.org/18689
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>

diff --git a/vcl/source/outdev/transparent.cxx b/vcl/source/outdev/transparent.cxx
index 69a4244..7f3b355 100644
--- a/vcl/source/outdev/transparent.cxx
+++ b/vcl/source/outdev/transparent.cxx
@@ -751,7 +751,7 @@ void OutputDevice::DrawTransparent( const GDIMetaFile& rMtf, const Point& rPos,
 
                     const AlphaMask aAlpha(xVDev->GetBitmap(aPoint, xVDev->GetOutputSizePixel()));
 
-                    xVDev.reset();
+                    xVDev.disposeAndClear();
 
                     // draw masked content to target and restore MapMode
                     DrawBitmapEx(aDstRect.TopLeft(), BitmapEx(aPaint, aAlpha));
@@ -799,7 +799,7 @@ void OutputDevice::DrawTransparent( const GDIMetaFile& rMtf, const Point& rPos,
 
                     aAlpha = xVDev->GetBitmap( Point(), xVDev->GetOutputSizePixel() );
 
-                    xVDev.reset();
+                    xVDev.disposeAndClear();
 
                     EnableMapMode( false );
                     DrawBitmapEx( aDstRect.TopLeft(), BitmapEx( aPaint, aAlpha ) );


More information about the Libreoffice-commits mailing list