Fix genererally retarded and broken note range / diskstream display / etc. related...
[ardour.git] / libs / ardour / smf_source.cc
index 8c2ee6031f760647b10f6c2430d06ca8767b50a0..bcf4aecae001d15db413c0e79a0e0cf20840c9ff 100644 (file)
 
 #include <glibmm/miscutils.h>
 
-#include <ardour/smf_source.h>
-#include <ardour/session.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_util.h>
+#include <ardour/session.h>
+#include <ardour/smf_source.h>
 #include <ardour/tempo.h>
-#include <ardour/audioengine.h>
-#include <ardour/smf_reader.h>
 
 #include "i18n.h"
 
@@ -46,55 +49,43 @@ using namespace ARDOUR;
 
 string SMFSource::_search_path;
 
-/*sigc::signal<void,struct tm*, time_t> SMFSource::HeaderPositionOffsetChanged;
-bool                                  SMFSource::header_position_negative;
-uint64_t                              SMFSource::header_position_offset;
-*/
-
-SMFSource::SMFSource (Session& s, std::string path, Flag flags)
-       : MidiSource (s, region_name_from_path(path, false))
-       , _flags (Flag(flags | Writable)) // FIXME: this needs to be writable for now
+/** Constructor used for new internal-to-session files.  File cannot exist. */
+SMFSource::SMFSource(Session& s, std::string path, Flag flags)
+       : MidiSource(s, region_name_from_path(path, false))
+       , Evoral::SMF()
+       , _flags(flags)
        , _allow_remove_if_empty(true)
