sc/source/filter/excel/xichart.cxx |    5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

New commits:
commit 5c95a2a1a86d2eabdc5a0cc23e836283d14bcccf
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Tue Oct 22 06:38:36 2024 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Thu Oct 24 11:52:41 2024 +0200

    tdf#163486: PVS: Identical branches
    
    V1037 Two or more case-branches perform the same actions. Check lines: 
2242, 2272
    
    Change-Id: Idcddf2a4d7522a368960e76256e28f1f0d36c9b3
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175394
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/sc/source/filter/excel/xichart.cxx 
b/sc/source/filter/excel/xichart.cxx
index 4b0b70dfe2f9..096f872fe7b2 100644
--- a/sc/source/filter/excel/xichart.cxx
+++ b/sc/source/filter/excel/xichart.cxx
@@ -2239,6 +2239,7 @@ void XclImpChType::ReadChType( XclImpStream& rStrm )
         case EXC_ID_CHAREA:
         case EXC_ID_CHRADARLINE:
         case EXC_ID_CHRADARAREA:
+        case EXC_ID_CHSURFACE:
             maData.mnFlags = rStrm.ReaduInt16();
         break;
 
@@ -2268,10 +2269,6 @@ void XclImpChType::ReadChType( XclImpStream& rStrm )
                 maData.mnFlags = 0;
         break;
 
-        case EXC_ID_CHSURFACE:
-            maData.mnFlags = rStrm.ReaduInt16();
-        break;
-
         default:
             bKnownType = false;
     }

Reply via email to