diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-19 21:22:23 +0200 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-20 08:49:13 +0200 |
| commit | 76a25639474c50c34a2b84ba1f357013de5ddae8 (patch) | |
| tree | 496f0231ebfaa14ddeea85527088d1256871e0cb /basic | |
| parent | 722cb06e71539e108f663ca1c2dd6baf7f4adc64 (diff) | |
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.kaganski@collabora.com>
Tested-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'basic')
| -rw-r--r-- | basic/source/sbx/sbxvalue.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
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; |