fix BeatFramesConverter so that it at least works even if its not optimally named...
[ardour.git] / libs / ardour / midi_region.cc
index 6fbb2e8923e6cc56f63893c8bc6415073e8343f3..e487cae2f5224426fc9a8947846de9f078e7d146 100644 (file)
@@ -1,6 +1,5 @@
 /*
-    Copyright (C) 2006 Paul Davis 
-       Written by Dave Robillard, 2006
+    Copyright (C) 2000-2006 Paul Davis
 
     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
@@ -15,6 +14,8 @@
     You should have received a copy of the GNU General Public License
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+
+    $Id: midiregion.cc 746 2006-08-02 02:44:23Z drobilla $
 */
 
 #include <cmath>
 
 #include <set>
 
-#include <sigc++/bind.h>
-#include <sigc++/class_slot.h>
-
 #include <glibmm/thread.h>
 
-#include <pbd/basename.h>
-#include <pbd/xml++.h>
+#include "pbd/basename.h"
+#include "pbd/xml++.h"
+#include "pbd/enumwriter.h"
 
-#include <ardour/midi_region.h>
-#include <ardour/session.h>
-#include <ardour/gain.h>
-#include <ardour/dB.h>
-#include <ardour/playlist.h>
-#include <ardour/midi_source.h>
+#include "ardour/midi_region.h"
+#include "ardour/session.h"
+#include "ardour/gain.h"
+#include "ardour/dB.h"
+#include "ardour/playlist.h"
+#include "ardour/midi_source.h"
+#include "ardour/region_factory.h"
+#include "ardour/types.h"
+#include "ardour/midi_ring_buffer.h"
 
 #include "i18n.h"
 #include <locale.h>
 
 using namespace std;
 using namespace ARDOUR;
+using namespace PBD;
 
