From 6725311151403e5bf5bef44e08c35ac76527aba4 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Wed, 3 Oct 2018 12:47:53 +0200 Subject: no need to call empty() before clear() found with git grep -A2 -nP '!.*\.empty()' | grep -B1 -w clear Change-Id: I87013eab9c6988048b891d20577e1f7efbce1b6c Reviewed-on: https://gerrit.libreoffice.org/61295 Tested-by: Jenkins Reviewed-by: Noel Grandin --- sd/source/ui/view/ToolBarManager.cxx | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'sd/source') diff --git a/sd/source/ui/view/ToolBarManager.cxx b/sd/source/ui/view/ToolBarManager.cxx index 607249f81023..d82b529ea24c 100644 --- a/sd/source/ui/view/ToolBarManager.cxx +++ b/sd/source/ui/view/ToolBarManager.cxx @@ -1203,10 +1203,7 @@ void ToolBarList::ClearGroup (sd::ToolBarManager::ToolBarGroup eGroup) Groups::iterator iGroup (maGroups.find(eGroup)); if (iGroup != maGroups.end()) { - if ( ! iGroup->second.empty()) - { - iGroup->second.clear(); - } + iGroup->second.clear(); } } -- cgit v1.2.3