Restore parent to 0 if an exception occurs while setting things up

[libs/gltk.git] / source / widget.cpp

diff --git a/source/widget.cpp b/source/widget.cpp
index 4b48e8ba11400d2f410790de92b8464e890b68b3..bbd2e8877af608b1255d21eb10d9b23d2506fba4 100644 (file)
--- a/source/widget.cpp
+++ b/source/widget.cpp
@@ -70,10 +70,20 @@ void Widget::set_parent(Container *p)
                throw hierarchy_error("widget already parented");
        else if(p==parent)
                return;
-       parent = p;
 
-       on_reparent();
-       update_style();
+       try
+       {
+               parent = p;
+
+               on_reparent();
+               update_style();
+       }
+       catch(...)
+       {
+               // The container has not yet added the widget as its child
+               parent = 0;
+               throw;
+       }
 }
 
 void Widget::set_style(const string &s)
@@ -160,24 +170,24 @@ void Widget::rebuild()
        }
 }
 
-void Widget::render() const
+void Widget::render(GL::Renderer &renderer) const
 {
        if(!style)
                throw logic_error(format("Attempt to render a widget with null style (class=\"%s\", style_name=\"%s\")", get_class(), style_name));
 
-       GL::MatrixStack::Push _pushm(GL::MatrixStack::modelview());
-       GL::MatrixStack::modelview() *= GL::Matrix::translation(geom.x, geom.y, 0);
+       GL::MatrixStack::Push _pushm(renderer.matrix_stack());
+       renderer.matrix_stack() *= GL::Matrix::translation(geom.x, geom.y, 0);
        const Style::PartSeq &parts = style->get_parts();
        list<CachedPart>::const_iterator j = cached_parts.begin();
        for(Style::PartSeq::const_iterator i=parts.begin(); (i!=parts.end() && j!=cached_parts.end()); ++i, ++j)
        {
                if(j->mesh && j->texture)
                {
-                       GL::Bind bind_tex(j->texture);
-                       j->mesh->draw();
+                       renderer.set_texture(j->texture);
+                       j->mesh->draw(renderer);
                }
                else if(!i->get_name().empty())
-                       render_special(*i);
+                       render_special(*i, renderer);
        }
 }