core.git: canvas/source

[Bogdan Buzea (via logerrit)]

 canvas/source/cairo/cairo_canvashelper.cxx |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

New commits:
commit 38dc12033fb609b3097b3100efb4a459aa233164
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Fri Oct 18 17:14:04 2024 +0200
Commit:     David Gilbert <freedesk...@treblig.org>
CommitDate: Sat Nov 2 01:00:21 2024 +0100

    tdf#163486: PVS: Identical branches
    
    
    Change-Id: I65edd548d9ed4fbc5589f354280dbfb90e2a9064
    V1037: Two or more case-branches perform the same actions. Check lines: 
171, 177
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175148
    Tested-by: Jenkins
    Reviewed-by: David Gilbert <freedesk...@treblig.org>

diff --git a/canvas/source/cairo/cairo_canvashelper.cxx 
b/canvas/source/cairo/cairo_canvashelper.cxx
index e4c781803e3f..99b56d719ae1 100644
--- a/canvas/source/cairo/cairo_canvashelper.cxx
+++ b/canvas/source/cairo/cairo_canvashelper.cxx
@@ -168,14 +168,12 @@ namespace cairocanvas
                 compositingMode = CAIRO_OPERATOR_SOURCE;
                 break;
             case rendering::CompositeOperation::DESTINATION:
+            case rendering::CompositeOperation::UNDER:
                 compositingMode = CAIRO_OPERATOR_DEST;
                 break;
             case rendering::CompositeOperation::OVER:
                 compositingMode = CAIRO_OPERATOR_OVER;
                 break;
-            case rendering::CompositeOperation::UNDER:
-                compositingMode = CAIRO_OPERATOR_DEST;
-                break;
             case rendering::CompositeOperation::INSIDE:
                 compositingMode = CAIRO_OPERATOR_IN;
                 break;