From d40a80b3d06f3b98a1caca2be6a8e34791149ea4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vladim=C3=ADr=20Vondru=C5=A1?= Date: Wed, 15 Feb 2012 23:24:17 +0100 Subject: [PATCH] Fixed crash on deletion of Object children. setParent(0) modifies list of children in parent object, thus the for cycle will break. Curious that I didn't notice that before. --- src/Object.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Object.cpp b/src/Object.cpp index 1a05cc84a..d525489f1 100644 --- a/src/Object.cpp +++ b/src/Object.cpp @@ -74,8 +74,8 @@ Object::~Object() { setParent(nullptr); /* Delete all children */ - for(set::const_iterator it = _children.begin(); it != _children.end(); ++it) - delete *it; + while(!_children.empty()) + delete *_children.begin(); } Scene* Object::scene() const {