Fix several MIDI timestamp related problems:
[ardour.git] / libs / ardour / midi_model.cc
index 7a09804aef85a5f8fb5636dd07294ec65b941815..d7167ed2ecfb8e43a57463213fec9aebf16f1816 100644 (file)
 
 */
 
+#define __STDC_LIMIT_MACROS 1
+
 #include <iostream>
 #include <algorithm>
-#include <queue>
+#include <stdexcept>
+#include <stdint.h>
 #include <pbd/enumwriter.h>
 #include <ardour/midi_model.h>
 #include <ardour/midi_events.h>
+#include <ardour/midi_source.h>
 #include <ardour/types.h>
 #include <ardour/session.h>
 
 using namespace std;
 using namespace ARDOUR;
 
-// Note
 
-MidiModel::Note::Note(double t, double d, uint8_t n, uint8_t v)
-       : _on_event(t, 3, NULL, true)
-       , _off_event(t + d, 3, NULL, true)
+// Read iterator (const_iterator)
+
+MidiModel::const_iterator::const_iterator(const MidiModel& model, double t)
+       : _model(&model)
+       , _is_end( (t == DBL_MAX) || model.empty())
+       , _locked( ! _is_end)
 {
-       _on_event.buffer()[0] = MIDI_CMD_NOTE_ON;
-       _on_event.buffer()[1] = n;
-       _on_event.buffer()[2] = v;
+       //cerr << "Created MIDI iterator @ " << t << "(is end: " << _is_end << ")" << endl;
        
-       _off_event.buffer()[0] = MIDI_CMD_NOTE_OFF;
-       _off_event.buffer()[1] = n;
-       _off_event.buffer()[2] = 0x40;
+       if (_is_end)
+               return;
+
+       model.read_lock();
        
-       assert(time() == t);
-       assert(duration() == d);
-       assert(note() == n);
-       assert(velocity() == v);
+       _note_iter = model.notes().end();
+
+       for (MidiModel::Notes::const_iterator i = model.notes().begin(); i != model.notes().end(); ++i) {
+               if ((*i).time() >= t) {
+                       _note_iter = i;
+                       break;
+               }
+       }
+                       
+       MidiControlIterator earliest_control = make_pair(boost::shared_ptr<AutomationList>(),
+                       make_pair(DBL_MAX, 0.0));
+
+       _control_iters.reserve(model.controls().size());
+       for (Automatable::Controls::const_iterator i = model.controls().begin();
+                       i != model.controls().end(); ++i) {
+
+               assert(i->first.type() == MidiCCAutomation);
+
+               double x, y;
+               bool ret = i->second->list()->rt_safe_earliest_event_unlocked(t, DBL_MAX, x, y);
+               if (!ret) {
+                       cerr << "MIDI Iterator: CC " << i->first.id() << " (size " << i->second->list()->size()
+                               << ") has no events past " << t << endl;
+                       continue;
+               } 
+
+               assert(x >= 0);
+               assert(y >= 0);
+               assert(y <= UINT8_MAX);
+               
+               const MidiControlIterator new_iter = make_pair(i->second->list(), make_pair(x, y));
+               
+               //cerr << "MIDI Iterator: CC " << i->first.id() << " added (" << x << ", " << y << ")" << endl;
+               _control_iters.push_back(new_iter);
+
+               if (x < earliest_control.second.first) {
+                       earliest_control = new_iter;
+                       _control_iter = _control_iters.end();
+                       --_control_iter;
+               }
+       }
+
+       if (_note_iter != model.notes().end()) {
+               _event = MidiEvent(_note_iter->on_event(), false);
+               ++_note_iter;
+       }
+
+       if (earliest_control.first && earliest_control.second.first < _event.time())
+               model.control_to_midi_event(_event, earliest_control);
+       else
+               _control_iter = _control_iters.end();
+
+       if (_event.size() == 0) {
+               //cerr << "Created MIDI iterator @ " << t << " is at end." << endl;
+               _is_end = true;
+               _model->read_unlock();
+               _locked = false;
+       //} else {
+       //      printf("MIDI Iterator = %X @ %lf\n", _event.type(), _event.time());
+       }
 }
 
 
