Plugin another menu memory leak.
authorRobin Gareus <robin@gareus.org>
Thu, 16 Mar 2017 00:50:30 +0000 (01:50 +0100)
committerRobin Gareus <robin@gareus.org>
Thu, 16 Mar 2017 01:36:48 +0000 (02:36 +0100)
gtk2_ardour/control_slave_ui.cc
gtk2_ardour/control_slave_ui.h

index 96ff49c3722eaf009f278d4836db8fad365d238b..8842590bb1d92658c487b66b2d9b373d5b39dc46 100644 (file)
@@ -44,6 +44,7 @@ using std::string;
 ControlSlaveUI::ControlSlaveUI (Session* s)
        : SessionHandlePtr (s)
        , initial_button (ArdourButton::default_elements)
+       , context_menu (0)
 {
        set_no_show_all (true);
 
@@ -59,6 +60,11 @@ ControlSlaveUI::ControlSlaveUI (Session* s)
        pack_start (initial_button, true, true);
 }
 
+ControlSlaveUI::~ControlSlaveUI ()
+{
+       delete context_menu;
+}
+
 void
 ControlSlaveUI::set_stripable (boost::shared_ptr<Stripable> s)
 {
@@ -177,8 +183,9 @@ ControlSlaveUI::vca_button_release (GdkEventButton* ev, uint32_t n)
                return true;
        }
 
-       Menu* menu = new Menu;
-       MenuList& items = menu->items();
+       delete context_menu;
+       context_menu = new Menu;
+       MenuList& items = context_menu->items();
        bool slaved = false;
 
        for (VCAList::iterator v = vcas.begin(); v != vcas.end(); ++v) {
@@ -212,7 +219,7 @@ ControlSlaveUI::vca_button_release (GdkEventButton* ev, uint32_t n)
        }
 
        if (!items.empty()) {
-               menu->popup (1, ev->time);
+               context_menu->popup (1, ev->time);
        }
 
        return true;
index 3443e1a79a8d57aee7af30e7ae66b84a814dab70..0c8de7a9728ba66b3a998e5ada4d278ff532257e 100644 (file)
@@ -43,6 +43,7 @@ class ControlSlaveUI : public Gtk::HBox, public ARDOUR::SessionHandlePtr
 {
    public:
        ControlSlaveUI (ARDOUR::Session*);
+       ~ControlSlaveUI ();
        void set_stripable (boost::shared_ptr<ARDOUR::Stripable>);
 
    private:
@@ -59,6 +60,8 @@ class ControlSlaveUI : public Gtk::HBox, public ARDOUR::SessionHandlePtr
        bool vca_button_release (GdkEventButton* ev, uint32_t n);
        void add_vca_button (boost::shared_ptr<ARDOUR::VCA>);
        void unassign_all ();
+
+       Gtk::Menu* context_menu;
 };
 
 #endif /* __gtk2_ardour_control_slave_ui_h__ */