ardour/ardour
ardour/ardour
13
1
Fork 0
Code Pull Requests Releases Activity

do not delete GtkUIManager-generated menu from ProcessorBox when the PB is destroyed. read the comment for more details

Browse Source 
git-svn-id: svn://localhost/ardour2/branches/3.0@11008 d708f5d6-7413-0410-9779-e7cbd77b26cf
This commit is contained in:
Paul Davis 2011-12-15 02:19:21 +00:00
parent 96585e532c
commit 38707ea06b
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
gtk2_ardour/processor_box.cc
View File

@ -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