-MidiModel::Note::Note(const Note& copy)
-       : _on_event(copy._on_event, true)
-       , _off_event(copy._off_event, true)
+MidiModel::const_iterator::~const_iterator()
 {
-       /*
-       assert(copy._on_event.size == 3);
-       _on_event.buffer = _on_event_buffer;
-       memcpy(_on_event_buffer, copy._on_event_buffer, 3);
-       
-       assert(copy._off_event.size == 3);
-       _off_event.buffer = _off_event_buffer;
-       memcpy(_off_event_buffer, copy._off_event_buffer, 3);
-       */
-
-       assert(time() == copy.time());
-       assert(end_time() == copy.end_time());
-       assert(note() == copy.note());
-       assert(velocity() == copy.velocity());
-       assert(duration() == copy.duration());
+       if (_locked)
+               _model->read_unlock();
 }
+               
 
-
-const MidiModel::Note&
-MidiModel::Note::operator=(const Note& copy)
+const MidiModel::const_iterator&
+MidiModel::const_iterator::operator++()
 {
-       _on_event = copy._on_event;
-       _off_event = copy._off_event;
-       /*_on_event.time = copy._on_event.time;
-       assert(copy._on_event.size == 3);
-       memcpy(_on_event_buffer, copy._on_event_buffer, 3);
+       if (_is_end)
+               throw std::logic_error("Attempt to iterate past end of MidiModel");
+
+       assert(_event.is_note() || _event.is_cc());
+
+       // Increment past current control event
+       if (_control_iter != _control_iters.end() && _control_iter->first && _event.is_cc()) {
+               double x, y;
+               const bool ret = _control_iter->first->rt_safe_earliest_event_unlocked(
+                               _control_iter->second.first, DBL_MAX, x, y, false);
+
+               if (ret) {
+                       //cerr << "Incremented " << _control_iter->first->parameter().id() << " to " << x << endl;
+                       _control_iter->second.first = x;
+                       _control_iter->second.second = y;
+               } else {
+                       //cerr << "Hit end of " << _control_iter->first->parameter().id() << endl;
+                       _control_iter->first.reset();
+                       _control_iter->second.first = DBL_MAX;
+               }
+       }
+
+       // Now find and point at the earliest event
+
+       _control_iter = _control_iters.begin();
+
+       for (std::vector<MidiControlIterator>::iterator i = _control_iters.begin();
+                       i != _control_iters.end(); ++i) {
+               if (i->second.first < _control_iter->second.first) {
+                       _control_iter = i;
+               }
+       }
        
-       _off_event.time = copy._off_event.time;
-       assert(copy._off_event.size == 3);
-       memcpy(_off_event_buffer, copy._off_event_buffer, 3);
-       */
+       enum Type { NIL, NOTE, CC };
+       Type type = NIL;
+
+       if (_note_iter != _model->notes().end())
+               type = NOTE;
        
-       assert(time() == copy.time());
-       assert(end_time() == copy.end_time());
-       assert(note() == copy.note());
-       assert(velocity() == copy.velocity());
-       assert(duration() == copy.duration());
+       if (_control_iter != _control_iters.end() && _control_iter->second.first != DBL_MAX)
+               if (_note_iter == _model->notes().end() || _control_iter->second.first < _note_iter->time())
+                       type = CC;
+
+       if (type == NOTE) {
+               //cerr << "MIDI Iterator = note" << endl;
+               _event = MidiEvent(_note_iter->on_event(), false);
+               ++_note_iter;
+       } else if (type == CC) {
+               //cerr << "MIDI Iterator = CC" << endl;
+               _model->control_to_midi_event(_event, *_control_iter);
+       } else {
+               //cerr << "MIDI Iterator = NIL" << endl;
+               _is_end = true;
+               _model->read_unlock();
+               _locked = false;
+       }
 
        return *this;
 }
+               
+
+bool
+MidiModel::const_iterator::operator==(const const_iterator& other) const
+{
+       if (_is_end)
+               if (other._is_end)
+                       return true;
+               else
+                       return false;
+       else
+               return (_event == other._event);
+}
 
+       
 // MidiModel
 
 MidiModel::MidiModel(Session& s, size_t size)
-       : _session(s)
+       : Automatable(s, "midi model")
        , _notes(size)
        , _note_mode(Sustained)
        , _writing(false)
+       , _edited(false)
+       //, _active_notes(LaterNoteEndComparator())
+       , _end_iter(*this, DBL_MAX)
+       , _next_read(UINT32_MAX)
+       , _read_iter(*this, DBL_MAX)
 {
 }
 
@@ -112,47 +217,168 @@ MidiModel::MidiModel(Session& s, size_t size)
  * \return number of events written to \a dst
  */
 size_t
