cleanup up cleanup at session destruction; clarify the meaning of 3 signals (DropReferences & Destroyed in libardour ; CatchDeletion in the GTK UI); clarify ownership of objects (session no longer pays attention to DropReferences for objects that it is considered to own, such as routes, sources, etc); fix up MIDI parsing and a couple of other places by correcting syntax for return of values from a boost::signals2::signal (possible danger elsewhere to be checked)

git-svn-id: svn://localhost/ardour2/branches/3.0@6389 d708f5d6-7413-0410-9779-e7cbd77b26cf

1 files changed, 5 insertions, 4 deletions

diff --git a/gtk2_ardour/editor_selection_list.cc b/gtk2_ardour/editor_selection_list.cc
index f3d11343bc..62b04d8efa 100644
--- a/gtk2_ardour/editor_selection_list.cc
+++ b/gtk2_ardour/editor_selection_list.cc
@@ -51,11 +51,11 @@ Editor::handle_new_named_selection ()
 }
 
 void
-Editor::add_named_selection_to_named_selection_display (NamedSelection& selection)
+Editor::add_named_selection_to_named_selection_display (boost::shared_ptr<NamedSelection> selection)
 {
         TreeModel::Row row = *(named_selection_model->append());
 	row[named_selection_columns.text] = selection.name;
-	row[named_selection_columns.selection] = &selection;
+	row[named_selection_columns.selection] = selection;
 }
 
 void
@@ -195,13 +195,14 @@ Editor::create_named_selection ()
 			return;
 		}
 
-		new NamedSelection (name, thelist); // creation will add it to the model
-
+		boost::shared_ptr<NamedSelection> ns (new NamedSelection (name, thelist));
+		
 		/* make the one we just added be selected */
 
 		TreeModel::Children::iterator added = named_selection_model->children().end();
 		--added;
 		named_selection_display.get_selection()->select (*added);
+
 	} else {
 		error << _("No selectable material found in the currently selected time range") << endmsg;
 	}