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/object/object-set.cpp | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) (limited to 'src/object/object-set.cpp') diff --git a/src/object/object-set.cpp b/src/object/object-set.cpp index aebd0804b..bf41bb092 100644 --- a/src/object/object-set.cpp +++ b/src/object/object-set.cpp @@ -333,8 +333,8 @@ boost::optional ObjectSet::center() const { std::list const ObjectSet::perspList() { std::list pl; - for (std::list::iterator i = _3dboxes.begin(); i != _3dboxes.end(); ++i) { - Persp3D *persp = box3d_get_perspective(*i); + for (auto & _3dboxe : _3dboxes) { + Persp3D *persp = box3d_get_perspective(_3dboxe); if (std::find(pl.begin(), pl.end(), persp) == pl.end()) pl.push_back(persp); } @@ -344,8 +344,7 @@ std::list const ObjectSet::perspList() { std::list const ObjectSet::box3DList(Persp3D *persp) { std::list boxes; if (persp) { - for (std::list::iterator i = _3dboxes.begin(); i != _3dboxes.end(); ++i) { - SPBox3D *box = *i; + for (auto box : _3dboxes) { if (persp == box3d_get_perspective(box)) { boxes.push_back(box); } @@ -359,8 +358,7 @@ std::list const ObjectSet::box3DList(Persp3D *persp) { void ObjectSet::_add3DBoxesRecursively(SPObject *obj) { std::list boxes = box3d_extract_boxes(obj); - for (std::list::iterator i = boxes.begin(); i != boxes.end(); ++i) { - SPBox3D *box = *i; + for (auto box : boxes) { _3dboxes.push_back(box); } } @@ -368,8 +366,7 @@ void ObjectSet::_add3DBoxesRecursively(SPObject *obj) { void ObjectSet::_remove3DBoxesRecursively(SPObject *obj) { std::list boxes = box3d_extract_boxes(obj); - for (std::list::iterator i = boxes.begin(); i != boxes.end(); ++i) { - SPBox3D *box = *i; + for (auto box : boxes) { std::list::iterator b = std::find(_3dboxes.begin(), _3dboxes.end(), box); if (b == _3dboxes.end()) { g_print ("Warning! Trying to remove unselected box from selection.\n"); -- cgit v1.2.3