Merge branch 'windows' of git.ardour.org:ardour/ardour into windows
[ardour.git] / libs / ardour / smf_source.cc
index 65759e7037c98e02918479fa696c236cc332fea0..d915bba8456025805284055bbd70604c12077a2c 100644 (file)
@@ -1,6 +1,6 @@
 /*
-    Copyright (C) 2006 Paul Davis 
-       Written by Dave Robillard, 2006
+    Copyright (C) 2006 Paul Davis
+    Author: David Robillard
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 #include <sys/stat.h>
 #include <unistd.h>
 #include <errno.h>
+#include <regex.h>
 
-#include "pbd/mountpoint.h"
 #include "pbd/pathscanner.h"
 #include "pbd/stl_delete.h"
 #include "pbd/strsplit.h"
 
+#include <glib/gstdio.h>
 #include <glibmm/miscutils.h>
 
-#include "evoral/SMFReader.hpp"
 #include "evoral/Control.hpp"
 
-#include "ardour/audioengine.h"
 #include "ardour/event_type_map.h"
 #include "ardour/midi_model.h"
 #include "ardour/midi_ring_buffer.h"
+#include "ardour/midi_state_tracker.h"
 #include "ardour/session.h"
 #include "ardour/smf_source.h"
+#include "ardour/debug.h"
 
 #include "i18n.h"
 
 using namespace ARDOUR;
 using namespace Glib;
+using namespace PBD;
 
 /** Constructor used for new internal-to-session files.  File cannot exist. */
-SMFSource::SMFSource (Session& s, const ustring& path, bool embedded, Source::Flag flags)
+SMFSource::SMFSource (Session& s, const string& path, Source::Flag flags)
        : Source(s, DataType::MIDI, path, flags)
-       , MidiSource(s, path)
-       , FileSource(s, DataType::MIDI, path, embedded, flags)
+       , MidiSource(s, path, flags)
+       , FileSource(s, DataType::MIDI, path, string(), flags)
        , Evoral::SMF()
        , _last_ev_time_beats(0.0)
        , _last_ev_time_frames(0)
        , _smf_last_read_end (0)
