From 5ba447bdcd13ba3d7c27c8609f207910227e4ab6 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Tue, 7 Nov 2017 13:55:06 +0200 Subject: new loplugin simplifydynamiccast simplify dynamic_cast followed by static_cast Change-Id: I965afcf05d1675094cfde53d3590a0fd00f26279 Reviewed-on: https://gerrit.libreoffice.org/44460 Reviewed-by: Noel Grandin Tested-by: Noel Grandin --- framework/source/layoutmanager/toolbarlayoutmanager.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'framework') diff --git a/framework/source/layoutmanager/toolbarlayoutmanager.cxx b/framework/source/layoutmanager/toolbarlayoutmanager.cxx index dd29eecba748..1e4612726dec 100644 --- a/framework/source/layoutmanager/toolbarlayoutmanager.cxx +++ b/framework/source/layoutmanager/toolbarlayoutmanager.cxx @@ -917,13 +917,13 @@ long ToolbarLayoutManager::childWindowEvent( VclSimpleEvent const * pEvent ) // To enable toolbar controllers to change their image when a sub-toolbar function // is activated, we need this mechanism. We have NO connection between these toolbars // anymore! - if ( dynamic_cast< const VclWindowEvent* >(pEvent) != nullptr ) + if ( auto pWindowEvent = dynamic_cast< const VclWindowEvent* >(pEvent) ) { if ( pEvent->GetId() == VclEventId::ToolboxSelect ) { OUString aToolbarName; OUString aCommand; - ToolBox* pToolBox = getToolboxPtr( static_cast(pEvent)->GetWindow() ); + ToolBox* pToolBox = getToolboxPtr( pWindowEvent->GetWindow() ); if ( pToolBox ) { -- cgit