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

Detach submenu before its parent is destroyed. Fixes #3134.

Browse Source 
git-svn-id: svn://localhost/ardour2/branches/3.0@7106 d708f5d6-7413-0410-9779-e7cbd77b26cf
This commit is contained in:
Carl Hetherington 2010-05-16 18:04:47 +00:00
parent 2deba15863
commit 20724294c2
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
gtk2_ardour/midi_time_axis.cc
View File

@ -377,6 +377,17 @@ MidiTimeAxisView::build_automation_action_menu ()
{
using namespace Menu_Helpers;
/* If we have a controller menu, we need to detach it before
RouteTimeAxis::build_automation_action_menu destroys the
menu it is attached to. Otherwise GTK destroys
controller_menu's gobj, meaning that it can't be reattached
below. See bug #3134.
*/
if (controller_menu) {
detach_menu (*controller_menu);
}
RouteTimeAxisView::build_automation_action_menu ();
MenuList& automation_items = automation_action_menu->items();
@ -403,7 +414,6 @@ MidiTimeAxisView::build_automation_action_menu ()
*/
build_controller_menu ();
detach_menu (*controller_menu);
automation_items.push_back (SeparatorElem());
automation_items.push_back (MenuElem (_("Controllers"), *controller_menu));