diff options
| author | Bogdan Buzea <buzea.bogdan@libreoffice.org> | 2024-10-19 17:59:06 +0200 |
|---|---|---|
| committer | David Gilbert <freedesktop@treblig.org> | 2024-10-31 02:00:24 +0100 |
| commit | aef81068f439be1be7dccbedc8548327f037167b (patch) | |
| tree | f9d262bd1af5c0eb94e477c089745158b2516d46 /sc | |
| parent | 06a2f461970288b569cc282c2913baab394c7075 (diff) | |
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 <freedesktop@treblig.org>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/filter/excel/excform8.cxx | 10 |
1 files changed, 3 insertions, 7 deletions
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] |