sc/source/filter/excel/excform8.cxx |   28 ++++++++++++----------------
 1 file changed, 12 insertions(+), 16 deletions(-)

New commits:
commit 3c32c9323968bd26e5b4524ed4045bc2829207ee
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Sat Oct 19 17:41:45 2024 +0200
Commit:     David Gilbert <freedesk...@treblig.org>
CommitDate: Wed Nov 6 18:16:19 2024 +0100

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

diff --git a/sc/source/filter/excel/excform8.cxx 
b/sc/source/filter/excel/excform8.cxx
index 0ab1ca02ab25..1984d2c362da 100644
--- a/sc/source/filter/excel/excform8.cxx
+++ b/sc/source/filter/excel/excform8.cxx
@@ -1092,17 +1092,6 @@ ConvErr ExcelToSc8::Convert( ScRangeListTabs& 
rRangeList, XclImpStream& aIn, std
                 rRangeList.Append(aCRD.toAbs(GetDocImport().getDoc(), 
aEingPos), nTab);
             }
                 break;
-            case 0x46:
-            case 0x66:
-            case 0x26: // Constant Reference Subexpression      [321 271]
-            case 0x47:
-            case 0x67:
-            case 0x27: // Erroneous Constant Reference Subexpr. [322 272]
-            case 0x48:
-            case 0x68:
-            case 0x28: // Incomplete Constant Reference Subexpr.[331 281]
-                aIn.Ignore( 6 );   // There isn't any more
-                break;
             case 0x49:
             case 0x69:
             case 0x29: // Variable Reference Subexpression      [331 281]
@@ -1113,11 +1102,6 @@ ConvErr ExcelToSc8::Convert( ScRangeListTabs& 
rRangeList, XclImpStream& aIn, std
             case 0x2A: // Deleted Cell Reference                [323 273]
                 aIn.Ignore( 3 );
                 break;
-            case 0x4B:
-            case 0x6B:
-            case 0x2B: // Deleted Area Reference                [323 273]
-                aIn.Ignore( 6 );
-                break;
             case 0x4C:
             case 0x6C:
             case 0x2C: // Cell Reference Within a Name          [323    ]
@@ -1245,6 +1229,18 @@ ConvErr ExcelToSc8::Convert( ScRangeListTabs& 
rRangeList, XclImpStream& aIn, std
                 }
             }
                 break;
+            case 0x46:
+            case 0x66:
+            case 0x26: // Constant Reference Subexpression      [321 271]
+            case 0x47:
+            case 0x67:
+            case 0x27: // Erroneous Constant Reference Subexpr. [322 272]
+            case 0x48:
+            case 0x68:
+            case 0x28: // Incomplete Constant Reference Subexpr.[331 281]
+            case 0x4B:
+            case 0x6B:
+            case 0x2B: // Deleted Area Reference                [323 273]
             case 0x5C:
             case 0x7C:
             case 0x3C: // Deleted 3-D Cell Reference            [    277]

Reply via email to