core.git: sc/source

[Bogdan Buzea (via logerrit)]

 sc/source/filter/excel/xeextlst.cxx |    4 ----
 1 file changed, 4 deletions(-)

New commits:
commit 0878b2a522e4f5fbba2842a3862a0de0e7159195
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Tue Oct 22 06:06:31 2024 +0200
Commit:     David Gilbert <freedesk...@treblig.org>
CommitDate: Sun Oct 27 01:18:22 2024 +0200

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

diff --git a/sc/source/filter/excel/xeextlst.cxx 
b/sc/source/filter/excel/xeextlst.cxx
index d32caecaffc2..8d36f639b1f3 100644
--- a/sc/source/filter/excel/xeextlst.cxx
+++ b/sc/source/filter/excel/xeextlst.cxx
@@ -328,8 +328,6 @@ const char* GetOperatorString(ScConditionMode eMode)
             pRet = "notBetween";
             break;
         case ScConditionMode::Duplicate:
-            pRet = nullptr;
-            break;
         case ScConditionMode::NotDuplicate:
             pRet = nullptr;
             break;
@@ -526,8 +524,6 @@ 
XclExpExtConditionalFormatting::XclExpExtConditionalFormatting( const XclExpRoot
             }
             break;
             case ScFormatEntry::Type::Databar:
-                maCfRules.AppendNewRecord(new XclExpExtCfRule( *this, *pEntry, 
aAddr, rItem.aGUID, rItem.nPriority));
-            break;
             case ScFormatEntry::Type::ExtCondition:
                 maCfRules.AppendNewRecord(new XclExpExtCfRule( *this, *pEntry, 
aAddr, rItem.aGUID, rItem.nPriority));
             break;