diff options
| author | Noel Grandin <noelgrandin@gmail.com> | 2018-08-09 16:30:57 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-08-10 08:21:55 +0200 |
| commit | d977e02ec6350115c39f03d588539e8bd423a1c3 (patch) | |
| tree | bd624f4037b1d498cde385e56d48e383335c32ce /svl | |
| parent | 76dd6bb9cc6dff0f59cdfce87b61210be6476965 (diff) | |
unnecessary null check before dynamic_cast, in various
Change-Id: I76ad0b3152030c29ee28f6a6cc80d0832188d02b
Reviewed-on: https://gerrit.libreoffice.org/58774
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'svl')
| -rw-r--r-- | svl/source/items/itempool.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/svl/source/items/itempool.cxx b/svl/source/items/itempool.cxx index 76e9d90b485f..f90588288163 100644 --- a/svl/source/items/itempool.cxx +++ b/svl/source/items/itempool.cxx @@ -482,7 +482,7 @@ void SfxItemPool::Delete() // from SfxItemPool // This causes chaos in Itempool! const SfxPoolItem* pStaticDefaultItem = (*pImpl->mpStaticDefaults)[n]; - if (pStaticDefaultItem && dynamic_cast<const SfxSetItem*>(pStaticDefaultItem) != nullptr) + if (dynamic_cast<const SfxSetItem*>(pStaticDefaultItem)) { // SfxSetItem found, remove PoolItems (and defaults) with same ID auto& rArrayPtr = pImpl->maPoolItems[n]; |