summaryrefslogtreecommitdiff
path: root/sfx2/source
diff options
context:
space:
mode:
authorNoel Grandin <noelgrandin@gmail.com>2018-08-09 16:30:57 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2018-08-10 08:21:55 +0200
commitd977e02ec6350115c39f03d588539e8bd423a1c3 (patch)
treebd624f4037b1d498cde385e56d48e383335c32ce /sfx2/source
parent76dd6bb9cc6dff0f59cdfce87b61210be6476965 (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 'sfx2/source')
-rw-r--r--sfx2/source/appl/appdde.cxx2
1 files changed, 1 insertions, 1 deletions
diff --git a/sfx2/source/appl/appdde.cxx b/sfx2/source/appl/appdde.cxx
index 7a572c94236d..92021f283ec5 100644
--- a/sfx2/source/appl/appdde.cxx
+++ b/sfx2/source/appl/appdde.cxx
@@ -154,7 +154,7 @@ bool ImplDdeService::MakeTopic( const OUString& rNm )
SfxCallMode::SYNCHRON,
{ &aName, &aNewView, &aSilent });
- if( pRet && dynamic_cast< const SfxViewFrameItem *>( pRet ) != nullptr &&
+ if( dynamic_cast< const SfxViewFrameItem *>( pRet ) &&
static_cast<SfxViewFrameItem const *>(pRet)->GetFrame() &&
nullptr != ( pShell = static_cast<SfxViewFrameItem const *>(pRet)
->GetFrame()->GetObjectShell() ) )