From 169dff19d4da8d76e69b8e896aa25b0013639c03 Mon Sep 17 00:00:00 2001 From: Marc Jeanmougin Date: Wed, 2 Jan 2019 10:41:30 +0100 Subject: modernize loops --- src/ui/contextmenu.cpp | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) (limited to 'src/ui/contextmenu.cpp') diff --git a/src/ui/contextmenu.cpp b/src/ui/contextmenu.cpp index 7d57e78a8..486af0dda 100644 --- a/src/ui/contextmenu.cpp +++ b/src/ui/contextmenu.cpp @@ -93,11 +93,11 @@ ContextMenu::ContextMenu(SPDesktop *desktop, SPItem *item) : std::vector< SPItem * > down_items = _desktop->getDocument()->getItemsPartiallyInBox( _desktop->dkey, b, true, true, true, true); bool has_down_hidden = false; bool has_down_locked = false; - for(std::vector< SPItem * >::iterator down = down_items.begin(); down != down_items.end(); ++down){ - if((*down)->isHidden()) { + for(auto & down_item : down_items){ + if(down_item->isHidden()) { has_down_hidden = true; } - if((*down)->isLocked()) { + if(down_item->isLocked()) { has_down_locked = true; } } @@ -216,10 +216,10 @@ void ContextMenu::HideSelected() void ContextMenu::UnLockBelow(std::vector items) { _desktop->selection->clear(); - for(auto i=items.begin();i!=items.end(); ++i) { - if ((*i)->isLocked()) { - (*i)->setLocked(false); - _desktop->selection->add(*i); + for(auto & item : items) { + if (item->isLocked()) { + item->setLocked(false); + _desktop->selection->add(item); } } } @@ -227,10 +227,10 @@ void ContextMenu::UnLockBelow(std::vector items) void ContextMenu::UnHideBelow(std::vector items) { _desktop->selection->clear(); - for(auto i=items.begin();i!=items.end(); ++i) { - if ((*i)->isHidden()) { - (*i)->setHidden(false); - _desktop->selection->add(*i); + for(auto & item : items) { + if (item->isHidden()) { + item->setHidden(false); + _desktop->selection->add(item); } } } -- cgit v1.2.3