fix BeatFramesConverter so that it at least works even if its not optimally named...
[ardour.git] / libs / ardour / midi_region.cc
index abb3191463446af0d9a69a92029daeb7386b4757..e487cae2f5224426fc9a8947846de9f078e7d146 100644 (file)
@@ -24,7 +24,6 @@
 
 #include <set>
 
-
 #include <glibmm/thread.h>
 
 #include "pbd/basename.h"
@@ -48,11 +47,38 @@ using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
 
+namespace ARDOUR {
+       namespace Properties {
+               PBD::PropertyDescriptor<void*>                midi_data;
+               PBD::PropertyDescriptor<Evoral::MusicalTime>  length_beats;
+        }
+}
+
+void
+MidiRegion::make_property_quarks ()
+{
+        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));
+}
+
+void
+MidiRegion::register_properties ()
+{
+       add_property (_length_beats);
+}
+
 /* Basic MidiRegion constructor (many channels) */
 MidiRegion::MidiRegion (const SourceList& srcs)
        : Region (srcs)
+        , _length_beats (Properties::length_beats, (Evoral::MusicalTime) 0)
 {
-       midi_source(0)->Switched.connect_same_thread (*this, boost::bind (&MidiRegion::switch_source, this, _1));
+        update_length_beats ();
+        register_properties ();
+
+       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);
 }
@@ -60,9 +86,14 @@ MidiRegion::MidiRegion (const SourceList& srcs)
 /** 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)
 {
+        update_length_beats ();
+        register_properties ();
+
        assert(_name.val().find("/") == string::npos);
-       midi_source(0)->Switched.connect_same_thread (*this, boost::bind (&MidiRegion::switch_source, this, _1));
+       midi_source(0)->ModelChanged.connect_same_thread (_source_connection, boost::bind (&MidiRegion::model_changed, this));
+       model_changed ();
 }
 
 MidiRegion::~MidiRegion ()
@@ -75,8 +106,8 @@ boost::shared_ptr<MidiRegion>
 MidiRegion::clone ()
 {
         BeatsFramesConverter bfc (_session.tempo_map(), _position);
-        double bbegin = bfc.from (_position);
-        double bend = bfc.from (last_frame() + 1);
+       Evoral::MusicalTime const bbegin = bfc.from (_start);
+       Evoral::MusicalTime const bend = bfc.from (_start + _length);
 
         boost::shared_ptr<MidiSource> ms = midi_source(0)->clone (bbegin, bend);
 
@@ -84,44 +115,63 @@ MidiRegion::clone ()
 
         plist.add (Properties::name, ms->name());
         plist.add (Properties::whole_file, true);
-        plist.add (Properties::start, 0);
+        plist.add (Properties::start, _start);
         plist.add (Properties::length, _length);
+        plist.add (Properties::length_beats, _length_beats);
         plist.add (Properties::layer, 0);
 
         return boost::dynamic_pointer_cast<MidiRegion> (RegionFactory::create (ms, plist, true));
 }
 
 void
-MidiRegion::set_position_internal (framepos_t pos, bool allow_bbt_recompute)
+MidiRegion::set_length_internal (framecnt_t len)
 {
-       BeatsFramesConverter old_converter(_session.tempo_map(), _position - _start);
-       double length_beats = old_converter.from(_length);
-
-       Region::set_position_internal(pos, allow_bbt_recompute);
+        Region::set_length_internal (len);
+        update_length_beats ();
+}
 
-       BeatsFramesConverter new_converter(_session.tempo_map(), pos - _start);
+void
+MidiRegion::update_length_beats ()
+{
+        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;
+}
 
-       set_length(new_converter.to(length_beats), 0);
+void
+MidiRegion::set_position_internal (framepos_t pos, bool allow_bbt_recompute)
+{
+       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));
+        }
 }
 
 framecnt_t
-MidiRegion::read_at (Evoral::EventSink<nframes_t>& out, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode, MidiStateTracker* tracker) const
+MidiRegion::read_at (Evoral::EventSink<framepos_t>& out, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode, MidiStateTracker* tracker) const
 {
        return _read_at (_sources, out, position, dur, chan_n, mode, tracker);
 }
 
 framecnt_t
-MidiRegion::master_read_at (MidiRingBuffer<nframes_t>& out, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode) const
+MidiRegion::master_read_at (MidiRingBuffer<framepos_t>& out, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode) const
 {
        return _read_at (_master_sources, out, position, dur, chan_n, mode); /* no tracker */
 }
 
 framecnt_t