-       , _fd (0)
-       , _last_ev_time(0)
-       , _track_size(4) // 4 bytes for the ever-present EOT event
-       , _header_size(22)
+       , _last_ev_time_beats(0.0)
+       , _last_ev_time_frames(0)
 {
-       /* constructor used for new internal-to-session files. file cannot exist */
-
-       if (init (path, false)) {
+       if (init(path, false)) {
                throw failed_constructor ();
        }
        
-       if (open()) {
+       if (create(path)) {
                throw failed_constructor ();
        }
 
-       cerr << "SMF Source path: " << path << endl;
-       
        assert(_name.find("/") == string::npos);
 }
 
-SMFSource::SMFSource (Session& s, const XMLNode& node)
-       : MidiSource (s, node)
-       , _flags (Flag (Writable|CanRename))
+/** Constructor used for existing internal-to-session files.  File must exist. */
+SMFSource::SMFSource(Session& s, const XMLNode& node)
+       : MidiSource(s, node)
+       , _flags(Flag(Writable|CanRename))
        , _allow_remove_if_empty(true)
-       , _fd (0)
-       , _last_ev_time(0)
-       , _track_size(4) // 4 bytes for the ever-present EOT event
-       , _header_size(22)
+       , _last_ev_time_beats(0.0)
+       , _last_ev_time_frames(0)
 {
-       /* constructor used for existing internal-to-session files. file must exist */
-
-       if (set_state (node)) {
+       if (set_state(node)) {
                throw failed_constructor ();
        }
        
-       if (init (_name, true)) {
+       if (init(_name, true)) {
                throw failed_constructor ();
        }
        
-       if (open()) {
+       if (open(_path)) {
                throw failed_constructor ();
        }
        
@@ -111,8 +102,8 @@ SMFSource::~SMFSource ()
 bool
 SMFSource::removable () const
 {
-       return (_flags & Removable) && ((_flags & RemoveAtDestroy) || 
-                                     ((_flags & RemovableIfEmpty) && is_empty()));
+       return (_flags & Removable) && ((_flags & RemoveAtDestroy) ||
+                       ((_flags & RemovableIfEmpty) && is_empty()));
 }
 
 int
@@ -133,326 +124,118 @@ SMFSource::init (string pathstr, bool must_exist)
        return 0;
 }
 
-/** Attempt to open the SMF file for reading and writing.
- *
- * Currently SMFSource is always read/write.
- *
- * \return  0 on success
- *         -1 if the file can not be opened for reading,
- *         -2 if the file can not be opened for writing
- */
-int
-SMFSource::open()
-{
-       //cerr << "Opening SMF file " << path() << " writeable: " << writable() << endl;
-
-       assert(writable()); // FIXME;
-
-       _fd = fopen(path().c_str(), "r+");
-
-       // File already exists
-       if (_fd) {
-               fseek(_fd, _header_size - 4, 0);
-               uint32_t track_size_be = 0;
-               fread(&track_size_be, 4, 1, _fd);
-               _track_size = GUINT32_FROM_BE(track_size_be);
-               //cerr << "SMF - read track size " << _track_size << endl;
-
-       // We're making a new file
-       } else {
-               _fd = fopen(path().c_str(), "w+");
-               if (_fd == NULL) {
-                       cerr << "ERROR: Can not open SMF file " << path() << " for writing: " <<
-                               strerror(errno) << endl;
-                       return -2;
-               }
-               _track_size = 4;
-
-               // Write a tentative header just to pad things out so writing happens in the right spot
-               flush_header();
-               flush_footer();
-       }
-               
-       return (_fd == 0) ? -1 : 0;
-}
-
-void
-SMFSource::close()
-{
-       if (_fd) {
-               flush_header();
-               flush_footer();
-               fclose(_fd);
-               _fd = NULL;
-       }
-}
-
-void
-SMFSource::seek_to_footer_position()
-{
-       uint8_t buffer[4];
-       
-       // lets check if there is a track end marker at the end of the data
-       fseek(_fd, -4, SEEK_END);
-       size_t read_bytes = fread(buffer, sizeof(uint8_t), 4, _fd);
-       //cerr << "SMFSource::seek_to_footer_position: read size: " << read_bytes << endl;
-       if( (read_bytes == 4) && 
-           buffer[0] == 0x00 && 
-           buffer[1] == 0xFF && 
-           buffer[2] == 0x2F && 
-           buffer[3] == 0x00) {
-               // there is one, so overwrite it
-               fseek(_fd, -4, SEEK_END);
-       } else {
-               // there is none, so append
-               fseek(_fd, 0, SEEK_END);
-       }
-}
-
-int
-SMFSource::flush_header()
-{
-       // FIXME: write timeline position somehow?
-       
-       //cerr << path() << " SMF Flushing header\n";
-
-       assert(_fd);
-
-       const uint16_t type     = GUINT16_TO_BE(0);     // SMF Type 0 (single track)
-       const uint16_t ntracks  = GUINT16_TO_BE(1);     // Number of tracks (always 1 for Type 0)
-       const uint16_t division = GUINT16_TO_BE(_ppqn); // Pulses per quarter note (beat)
-
-       char data[6];
-       memcpy(data, &type, 2);
-       memcpy(data+2, &ntracks, 2);
-       memcpy(data+4, &division, 2);
-
-       _fd = freopen(path().c_str(), "r+", _fd);
-       assert(_fd);
-       fseek(_fd, 0, SEEK_SET);
-       write_chunk("MThd", 6, data);
-       write_chunk_header("MTrk", _track_size); 
-
-       fflush(_fd);
-
-       return 0;
-}
-
-int
-SMFSource::flush_footer()
-{
-       //cerr << path() << " SMF Flushing footer\n";
-       seek_to_footer_position();
-       write_footer();
-
-       return 0;
-}
-
-void
-SMFSource::write_footer()
-{
-       write_var_len(0);
-       char eot[3] = { 0xFF, 0x2F, 0x00 }; // end-of-track meta-event
-       fwrite(eot, 1, 3, _fd);
-       fflush(_fd);
-}
-
-/** Returns the offset of the first event in the file with a time past @a start,
- * relative to the start of the source.
- *
- * Returns -1 if not found.
- */
-/*
-long
-SMFSource::find_first_event_after(nframes_t start)
-{
-       // FIXME: obviously this is slooow
-       
-       fseek(_fd, _header_size, 0);
-
-       while ( ! feof(_fd) ) {
-               const uint32_t delta_time = read_var_len();
-
-               if (delta_time > start)
-                       return delta_time;
-       }
-
-       return -1;
-}
-*/
-
-/** Read an event from the current position in file.
- *
- * File position MUST be at the beginning of a delta time, or this will die very messily.
- * ev.buffer must be of size ev.size, and large enough for the event.  The returned event
- * will have it's time field set to it's delta time, in SMF tempo-based ticks, using the
- * rate given by ppqn() (it is the caller's responsibility to calculate a real time).
- *
- * \a size should be the capacity of \a buf.  If it is not large enough, \a buf will
- * be freed and a new buffer allocated in its place, the size of which will be placed
- * in size.
- *
- * Returns event length (including status byte) on success, 0 if event was
- * skipped (eg a meta event), or -1 on EOF (or end of track).
- */
-int
-SMFSource::read_event(uint32_t* delta_t, uint32_t* size, Byte** buf) const
-{
-       if (feof(_fd)) {
-               return -1;
-       }
-
-       assert(delta_t);
-       assert(size);
-       assert(buf);
-
-       try {
-               *delta_t = SMFReader::read_var_len(_fd);
-       } catch (...) {
-               return -1; // Premature EOF
-       }
-       
-       if (feof(_fd)) {
-               return -1; // Premature EOF
-       }
-
-       const int status = fgetc(_fd);
-
-       if (status == EOF) {
-               return -1; // Premature EOF
-       }
-
-       //printf("Status @ %X = %X\n", (unsigned)ftell(_fd) - 1, status);
-
-       if (status == 0xFF) {
-               if (feof(_fd)) {
-                       return -1; // Premature EOF
-               }
-               const int type = fgetc(_fd);
-               if ((unsigned char)type == 0x2F) {
-                       return -1; // hit end of track
-               } else {
-                       *size = 0;
-                       return 0;
-               }
-       }
-       
-       const int event_size = midi_event_size((unsigned char)status) + 1;
-       if (event_size <= 0) {
-               *size = 0;
-               return 0;
-       }
-       
-       // Make sure we have enough scratch buffer
-       if (*size < (unsigned)event_size)
-               *buf = (Byte*)realloc(*buf, event_size);
-       
-       *size = event_size;
-
-       (*buf)[0] = (unsigned char)status;
-       if (event_size > 1)
-               fread((*buf) + 1, 1, *size - 1, _fd);
-
-       printf("SMFSource %s read event: delta = %u, size = %u, data = ", _name.c_str(), *delta_t, *size);
-       for (size_t i=0; i < *size; ++i) {
-               printf("%X ", (*buf)[i]);
-       }
-       printf("\n"); 
-       
-       return (int)*size;
-}
-
 /** All stamps in audio frames */
 nframes_t
-SMFSource::read_unlocked (MidiRingBuffer& dst, nframes_t start, nframes_t cnt, nframes_t stamp_offset, nframes_t negative_stamp_offset) const
+SMFSource::read_unlocked (MidiRingBuffer<nframes_t>& dst, nframes_t start, nframes_t dur,
+               nframes_t stamp_offset, nframes_t negative_stamp_offset) const
 {
-       cerr << "SMF read_unlocked " << name() << " read " << start << ", count=" << cnt << ", offset=" << stamp_offset << endl;
+       //cerr << "SMF read_unlocked " << name() << " read "
+       //<< start << ", count=" << dur << ", offset=" << stamp_offset << endl;
 
-       // 64 bits ought to be enough for anybody
+       int ret;
        uint64_t time = 0; // in SMF ticks, 1 tick per _ppqn
 
        _read_data_count = 0;
 
        // Output parameters for read_event (which will allocate scratch in buffer as needed)
        uint32_t ev_delta_t = 0;
-       uint32_t ev_size = 0;
-       Byte*    ev_buffer = 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
 
-       // FIXME: don't seek to start and search every read (brutal!)
-       fseek(_fd, _header_size, SEEK_SET);
-       
        // FIXME: assumes tempo never changes after start
-       const double frames_per_beat = _session.tempo_map().tempo_at(_timeline_position).frames_per_beat(
+       const Tempo& tempo = _session.tempo_map().tempo_at(_timeline_position);
+       const double frames_per_beat = tempo.frames_per_beat(
                        _session.engine().frame_rate(),
                        _session.tempo_map().meter_at(_timeline_position));
        
-       const uint64_t start_ticks = (uint64_t)((start / frames_per_beat) * _ppqn);
+       const uint64_t start_ticks = (uint64_t)((start / frames_per_beat) * ppqn());
+
+       if (_last_read_end == 0 || start != _last_read_end) {
+               cerr << "SMFSource::read_unlocked seeking to " << start << endl;
+               Evoral::SMF::seek_to_start();
+               while (time < start_ticks) {
+                       ret = read_event(&ev_delta_t, &ev_size, &ev_buffer);
+                       if (ret == -1) { // EOF
+                               _last_read_end = start + dur;
+                               return dur;
+                       }
+                       time += ev_delta_t; // accumulate delta time
+               }
+       }
+       
+       _last_read_end = start + dur;
 
-       while (!feof(_fd)) {
-               int ret = read_event(&ev_delta_t, &ev_size, &ev_buffer);
+       while (true) {
+               ret = read_event(&ev_delta_t, &ev_size, &ev_buffer);
                if (ret == -1) { // EOF
-                       //cerr << "SMF - EOF\n";
                        break;
                }
+               
+               ev_type = EventTypeMap::instance().midi_event_type(ev_buffer[0]);
+               
+               time += ev_delta_t; // accumulate delta time
 
-               if (ret == 0) { // meta-event (skipped)
-                       //cerr << "SMF - META\n";
-                       time += ev_delta_t; // just accumulate delta time and ignore event
+               if (ret == 0) { // meta-event (skipped, just accumulate time)
                        continue;
                }
 
-               time += ev_delta_t; // accumulate delta time
-
-               if (time >= start_ticks) {
-                       const nframes_t ev_frame_time = (nframes_t)(
-                                       ((time / (double)_ppqn) * frames_per_beat)) + stamp_offset;
+               assert(time >= start_ticks);
+               const nframes_t ev_frame_time = (nframes_t)(
+                               ((time / (double)ppqn()) * frames_per_beat)) + stamp_offset;
 
-                       if (ev_frame_time <= start + cnt)
-                               dst.write(ev_frame_time - negative_stamp_offset, ev_size, ev_buffer);
-                       else
-                               break;
+               if (ev_frame_time < start + dur) {
+                       dst.write(ev_frame_time - negative_stamp_offset, ev_type, ev_size, ev_buffer);
+               } else {
+                       break;
                }
 
                _read_data_count += ev_size;
 
-               if (ev_size > scratch_size)
+               if (ev_size > scratch_size) {
                        scratch_size = ev_size;
-               else
-                       ev_size = scratch_size; // minimize realloc in read_event
+               }
+               ev_size = scratch_size; // ensure read_event only allocates if necessary
        }
        
-       return cnt;
+       return dur;
 }
 
 /** All stamps in audio frames */
 nframes_t
-SMFSource::write_unlocked (MidiRingBuffer& src, nframes_t cnt)
+SMFSource::write_unlocked (MidiRingBuffer<nframes_t>& src, nframes_t dur)
 {
        _write_data_count = 0;
                
-       double time;
-       size_t size;
+       nframes_t         time;
+       Evoral::EventType type;
+       uint32_t          size;
 
-       size_t buf_capacity = 4;
-       Byte* buf = (Byte*)malloc(buf_capacity);
+       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(0, 0.0, 4, NULL, true);
 
        while (true) {
-               bool ret = src.full_peek(sizeof(double), (Byte*)&time);
-               if (!ret || time - _timeline_position > _length + cnt)
+               bool ret = src.peek_time(&time);
+               if (!ret || time - _timeline_position > _length + dur) {
                        break;
+               }
 
-               ret = src.read_prefix(&time, &size);
-               if (!ret)
+               ret = src.read_prefix(&time, &type, &size);
+               if (!ret) {
                        break;
+               }
 
                if (size > buf_capacity) {
                        buf_capacity = size;
-                       buf = (Byte*)realloc(buf, size);
+                       buf = (uint8_t*)realloc(buf, size);
                }
 
                ret = src.read_contents(size, buf);
@@ -464,65 +247,102 @@ SMFSource::write_unlocked (MidiRingBuffer& src, nframes_t cnt)
                assert(time >= _timeline_position);
                time -= _timeline_position;
                
-               const MIDI::Event ev(time, size, buf);
-               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;
+               ev.set(buf, size, time);
+               ev.set_event_type(EventTypeMap::instance().midi_event_type(ev.buffer()[0]));
+               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);
+               append_event_unlocked_frames(ev);
+       }
 
-               if (_model)
-                       _model->append(ev);
+       if (_model) {
+               set_default_controls_interpolation();
        }
 
-       fflush(_fd);
+       Evoral::SMF::flush();
        free(buf);
 
        const nframes_t oldlen = _length;
-       update_length(oldlen, cnt);
+       update_length(oldlen, dur);
 
-       ViewDataRangeReady (_timeline_position + oldlen, cnt); /* EMIT SIGNAL */
-       
-       return cnt;
+       ViewDataRangeReady(_timeline_position + oldlen, dur); /* EMIT SIGNAL */
+
+       return dur;
 }
                
 
+/** Append an event with a timestamp in beats (double) */
 void
-SMFSource::append_event_unlocked(EventTimeUnit unit, const MIDI::Event& ev)
+SMFSource::append_event_unlocked_beats(const Evoral::Event<double>& ev)
 {
-       /*printf("SMFSource: %s - append_event_unlocked chan = %u, time = %lf, size = %u, data = ",
-                       name().c_str(), (unsigned)ev.channel(), ev.time(), ev.size()); */
-       for (size_t i=0; i < ev.size(); ++i) {
-               printf("%X ", ev.buffer()[i]);
+       if (ev.size() == 0)  {
+               return;
        }
-       printf("\n");
+
+       /*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);
-       assert(ev.time() >= _last_ev_time);
-       
-       uint32_t delta_time = 0;
+       if (ev.time() < _last_ev_time_beats) {
+               cerr << "SMFSource: Warning: Skipping event with non-monotonic time" << endl;
+               return;
+       }
        
-       if (unit == Frames) {
-               // FIXME: assumes tempo never changes after start
-               const double frames_per_beat = _session.tempo_map().tempo_at(_timeline_position).frames_per_beat(
-                               _session.engine().frame_rate(),
-                               _session.tempo_map().meter_at(_timeline_position));
+       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());
 
-               delta_time = (uint32_t)((ev.time() - _last_ev_time) / frames_per_beat * _ppqn);
-       } else {
-               assert(unit == Beats);
-               delta_time = (uint32_t)((ev.time() - _last_ev_time) * _ppqn);
+       Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer());
+       _last_ev_time_beats = ev.time();
+
+       _write_data_count += ev.size();
+
+       if (_model) {
+               _model->append(ev);
+       }
+}
+
+/** Append an event with a timestamp in frames (nframes_t) */
+void
+SMFSource::append_event_unlocked_frames(const Evoral::Event<nframes_t>& ev)
+{
+       if (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");*/
        
-       const size_t stamp_size = write_var_len(delta_time);
-       fwrite(ev.buffer(), 1, ev.size(), _fd);
+       assert(ev.time() >= 0);
+       if (ev.time() < _last_ev_time_frames) {
+               cerr << "SMFSource: Warning: Skipping event with non-monotonic time" << endl;
+               return;
+       }
+       
+       // FIXME: assumes tempo never changes after start
+       const Tempo& tempo = _session.tempo_map().tempo_at(_timeline_position);
+       const double frames_per_beat = tempo.frames_per_beat(
+                       _session.engine().frame_rate(),
+                       _session.tempo_map().meter_at(_timeline_position));
+
+       uint32_t delta_time = (uint32_t)((ev.time() - _last_ev_time_frames)
+                       / frames_per_beat * (double)ppqn());
+
+       Evoral::SMF::append_event_delta(delta_time, ev.size(), ev.buffer());
+       _last_ev_time_frames = ev.time();
 
-       _track_size += stamp_size + ev.size();
        _write_data_count += ev.size();
 
-       _last_ev_time = ev.time();
+       if (_model) {
+               double beat_time = ev.time() / frames_per_beat;
+               const Evoral::Event<double> beat_ev(
+                               ev.event_type(), beat_time, ev.size(), (uint8_t*)ev.buffer());
+               _model->append(beat_ev);
+       }
 }
 
 
@@ -546,15 +366,11 @@ SMFSource::set_state (const XMLNode& node)
        }
 
        if ((prop = node.property (X_("flags"))) != 0) {
-
                int ival;
                sscanf (prop->value().c_str(), "0x%x", &ival);
                _flags = Flag (ival);
-
        } else {
-
                _flags = Flag (0);
-
        }
 
        assert(_name.find("/") == string::npos);
@@ -575,8 +391,9 @@ void
 SMFSource::mark_streaming_midi_write_started (NoteMode mode, nframes_t start_frame)
 {
        MidiSource::mark_streaming_midi_write_started (mode, start_frame);
-       _last_ev_time = 0;
-       fseek(_fd, _header_size, SEEK_SET);
+       Evoral::SMF::begin_write ();
+       _last_ev_time_beats = 0.0;
+       _last_ev_time_frames = 0;
 }
 
 void
@@ -589,8 +406,7 @@ SMFSource::mark_streaming_write_completed ()
        }
        
        _model->set_edited(false);
-       flush_header();
-       flush_footer();
+       Evoral::SMF::end_write ();
 }
 
 void
@@ -604,29 +420,23 @@ SMFSource::mark_take (string id)
 int
 SMFSource::move_to_trash (const string trash_dir_name)
 {
-       string newpath;
-
        if (!writable()) {
                return -1;
        }
 
-       /* don't move the file across filesystems, just
-          stick it in the 'trash_dir_name' directory
-          on whichever filesystem it was already on.
+       /* don't move the file across filesystems, just stick it in the
+          trash_dir_name directory on whichever filesystem it was already on
        */
-
+       
+       Glib::ustring newpath;
        newpath = Glib::path_get_dirname (_path);
-       newpath = Glib::path_get_dirname (newpath);
+       newpath = Glib::path_get_dirname (newpath); 
 
-       newpath += '/';
-       newpath += trash_dir_name;
-       newpath += '/';
+       newpath += string("/") + trash_dir_name + "/";
        newpath += Glib::path_get_basename (_path);
 
+       /* the new path already exists, try versioning */
        if (access (newpath.c_str(), F_OK) == 0) {
-
-               /* the new path already exists, try versioning */
-               
                char buf[PATH_MAX+1];
                int version = 1;
                string newpath_v;
@@ -640,40 +450,24 @@ SMFSource::move_to_trash (const string trash_dir_name)
                }
                
                if (version == 999) {
-                       PBD::error << string_compose (_("there are already 1000 files with names like %1; versioning discontinued"),
-                                         newpath)
-                             << endmsg;
+                       PBD::error << string_compose (
+                                       _("there are already 1000 files with names like %1; versioning discontinued"),
+                                       newpath) << endmsg;
                } else {
                        newpath = newpath_v;
                }
-
-       } else {
-
-               /* it doesn't exist, or we can't read it or something */
-
        }
 
        if (::rename (_path.c_str(), newpath.c_str()) != 0) {
-               PBD::error << string_compose (_("cannot rename midi file source from %1 to %2 (%3)"),
-                                 _path, newpath, strerror (errno))
-                     << endmsg;
+               PBD::error << string_compose (
+                               _("cannot rename midi file source from %1 to %2 (%3)"),
+                               _path, newpath, strerror (errno)) << endmsg;
                return -1;
        }
-#if 0
-       if (::unlink (peakpath.c_str()) != 0) {
-               PBD::error << string_compose (_("cannot remove peakfile %1 for %2 (%3)"),
-                                 peakpath, _path, strerror (errno))
-                     << endmsg;
-               /* try to back out */
-               rename (newpath.c_str(), _path.c_str());
-               return -1;
-       }
-           
+       
        _path = newpath;
-       peakpath = "";
-#endif 
+       
        /* file can not be removed twice, since the operation is not idempotent */
-
        _flags = Flag (_flags & ~(RemoveAtDestroy|Removable|RemovableIfEmpty));
 
        return 0;
@@ -689,19 +483,10 @@ SMFSource::safe_file_extension(const Glib::ustring& file)
 bool
 SMFSource::find (string pathstr, bool must_exist, bool& isnew)
 {
-       string::size_type pos;
        bool ret = false;
 
        isnew = false;
 
-       /* clean up PATH:CHANNEL notation so that we are looking for the correct path */
-
-       if ((pos = pathstr.find_last_of (':')) == string::npos) {
-               pathstr = pathstr;
-       } else {
-               pathstr = pathstr.substr (0, pos);
-       }
-
        if (pathstr[0] != '/') {
 
                /* non-absolute pathname: find pathstr in search path */
@@ -721,7 +506,6 @@ SMFSource::find (string pathstr, bool must_exist, bool& isnew)
                cnt = 0;
                
                for (vector<string>::iterator i = dirs.begin(); i != dirs.end(); ++i) {
-
                        fullpath = *i;
                        if (fullpath[fullpath.length()-1] != '/') {
                                fullpath += '/';
@@ -826,74 +610,23 @@ SMFSource::set_source_name (string newname, bool destructive)
        _name = Glib::path_get_basename (newpath);
        _path = newpath;
 
-       return 0;//rename_peakfile (peak_path (_path));
-}
-
-bool
-SMFSource::is_empty () const
-{
-       bool ret = (_track_size > 4);
-
-       //cerr << name() << " IS EMPTY: " << ret << endl;
-
-       return ret;
-}
-
-
-void
-SMFSource::write_chunk_header(const char id[4], uint32_t length)
-{
-       const uint32_t length_be = GUINT32_TO_BE(length);
-
-       fwrite(id, 1, 4, _fd);
-       fwrite(&length_be, 4, 1, _fd);
-}
-
-void
-SMFSource::write_chunk(const char id[4], uint32_t length, void* data)
-{
-       write_chunk_header(id, length);
-       
-       fwrite(data, 1, length, _fd);
-}
-
-/** Returns the size (in bytes) of the value written. */
-size_t
-SMFSource::write_var_len(uint32_t value)
-{
-       size_t ret = 0;
-
-       uint32_t buffer = value & 0x7F;
-
-       while ( (value >>= 7) ) {
-               buffer <<= 8;
-               buffer |= ((value & 0x7F) | 0x80);
-       }
-
-       while (true) {
-               //printf("Writing var len byte %X\n", (unsigned char)buffer);
-               ++ret;
-               fputc(buffer, _fd);
-               if (buffer & 0x80)
-                       buffer >>= 8;
-               else
-                       break;
-       }
-
-       return ret;
+       return 0;
 }
 
 void
 SMFSource::load_model(bool lock, bool force_reload)
 {
-       if (_writing)
+       if (_writing) {
                return;
-
-       if (lock)
+       }
+       
+       if (lock) {
                Glib::Mutex::Lock lm (_lock);
+       }
 
-       if (_model && !force_reload && !_model->empty())
+       if (_model && !force_reload) {
                return;
+       }
 
        if (! _model) {
                _model = boost::shared_ptr<MidiModel>(new MidiModel(this));
@@ -905,42 +638,56 @@ SMFSource::load_model(bool lock, bool force_reload)
        }
 
        _model->start_write();
-
-       fseek(_fd, _header_size, SEEK_SET);
+       Evoral::SMF::seek_to_start();
 
        uint64_t time = 0; /* in SMF ticks */
-       MIDI::Event ev;
+       Evoral::Event<double> ev;
        
        size_t scratch_size = 0; // keep track of scratch and minimize reallocs
        
-       // FIXME: assumes tempo never changes after start
-       const double frames_per_beat = _session.tempo_map().tempo_at(_timeline_position).frames_per_beat(
-                       _session.engine().frame_rate(),
-                       _session.tempo_map().meter_at(_timeline_position));
-       
        uint32_t delta_t = 0;
+       uint32_t size    = 0;
+       uint8_t* buf     = NULL;
        int ret;
-       while ((ret = read_event(&delta_t, &ev.size(), &ev.buffer())) >= 0) {
-               
+       while ((ret = read_event(&delta_t, &size, &buf)) >= 0) {
                time += delta_t;
+               ev.set(buf, size, time / (double)ppqn());
                
                if (ret > 0) { // didn't skip (meta) event
-                       // make ev.time absolute time in frames
-                       ev.time() = (double)time * frames_per_beat / (double)_ppqn;
-
+                       ev.set_event_type(EventTypeMap::instance().midi_event_type(buf[0]));
                        _model->append(ev);
                }
 
-               if (ev.size() > scratch_size)
+               if (ev.size() > scratch_size) {
                        scratch_size = ev.size();
-               else
-                       ev.size() = scratch_size;
+               }
+               ev.size() = scratch_size; // ensure read_event only allocates if necessary
        }
+
+       set_default_controls_interpolation();
        
        _model->end_write(false);
        _model->set_edited(false);
 
-       free(ev.buffer());
+       free(buf);
+}
+
+#define LINEAR_INTERPOLATION_MODE_WORKS_PROPERLY 0
+
+void
+SMFSource::set_default_controls_interpolation()
+{
+       // 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
+       }
 }
 
 
@@ -951,3 +698,9 @@ SMFSource::destroy_model()
        _model.reset();
 }
 
+void
+SMFSource::flush_midi()
+{
+       Evoral::SMF::end_write();
+}
+