-MidiModel::read (MidiRingBuffer& dst, nframes_t start, nframes_t nframes, nframes_t stamp_offset) const
+MidiModel::read(MidiRingBuffer& dst, nframes_t start, nframes_t nframes, nframes_t stamp_offset) const
 {
        size_t read_events = 0;
 
-       //cerr << "MM READ @ " << start << " + " << nframes << endl;
+       if (start != _next_read) {
+               _read_iter = const_iterator(*this, (double)start);
+       //      cerr << "Repositioning iterator from " << _next_read << " to " << start << endl;
+       //} else {
+       //      cerr << "Using cached iterator at " << _next_read << endl;
+       }
 
-       /* FIXME: cache last lookup value to avoid the search */
+       _next_read = start + nframes;
+
+       while (_read_iter != end() && _read_iter->time() < start + nframes) {
+               dst.write(_read_iter->time() + stamp_offset, _read_iter->size(), _read_iter->buffer());
+               ++_read_iter;
+               ++read_events;
+       }
+
+#if 0
+       /* FIXME: cache last lookup value to avoid O(n) search every time */
 
        if (_note_mode == Sustained) {
-               LaterNoteEndComparator cmp;
-               priority_queue<const Note*,vector<const Note*>,LaterNoteEndComparator> active_notes(cmp);
 
-               /* FIXME: cache last lookup value to avoid the search */
                for (Notes::const_iterator n = _notes.begin(); n != _notes.end(); ++n) {
-       
-                       //cerr << "MM ON " << n->time() << endl;
+
+                       while ( ! _active_notes.empty() ) {
+                               const Note* const earliest_off = _active_notes.top();
+                               const MidiEvent&  off_ev       = earliest_off->off_event();
+                               if (off_ev.time() < start + nframes && off_ev.time() <= n->time()) {
+                                       dst.write(off_ev.time() + stamp_offset, off_ev.size(), off_ev.buffer());
+                                       _active_notes.pop();
+                                       ++read_events;
+                               } else {
+                                       break;
+                               }
+                       }
 
                        if (n->time() >= start + nframes)
                                break;
 
-                       while ( ! active_notes.empty() ) {
-                               const Note* const earliest_off = active_notes.top();
-                               const MidiEvent& ev = earliest_off->off_event();
-                               if (ev.time() < start + nframes && ev.time() <= n->time()) {
+                       // Note on
+                       if (n->time() >= start) {
+                               const MidiEvent& on_ev = n->on_event();
+                               dst.write(on_ev.time() + stamp_offset, on_ev.size(), on_ev.buffer());
+                               _active_notes.push(&(*n));
+                               ++read_events;
+                       }
+
+               }
+                       
+               // Write any trailing note offs
+               while ( ! _active_notes.empty() ) {
+                       const Note* const earliest_off = _active_notes.top();
+                       const MidiEvent&  off_ev       = earliest_off->off_event();
+                       if (off_ev.time() < start + nframes) {
+                               dst.write(off_ev.time() + stamp_offset, off_ev.size(), off_ev.buffer());
+                               _active_notes.pop();
+                               ++read_events;
+                       } else {
+                               break;
+                       }
+               }
+
+       // Percussive
+       } else {
+               for (Notes::const_iterator n = _notes.begin(); n != _notes.end(); ++n) {
+                       // Note on
+                       if (n->time() >= start) {
+                               if (n->time() < start + nframes) {
+                                       const MidiEvent& ev = n->on_event();
                                        dst.write(ev.time() + stamp_offset, ev.size(), ev.buffer());
-                                       active_notes.pop();
                                        ++read_events;
                                } else {
                                        break;
                                }
                        }
+               }
+       }
+#endif
+       return read_events;
+}
+       
+
+bool
+MidiModel::control_to_midi_event(MidiEvent& ev, const MidiControlIterator& iter) const
+{
+       if (iter.first->parameter().type() == MidiCCAutomation) {
+               if (ev.size() < 3)
+                       ev.set_buffer((Byte*)malloc(3), true);
+
+               assert(iter.first);
+               assert(iter.first->parameter().id() <= INT8_MAX);
+               assert(iter.second.second <= INT8_MAX);
+               ev.buffer()[0] = MIDI_CMD_CONTROL;
+               ev.buffer()[1] = (Byte)iter.first->parameter().id();
+               ev.buffer()[2] = (Byte)iter.second.second;
+               ev.time() = iter.second.first; // x
+               ev.size() = 3;
+               return true;
+       } else {
+               return false;
+       }
+}
+
+       
+/** Return the earliest MIDI event in the given range.
+ *
+ * \return true if \a output has been set to the earliest event in the given range.
+ */
+#if 0
+bool
+MidiModel::earliest_note_event(MidiEvent& output, nframes_t start, nframes_t nframes) const
+{
+       /* FIXME: cache last lookup value to avoid O(n) search every time */
+               
+       const Note*      const earliest_on  = NULL;
+       const Note*      const earliest_off = NULL;
+       const MidiEvent* const earliest_cc = NULL;
+
+       /* Notes */
+
+       if (_note_mode == Sustained) {
+                               
+               for (Notes::const_iterator n = _notes.begin(); n != _notes.end(); ++n) {
+
+                       if ( ! _active_notes.empty() ) {
+                               const Note* const earliest_off = _active_notes.top();
+                               const MidiEvent&  off_ev       = earliest_off->off_event();
+                               if (off_ev.time() < start + nframes && off_ev.time() <= n->time()) {
+                                       output = off_ev;
+                                       //dst.write(off_ev.time() + stamp_offset, off_ev.size(), off_ev.buffer());
+                                       _active_notes.pop();
+                                       return true;
+                               }
+                       }
+
+                       if (n->time() >= start + nframes)
+                               break;
 
                        // Note on
                        if (n->time() >= start) {
-                               const MidiEvent& ev = n->on_event();
-                               dst.write(ev.time() + stamp_offset, ev.size(), ev.buffer());
-                               active_notes.push(&(*n));
-                               ++read_events;
+                               earliest_on = &n->on_event();
+                               //dst.write(on_ev.time() + stamp_offset, on_ev.size(), on_ev.buffer());
+                               _active_notes.push(&(*n));
+                               return true;
                        }
 
                }
+                       
+               // Write any trailing note offs
+               while ( ! _active_notes.empty() ) {
+                       const Note* const earliest_off = _active_notes.top();
+                       const MidiEvent&  off_ev       = earliest_off->off_event();
+                       if (off_ev.time() < start + nframes) {
+                               dst.write(off_ev.time() + stamp_offset, off_ev.size(), off_ev.buffer());
+                               _active_notes.pop();
+                               ++read_events;
+                       } else {
+                               break;
+                       }
+               }
 
        // Percussive
        } else {
@@ -170,12 +396,9 @@ MidiModel::read (MidiRingBuffer& dst, nframes_t start, nframes_t nframes, nframe
                }
        }
 
-       //if (read_events > 0)
-       //      cerr << "MM READ " << read_events << " EVENTS" << endl;
-
        return read_events;
 }
-
+#endif
 
 /** Begin a write of events to the model.
  *
@@ -187,9 +410,11 @@ MidiModel::read (MidiRingBuffer& dst, nframes_t start, nframes_t nframes, nframe
 void
 MidiModel::start_write()
 {
-       //cerr << "MM START WRITE, MODE = " << enum_2_string(_note_mode) << endl;
-       _write_notes.clear();
+       //cerr << "MM " << this << " START WRITE, MODE = " << enum_2_string(_note_mode) << endl;
+       write_lock();
        _writing = true;
+       _write_notes.clear();
+       write_unlock();
 }
 
 
@@ -203,9 +428,10 @@ MidiModel::start_write()
 void
 MidiModel::end_write(bool delete_stuck)
 {
+       write_lock();
        assert(_writing);
        
-       //cerr << "MM END WRITE\n";
+       //cerr << "MM " << this << " END WRITE: " << _notes.size() << " NOTES\n";
 
        if (_note_mode == Sustained && delete_stuck) {
                for (Notes::iterator n = _notes.begin(); n != _notes.end() ; ) {
@@ -220,32 +446,7 @@ MidiModel::end_write(bool delete_stuck)
 
        _write_notes.clear();
        _writing = false;
-}
-
-
-/** Append contents of \a buf to model.  NOT realtime safe.
- *
- * Timestamps of events in \a buf are expected to be relative to
- * the start of this model (t=0) and MUST be monotonically increasing
- * and MUST be >= the latest event currently in the model.
- *
- * Events in buf are deep copied.
- */
-void
-MidiModel::append(const MidiBuffer& buf)
-{ 
-       assert(_writing);
-
-       for (MidiBuffer::const_iterator i = buf.begin(); i != buf.end(); ++i) {
-               const MidiEvent& ev = *i;
-               
-               assert(_notes.empty() || ev.time() >= _notes.back().time());
-
-               if (ev.type() == MIDI_CMD_NOTE_ON)
-                       append_note_on(ev.time(), ev.note(), ev.velocity());
-               else if (ev.type() == MIDI_CMD_NOTE_OFF)
-                       append_note_off(ev.time(), ev.note());
-       }
+       write_unlock();
 }
 
 
@@ -256,35 +457,47 @@ MidiModel::append(const MidiBuffer& buf)
  * and MUST be >= the latest event currently in the model.
  */
 void
-MidiModel::append(double time, size_t size, const Byte* buf)
+MidiModel::append(const MidiEvent& ev)
 {
-       assert(_notes.empty() || time >= _notes.back().time());
+       write_lock();
+
+       assert(_notes.empty() || ev.time() >= _notes.back().time());
        assert(_writing);
 
-       if ((buf[0] & 0xF0) == MIDI_CMD_NOTE_ON)
-               append_note_on(time, buf[1], buf[2]);
-       else if ((buf[0] & 0xF0) == MIDI_CMD_NOTE_OFF)
-               append_note_off(time, buf[1]);
+       if (ev.is_note_on())
+               append_note_on_unlocked(ev.time(), ev.note(), ev.velocity());
+       else if (ev.is_note_off())
+               append_note_off_unlocked(ev.time(), ev.note());
+       else if (ev.is_cc())
+               append_cc_unlocked(ev.time(), ev.cc_number(), ev.cc_value());
+       else
+               printf("MM Unknown event type %X\n", ev.type());
+       
+       write_unlock();
 }
 
 
 void
-MidiModel::append_note_on(double time, uint8_t note_num, uint8_t velocity)
+MidiModel::append_note_on_unlocked(double time, uint8_t note_num, uint8_t velocity)
 {
+       //cerr << "MidiModel " << this << " note " << (int)note_num << " on @ " << time << endl;
+
        assert(_writing);
        _notes.push_back(Note(time, 0, note_num, velocity));
        if (_note_mode == Sustained) {
-               //cerr << "MM Appending note on " << (unsigned)(uint8_t)note_num << endl;
+               //cerr << "MM Sustained: Appending active note on " << (unsigned)(uint8_t)note_num << endl;
                _write_notes.push_back(_notes.size() - 1);
        } else {
-               //cerr << "MM NOT appending note on" << endl;
+               //cerr << "MM Percussive: NOT appending active note on" << endl;
        }
 }
 
 
 void
-MidiModel::append_note_off(double time, uint8_t note_num)
+MidiModel::append_note_off_unlocked(double time, uint8_t note_num)
 {
+       //cerr << "MidiModel " << this << " note " << (int)note_num << " off @ " << time << endl;
+
        assert(_writing);
        if (_note_mode == Percussive) {
                //cerr << "MM Ignoring note off (percussive mode)" << endl;
@@ -305,7 +518,7 @@ MidiModel::append_note_off(double time, uint8_t note_num)
                        assert(time > note.time());
                        note.set_duration(time - note.time());
                        _write_notes.erase(n);
-                       //cerr << "MidiModel resolved note, duration: " << note.duration() << endl;
+                       //cerr << "MM resolved note, duration: " << note.duration() << endl;
                        break;
                }
        }
@@ -313,22 +526,37 @@ MidiModel::append_note_off(double time, uint8_t note_num)
 
 
 void
-MidiModel::add_note(const Note& note)
+MidiModel::append_cc_unlocked(double time, uint8_t number, uint8_t value)
+{
+       Parameter param(MidiCCAutomation, number);
+       
+       boost::shared_ptr<AutomationControl> control = Automatable::control(param, true);
+       //cerr << "MidiModel " << this << "(" << control.get() << ") add CC " << (int)number << " = " << (int)value
+       //      << " @ " << time << endl;
+       control->list()->fast_simple_add(time, (double)value);
+}
+
+
+void
+MidiModel::add_note_unlocked(const Note& note)
 {
+       //cerr << "MidiModel " << this << " add note " << (int)note.note() << " @ " << note.time() << endl;
        Notes::iterator i = upper_bound(_notes.begin(), _notes.end(), note, note_time_comparator);
        _notes.insert(i, note);
 }
 
 
 void
-MidiModel::remove_note(const Note& note)
+MidiModel::remove_note_unlocked(const Note& note)
 {
+       //cerr << "MidiModel " << this << " remove note " << (int)note.note() << " @ " << note.time() << endl;
        Notes::iterator n = find(_notes.begin(), _notes.end(), note);
        if (n != _notes.end())
                _notes.erase(n);
 }
 
 /** Slow!  for debugging only. */
+#ifndef NDEBUG
 bool
 MidiModel::is_sorted() const
 {
@@ -341,7 +569,7 @@ MidiModel::is_sorted() const
 
        return true;
 }
-
+#endif
 
 /** Start a new command.
  *
@@ -369,6 +597,7 @@ MidiModel::apply_command(Command* cmd)
        (*cmd)();
        assert(is_sorted());
        _session.commit_reversible_command(cmd);
+       _edited = true;
 }
 
 
@@ -401,11 +630,15 @@ MidiModel::DeltaCommand::operator()()
        // This could be made much faster by using a priority_queue for added and
        // removed notes (or sort here), and doing a single iteration over _model
        
+       _model.write_lock();
+       
        for (std::list<Note>::iterator i = _added_notes.begin(); i != _added_notes.end(); ++i)
-               _model.add_note(*i);
+               _model.add_note_unlocked(*i);
        
        for (std::list<Note>::iterator i = _removed_notes.begin(); i != _removed_notes.end(); ++i)
-               _model.remove_note(*i);
+               _model.remove_note_unlocked(*i);
+       
+       _model.write_unlock();
        
        _model.ContentsChanged(); /* EMIT SIGNAL */
 }
@@ -416,13 +649,87 @@ MidiModel::DeltaCommand::undo()
 {
        // This could be made much faster by using a priority_queue for added and
        // removed notes (or sort here), and doing a single iteration over _model
+       
+       _model.write_lock();
 
        for (std::list<Note>::iterator i = _added_notes.begin(); i != _added_notes.end(); ++i)
-               _model.remove_note(*i);
+               _model.remove_note_unlocked(*i);
        
        for (std::list<Note>::iterator i = _removed_notes.begin(); i != _removed_notes.end(); ++i)
-               _model.add_note(*i);
+               _model.add_note_unlocked(*i);
+       
+       _model.write_unlock();
        
        _model.ContentsChanged(); /* EMIT SIGNAL */
 }
 
+
+bool
+MidiModel::write_to(boost::shared_ptr<MidiSource> source)
+{
+       //cerr << "Writing model to " << source->name() << endl;
+
+       /* This could be done using a temporary MidiRingBuffer and using
+        * MidiModel::read and MidiSource::write, but this is more efficient
+        * and doesn't require any buffer size assumptions (ie it's worth
+        * the code duplication).
+        *
+        * This is also different from read in that note off events are written
+        * regardless of the track mode.  This is so the user can switch a
+        * recorded track (with note durations from some instrument) to percussive,
+        * save, reload, then switch it back to sustained preserving the original
+        * note durations.
+        */
+
+       /* Percussive 
+       for (Notes::const_iterator n = _notes.begin(); n != _notes.end(); ++n) {
+               const MidiEvent& ev = n->on_event();
+               source->append_event_unlocked(ev);
+       }*/
+
+       read_lock();
+
+       LaterNoteEndComparator cmp;
+       ActiveNotes active_notes(cmp);
+               
+       // Foreach note
+       for (Notes::const_iterator n = _notes.begin(); n != _notes.end(); ++n) {
+
+               // Write any pending note offs earlier than this note on
+               while ( ! active_notes.empty() ) {
+                       const Note* const earliest_off = active_notes.top();
+                       const MidiEvent&  off_ev       = earliest_off->off_event();
+                       if (off_ev.time() <= n->time()) {
+                               source->append_event_unlocked(off_ev);
+                               active_notes.pop();
+                       } else {
+                               break;
+                       }
+               }
+
+               // Write this note on
+               source->append_event_unlocked(n->on_event());
+               if (n->duration() > 0)
+                       active_notes.push(&(*n));
+       }
+               
+       // Write any trailing note offs
+       while ( ! active_notes.empty() ) {
+               source->append_event_unlocked(active_notes.top()->off_event());
+               active_notes.pop();
+       }
+
+       _edited = false;
+       
+       read_unlock();
+
+       return true;
+}
+
+XMLNode&
+MidiModel::get_state()
+{
+       XMLNode *node = new XMLNode("MidiModel");
+       return *node;
+}
+