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

New commits:
commit aef81068f439be1be7dccbedc8548327f037167b
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Sat Oct 19 17:59:06 2024 +0200
Commit:     David Gilbert <freedesk...@treblig.org>
CommitDate: Thu Oct 31 02:00:24 2024 +0100

    tdf#163486: PVS: Identical branches
    
    V1037 Two or more case-branches perform the same actions. Check lines: 981, 
984, 1056
    
    Change-Id: I5e1578341075dfb8f6699fca3f5468de1eeef70b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175212
    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 7de536469cff..4e1e8c5d313b 100644
--- a/sc/source/filter/excel/excform8.cxx
+++ b/sc/source/filter/excel/excform8.cxx
@@ -972,13 +972,6 @@ ConvErr ExcelToSc8::Convert( ScRangeListTabs& rRangeList, 
XclImpStream& aIn, std
 
         switch( nOp )   //                              book page:
         {           //                                      SDK4 SDK5
-            case 0x01: // Array Formula                         [325    ]
-                       // Array Formula or Shared Formula       [    277]
-                aIn.Ignore( 4 );
-                break;
-            case 0x02: // Data Table                            [325 277]
-                aIn.Ignore( 4 );
-                break;
             case 0x03: // Addition                              [312 264]
             case 0x04: // Subtraction                           [313 264]
             case 0x05: // Multiplication                        [313 264]
@@ -1046,6 +1039,9 @@ ConvErr ExcelToSc8::Convert( ScRangeListTabs& rRangeList, 
XclImpStream& aIn, std
             case 0x22: // Function, Variable Number of Arg.     [333 283]
                 aIn.Ignore( 3 );
                 break;
+            case 0x01: // Array Formula                         [325    ]
+                       // Array Formula or Shared Formula       [    277]
+            case 0x02: // Data Table                            [325 277]
             case 0x43:
             case 0x63:
             case 0x23: // Name                                  [318 269]

Reply via email to