sc/source/filter/excel/excform.cxx |   10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

New commits:
commit bb70ecfdfd2117985627cc1b5e0de4b787969e7f
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Sun Oct 20 08:02:42 2024 +0200
Commit:     David Gilbert <freedesk...@treblig.org>
CommitDate: Tue Nov 12 01:57:56 2024 +0100

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

diff --git a/sc/source/filter/excel/excform.cxx 
b/sc/source/filter/excel/excform.cxx
index ee7b4a65e542..e9e59b5fe6da 100644
--- a/sc/source/filter/excel/excform.cxx
+++ b/sc/source/filter/excel/excform.cxx
@@ -918,8 +918,6 @@ ConvErr ExcelToSc::Convert( ScRangeListTabs& rRangeList, 
XclImpStream& aIn, std:
         {           //                                      SDK4 SDK5
             case 0x01: // Array Formula                         [325    ]
                        // Array Formula or Shared Formula       [    277]
-                nIgnore = (meBiff == EXC_BIFF2) ? 3 : 4;
-                break;
             case 0x02: // Data Table                            [325 277]
                 nIgnore = (meBiff == EXC_BIFF2) ? 3 : 4;
                 break;
@@ -1000,6 +998,9 @@ ConvErr ExcelToSc::Convert( ScRangeListTabs& rRangeList, 
XclImpStream& aIn, std:
                 nIgnore = 1;
                 break;
             case 0x1E: // Integer                               [315 266]
+            case 0x58:
+            case 0x78:
+            case 0x38: // Command-Equivalent Function           [333    ]
                 nIgnore = 2;
                 break;
             case 0x1F: // Number                                [315 266]
@@ -1154,11 +1155,6 @@ ConvErr ExcelToSc::Convert( ScRangeListTabs& rRangeList, 
XclImpStream& aIn, std:
             case 0x2F: // Incomplete Reference Subexpression... [332 282]
                 nIgnore = (meBiff == EXC_BIFF2) ? 1 : 2;
                 break;
-            case 0x58:
-            case 0x78:
-            case 0x38: // Command-Equivalent Function           [333    ]
-                nIgnore = 2;
-                break;
             case 0x59:
             case 0x79:
             case 0x39: // Name or External Name                 [    275]

Reply via email to