diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-03 12:47:53 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-03 15:29:19 +0200 |
| commit | 6725311151403e5bf5bef44e08c35ac76527aba4 (patch) | |
| tree | 62dda4a35d0cc2b8f96be6063e9c66c28957a0f4 /sd | |
| parent | 0de15abda691b78bef38862f078e60d97da549a5 (diff) | |
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 <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sd')
| -rw-r--r-- | sd/source/ui/view/ToolBarManager.cxx | 5 |
1 files changed, 1 insertions, 4 deletions
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(); } } |