Only show user-presets in favorite sidebar
[ardour.git] / libs / ardour / smf_source.cc
index 34eec3b4a6c16acce77c4c9b04a26cd6c21d62e1..078113fb9b635bd7dde1da2bee60fae115579d5a 100644 (file)
 #include <errno.h>
 #include <regex.h>
 
-#include "pbd/pathscanner.h"
+#include "pbd/file_utils.h"
 #include "pbd/stl_delete.h"
 #include "pbd/strsplit.h"
 
-#include <glib/gstdio.h>
+#include "pbd/gstdio_compat.h"
 #include <glibmm/miscutils.h>
 #include <glibmm/fileutils.h>
 
 #include "evoral/Control.hpp"
 #include "evoral/SMF.hpp"
 
-#include "ardour/event_type_map.h"
+#include "ardour/debug.h"
+#include "ardour/midi_channel_filter.h"
 #include "ardour/midi_model.h"
 #include "ardour/midi_ring_buffer.h"
 #include "ardour/midi_state_tracker.h"
+#include "ardour/parameter_types.h"
 #include "ardour/session.h"
 #include "ardour/smf_source.h"
-#include "ardour/debug.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace ARDOUR;
 using namespace Glib;
 using namespace PBD;
 using namespace Evoral;
+using namespace std;
 
 /** Constructor used for new internal-to-session files.  File cannot exist. */
 SMFSource::SMFSource (Session& s, const string& path, Source::Flag flags)
@@ -58,8 +60,9 @@ SMFSource::SMFSource (Session& s, const string& path, Source::Flag flags)
        , MidiSource(s, path, flags)
        , FileSource(s, DataType::MIDI, path, string(), flags)
        , Evoral::SMF()
+       , _open (false)
        , _last_ev_time_beats(0.0)
