Fix some more Gtk::Menu memory leaks

1 files changed, 5 insertions, 10 deletions

diff --git a/gtk2_ardour/visibility_group.cc b/gtk2_ardour/visibility_group.cc
index 10d5f98b73..0090e39019 100644
--- a/gtk2_ardour/visibility_group.cc
+++ b/gtk2_ardour/visibility_group.cc
@@ -25,6 +25,7 @@
 #include "pbd/strsplit.h"
 #include "pbd/xml++.h"
 
+#include "ardour_ui.h"
 #include "visibility_group.h"
 
 #include "pbd/i18n.h"
@@ -69,17 +70,9 @@ VisibilityGroup::button_press_event (GdkEventButton* ev)
 		return false;
 	}
 
-	/* memory leak: Gtk::Menu* */
-	menu()->popup (1, ev->time);
-	return true;
-}
-
-Gtk::Menu*
-VisibilityGroup::menu ()
-{
 	using namespace Gtk::Menu_Helpers;
 
-	Gtk::Menu* m = Gtk::manage (new Gtk::Menu);
+	Gtk::Menu* m = ARDOUR_UI::instance()->shared_popup_menu ();
 	MenuList& items = m->items ();
 
 	for (vector<Member>::iterator i = _members.begin(); i != _members.end(); ++i) {
@@ -89,9 +82,11 @@ VisibilityGroup::menu ()
 		j->signal_activate().connect (sigc::bind (sigc::mem_fun (*this, &VisibilityGroup::toggle), i));
 	}
 
-	return m;
+	m->popup (1, ev->time);
+	return true;
 }
 
+
 /** @return true if the member should be visible, even taking into account any override functor */
 bool
 VisibilityGroup::should_actually_be_visible (Member const & m) const