From d977e02ec6350115c39f03d588539e8bd423a1c3 Mon Sep 17 00:00:00 2001
From: Noel Grandin <noelgrandin@gmail.com>
Date: Thu, 9 Aug 2018 16:30:57 +0200
Subject: 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>
---
 sfx2/source/appl/appdde.cxx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'sfx2/source')

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() ) )
-- 
cgit