Use std::unique_ptr for managing memory

[libs/gltk.git] / source / container.h

diff --git a/source/container.h b/source/container.h
index a0cba3b776f8ffe3fce88443775bff461304d85b..6311a411a57fb1b16e03dcdd24564242a2ac5902 100644 (file)
--- a/source/container.h
+++ b/source/container.h
@@ -1,8 +1,9 @@
 #ifndef MSP_GLTK_CONTAINER_H_
 #define MSP_GLTK_CONTAINER_H_
 
-#include <list>
+#include <memory>
 #include <stdexcept>
+#include <vector>
 #include <sigc++/trackable.h>
 #include "mspgltk_api.h"
 #include "widget.h"
@@ -37,7 +38,7 @@ protected:
                void rebuild_needed();
        };
 
-       std::list<Child *> children;
+       std::vector<std::unique_ptr<Child>> children;
        Widget *click_focus = nullptr;
        unsigned click_button = 0;
        Widget *pointer_focus = nullptr;
@@ -54,14 +55,13 @@ public:
        void add(Widget &);
        void remove(Widget &);
 protected:
-       virtual Child *create_child(Widget *);
        Geometry determine_child_geometry(const Widget &, const Part &) const;
        void autosize_child(const Widget &, const Part &, Geometry &) const;
        void reposition_child(Widget &, const Part &) const;
 public:
-       std::list<Widget *> get_children() const;
-       Widget *get_child_at(int, int) const;
-       Widget *get_descendant_at(int, int) const;
+       std::vector<Widget *> get_children() const;
+       Widget *find_child_at(int, int) const;
+       Widget *find_descendant_at(int, int) const;
        void raise(Widget &);
 
 protected:
@@ -75,28 +75,28 @@ private:
        void check_animation_interval();
 
 protected:
-       virtual void rebuild_hierarchy();
+       void rebuild_hierarchy() override;
 
 public:
-       virtual void button_press(int, int, unsigned);
-       virtual void button_release(int, int, unsigned);
-       virtual void pointer_motion(int, int);
+       void button_press(int, int, unsigned) override;
+       void button_release(int, int, unsigned) override;
+       void pointer_motion(int, int) override;
 private:
        Widget *get_pointer_target(int, int, bool) const;
 public:
-       virtual void pointer_leave();
-       virtual void touch_press(int, int, unsigned);
-       virtual void touch_release(int, int, unsigned);
-       virtual void touch_motion(int, int, unsigned);
-       virtual bool key_press(unsigned, unsigned);
-       virtual bool key_release(unsigned, unsigned);
-       virtual bool character(wchar_t);
-       virtual void focus_in();
-       virtual void focus_out();
-       virtual bool navigate(Navigation);
-       virtual void animate(const Time::TimeDelta &);
+       void pointer_leave() override;
+       void touch_press(int, int, unsigned) override;
+       void touch_release(int, int, unsigned) override;
+       void touch_motion(int, int, unsigned) override;
+       bool key_press(unsigned, unsigned) override;
+       bool key_release(unsigned, unsigned) override;
+       bool character(wchar_t) override;
+       void focus_in() override;
+       void focus_out() override;
+       bool navigate(Navigation) override;
+       void animate(const Time::TimeDelta &) override;
 protected:
-       virtual void on_reparent();
+       void on_reparent() override;
        virtual void on_child_added(Widget &) { }
        virtual void on_child_removed(Widget &) { }
        virtual void on_input_focus_changed(Widget *);