basic/source/sbx/sbxvalue.cxx | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
New commits:
commit 76a25639474c50c34a2b84ba1f357013de5ddae8
Author: Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Fri Oct 19 21:22:23 2018 +0200
Commit: Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Sat Oct 20 08:49:13 2018 +0200
tdf#120703 (PVS): redundant check
V547 Expression 'aData.eType == SbxDECIMAL' is always true
Change-Id: I37bf367221ced7bf1c53bcd5138ae7bac656da52
Reviewed-on: https://gerrit.libreoffice.org/62030
Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>
Tested-by: Mike Kaganski <mike.kagan...@collabora.com>
diff --git a/basic/source/sbx/sbxvalue.cxx b/basic/source/sbx/sbxvalue.cxx
index dba938e96eec..05e8773aab9f 100644
--- a/basic/source/sbx/sbxvalue.cxx
+++ b/basic/source/sbx/sbxvalue.cxx
@@ -164,8 +164,7 @@ void SbxValue::Clear()
}
break;
case SbxDECIMAL:
- if( aData.eType == SbxDECIMAL )
- releaseDecimalPtr( aData.pDecimal );
+ releaseDecimalPtr( aData.pDecimal );
break;
case SbxDATAOBJECT:
aData.pData = nullptr; break;
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