+       , _smf_last_read_time (0)
 {
-       if (init(_name, false)) {
-               throw failed_constructor ();
-       }
-       
-       if (create(path)) {
+       /* note that origin remains empty */
+
+       if (init(_path, false)) {
                throw failed_constructor ();
        }
+
+       /* file is not opened until write */
 }
 
 /** Constructor used for existing internal-to-session files. */
@@ -74,37 +77,57 @@ SMFSource::SMFSource (Session& s, const XMLNode& node, bool must_exist)
        , _last_ev_time_beats(0.0)
        , _last_ev_time_frames(0)
        , _smf_last_read_end (0)
+       , _smf_last_read_time (0)
 {
-       if (set_state(node)) {
+       if (set_state(node, Stateful::loading_state_version)) {
                throw failed_constructor ();
        }
-       
-       if (init(_name, true)) {
+
+       if (init(_path, true)) {
                throw failed_constructor ();
        }
-       
+
        if (open(_path)) {
                throw failed_constructor ();
        }
+
+       _open = true;
 }
 
 SMFSource::~SMFSource ()
 {
        if (removable()) {
-               unlink (_path.c_str());
+               ::g_unlink (_path.c_str());
        }
 }
 
+int
+SMFSource::open_for_write ()
+{
+       if (create (_path)) {
+               return -1;
+       }
+       _open = true;
+       return 0;
+}
+
 /** All stamps in audio frames */
-nframes_t
-SMFSource::read_unlocked (MidiRingBuffer<nframes_t>& destination, sframes_t source_start,
-               sframes_t start, nframes_t duration,
-               sframes_t stamp_offset, sframes_t negative_stamp_offset) const
+framecnt_t
+SMFSource::read_unlocked (Evoral::EventSink<framepos_t>& destination,
+                          framepos_t const               source_start,
+                          framepos_t                     start,
+                          framecnt_t                     duration,
+                          MidiStateTracker*              tracker) const
 {
        int      ret  = 0;
        uint64_t time = 0; // in SMF ticks, 1 tick per _ppqn
 
-       _read_data_count = 0;
+       if (writable() && !_open) {
+               /* nothing to read since nothing has ben written */
+               return duration;
+       }
+
+       DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked: start %1 duration %2\n", start, duration));
 
        // Output parameters for read_event (which will allocate scratch in buffer as needed)
        uint32_t ev_delta_t = 0;
@@ -113,218 +136,266 @@ SMFSource::read_unlocked (MidiRingBuffer<nframes_t>& destination, sframes_t sour
        uint8_t* ev_buffer  = 0;
 
        size_t scratch_size = 0; // keep track of scratch to minimize reallocs
-       
-       BeatsFramesConverter converter(_session, source_start);
+
+       BeatsFramesConverter converter(_session.tempo_map(), source_start);
 
        const uint64_t start_ticks = (uint64_t)(converter.from(start) * ppqn());
+       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) {
-               //cerr << "SMFSource::read_unlocked seeking to " << start << endl;
+       if (_smf_last_read_end == 0 || start != _smf_last_read_end) {
+               DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked: seek to %1\n", start));
                Evoral::SMF::seek_to_start();
                while (time < start_ticks) {
-                       ret = read_event(&ev_delta_t, &ev_size, &ev_buffer);
+                       gint ignored;
+
+                       ret = read_event(&ev_delta_t, &ev_size, &ev_buffer, &ignored);
                        if (ret == -1) { // EOF
                                _smf_last_read_end = start + duration;
                                return duration;
                        }
                        time += ev_delta_t; // accumulate delta time
                }
-//     }
-       
+       } else {
+               DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked: set time to %1\n", _smf_last_read_time));
+               time = _smf_last_read_time;
+       }
+
        _smf_last_read_end = start + duration;
 
        while (true) {
-               ret = read_event(&ev_delta_t, &ev_size, &ev_buffer);
+               gint ignored; /* XXX don't ignore note id's ??*/
+
+               ret = read_event(&ev_delta_t, &ev_size, &ev_buffer, &ignored);
                if (ret == -1) { // EOF
                        break;
                }
-               
+
                time += ev_delta_t; // accumulate delta time
+               _smf_last_read_time = time;
 
                if (ret == 0) { // meta-event (skipped, just accumulate time)
                        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));
+
                assert(time >= start_ticks);
-               const sframes_t ev_frame_time = converter.to(time / (double)ppqn()) + stamp_offset;
+
+               /* Note that we add on the source start time (in session frames) here so that ev_frame_time
+                  is in session frames.
+               */
+               const framepos_t ev_frame_time = converter.to(time / (double)ppqn()) + source_start;
 
                if (ev_frame_time < start + duration) {
-                       destination.write(ev_frame_time - negative_stamp_offset, ev_type, ev_size, ev_buffer);
+                       destination.write (ev_frame_time, ev_type, ev_size, ev_buffer);
+
+                       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);
+                               }
+                       }
                } else {
                        break;
                }
 
-               _read_data_count += ev_size;
-
                if (ev_size > scratch_size) {
                        scratch_size = ev_size;
                }
                ev_size = scratch_size; // ensure read_event only allocates if necessary
        }
-       
+
        return duration;
 }
 