-       , _last_ev_time_frames(0)
+       , _last_ev_time_samples(0)
        , _smf_last_read_end (0)
        , _smf_last_read_time (0)
 {
@@ -68,10 +71,12 @@ SMFSource::SMFSource (Session& s, const string& path, Source::Flag flags)
        if (init (_path, false)) {
                throw failed_constructor ();
        }
+
         assert (!Glib::file_test (_path, Glib::FILE_TEST_EXISTS));
        existence_check ();
 
+       _flags = Source::Flag (_flags | Empty);
+
        /* file is not opened until write */
 
        if (flags & Writable) {
@@ -91,23 +96,23 @@ SMFSource::SMFSource (Session& s, const string& path)
        , MidiSource(s, path, Source::Flag (0))
        , FileSource(s, DataType::MIDI, path, string(), Source::Flag (0))
        , Evoral::SMF()
+       , _open (false)
        , _last_ev_time_beats(0.0)
-       , _last_ev_time_frames(0)
+       , _last_ev_time_samples(0)
        , _smf_last_read_end (0)
        , _smf_last_read_time (0)
 {
        /* note that origin remains empty */
 
-       if (init (_path, false)) {
+       if (init (_path, true)) {
                throw failed_constructor ();
        }
+
         assert (Glib::file_test (_path, Glib::FILE_TEST_EXISTS));
        existence_check ();
 
-       /* file is not opened until write */
-
        if (_flags & Writable) {
+               /* file is not opened until write */
                return;
        }
 
@@ -123,8 +128,9 @@ SMFSource::SMFSource (Session& s, const XMLNode& node, bool must_exist)
        : Source(s, node)
        , MidiSource(s, node)
        , FileSource(s, node, must_exist)
+       , _open (false)
        , _last_ev_time_beats(0.0)
-       , _last_ev_time_frames(0)
+       , _last_ev_time_samples(0)
        , _smf_last_read_end (0)
        , _smf_last_read_time (0)
 {
@@ -132,14 +138,47 @@ SMFSource::SMFSource (Session& s, const XMLNode& node, bool must_exist)
                throw failed_constructor ();
        }
 
-       if (init (_path, true)) {
-               throw failed_constructor ();
+       /* we expect the file to exist, but if no MIDI data was ever added
+          it will have been removed at last session close. so, we don't
+          require it to exist if it was marked Empty.
+       */
+
+       try {
+
+               if (init (_path, true)) {
+                       throw failed_constructor ();
+               }
+
+       } catch (MissingSource& err) {
+
+               if (_flags & Source::Empty) {
+                       /* we don't care that the file was not found, because
+                          it was empty. But FileSource::init() will have
+                          failed to set our _path correctly, so we have to do
+                          this ourselves. Use the first entry in the search
+                          path for MIDI files, which is assumed to be the
+                          correct "main" location.
+                       */
+                       std::vector<string> sdirs = s.source_search_path (DataType::MIDI);
+                       _path = Glib::build_filename (sdirs.front(), _path);
+                       /* This might be important, too */
+                       _file_is_new = true;
+               } else {
+                       /* pass it on */
+                       throw;
+               }
        }
 
-        assert (Glib::file_test (_path, Glib::FILE_TEST_EXISTS));
-       existence_check ();
+       if (!(_flags & Source::Empty)) {
+               assert (Glib::file_test (_path, Glib::FILE_TEST_EXISTS));
+               existence_check ();
+       } else {
+               assert (_flags & Source::Writable);
+               /* file will be opened on write */
+               return;
+       }
 
-       if (open(_path)) {
+       if (open (_path)) {
                throw failed_constructor ();
        }
 
@@ -163,13 +202,22 @@ SMFSource::open_for_write ()
        return 0;
 }
 
-/** All stamps in audio frames */
-framecnt_t
-SMFSource::read_unlocked (Evoral::EventSink<framepos_t>& destination,
-                          framepos_t const               source_start,
-                          framepos_t                     start,
-                          framecnt_t                     duration,
-                          MidiStateTracker*              tracker) const
+void
+SMFSource::close ()
+{
+       /* nothing to do: file descriptor is never kept open */
+}
+
+/** All stamps in audio samples */
+samplecnt_t
+SMFSource::read_unlocked (const Lock&                    lock,
+                          Evoral::EventSink<samplepos_t>& destination,
+                          samplepos_t const               source_start,
+                          samplepos_t                     start,
+                          samplecnt_t                     duration,
+                          Evoral::Range<samplepos_t>*     loop_range,
+                          MidiStateTracker*              tracker,
+                          MidiChannelFilter*             filter) const
 {
        int      ret  = 0;
        uint64_t time = 0; // in SMF ticks, 1 tick per _ppqn
@@ -183,15 +231,14 @@ SMFSource::read_unlocked (Evoral::EventSink<framepos_t>& destination,
 
        // Output parameters for read_event (which will allocate scratch in buffer as needed)
        uint32_t ev_delta_t = 0;
-       uint32_t ev_type    = 0;
        uint32_t ev_size    = 0;
        uint8_t* ev_buffer  = 0;
 
        size_t scratch_size = 0; // keep track of scratch to minimize reallocs
 
-       BeatsFramesConverter converter(_session.tempo_map(), source_start);
+       BeatsSamplesConverter converter(_session.tempo_map(), source_start);
 
-       const uint64_t start_ticks = (uint64_t)(converter.from(start) * ppqn());
+       const uint64_t start_ticks = converter.from(start).to_ticks();
        DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked: start in ticks %1\n", start_ticks));
 
        if (_smf_last_read_end == 0 || start != _smf_last_read_end) {
@@ -229,26 +276,25 @@ SMFSource::read_unlocked (Evoral::EventSink<framepos_t>& destination,
                        continue;
                }
 
-               ev_type = EventTypeMap::instance().midi_event_type(ev_buffer[0]);
-
-               DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked delta %1, time %2, buf[0] %3, type %4\n",
-                                                                 ev_delta_t, time, ev_buffer[0], ev_type));
+               DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked delta %1, time %2, buf[0] %3\n",
+                                                                 ev_delta_t, time, ev_buffer[0]));
 
                assert(time >= start_ticks);
 
-               /* Note that we add on the source start time (in session frames) here so that ev_frame_time
-                  is in session frames.
+               /* Note that we add on the source start time (in session samples) here so that ev_sample_time
+                  is in session samples.
                */
-               const framepos_t ev_frame_time = converter.to(time / (double)ppqn()) + source_start;
+               const samplepos_t ev_sample_time = converter.to(Temporal::Beats::ticks_at_rate(time, ppqn())) + source_start;
 
-               if (ev_frame_time < start + duration) {
-                       destination.write (ev_frame_time, ev_type, ev_size, ev_buffer);
+               if (loop_range) {
+                       loop_range->squish (ev_sample_time);
+               }
 
-                       if (tracker) {
-                               if (ev_buffer[0] & MIDI_CMD_NOTE_ON) {
-                                       tracker->add (ev_buffer[1], ev_buffer[0] & 0xf);
-                               } else if (ev_buffer[0] & MIDI_CMD_NOTE_OFF) {
-                                       tracker->remove (ev_buffer[1], ev_buffer[0] & 0xf);
+               if (ev_sample_time < start + duration) {
+                       if (!filter || !filter->filter(ev_buffer, ev_size)) {
+                               destination.write (ev_sample_time, Evoral::MIDI_EVENT, ev_size, ev_buffer);
+                               if (tracker) {
+                                       tracker->track(ev_buffer);
                                }
                        }
                } else {
@@ -264,16 +310,17 @@ SMFSource::read_unlocked (Evoral::EventSink<framepos_t>& destination,
        return duration;
 }
 
-framecnt_t
-SMFSource::write_unlocked (MidiRingBuffer<framepos_t>& source,
-                           framepos_t                  position,
-                           framecnt_t                  cnt)
+samplecnt_t
+SMFSource::write_unlocked (const Lock&                 lock,
+                           MidiRingBuffer<samplepos_t>& source,
+                           samplepos_t                  position,
+                           samplecnt_t                  cnt)
 {
        if (!_writing) {
-               mark_streaming_write_started ();
+               mark_streaming_write_started (lock);
        }
 
-       framepos_t        time;
+       samplepos_t        time;
        Evoral::EventType type;
        uint32_t          size;
 
@@ -284,16 +331,16 @@ SMFSource::write_unlocked (MidiRingBuffer<framepos_t>& source,
                _model->start_write();
        }
 
-       Evoral::MIDIEvent<framepos_t> ev;
+       Evoral::Event<samplepos_t> ev;
        while (true) {
-               /* Get the event time, in frames since session start but ignoring looping. */
+               /* Get the event time, in samples since session start but ignoring looping. */
                bool ret;
                if (!(ret = source.peek ((uint8_t*)&time, sizeof (time)))) {
                        /* Ring is empty, no more events. */
                        break;
                }
 
-               if ((cnt != max_framecnt) &&
+               if ((cnt != max_samplecnt) &&
                    (time > position + _capture_length + cnt)) {
                        /* The diskstream doesn't want us to write everything, and this
                           event is past the end of this block, so we're done for now. */
@@ -325,16 +372,16 @@ SMFSource::write_unlocked (MidiRingBuffer<framepos_t>& source,
                        break;
                }
                time -= position;
-                       
+
                ev.set(buf, size, time);
-               ev.set_event_type(EventTypeMap::instance().midi_event_type(ev.buffer()[0]));
+               ev.set_event_type(Evoral::MIDI_EVENT);
                ev.set_id(Evoral::next_event_id());
 
                if (!(ev.is_channel_event() || ev.is_smf_meta_event() || ev.is_sysex())) {
                        continue;
                }
 
-               append_event_unlocked_frames(ev, position);
+               append_event_samples(lock, ev, position);
        }
 
        Evoral::SMF::flush ();
@@ -343,22 +390,37 @@ SMFSource::write_unlocked (MidiRingBuffer<framepos_t>& source,
        return cnt;
 }
 
-/** Append an event with a timestamp in beats (double) */
+/** Append an event with a timestamp in beats */
 void
-SMFSource::append_event_unlocked_beats (const Evoral::Event<double>& ev)
+SMFSource::append_event_beats (const Glib::Threads::Mutex::Lock&   lock,
+                               const Evoral::Event<Temporal::Beats>& ev)
 {
        if (!_writing || ev.size() == 0)  {
                return;
        }
 
-       /*printf("SMFSource: %s - append_event_unlocked_beats ID = %d time = %lf, size = %u, data = ",
+#if 0
+       printf("SMFSource: %s - append_event_beats ID = %d time = %lf, size = %u, data = ",
                name().c_str(), ev.id(), ev.time(), ev.size());
-              for (size_t i = 0; i < ev.size(); ++i) printf("%X ", ev.buffer()[i]); printf("\n");*/
+              for (size_t i = 0; i < ev.size(); ++i) printf("%X ", ev.buffer()[i]); printf("\n");
+#endif
 
-       if (ev.time() < _last_ev_time_beats) {
-               warning << string_compose(_("Skipping event with unordered time %1"), ev.time())
-                       << endmsg;
-               return;
+       Temporal::Beats time = ev.time();
+       if (time < _last_ev_time_beats) {
+               const Temporal::Beats difference = _last_ev_time_beats - time;
+               if (difference.to_double() / (double)ppqn() < 1.0) {
+                       /* Close enough.  This problem occurs because Sequence is not
+                          actually ordered due to fuzzy time comparison.  I'm pretty sure
+                          this is inherently a bad idea which causes problems all over the
+                          place, but tolerate it here for now anyway. */
+                       time = _last_ev_time_beats;
+               } else {
+                       /* Out of order by more than a tick. */
+                       warning << string_compose(_("Skipping event with unordered beat time %1 < %2 (off by %3 beats, %4 ticks)"),
+                                                 ev.time(), _last_ev_time_beats, difference, difference.to_double() / (double)ppqn())
+                               << endmsg;
+                       return;
+               }
        }
 
        Evoral::event_id_t event_id;
@@ -373,36 +435,40 @@ SMFSource::append_event_unlocked_beats (const Evoral::Event<double>& ev)
                _model->append (ev, event_id);
        }
 
-       _length_beats = max(_length_beats, ev.time());
+       _length_beats = max(_length_beats, time);
 
-       const double delta_time_beats   = ev.time() - _last_ev_time_beats;
-       const uint32_t delta_time_ticks = (uint32_t)lrint(delta_time_beats * (double)ppqn());
+       const Temporal::Beats delta_time_beats = time - _last_ev_time_beats;
+       const uint32_t      delta_time_ticks = delta_time_beats.to_ticks(ppqn());
 
        Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer(), event_id);
-       _last_ev_time_beats = ev.time();
+       _last_ev_time_beats = time;
+       _flags = Source::Flag (_flags & ~Empty);
 }
 
-/** Append an event with a timestamp in frames (framepos_t) */
+/** Append an event with a timestamp in samples (samplepos_t) */
 void
-SMFSource::append_event_unlocked_frames (const Evoral::Event<framepos_t>& ev, framepos_t position)
+SMFSource::append_event_samples (const Glib::Threads::Mutex::Lock& lock,
+                                const Evoral::Event<samplepos_t>&  ev,
+                                samplepos_t                        position)
 {
        if (!_writing || ev.size() == 0)  {
                return;
        }
 
-       // printf("SMFSource: %s - append_event_unlocked_frames ID = %d time = %u, size = %u, data = ",
+       // printf("SMFSource: %s - append_event_samples ID = %d time = %u, size = %u, data = ",
        // name().c_str(), ev.id(), ev.time(), ev.size());
        // for (size_t i=0; i < ev.size(); ++i) printf("%X ", ev.buffer()[i]); printf("\n");
 
-       if (ev.time() < _last_ev_time_frames) {
-               warning << string_compose(_("Skipping event with unordered time %1"), ev.time())
+       if (ev.time() < _last_ev_time_samples) {
+               warning << string_compose(_("Skipping event with unordered sample time %1 < %2"),
+                                         ev.time(), _last_ev_time_samples)
                        << endmsg;
                return;
        }
 
-       BeatsFramesConverter converter(_session.tempo_map(), position);
-       const double ev_time_beats = converter.from(ev.time());
-       Evoral::event_id_t event_id;
+       BeatsSamplesConverter converter(_session.tempo_map(), position);
+       const Temporal::Beats  ev_time_beats = converter.from(ev.time());
+       Evoral::event_id_t   event_id;
 
        if (ev.id() < 0) {
                event_id  = Evoral::next_event_id();
@@ -411,28 +477,29 @@ SMFSource::append_event_unlocked_frames (const Evoral::Event<framepos_t>& ev, fr
        }
 
        if (_model) {
-               const Evoral::Event<double> beat_ev (ev.event_type(),
-                                                    ev_time_beats,
-                                                    ev.size(),
-                                                    const_cast<uint8_t*>(ev.buffer()));
+               const Evoral::Event<Temporal::Beats> beat_ev (ev.event_type(),
+                                                           ev_time_beats,
+                                                           ev.size(),
+                                                           const_cast<uint8_t*>(ev.buffer()));
                _model->append (beat_ev, event_id);
        }
 
        _length_beats = max(_length_beats, ev_time_beats);
 
-       const Evoral::MusicalTime last_time_beats  = converter.from (_last_ev_time_frames);
-       const Evoral::MusicalTime delta_time_beats = ev_time_beats - last_time_beats;
-       const uint32_t            delta_time_ticks = (uint32_t)(lrint(delta_time_beats * (double)ppqn()));
+       const Temporal::Beats last_time_beats  = converter.from (_last_ev_time_samples);
+       const Temporal::Beats delta_time_beats = ev_time_beats - last_time_beats;
+       const uint32_t      delta_time_ticks = delta_time_beats.to_ticks(ppqn());
 
        Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer(), event_id);
-       _last_ev_time_frames = ev.time();
+       _last_ev_time_samples = ev.time();
+       _flags = Source::Flag (_flags & ~Empty);
 }
 
 XMLNode&
 SMFSource::get_state ()
 {
        XMLNode& node = MidiSource::get_state();
-       node.add_property (X_("origin"), _origin);
+       node.set_property (X_("origin"), _origin);
        return node;
 }
 
@@ -455,33 +522,30 @@ SMFSource::set_state (const XMLNode& node, int version)
 }
 
 void
-SMFSource::mark_streaming_midi_write_started (NoteMode mode)
+SMFSource::mark_streaming_midi_write_started (const Lock& lock, NoteMode mode)
 {
-       /* CALLER MUST HOLD LOCK */
-
        if (!_open && open_for_write()) {
                error << string_compose (_("cannot open MIDI file %1 for write"), _path) << endmsg;
                /* XXX should probably throw or return something */
                return;
        }
 
-       MidiSource::mark_streaming_midi_write_started (mode);
+       MidiSource::mark_streaming_midi_write_started (lock, mode);
        Evoral::SMF::begin_write ();
-       _last_ev_time_beats = 0.0;
-       _last_ev_time_frames = 0;
+       _last_ev_time_beats  = Temporal::Beats();
+       _last_ev_time_samples = 0;
 }
 
 void
-SMFSource::mark_streaming_write_completed ()
+SMFSource::mark_streaming_write_completed (const Lock& lock)
 {
-       mark_midi_streaming_write_completed (Evoral::Sequence<Evoral::MusicalTime>::DeleteStuckNotes);
+       mark_midi_streaming_write_completed (lock, Evoral::Sequence<Temporal::Beats>::DeleteStuckNotes);
 }
 
 void
-SMFSource::mark_midi_streaming_write_completed (Evoral::Sequence<Evoral::MusicalTime>::StuckNoteOption stuck_notes_option, Evoral::MusicalTime when)
+SMFSource::mark_midi_streaming_write_completed (const Lock& lm, Evoral::Sequence<Temporal::Beats>::StuckNoteOption stuck_notes_option, Temporal::Beats when)
 {
-       Glib::Threads::Mutex::Lock lm (_lock);
-       MidiSource::mark_midi_streaming_write_completed (stuck_notes_option, when);
+       MidiSource::mark_midi_streaming_write_completed (lm, stuck_notes_option, when);
 
        if (!writable()) {
                warning << string_compose ("attempt to write to unwritable SMF file %1", _path) << endmsg;
@@ -492,7 +556,11 @@ SMFSource::mark_midi_streaming_write_completed (Evoral::Sequence<Evoral::Musical
                _model->set_edited(false);
        }
 
-       Evoral::SMF::end_write ();
+       try {
+               Evoral::SMF::end_write (_path);
+       } catch (std::exception & e) {
+               error << string_compose (_("Exception while writing %1, file may be corrupt/unusable"), _path) << endmsg;
+       }
 
        /* data in the file now, not removable */
 
@@ -515,7 +583,7 @@ SMFSource::safe_midi_file_extension (const string& file)
        static bool compile = true;
        const int nmatches = 2;
        regmatch_t matches[nmatches];
-       
+
        if (Glib::file_test (file, Glib::FILE_TEST_EXISTS)) {
                if (!Glib::file_test (file, Glib::FILE_TEST_IS_REGULAR)) {
                        /* exists but is not a regular file */
@@ -528,7 +596,7 @@ SMFSource::safe_midi_file_extension (const string& file)
        } else {
                compile = false;
        }
-       
+
        if (regexec (&compiled_pattern, file.c_str(), nmatches, matches, 0)) {
                return false;
        }
@@ -537,22 +605,18 @@ SMFSource::safe_midi_file_extension (const string& file)
 }
 
 static bool compare_eventlist (
-               const std::pair< Evoral::Event<double>*, gint >& a,
-               const std::pair< Evoral::Event<double>*, gint >& b) {
+       const std::pair< const Evoral::Event<Temporal::Beats>*, gint >& a,
+       const std::pair< const Evoral::Event<Temporal::Beats>*, gint >& b) {
        return ( a.first->time() < b.first->time() );
 }
 
 void
-SMFSource::load_model (bool lock, bool force_reload)
+SMFSource::load_model (const Glib::Threads::Mutex::Lock& lock, bool force_reload)
 {
        if (_writing) {
                return;
        }
 
-       boost::shared_ptr<Glib::Threads::Mutex::Lock> lm;
-       if (lock)
-               lm = boost::shared_ptr<Glib::Threads::Mutex::Lock>(new Glib::Threads::Mutex::Lock(_lock));
-
        if (_model && !force_reload) {
                return;
        }
@@ -563,6 +627,8 @@ SMFSource::load_model (bool lock, bool force_reload)
                _model->clear();
        }
 
+       invalidate(lock);
+
        if (writable() && !_open) {
                return;
        }
@@ -571,7 +637,7 @@ SMFSource::load_model (bool lock, bool force_reload)
        Evoral::SMF::seek_to_start();
 
        uint64_t time = 0; /* in SMF ticks */
-       Evoral::Event<double> ev;
+       Evoral::Event<Temporal::Beats> ev;
 
        uint32_t scratch_size = 0; // keep track of scratch and minimize reallocs
 
@@ -583,7 +649,7 @@ SMFSource::load_model (bool lock, bool force_reload)
        bool have_event_id;
 
        // TODO simplify event allocation
-       std::list< std::pair< Evoral::Event<double>*, gint > > eventlist;
+       std::list< std::pair< Evoral::Event<Temporal::Beats>*, gint > > eventlist;
 
        for (unsigned i = 1; i <= num_tracks(); ++i) {
                if (seek_to_track(i)) continue;
@@ -609,8 +675,7 @@ SMFSource::load_model (bool lock, bool force_reload)
                                if (!have_event_id) {
                                        event_id = Evoral::next_event_id();
                                }
-                               uint32_t event_type = EventTypeMap::instance().midi_event_type(buf[0]);
-                               double   event_time = time / (double) ppqn();
+                               const Temporal::Beats event_time = Temporal::Beats::ticks_at_rate(time, ppqn());
 #ifndef NDEBUG
                                std::string ss;
 
@@ -620,13 +685,13 @@ SMFSource::load_model (bool lock, bool force_reload)
                                        ss += b;
                                }
 
-                               DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF %6 load model delta %1, time %2, size %3 buf %4, type %5\n",
-                                                       delta_t, time, size, ss , event_type, name()));
+                               DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF %7 load model delta %1, time %2, size %3 buf %4, id %6\n",
+                                                       delta_t, time, size, ss, event_id, name()));
 #endif
 
                                eventlist.push_back(make_pair (
-                                                       new Evoral::Event<double> (
-                                                               event_type, event_time,
+                                                       new Evoral::Event<Temporal::Beats> (
+                                                               Evoral::MIDI_EVENT, event_time,
                                                                size, buf, true)
                                                        , event_id));
 
@@ -644,58 +709,70 @@ SMFSource::load_model (bool lock, bool force_reload)
 
        eventlist.sort(compare_eventlist);
 
-       std::list< std::pair< Evoral::Event<double>*, gint > >::iterator it;
+       std::list< std::pair< Evoral::Event<Temporal::Beats>*, gint > >::iterator it;
        for (it=eventlist.begin(); it!=eventlist.end(); ++it) {
                _model->append (*it->first, it->second);
                delete it->first;
        }
 
-       _model->end_write (Evoral::Sequence<Evoral::MusicalTime>::ResolveStuckNotes, _length_beats);
-       _model->set_edited (false);
+        // cerr << "----SMF-SRC-----\n";
+        // _playback_buf->dump (cerr);
+        // cerr << "----------------\n";
 
-       _model_iter = _model->begin();
+       _model->end_write (Evoral::Sequence<Temporal::Beats>::ResolveStuckNotes, _length_beats);
+       _model->set_edited (false);
+       invalidate(lock);
 
        free(buf);
 }
 
 void
-SMFSource::destroy_model ()
+SMFSource::destroy_model (const Glib::Threads::Mutex::Lock& lock)
 {
        //cerr << _name << " destroying model " << _model.get() << endl;
        _model.reset();
+       invalidate(lock);
 }
 
 void
-SMFSource::flush_midi ()
+SMFSource::flush_midi (const Lock& lock)
 {
-       if (!writable() || (writable() && !_open)) {
+       if (!writable() || _length_beats == 0.0) {
                return;
        }
 
-       Evoral::SMF::end_write ();
+       ensure_disk_file (lock);
+
+       Evoral::SMF::end_write (_path);
        /* data in the file means its no longer removable */
        mark_nonremovable ();
+
+       invalidate(lock);
 }
 
 void
 SMFSource::set_path (const string& p)
 {
        FileSource::set_path (p);
-       SMF::set_path (_path);
 }
 
 /** Ensure that this source has some file on disk, even if it's just a SMF header */
 void
-SMFSource::ensure_disk_file ()
+SMFSource::ensure_disk_file (const Lock& lock)
 {
+       if (!writable()) {
+               return;
+       }
+
        if (_model) {
                /* We have a model, so write it to disk; see MidiSource::session_saved
                   for an explanation of what we are doing here.
                */
                boost::shared_ptr<MidiModel> mm = _model;
                _model.reset ();
-               mm->sync_to_source ();
+               mm->sync_to_source (lock);
                _model = mm;
+               invalidate(lock);
        } else {
                /* No model; if it's not already open, it's an empty source, so create
                   and open it for writing.
@@ -703,9 +780,6 @@ SMFSource::ensure_disk_file ()
                if (!_open) {
                        open_for_write ();
                }
-
-               /* Flush, which will definitely put something on disk */
-               flush_midi ();
        }
 }
 
@@ -715,38 +789,6 @@ SMFSource::prevent_deletion ()
        /* Unlike the audio case, the MIDI file remains mutable (because we can
           edit MIDI data)
        */
-  
-       _flags = Flag (_flags & ~(Removable|RemovableIfEmpty|RemoveAtDestroy));
-}
-
-int
-SMFSource::rename (const string& newname)
-{
-       Glib::Threads::Mutex::Lock lm (_lock);
-       string oldpath = _path;
-       string newpath = _session.new_source_path_from_name (DataType::MIDI, newname);
 
-       if (newpath.empty()) {
-               error << string_compose (_("programming error: %1"), "cannot generate a changed file path") << endmsg;
-               return -1;
-       }
-
-       // Test whether newpath exists, if yes notify the user but continue.
-       if (Glib::file_test (newpath, Glib::FILE_TEST_EXISTS)) {
-               error << string_compose (_("Programming error! %1 tried to rename a file over another file! It's safe to continue working, but please report this to the developers."), PROGRAM_NAME) << endmsg;
-               return -1;
-       }
-
-       if (Glib::file_test (oldpath.c_str(), Glib::FILE_TEST_EXISTS)) { 
-               /* rename only needed if file exists on disk */
-               if (::rename (oldpath.c_str(), newpath.c_str()) != 0) {
-                       error << string_compose (_("cannot rename file %1 to %2 (%3)"), oldpath, newpath, strerror(errno)) << endmsg;
-                       return -1;
-               }
-       }
-
-       _name = Glib::path_get_basename (newpath);
-       _path = newpath;
-
-       return 0;
+       _flags = Flag (_flags & ~(Removable|RemovableIfEmpty|RemoveAtDestroy));
 }