X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;ds=sidebyside;f=source%2Fwidget.cpp;h=1cb42e5cae15598cdd82ea25809913e59a8359e7;hb=c1faa54a3218b53757b8b55de0ff8aa64412253b;hp=17bc4e3c95c96a6aed1f3e5eb14129b6951fdd72;hpb=b92c878a286036af106e969a29b2689876aa5f65;p=libs%2Fgltk.git

diff --git a/source/widget.cpp b/source/widget.cpp
index 17bc4e3..1cb42e5 100644
--- a/source/widget.cpp
+++ b/source/widget.cpp
@@ -1,10 +1,3 @@
-/* $Id$
-
-This file is part of libmspgltk
-Copyright © 2007-2011  Mikko Rasa, Mikkosoft Productions
-Distributed under the LGPL
-*/
-
 #include <msp/gl/immediate.h>
 #include <msp/gl/matrix.h>
 #include <msp/gl/transform.h>
@@ -30,7 +23,11 @@ Widget::Widget():
 Widget::~Widget()
 {
 	if(parent)
-		parent->remove(*this);
+	{
+		Container *p = parent;
+		parent = 0;
+		p->remove(*this);
+	}
 }
 
 void Widget::set_position(int x, int y)
@@ -69,7 +66,9 @@ void Widget::set_geometry(const Geometry &g)
 void Widget::set_parent(Container *p)
 {
 	if(parent && p)
-		throw InvalidState("Widget is already in a Container");
+		throw hierarchy_error("widget already parented");
+	else if(p==parent)
+		return;
 	parent = p;
 
 	on_reparent();
@@ -128,9 +127,9 @@ void Widget::set_focusable(bool f)
 void Widget::set_focus()
 {
 	if(!parent)
-		throw InvalidState("No parent");
+		throw hierarchy_error("no parent");
 	if(!visible)
-		throw InvalidState("Can't set focus on invisible widget");
+		throw logic_error("!visible");
 
 	signal_request_focus.emit();
 }
@@ -138,7 +137,7 @@ void Widget::set_focus()
 void Widget::render() const
 {
 	if(!style)
-		throw InvalidState(format("Attempt to render a widget with null style (class=\"%s\", style_name=\"%s\")", get_class(), style_name));
+		throw logic_error(format("Attempt to render a widget with null style (class=\"%s\", style_name=\"%s\")", get_class(), style_name));
 
 	GL::push_matrix();
 	GL::translate(geom.x, geom.y, 0);