[Libreoffice-commits] core.git: vcl/source

[Caolán McNamara (via logerrit)]

 vcl/source/outdev/transparent.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit e56479ee8a88f538c2163ddd58c5dc49a8744500
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Sat Nov 12 16:15:51 2022 +0000
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Sun Nov 13 17:29:16 2022 +0100

    ofz#53338 Abrt
    
    we're turning "false" to indicate nothing was drawn, but nothing was
    drawn because the entire output would have been clipped out, so there's
    no point drawing anyway, so return true instead to indicate the draw
    was successfully short-circuited to give a correct no-op result
    
    probably went wrong with:
    
    commit 9153a36b59f2efc2bfb2c9256c39eb5687808deb
    Date:   Fri Mar 28 23:30:02 2014 +1100
    
        fdo#74702 Refactor OutputDevice::DrawTransparent()
    
    Change-Id: I989ebffff267f57c036560cda59eb7abe14d405a
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142635
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/vcl/source/outdev/transparent.cxx 
b/vcl/source/outdev/transparent.cxx
index d19f2cfc18bd..58e12457f9e9 100644
--- a/vcl/source/outdev/transparent.cxx
+++ b/vcl/source/outdev/transparent.cxx
@@ -213,7 +213,7 @@ bool OutputDevice::DrawTransparentNatively ( const 
tools::PolyPolygon& rPolyPoly
             InitClipRegion();
 
         if( mbOutputClipped )
-            return false;
+            return true;
 
         if( mbInitLineColor )
             InitLineColor();