sd/source/filter/eppt/pptx-epptbase.cxx |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

New commits:
commit e2479ebd5c727a1e7d0eb40210f9fec9a1ad75ad
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Sat Oct 19 19:31:01 2024 +0200
Commit:     David Gilbert <freedesk...@treblig.org>
CommitDate: Thu Oct 31 02:03:04 2024 +0100

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

diff --git a/sd/source/filter/eppt/pptx-epptbase.cxx 
b/sd/source/filter/eppt/pptx-epptbase.cxx
index eebf01410c15..c548ff09f1d9 100644
--- a/sd/source/filter/eppt/pptx-epptbase.cxx
+++ b/sd/source/filter/eppt/pptx-epptbase.cxx
@@ -941,6 +941,7 @@ sal_Int8 PPTWriterBase::GetTransition( FadeEffect eEffect, 
sal_uInt8& nDirection
         nDirection++;
         [[fallthrough]];
     case FadeEffect_FADE_FROM_RIGHT :
+    case FadeEffect_ROLL_FROM_RIGHT :
         nPPTTransitionType = PPT_TRANSITION_TYPE_WIPE;
         break;
 
@@ -953,9 +954,6 @@ sal_Int8 PPTWriterBase::GetTransition( FadeEffect eEffect, 
sal_uInt8& nDirection
     case FadeEffect_ROLL_FROM_BOTTOM :
         nDirection++;
         [[fallthrough]];
-    case FadeEffect_ROLL_FROM_RIGHT :
-        nPPTTransitionType = PPT_TRANSITION_TYPE_WIPE;
-        break;
 
     case FadeEffect_FADE_TO_CENTER :
         nDirection++;

Reply via email to