-MidiRegion::_read_at (const SourceList& /*srcs*/, Evoral::EventSink<nframes_t>& dst, framepos_t position, framecnt_t dur, uint32_t chan_n, 
+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
 {
        frameoffset_t internal_offset = 0;
-       frameoffset_t src_offset      = 0;
        framecnt_t to_read         = 0;
 
        /* precondition: caller has verified that we cover the desired section */
@@ -133,12 +183,12 @@ MidiRegion::_read_at (const SourceList& /*srcs*/, Evoral::EventSink<nframes_t>&
        }
 
        if (position < _position) {
+               /* we are starting the read from before the start of the region */
                internal_offset = 0;
-               src_offset = _position - position;
-               dur -= src_offset;
+               dur -= _position - position;
        } else {
+               /* we are starting the read from after the start of the region */
                internal_offset = position - _position;
-               src_offset = 0;
        }
 
        if (internal_offset >= _length) {
@@ -153,34 +203,24 @@ MidiRegion::_read_at (const SourceList& /*srcs*/, Evoral::EventSink<nframes_t>&
 
        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;
+        */
 
-       framepos_t output_buffer_position = 0;
-       framepos_t negative_output_buffer_position = 0;
-       if (_position >= _start) {
-               // handle resizing of beginnings of regions correctly
-               output_buffer_position = _position - _start;
-       } else {
-               // when _start is greater than _position, we have to subtract
-               // _start from the note times in the midi source
-               negative_output_buffer_position = _start;
-       }
-
-       /*cerr << "MR read @ " << position << " * " << to_read
-               << " _position = " << _position
-           << " _start = " << _start
-           << " offset = " << output_buffer_position
-           << " negoffset = " << negative_output_buffer_position
-           << " 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 this read context
+                       _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
-                       output_buffer_position, // the offset in the output buffer
-                       negative_output_buffer_position, // amount to substract from note times
-                       tracker
+                       tracker,
+                       _filtered_parameters
                    ) != to_read) {
                return 0; /* "read nothing" */
        }
@@ -191,15 +231,21 @@ MidiRegion::_read_at (const SourceList& /*srcs*/, Evoral::EventSink<nframes_t>&
 }
 
 XMLNode&
-MidiRegion::state (bool full)
+MidiRegion::state ()
 {
-       return Region::state (full);
+       return Region::state ();
 }
 
 int
 MidiRegion::set_state (const XMLNode& node, int version)
 {
-       return Region::set_state (node, version);
+       int ret = Region::set_state (node, version);
+
+        if (ret == 0) {
+                update_length_beats ();
+        }
+
+        return ret;
 }
 
 void
@@ -242,18 +288,72 @@ MidiRegion::midi_source (uint32_t n) const
        return boost::dynamic_pointer_cast<MidiSource>(source(n));
 }
 
-
 void
-MidiRegion::switch_source(boost::shared_ptr<Source> src)
+MidiRegion::model_changed ()
 {
-       boost::shared_ptr<MidiSource> msrc = boost::dynamic_pointer_cast<MidiSource>(src);
-       if (!msrc)
+       if (!model()) {
                return;
+       }
+       
+       /* build list of filtered Parameters, being those whose automation state is not `Play' */
 
-       // MIDI regions have only one source
-       _sources.clear();
-       _sources.push_back(msrc);
+       _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 ());
+               }
+       }
 
-       set_name(msrc->name());
+       /* watch for changes to controls' AutoState */
+       midi_source()->AutomationStateChanged.connect_same_thread (
+               _model_connection, boost::bind (&MidiRegion::model_automation_state_changed, this, _1)
+               );
+
+        model()->ContentsChanged.connect_same_thread (
+                _model_contents_connection, boost::bind (&MidiRegion::model_contents_changed, this));
+}
+
+void
+MidiRegion::model_contents_changed ()
+{
+        send_change (PropertyChange (Properties::midi_data));        
 }
 
+void
+MidiRegion::model_automation_state_changed (Evoral::Parameter const & p)
+{
+       /* Update our filtered parameters list after a change to a parameter's AutoState */
+       
+       boost::shared_ptr<AutomationControl> ac = model()->automation_control (p);
+       assert (ac);
+
+       if (ac->alist()->automation_state() == Play) {
+               _filtered_parameters.erase (p);
+       } else {
+               _filtered_parameters.insert (p);
+       }
+
+       /* 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 ();
+}
+
+/** 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 ()
+{
+        BeatsFramesConverter c (_session.tempo_map(), _position);
+
+       model()->insert_silence_at_start (c.from (-_start));
+       _start = 0;
+}