From 38707ea06b6b4c786c7679a2cba96811d4beef27 Mon Sep 17 00:00:00 2001
From: Paul Davis <paul@linuxaudiosystems.com>
Date: Thu, 15 Dec 2011 02:19:21 +0000
Subject: [PATCH] do not delete GtkUIManager-generated menu from ProcessorBox
 when the PB is destroyed. read the comment for more details

git-svn-id: svn://localhost/ardour2/branches/3.0@11008 d708f5d6-7413-0410-9779-e7cbd77b26cf
---
 gtk2_ardour/processor_box.cc | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/gtk2_ardour/processor_box.cc b/gtk2_ardour/processor_box.cc
index 7ad56cced3..3040061dc9 100644
--- a/gtk2_ardour/processor_box.cc
+++ b/gtk2_ardour/processor_box.cc
@@ -505,7 +505,15 @@ ProcessorBox::ProcessorBox (ARDOUR::Session* sess, boost::function<PluginSelecto
 
 ProcessorBox::~ProcessorBox ()
 {
-	delete processor_menu;
+	/* it may appear as if we should delete processor_menu but that is a
+	 * pointer to a widget owned by the UI Manager, and has potentially
+	 * be returned to many other ProcessorBoxes. GTK doesn't really make
+	 * clear the ownership of this widget, which is a menu and thus is
+	 * never packed into any container other than an implict GtkWindow.
+	 *
+	 * For now, until or if we ever get clarification over the ownership
+	 * story just let it continue to exist. At worst, its a small memory leak.
+	 */
 }
 
 void