Bugfix: IGUIElement::addChild now prevents setting an element as it's own...

Bugfix: IGUIElement::addChild now prevents setting an element as it's own child (fixes for example a crash in the GUI-editor).


git-svn-id: svn://svn.code.sf.net/p/irrlicht/code/trunk@4694 dfc29bdd-3216-0410-991c-e03cc46cb475

changes.txt

 --------------------------
 Changes in 1.9 (not yet released)
 
+- Bugfix: IGUIElement::addChild now prevents setting an element as it's own child.
 - Bugfix: getFont for xml-fonts now also works for fonts inside archives (thx @Neirdan for bugreport)
 - Added function irr::core::mergeFilename 
 - Add ISceneNodeAnimator::setStartTime/getStartTime to allow resetting movement animators.

include/IGUIElement.h

@@ -278,9 +278,9 @@ public:
 	//! Adds a GUI element as new child of this element.
 	virtual void addChild(IGUIElement* child)
 	{
-		addChildToEnd(child);
-		if (child)
+		if ( child && child != this )
 		{
+			addChildToEnd(child);
 			child->updateAbsolutePosition();
 		}
 	}