-MidiRegionState::MidiRegionState (string why)
-       : RegionState (why)
-{
+namespace ARDOUR {
+       namespace Properties {
+               PBD::PropertyDescriptor<void*>                midi_data;
+               PBD::PropertyDescriptor<Evoral::MusicalTime>  length_beats;
+        }
 }
 
-MidiRegion::MidiRegion (MidiSource& src, jack_nframes_t start, jack_nframes_t length, bool announce)
-       : Region (start, length, PBD::basename_nosuffix(src.name()), 0,  Region::Flag(Region::DefaultFlags|Region::External))
+void
+MidiRegion::make_property_quarks ()
 {
-       /* basic MidiRegion constructor */
-
-       sources.push_back (&src);
-       master_sources.push_back (&src);
-       src.GoingAway.connect (mem_fun (*this, &MidiRegion::source_deleted));
-
-       save_state ("initial state");
-
-       if (announce) {
-                CheckNewRegion (this); /* EMIT SIGNAL */
-       }
+        Properties::midi_data.property_id = g_quark_from_static_string (X_("midi-data"));
+        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for midi-data = %1\n", Properties::midi_data.property_id));
+        Properties::length_beats.property_id = g_quark_from_static_string (X_("length-beats"));
+        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for length-beats = %1\n", Properties::length_beats.property_id));
 }
 
-MidiRegion::MidiRegion (MidiSource& src, jack_nframes_t start, jack_nframes_t length, const string& name, layer_t layer, Flag flags, bool announce)
-       : Region (start, length, name, layer, flags)
-{
-       /* basic MidiRegion constructor */
-
-       sources.push_back (&src);
-       master_sources.push_back (&src);
-       src.GoingAway.connect (mem_fun (*this, &MidiRegion::source_deleted));
-
-       save_state ("initial state");
-
-       if (announce) {
-                CheckNewRegion (this); /* EMIT SIGNAL */
-       }
-}
-
-MidiRegion::MidiRegion (SourceList& srcs, jack_nframes_t start, jack_nframes_t length, const string& name, layer_t layer, Flag flags, bool announce)
-       : Region (start, length, name, layer, flags)
-{
-       /* basic MidiRegion constructor */
-#if 0
-       for (SourceList::iterator i=srcs.begin(); i != srcs.end(); ++i) {
-               sources.push_back (*i);
-               master_sources.push_back (*i);
-               (*i)->GoingAway.connect (mem_fun (*this, &MidiRegion::source_deleted));
-       }
-
+void
+MidiRegion::register_properties ()
 {
-       /* create a new MidiRegion, that is part of an existing one */
-       
-       set<MidiSource*> unique_srcs;
-
-       for (SourceList::const_iterator i= other.sources.begin(); i != other.sources.end(); ++i) {
-               sources.push_back (*i);
-               (*i)->GoingAway.connect (mem_fun (*this, &MidiRegion::source_deleted));
-               unique_srcs.insert (*i);
-       }
-
-       for (SourceList::const_iterator i = other.master_sources.begin(); i != other.master_sources.end(); ++i) {
-               if (unique_srcs.find (*i) == unique_srcs.end()) {
-                       (*i)->GoingAway.connect (mem_fun (*this, &MidiRegion::source_deleted));
-               }
-               master_sources.push_back (*i);
-       }
-
-       save_state ("initial state");
-
-       if (announce) {
-               CheckNewRegion (this); /* EMIT SIGNAL */
-       }
-#endif
+       add_property (_length_beats);
 }
 
-MidiRegion::MidiRegion (const MidiRegion &other)
-       : Region (other)
+/* Basic MidiRegion constructor (many channels) */
+MidiRegion::MidiRegion (const SourceList& srcs)
+       : Region (srcs)
+        , _length_beats (Properties::length_beats, (Evoral::MusicalTime) 0)
 {
-       /* Pure copy constructor */
-
-       set<MidiSource*> unique_srcs;
+        update_length_beats ();
+        register_properties ();
 
-       for (SourceList::const_iterator i = other.sources.begin(); i != other.sources.end(); ++i) {
-               sources.push_back (*i);
-               (*i)->GoingAway.connect (mem_fun (*this, &MidiRegion::source_deleted));
-               unique_srcs.insert (*i);
-       }
-
-       for (SourceList::const_iterator i = other.master_sources.begin(); i != other.master_sources.end(); ++i) {
-               master_sources.push_back (*i);
-               if (unique_srcs.find (*i) == unique_srcs.end()) {
-                       (*i)->GoingAway.connect (mem_fun (*this, &MidiRegion::source_deleted));
-               }
-       }
-
-       save_state ("initial state");
-
-       /* NOTE: no CheckNewRegion signal emitted here. This is the copy constructor */
+       midi_source(0)->ModelChanged.connect_same_thread (_source_connection, boost::bind (&MidiRegion::model_changed, this));
+       model_changed ();
+       assert(_name.val().find("/") == string::npos);
+       assert(_type == DataType::MIDI);
 }
 
-MidiRegion::MidiRegion (MidiSource& src, const XMLNode& node)
-       : Region (node)
+/** Create a new MidiRegion, that is part of an existing one */
+MidiRegion::MidiRegion (boost::shared_ptr<const MidiRegion> other, frameoffset_t offset, bool offset_relative)
+       : Region (other, offset, offset_relative)
+        , _length_beats (Properties::length_beats, (Evoral::MusicalTime) 0)
 {
-       sources.push_back (&src);
-       master_sources.push_back (&src);
-       src.GoingAway.connect (mem_fun (*this, &MidiRegion::source_deleted));
-
-       if (set_state (node)) {
-               throw failed_constructor();
-       }
+        update_length_beats ();
+        register_properties ();
 
-       save_state ("initial state");
-
-       CheckNewRegion (this); /* EMIT SIGNAL */
-}
-
-MidiRegion::MidiRegion (SourceList& srcs, const XMLNode& node)
-       : Region (node)
-{
-       /* basic MidiRegion constructor */
-
-       set<MidiSource*> unique_srcs;
-
-       for (SourceList::iterator i=srcs.begin(); i != srcs.end(); ++i) {
-               sources.push_back (*i);
-               (*i)->GoingAway.connect (mem_fun (*this, &MidiRegion::source_deleted));
-               unique_srcs.insert (*i);
-       }
-
-       for (SourceList::iterator i = srcs.begin(); i != srcs.end(); ++i) {
-               master_sources.push_back (*i);
-               if (unique_srcs.find (*i) == unique_srcs.end()) {
-                       (*i)->GoingAway.connect (mem_fun (*this, &MidiRegion::source_deleted));
-               }
-       }
-
-       if (set_state (node)) {
-               throw failed_constructor();
-       }
-
-       save_state ("initial state");
-
-       CheckNewRegion (this); /* EMIT SIGNAL */
+       assert(_name.val().find("/") == string::npos);
+       midi_source(0)->ModelChanged.connect_same_thread (_source_connection, boost::bind (&MidiRegion::model_changed, this));
+       model_changed ();
 }
 
 MidiRegion::~MidiRegion ()
 {
-       GoingAway (this);
 }
 
-StateManager::State*
-MidiRegion::state_factory (std::string why) const
+/** Create a new MidiRegion that has its own version of some/all of the Source used by another. 
+ */
+boost::shared_ptr<MidiRegion>
+MidiRegion::clone ()
 {
-       MidiRegionState* state = new MidiRegionState (why);
+        BeatsFramesConverter bfc (_session.tempo_map(), _position);
+       Evoral::MusicalTime const bbegin = bfc.from (_start);
+       Evoral::MusicalTime const bend = bfc.from (_start + _length);
 
-       Region::store_state (*state);
+        boost::shared_ptr<MidiSource> ms = midi_source(0)->clone (bbegin, bend);
 
-       return state;
-}      
+        PropertyList plist;
 
-Change
-MidiRegion::restore_state (StateManager::State& sstate) 
-{
-       MidiRegionState* state = dynamic_cast<MidiRegionState*> (&sstate);
+        plist.add (Properties::name, ms->name());
+        plist.add (Properties::whole_file, true);
+        plist.add (Properties::start, _start);
+        plist.add (Properties::length, _length);
+        plist.add (Properties::length_beats, _length_beats);
+        plist.add (Properties::layer, 0);
 
-       Change what_changed = Region::restore_and_return_flags (*state);
-       
-       if (_flags != Flag (state->_flags)) {
-               
-               //uint32_t old_flags = _flags;
-               
-               _flags = Flag (state->_flags);
-               
-       }
-               
-       /* XXX need a way to test stored state versus current for envelopes */
-
-       what_changed = Change (what_changed);
-
-       return what_changed;
+        return boost::dynamic_pointer_cast<MidiRegion> (RegionFactory::create (ms, plist, true));
 }
 
-UndoAction
-MidiRegion::get_memento() const
+void
+MidiRegion::set_length_internal (framecnt_t len)
 {
-       return sigc::bind (mem_fun (*(const_cast<MidiRegion *> (this)), &StateManager::use_state), _current_state_id);
+        Region::set_length_internal (len);
+        update_length_beats ();
 }
 
-bool
-MidiRegion::verify_length (jack_nframes_t len)
+void
+MidiRegion::update_length_beats ()
 {
-       for (uint32_t n=0; n < sources.size(); ++n) {
-               if (_start > sources[n]->length() - len) {
-                       return false;
-               }
-       }
-       return true;
+        cerr << name() << " Updating length beats, currently = " << _length_beats << " w/length = " << _length << endl;
+       BeatsFramesConverter converter (_session.tempo_map(), _position);
+        _length_beats = converter.from (_length);
+        cerr << "\tnew value: " << _length_beats << endl;
 }
 
-bool
-MidiRegion::verify_start_and_length (jack_nframes_t new_start, jack_nframes_t new_length)
-{
-       for (uint32_t n=0; n < sources.size(); ++n) {
-               if (new_length > sources[n]->length() - new_start) {
-                       return false;
-               }
-       }
-       return true;
-}
-bool
-MidiRegion::verify_start (jack_nframes_t pos)
+void
+MidiRegion::set_position_internal (framepos_t pos, bool allow_bbt_recompute)
 {
-       for (uint32_t n=0; n < sources.size(); ++n) {
-               if (pos > sources[n]->length() - _length) {
-                       return false;
-               }
-       }
-       return true;
+       Region::set_position_internal (pos, allow_bbt_recompute);
+        /* zero length regions don't exist - so if _length_beats is zero, this object
+           is under construction.
+        */
+        if (_length_beats) {
+                /* leave _length_beats alone, and change _length to reflect the state of things
+                   at the new position (tempo map may dictate a different number of frames
+                */
+                BeatsFramesConverter converter (_session.tempo_map(), _position - _start);
+                Region::set_length_internal (converter.to (_length_beats));
+        }
 }
 
-bool
-MidiRegion::verify_start_mutable (jack_nframes_t& new_start)
+framecnt_t
+MidiRegion::read_at (Evoral::EventSink<framepos_t>& out, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode, MidiStateTracker* tracker) const
 {
-       for (uint32_t n=0; n < sources.size(); ++n) {
-               if (new_start > sources[n]->length() - _length) {
-                       new_start = sources[n]->length() - _length;
-               }
-       }
-       return true;
+       return _read_at (_sources, out, position, dur, chan_n, mode, tracker);
 }
 
-jack_nframes_t
-MidiRegion::read_at (unsigned char *buf, unsigned char *mixdown_buffer, char * workbuf, jack_nframes_t position, 
-                     jack_nframes_t cnt, 
-                     uint32_t chan_n, jack_nframes_t read_frames, jack_nframes_t skip_frames) const
+framecnt_t
+MidiRegion::master_read_at (MidiRingBuffer<framepos_t>& out, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode) const
 {
-       return _read_at (sources, buf, mixdown_buffer, workbuf, position, cnt, chan_n, read_frames, skip_frames);
+       return _read_at (_master_sources, out, position, dur, chan_n, mode); /* no tracker */
 }
 
-jack_nframes_t
-MidiRegion::master_read_at (unsigned char *buf, unsigned char *mixdown_buffer, char * workbuf, jack_nframes_t position, 
-                            jack_nframes_t cnt, uint32_t chan_n) const
+framecnt_t
+MidiRegion::_read_at (const SourceList& /*srcs*/, Evoral::EventSink<framepos_t>& dst, framepos_t position, framecnt_t dur, uint32_t chan_n, 
+                     NoteMode mode, MidiStateTracker* tracker) const
 {
-       return _read_at (master_sources, buf, mixdown_buffer, workbuf, position, cnt, chan_n, 0, 0);
-}
+       frameoffset_t internal_offset = 0;
+       framecnt_t to_read         = 0;
 
-jack_nframes_t
-MidiRegion::_read_at (const SourceList& srcs, unsigned char *buf, unsigned char *mixdown_buffer, char * workbuf,
-                      jack_nframes_t position, jack_nframes_t cnt, 
-                      uint32_t chan_n, jack_nframes_t read_frames, jack_nframes_t skip_frames) const
-{
-       jack_nframes_t internal_offset;
-       jack_nframes_t buf_offset;
-       jack_nframes_t to_read;
-       
        /* precondition: caller has verified that we cover the desired section */
 
-       if (chan_n >= sources.size()) {
+       assert(chan_n == 0);
+
+       if (muted()) {
                return 0; /* read nothing */
        }
-       
+
        if (position < _position) {
+               /* we are starting the read from before the start of the region */
                internal_offset = 0;
-               buf_offset = _position - position;
-               cnt -= buf_offset;
+               dur -= _position - position;
        } else {
+               /* we are starting the read from after the start of the region */
                internal_offset = position - _position;
-               buf_offset = 0;
        }
 
        if (internal_offset >= _length) {
                return 0; /* read nothing */
        }
-       
-
-       if ((to_read = min (cnt, _length - internal_offset)) == 0) {
-               return 0; /* read nothing */
-       }
-
-       if (opaque()) {
-               /* overwrite whatever is there */
-               mixdown_buffer = buf + buf_offset;
-       } else {
-               mixdown_buffer += buf_offset;
-       }
 
-       if (muted()) {
+       if ((to_read = min (dur, _length - internal_offset)) == 0) {
                return 0; /* read nothing */
        }
 
        _read_data_count = 0;
 
-       if (srcs[chan_n]->read (mixdown_buffer, _start + internal_offset, to_read, workbuf) != to_read) {
+       boost::shared_ptr<MidiSource> src = midi_source(chan_n);
+       src->set_note_mode(mode);
+        
+        /*
+          cerr << "MR read @ " << position << " * " << to_read
+          << " _position = " << _position
+          << " _start = " << _start
+          << " intoffset = " << internal_offset
+          << endl;
+        */
+
+       /* This call reads events from a source and writes them to `dst' timed in session frames */
+
+       if (src->midi_read (
+                       dst, // destination buffer
+                       _position - _start, // start position of the source in session frames
+                       _start + internal_offset, // where to start reading in the source
+                       to_read, // read duration in frames
+                       tracker,
+                       _filtered_parameters
+                   ) != to_read) {
                return 0; /* "read nothing" */
        }
 
-       _read_data_count += srcs[chan_n]->read_data_count();
-
-       if (!opaque()) {
-
-               /* gack. the things we do for users.
-                */
+       _read_data_count += src->read_data_count();
 
-               buf += buf_offset;
-
-               for (jack_nframes_t n = 0; n < to_read; ++n) {
-                       buf[n] += mixdown_buffer[n];
-               }
-       } 
-       
        return to_read;
 }
-       
-XMLNode&
-MidiRegion::get_state ()
-{
-       return state (true);
-}
 
 XMLNode&
-MidiRegion::state (bool full)
+MidiRegion::state ()
 {
-       XMLNode& node (Region::state (full));
-#if 0
-//XMLNode *child;
-       char buf[64];
-       char buf2[64];
-       LocaleGuard lg (X_("POSIX"));
-       
-       snprintf (buf, sizeof (buf), "0x%x", (int) _flags);
-       node.add_property ("flags", buf);
-
-       for (uint32_t n=0; n < sources.size(); ++n) {
-               snprintf (buf2, sizeof(buf2), "source-%d", n);
-               snprintf (buf, sizeof(buf), "%" PRIu64, sources[n]->id());
-               node.add_property (buf2, buf);
-       }
-
-       snprintf (buf, sizeof (buf), "%u", (uint32_t) sources.size());
-       node.add_property ("channels", buf);
-
-       if (full && _extra_xml) {
-               node.add_child_copy (*_extra_xml);
-       }
-#endif
-       return node;
-}
-
-int
-MidiRegion::set_state (const XMLNode& node)
-{
-       const XMLNodeList& nlist = node.children();
-       const XMLProperty *prop;
-       LocaleGuard lg (X_("POSIX"));
-
-       Region::set_state (node);
-
-       if ((prop = node.property ("flags")) != 0) {
-               _flags = Flag (strtol (prop->value().c_str(), (char **) 0, 16));
-
-               _flags = Flag (_flags & ~Region::LeftOfSplit);
-               _flags = Flag (_flags & ~Region::RightOfSplit);
-       }
-
-       /* Now find envelope description and other misc child items */
-                               
-       for (XMLNodeConstIterator niter = nlist.begin(); niter != nlist.end(); ++niter) {
-               
-               XMLNode *child;
-               //XMLProperty *prop;
-               
-               child = (*niter);
-       }
-
-       return 0;
+       return Region::state ();
 }
 
 int
-MidiRegion::separate_by_channel (Session& session, vector<MidiRegion*>& v) const
+MidiRegion::set_state (const XMLNode& node, int version)
 {
-       SourceList srcs;
-       string new_name;
+       int ret = Region::set_state (node, version);
 
-       for (SourceList::const_iterator i = master_sources.begin(); i != master_sources.end(); ++i) {
+        if (ret == 0) {
+                update_length_beats ();
+        }
 
-               srcs.clear ();
-               srcs.push_back (*i);
-
-               /* generate a new name */
-               
-               if (session.region_name (new_name, _name)) {
-                       return -1;
-               }
-
-               /* create a copy with just one source */
-
-               v.push_back (new MidiRegion (srcs, _start, _length, new_name, _layer, _flags));
-       }
-
-       return 0;
+        return ret;
 }
 
 void
-MidiRegion::source_deleted (Source* ignored)
+MidiRegion::recompute_at_end ()
 {
-       delete this;
+       /* our length has changed
+         * so what? stuck notes are dealt with via a note state tracker
+        */
 }
 
 void
-MidiRegion::lock_sources ()
+MidiRegion::recompute_at_start ()
 {
-       SourceList::iterator i;
-       set<MidiSource*> unique_srcs;
-
-       for (i = sources.begin(); i != sources.end(); ++i) {
-               unique_srcs.insert (*i);
-               (*i)->use ();
-       }
-
-       for (i = master_sources.begin(); i != master_sources.end(); ++i) {
-               if (unique_srcs.find (*i) == unique_srcs.end()) {
-                       (*i)->use ();
-               }
-       }
+       /* as above, but the shift was from the front
+        * maybe bump currently active note's note-ons up so they sound here?
+        * that could be undesireable in certain situations though.. maybe
+        * remove the note entirely, including it's note off?  something needs to
+        * be done to keep the played MIDI sane to avoid messing up voices of
+        * polyhonic things etc........
+        */
 }
 
-void
-MidiRegion::unlock_sources ()
+int
+MidiRegion::separate_by_channel (ARDOUR::Session&, vector< boost::shared_ptr<Region> >&) const
 {
-       SourceList::iterator i;
-       set<MidiSource*> unique_srcs;
-
-       for (i = sources.begin(); i != sources.end(); ++i) {
-               unique_srcs.insert (*i);
-               (*i)->release ();
-       }
-
-       for (i = master_sources.begin(); i != master_sources.end(); ++i) {
-               if (unique_srcs.find (*i) == unique_srcs.end()) {
-                       (*i)->release ();
-               }
-       }
+       // TODO
+       return -1;
 }
 
-vector<string>
-MidiRegion::master_source_names ()
+int
+MidiRegion::exportme (ARDOUR::Session&, ARDOUR::ExportSpecification&)
 {
-       SourceList::iterator i;
-
-       vector<string> names;
-       for (i = master_sources.begin(); i != master_sources.end(); ++i) {
-               names.push_back((*i)->name());
-       }
-
-       return names;
+       return -1;
 }
 
-bool
-MidiRegion::region_list_equivalent (const MidiRegion& other) const
+boost::shared_ptr<MidiSource>
+MidiRegion::midi_source (uint32_t n) const
 {
-       return size_equivalent (other) && source_equivalent (other) && _name == other._name;
+       // Guaranteed to succeed (use a static cast?)
+       return boost::dynamic_pointer_cast<MidiSource>(source(n));
 }
 
-bool
-MidiRegion::source_equivalent (const MidiRegion& other) const
+void
+MidiRegion::model_changed ()
 {
-       SourceList::const_iterator i;
-       SourceList::const_iterator io;
-
-       for (i = sources.begin(), io = other.sources.begin(); i != sources.end() && io != other.sources.end(); ++i, ++io) {
-               if ((*i)->id() != (*io)->id()) {
-                       return false;
-               }
+       if (!model()) {
+               return;
        }
+       
+       /* build list of filtered Parameters, being those whose automation state is not `Play' */
 
-       for (i = master_sources.begin(), io = other.master_sources.begin(); i != master_sources.end() && io != other.master_sources.end(); ++i, ++io) {
-               if ((*i)->id() != (*io)->id()) {
-                       return false;
+       _filtered_parameters.clear ();
+
+       Automatable::Controls const & c = model()->controls();
+
+       for (Automatable::Controls::const_iterator i = c.begin(); i != c.end(); ++i) {
+               boost::shared_ptr<AutomationControl> ac = boost::dynamic_pointer_cast<AutomationControl> (i->second);
+               assert (ac);
+               if (ac->alist()->automation_state() != Play) {
+                       _filtered_parameters.insert (ac->parameter ());
                }
        }
 
-       return true;
-}
-
-bool
-MidiRegion::overlap_equivalent (const MidiRegion& other) const
-{
-       return coverage (other.first_frame(), other.last_frame()) != OverlapNone;
-}
+       /* watch for changes to controls' AutoState */
+       midi_source()->AutomationStateChanged.connect_same_thread (
+               _model_connection, boost::bind (&MidiRegion::model_automation_state_changed, this, _1)
+               );
 
-bool
-MidiRegion::equivalent (const MidiRegion& other) const
-{
-       return _start == other._start &&
-               _position == other._position &&
-               _length == other._length;
+        model()->ContentsChanged.connect_same_thread (
+                _model_contents_connection, boost::bind (&MidiRegion::model_contents_changed, this));
 }
 
-bool
-MidiRegion::size_equivalent (const MidiRegion& other) const
+void
+MidiRegion::model_contents_changed ()
 {
-       return _start == other._start &&
-               _length == other._length;
+        send_change (PropertyChange (Properties::midi_data));        
 }
 
-#if 0
-int
-MidiRegion::exportme (Session& session, AudioExportSpecification& spec)
+void
+MidiRegion::model_automation_state_changed (Evoral::Parameter const & p)
 {
-       const jack_nframes_t blocksize = 4096;
-       jack_nframes_t to_read;
-       int status = -1;
-
-       spec.channels = sources.size();
-
-       if (spec.prepare (blocksize, session.frame_rate())) {
-               goto out;
-       }
-
-       spec.pos = 0;
-       spec.total_frames = _length;
-
-       while (spec.pos < _length && !spec.stop) {
-               
-               
-               /* step 1: interleave */
-               
-               to_read = min (_length - spec.pos, blocksize);
-               
-               if (spec.channels == 1) {
-
-                       if (sources.front()->read (spec.dataF, _start + spec.pos, to_read, 0) != to_read) {
-                               goto out;
-                       }
-
-               } else {
-
-                       Sample buf[blocksize];
-
-                       for (uint32_t chan = 0; chan < spec.channels; ++chan) {
-                               
-                               if (sources[chan]->read (buf, _start + spec.pos, to_read, 0) != to_read) {
-                                       goto out;
-                               }
-                               
-                               for (jack_nframes_t x = 0; x < to_read; ++x) {
-                                       spec.dataF[chan+(x*spec.channels)] = buf[x];
-                               }
-                       }
-               }
-               
-               if (spec.process (to_read)) {
-                       goto out;
-               }
-               
-               spec.pos += to_read;
-               spec.progress = (double) spec.pos /_length;
-               
-       }
+       /* Update our filtered parameters list after a change to a parameter's AutoState */
        
-       status = 0;
-
-  out: 
-       spec.running = false;
-       spec.status = status;
-       spec.clear();
-       
-       return status;
-}
-#endif
+       boost::shared_ptr<AutomationControl> ac = model()->automation_control (p);
+       assert (ac);
 
-Region*
-MidiRegion::get_parent()
-{
-#if 0
-       Region* r = 0;
-
-       if (_playlist) {
-               r = _playlist->session().find_whole_file_parent (*this);
+       if (ac->alist()->automation_state() == Play) {
+               _filtered_parameters.erase (p);
+       } else {
+               _filtered_parameters.insert (p);
        }
-       
-       return r;
-#endif
-       return NULL;
-}
 
+       /* the source will have an iterator into the model, and that iterator will have been set up
+          for a given set of filtered_parameters, so now that we've changed that list we must invalidate
+          the iterator.
+       */
+       Glib::Mutex::Lock lm (midi_source(0)->mutex());
+       midi_source(0)->invalidate ();
+}
 
-bool
-MidiRegion::speed_mismatch (float sr) const
+/** This is called when a trim drag has resulted in a -ve _start time for this region.
+ *  Fix it up by adding some empty space to the source.
+ */
+void
+MidiRegion::fix_negative_start ()
 {
-#if 0
-       if (sources.empty()) {
-               /* impossible, but ... */
-               return false;
-       }
+        BeatsFramesConverter c (_session.tempo_map(), _position);
 
-       float fsr = sources.front()->sample_rate();
-
-       return fsr == sr;
-#endif
-       return false;
+       model()->insert_silence_at_start (c.from (-_start));
+       _start = 0;
 }
-