-/** All stamps in audio frames */
-nframes_t
-SMFSource::write_unlocked (MidiRingBuffer<nframes_t>& source, sframes_t position, nframes_t duration)
+framecnt_t
+SMFSource::write_unlocked (MidiRingBuffer<framepos_t>& source,
+                           framepos_t                  position,
+                           framecnt_t                  cnt)
 {
-       _write_data_count = 0;
-               
-       nframes_t         time;
+       if (!_writing) {
+               mark_streaming_write_started ();
+       }
+
+       framepos_t        time;
        Evoral::EventType type;
        uint32_t          size;
 
        size_t   buf_capacity = 4;
        uint8_t* buf          = (uint8_t*)malloc(buf_capacity);
-       
-       if (_model && ! _model->writing()) {
+
+       if (_model && !_model->writing()) {
                _model->start_write();
        }
 
-       Evoral::MIDIEvent<nframes_t> ev;
-
+       Evoral::MIDIEvent<framepos_t> ev;
        while (true) {
-               bool ret = source.peek_time(&time);
-               if (!ret || time > _last_write_end + duration) {
+               /* Get the event time, in frames since session start but ignoring looping. */
+               bool ret;
+               if (!(ret = source.peek ((uint8_t*)&time, sizeof (time)))) {
+                       /* Ring is empty, no more events. */
                        break;
                }
 
-               ret = source.read_prefix(&time, &type, &size);
-               if (!ret) {
-                       cerr << "ERROR: Unable to read event prefix, corrupt MIDI ring buffer" << endl;
+               if ((cnt != max_framecnt) &&
+                   (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. */
                        break;
                }
 
+               /* Read the time, type, and size of the event. */
+               if (!(ret = source.read_prefix (&time, &type, &size))) {
+                       error << _("Unable to read event prefix, corrupt MIDI ring") << endmsg;
+                       break;
+               }
+
+               /* Enlarge body buffer if necessary now that we know the size. */
                if (size > buf_capacity) {
                        buf_capacity = size;
                        buf = (uint8_t*)realloc(buf, size);
                }
 
+               /* Read the event body into buffer. */
                ret = source.read_contents(size, buf);
                if (!ret) {
-                       cerr << "ERROR: Read time/size but not buffer, corrupt MIDI ring buffer" << endl;
+                       error << _("Event has time and size but no body, corrupt MIDI ring") << endmsg;
+                       break;
+               }
+
+               /* Convert event time from absolute to source relative. */
+               if (time < position) {
+                       error << _("Event time is before MIDI source position") << endmsg;
                        break;
                }
-               
-               assert(time >= position);
                time -= position;
-               
+                       
                ev.set(buf, size, time);
                ev.set_event_type(EventTypeMap::instance().midi_event_type(ev.buffer()[0]));
+               ev.set_id(Evoral::next_event_id());
+
                if (!(ev.is_channel_event() || ev.is_smf_meta_event() || ev.is_sysex())) {
-                       cerr << "SMFSource: WARNING: caller tried to write non SMF-Event of type "
-                                       << std::hex << int(ev.buffer()[0]) << endl;
                        continue;
                }
-               
-               append_event_unlocked_frames(ev, position);
-       }
 
-       if (_model) {
-               set_default_controls_interpolation();
+               append_event_unlocked_frames(ev, position);
        }
 
-       Evoral::SMF::flush();
-       free(buf);
+       Evoral::SMF::flush ();
+       free (buf);
 
-       ViewDataRangeReady(position + _last_write_end, duration); /* EMIT SIGNAL */
-
-       return duration;
+       return cnt;
 }
-               
 
 /** Append an event with a timestamp in beats (double) */
 void
 SMFSource::append_event_unlocked_beats (const Evoral::Event<double>& ev)
 {
-       if (ev.size() == 0)  {
+       if (!_writing || ev.size() == 0)  {
                return;
        }
 
-       /*printf("SMFSource: %s - append_event_unlocked_beats time = %lf, size = %u, data = ",
-                       name().c_str(), ev.time(), ev.size()); 
-       for (size_t i = 0; i < ev.size(); ++i) printf("%X ", ev.buffer()[i]); printf("\n");*/
-       
-       assert(ev.time() >= 0);
+       /*printf("SMFSource: %s - append_event_unlocked_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");*/
+
        if (ev.time() < _last_ev_time_beats) {
-               cerr << "SMFSource: Warning: Skipping event with non-monotonic time" << endl;
+               warning << string_compose(_("Skipping event with unordered time %1"), ev.time())
+                       << endmsg;
                return;
        }
-       
-       _length_beats = max(_length_beats, ev.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());
-       
-       Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer());
-       _last_ev_time_beats = ev.time();
 
-       _write_data_count += ev.size();
+       Evoral::event_id_t event_id;
+
+       if (ev.id() < 0) {
+               event_id  = Evoral::next_event_id();
+       } else {
+               event_id = ev.id();
+       }
 
        if (_model) {
-               _model->append(ev);
+               _model->append (ev, event_id);
        }
+
+       _length_beats = max(_length_beats, ev.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());
+
+       Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer(), event_id);
+       _last_ev_time_beats = ev.time();
 }
 
-/** Append an event with a timestamp in frames (nframes_t) */
+/** Append an event with a timestamp in frames (framepos_t) */
 void
-SMFSource::append_event_unlocked_frames (const Evoral::Event<nframes_t>& ev, sframes_t position)
+SMFSource::append_event_unlocked_frames (const Evoral::Event<framepos_t>& ev, framepos_t position)
 {
-       if (ev.size() == 0)  {
+       if (!_writing || ev.size() == 0)  {
                return;
        }
 
-       /*printf("SMFSource: %s - append_event_unlocked_frames time = %u, size = %u, data = ",
-                       name().c_str(), ev.time(), ev.size()); 
-       for (size_t i=0; i < ev.size(); ++i) printf("%X ", ev.buffer()[i]); printf("\n");*/
-       
+       // printf("SMFSource: %s - append_event_unlocked_frames 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) {
-               cerr << "SMFSource: Warning: Skipping event with non-monotonic time" << endl;
+               warning << string_compose(_("Skipping event with unordered time %1"), ev.time())
+                       << endmsg;
                return;
        }
-       
-       BeatsFramesConverter converter(_session, position);
-       
-       _length_beats = max(_length_beats, converter.from(ev.time()));
-       
-       const sframes_t delta_time_frames = ev.time() - _last_ev_time_frames;
-       const double    delta_time_beats  = converter.from(delta_time_frames);
-       const uint32_t  delta_time_ticks  = (uint32_t)(lrint(delta_time_beats * (double)ppqn()));
 
-       Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer());
-       _last_ev_time_frames = ev.time();
+       BeatsFramesConverter converter(_session.tempo_map(), position);
+       const double ev_time_beats = converter.from(ev.time());
+       Evoral::event_id_t event_id;
 
-       _write_data_count += ev.size();
+       if (ev.id() < 0) {
+               event_id  = Evoral::next_event_id();
+       } else {
+               event_id = ev.id();
+       }
 
        if (_model) {
-               const double ev_time_beats = converter.from(ev.time());
-               const Evoral::Event<double> beat_ev(
-                               ev.event_type(), ev_time_beats, ev.size(), (uint8_t*)ev.buffer());
-               _model->append(beat_ev);
+               const Evoral::Event<double> 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()));
+
+       Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer(), event_id);
+       _last_ev_time_frames = ev.time();
 }
 
 XMLNode&
 SMFSource::get_state ()
 {
-       return MidiSource::get_state();
+       XMLNode& node = MidiSource::get_state();
+       node.add_property (X_("origin"), _origin);
+       return node;
 }
 
 int
