basic/source/runtime/methods1.cxx | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
New commits:
commit c2308be74756ea024801c1449d77361cb65ca6bf
Author: Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Fri Oct 18 17:33:10 2024 +0200
Commit: David Gilbert <freedesk...@treblig.org>
CommitDate: Sat Oct 19 19:52:19 2024 +0200
tdf#163486: PVS: Identical branches
Change-Id: I01aae9395213eb09c8bea990b2b6a811f7ab2ee2
V1037: Two or more case-branches perform the same actions. Check lines:
1249, 1277
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175150
Tested-by: Jenkins
Reviewed-by: David Gilbert <freedesk...@treblig.org>
diff --git a/basic/source/runtime/methods1.cxx
b/basic/source/runtime/methods1.cxx
index fa6527b5afd2..8787ca4dc430 100644
--- a/basic/source/runtime/methods1.cxx
+++ b/basic/source/runtime/methods1.cxx
@@ -1246,6 +1246,9 @@ void SbRtl_TypeLen(StarBASIC *, SbxArray & rPar, bool)
case SbxDIMARRAY:
case SbxCARRAY:
case SbxUSERDEF:
+ case SbxOBJECT:
+ case SbxVARIANT:
+ case SbxDATAOBJECT:
nLen = 0;
break;
@@ -1271,12 +1274,6 @@ void SbRtl_TypeLen(StarBASIC *, SbxArray & rPar, bool)
nLen = 8;
break;
- case SbxOBJECT:
- case SbxVARIANT:
- case SbxDATAOBJECT:
- nLen = 0;
- break;
-
case SbxCHAR:
case SbxBYTE:
case SbxBOOL:
