sc/source/filter/excel/excform8.cxx | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
New commits:
commit fa1902c464f776d2a79a16598e5d713d13f3bcab
Author: Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Sat Oct 19 17:55:39 2024 +0200
Commit: David Gilbert <freedesk...@treblig.org>
CommitDate: Tue Nov 12 01:09:25 2024 +0100
tdf#163486: PVS: Identical branches
V1037 Two or more case-branches perform the same actions. Check lines:
1051, 1124
Change-Id: Iffebecc9ae82633734fd585e40fe51b30fde56b9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175211
Reviewed-by: David Gilbert <freedesk...@treblig.org>
Tested-by: Jenkins
diff --git a/sc/source/filter/excel/excform8.cxx
b/sc/source/filter/excel/excform8.cxx
index ac0db70112af..144b4da1e7d4 100644
--- a/sc/source/filter/excel/excform8.cxx
+++ b/sc/source/filter/excel/excform8.cxx
@@ -1028,11 +1028,6 @@ ConvErr ExcelToSc8::Convert( ScRangeListTabs&
rRangeList, XclImpStream& aIn, std
case 0x21: // Function, Fixed Number of Arguments [333 282]
aIn.Ignore( 2 );
break;
- case 0x42:
- case 0x62:
- 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]
@@ -1093,6 +1088,9 @@ ConvErr ExcelToSc8::Convert( ScRangeListTabs& rRangeList,
XclImpStream& aIn, std
case 0x29: // Variable Reference Subexpression [331 281]
aIn.Ignore( 2 ); // There isn't any more
break;
+ case 0x42:
+ case 0x62:
+ case 0x22: // Function, Variable Number of Arg. [333 283]
case 0x4A:
case 0x6A:
case 0x2A: // Deleted Cell Reference [323 273]