-SMFSource::set_state (const XMLNode& node)
+SMFSource::set_state (const XMLNode& node, int version)
 {
-       if (Source::set_state (node)) {
+       if (Source::set_state (node, version)) {
                return -1;
        }
 
-       if (MidiSource::set_state (node)) {
+       if (MidiSource::set_state (node, version)) {
                return -1;
        }
-       
-       if (FileSource::set_state (node)) {
+
+       if (FileSource::set_state (node, version)) {
                return -1;
        }
 
@@ -332,9 +403,17 @@ SMFSource::set_state (const XMLNode& node)
 }
 
 void
-SMFSource::mark_streaming_midi_write_started (NoteMode mode, sframes_t start_frame)
+SMFSource::mark_streaming_midi_write_started (NoteMode mode)
 {
-       MidiSource::mark_streaming_midi_write_started (mode, start_frame);
+       /* 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);
        Evoral::SMF::begin_write ();
        _last_ev_time_beats = 0.0;
        _last_ev_time_frames = 0;
@@ -343,20 +422,50 @@ SMFSource::mark_streaming_midi_write_started (NoteMode mode, sframes_t start_fra
 void
 SMFSource::mark_streaming_write_completed ()
 {
-       MidiSource::mark_streaming_write_completed();
+       mark_midi_streaming_write_completed (Evoral::Sequence<Evoral::MusicalTime>::DeleteStuckNotes);
+}
+
+void
+SMFSource::mark_midi_streaming_write_completed (Evoral::Sequence<Evoral::MusicalTime>::StuckNoteOption stuck_notes_option, Evoral::MusicalTime when)
+{
+       Glib::Threads::Mutex::Lock lm (_lock);
+       MidiSource::mark_midi_streaming_write_completed (stuck_notes_option, when);
 
        if (!writable()) {
+               warning << string_compose ("attempt to write to unwritable SMF file %1", _path) << endmsg;
                return;
        }
-       
-       _model->set_edited(false);
+
+       if (_model) {
+               _model->set_edited(false);
+       }
+
        Evoral::SMF::end_write ();
+
+       /* data in the file now, not removable */
+
+       mark_nonremovable ();
 }
 
 bool
-SMFSource::safe_midi_file_extension (const Glib::ustring& file)
+SMFSource::safe_midi_file_extension (const string& file)
 {
-       return (file.rfind(".mid") != Glib::ustring::npos);
+       static regex_t compiled_pattern;
+       static bool compile = true;
+       const int nmatches = 2;
+       regmatch_t matches[nmatches];
+       
+       if (compile && regcomp (&compiled_pattern, "[mM][iI][dD][iI]?$", REG_EXTENDED)) {
+               return false;
+       } else {
+               compile = false;
+       }
+       
+       if (regexec (&compiled_pattern, file.c_str(), nmatches, matches, 0)) {
+               return false;
+       }
+
+       return true;
 }
 
 void
@@ -365,92 +474,150 @@ SMFSource::load_model (bool lock, bool force_reload)
        if (_writing) {
                return;
        }
-       
-       if (lock) {
-               Glib::Mutex::Lock lm (_lock);
-       }
+
+       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;
        }
 
-       if (! _model) {
-               _model = boost::shared_ptr<MidiModel>(new MidiModel(this));
-               //cerr << _name << " loaded new model " << _model.get() << endl;
+       if (!_model) {
+               _model = boost::shared_ptr<MidiModel> (new MidiModel (shared_from_this ()));
        } else {
-               /*cerr << _name << " reloading model " << _model.get()
-                       << " (" << _model->n_notes() << " notes)" << endl;*/
                _model->clear();
        }
 
+       if (writable() && !_open) {
+               return;
+       }
+
        _model->start_write();
        Evoral::SMF::seek_to_start();
 
        uint64_t time = 0; /* in SMF ticks */
        Evoral::Event<double> ev;
-       
-       size_t scratch_size = 0; // keep track of scratch and minimize reallocs
-       
+
+       uint32_t scratch_size = 0; // keep track of scratch and minimize reallocs
+
        uint32_t delta_t = 0;
        uint32_t size    = 0;
        uint8_t* buf     = NULL;
        int ret;
-       while ((ret = read_event(&delta_t, &size, &buf)) >= 0) {
+       gint event_id;
+       bool have_event_id = false;
+
+       while ((ret = read_event (&delta_t, &size, &buf, &event_id)) >= 0) {
+
                time += delta_t;
-               ev.set(buf, size, time / (double)ppqn());
 
-               if (ret > 0) { // didn't skip (meta) event
-                       ev.set_event_type(EventTypeMap::instance().midi_event_type(buf[0]));
-                       _model->append(ev);
+               if (ret == 0) {
+
+                       /* meta-event : did we get an event ID ?
+                        */
+
+                       if (event_id >= 0) {
+                               have_event_id = true;
+                       }
+
+                       continue;
                }
 
-               if (ev.size() > scratch_size) {
-                       scratch_size = ev.size();
+               if (ret > 0) {
+
+                       /* not a meta-event */
+
+                       ev.set (buf, size, time / (double)ppqn());
+                       ev.set_event_type(EventTypeMap::instance().midi_event_type(buf[0]));
+
+                       if (!have_event_id) {
+                               event_id = Evoral::next_event_id();
+                       }
+#ifndef NDEBUG
+                       std::string ss;
+
+                       for (uint32_t xx = 0; xx < size; ++xx) {
+                               char b[8];
+                               snprintf (b, sizeof (b), "0x%x ", buf[xx]);
+                               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 , ev.event_type(), name()));
+#endif
+
+                       _model->append (ev, event_id);
+
+                       // Set size to max capacity to minimize allocs in read_event
+                       scratch_size = std::max(size, scratch_size);
+                       size = scratch_size;
+
+                       _length_beats = max(_length_beats, ev.time());
                }
-               ev.size() = scratch_size; // ensure read_event only allocates if necessary
-               
-               _length_beats = max(_length_beats, ev.time());
+
+               /* event ID's must immediately precede the event they are for
+                */
+
+               have_event_id = false;
        }
 
-       set_default_controls_interpolation();
-       
-       _model->end_write(false);
-       _model->set_edited(false);
+       _model->end_write (Evoral::Sequence<Evoral::MusicalTime>::ResolveStuckNotes, _length_beats);
+       _model->set_edited (false);
 
        _model_iter = _model->begin();
 
        free(buf);
 }
 
-#define LINEAR_INTERPOLATION_MODE_WORKS_PROPERLY 0
+void
+SMFSource::destroy_model ()
+{
+       //cerr << _name << " destroying model " << _model.get() << endl;
+       _model.reset();
+}
 
 void
-SMFSource::set_default_controls_interpolation ()
+SMFSource::flush_midi ()
 {
-       // set interpolation style to defaults, can be changed by the GUI later
-       Evoral::ControlSet::Controls controls = _model->controls();
-       for (Evoral::ControlSet::Controls::iterator c = controls.begin(); c != controls.end(); ++c) {
-               (*c).second->list()->set_interpolation(
-                       // to be enabled when ControlList::rt_safe_earliest_event_linear_unlocked works properly
-                       #if LINEAR_INTERPOLATION_MODE_WORKS_PROPERLY
-                       EventTypeMap::instance().interpolation_of((*c).first));
-                       #else
-                       Evoral::ControlList::Discrete);
-                       #endif
+       if (!writable() || (writable() && !_open)) {
+               return;
        }
-}
 
+       Evoral::SMF::end_write ();
+       /* data in the file means its no longer removable */
+       mark_nonremovable ();
+}
 
 void
-SMFSource::destroy_model ()
+SMFSource::set_path (const string& p)
 {
-       //cerr << _name << " destroying model " << _model.get() << endl;
-       _model.reset();
+       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::flush_midi ()
+SMFSource::ensure_disk_file ()
 {
-       Evoral::SMF::end_write();
+       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 ();
+               _model = mm;
+       } else {
+               /* No model; if it's not already open, it's an empty source, so create
+                  and open it for writing.
+               */
+               if (!_open) {
+                       open_for_write ();
+               }
+
+               /* Flush, which will definitely put something on disk */
+               flush_midi ();
+       }
 }