better handling of the inverse-push of solo-by-upstream. still not quite right, but...

[ardour.git] / libs / ardour / midi_patch_manager.cc

diff --git a/libs/ardour/midi_patch_manager.cc b/libs/ardour/midi_patch_manager.cc
index a0ccbdec9edead484fe9b772d272e2ef53a49b5e..a2522fe997099ba2fec55a04ef15156f070632c5 100644 (file)
--- a/libs/ardour/midi_patch_manager.cc
+++ b/libs/ardour/midi_patch_manager.cc
@@ -18,7 +18,6 @@
     $Id$
 */
 
-#include <sigc++/sigc++.h>
 #include <boost/shared_ptr.hpp>
 
 #include "pbd/compose.h"
@@ -32,7 +31,6 @@
 #include "i18n.h"
 
 using namespace std;
-using namespace sigc;
 using namespace ARDOUR;
 using namespace MIDI;
 using namespace MIDI::Name;
@@ -42,11 +40,9 @@ using namespace PBD::sys;
 MidiPatchManager* MidiPatchManager::_manager = 0;
 
 void
-MidiPatchManager::set_session (Session& s)
+MidiPatchManager::set_session (Session* s)
 {
-       _session = &s;
-       _session->GoingAway.connect (mem_fun (*this, &MidiPatchManager::drop_session));
-
+       SessionHandlePtr::set_session (s);
        refresh();
 }
 
@@ -57,6 +53,10 @@ MidiPatchManager::refresh()
        _master_devices_by_model.clear();
        _all_models.clear();
 
+       if (!_session) {
+               return;
+       }
+
        path path_to_patches = _session->session_directory().midi_patch_path();
 
        if (!exists(path_to_patches)) {
@@ -70,7 +70,7 @@ MidiPatchManager::refresh()
 
        find_matching_files_in_directory(path_to_patches, pattern, result);
 
-       cerr << "Loading " << result.size() << " MIDI patches from " << path_to_patches.to_string() << endl;
+       info << "Loading " << result.size() << " MIDI patches from " << path_to_patches.to_string() << endl;
 
        for (vector<path>::iterator i = result.begin(); i != result.end(); ++i) {
                boost::shared_ptr<MIDINameDocument> document(new MIDINameDocument(i->to_string()));
@@ -94,10 +94,8 @@ MidiPatchManager::refresh()
 }
 
 void
-MidiPatchManager::drop_session ()
+MidiPatchManager::session_going_away ()
 {
-       _session = 0;
-       _documents.clear();
-       _master_devices_by_model.clear();
-       _all_models.clear();
+       SessionHandlePtr::session_going_away ();
+       refresh ();
 }