fix BeatFramesConverter so that it at least works even if its not optimally named...
[ardour.git] / libs / ardour / midi_region.cc
index e8d6229e911496ed2f2e4d4ca9b8f9772a3c375c..e487cae2f5224426fc9a8947846de9f078e7d146 100644 (file)
@@ -49,7 +49,8 @@ using namespace PBD;
 
 namespace ARDOUR {
        namespace Properties {
-               PBD::PropertyDescriptor<void*> midi_data;
+               PBD::PropertyDescriptor<void*>                midi_data;
+               PBD::PropertyDescriptor<Evoral::MusicalTime>  length_beats;
         }
 }
 
@@ -58,18 +59,22 @@ 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 ()
 {
-        /* none yet, but its only a matter of time */
+       add_property (_length_beats);
 }
 
 /* Basic MidiRegion constructor (many channels) */
 MidiRegion::MidiRegion (const SourceList& srcs)
        : Region (srcs)
+        , _length_beats (Properties::length_beats, (Evoral::MusicalTime) 0)
 {
+        update_length_beats ();
         register_properties ();
 
        midi_source(0)->ModelChanged.connect_same_thread (_source_connection, boost::bind (&MidiRegion::model_changed, this));
@@ -81,7 +86,9 @@ 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);
@@ -99,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);
 
@@ -108,40 +115,60 @@ 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;
@@ -212,7 +239,13 @@ MidiRegion::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
@@ -306,9 +339,21 @@ MidiRegion::model_automation_state_changed (Evoral::Parameter const & p)
        }
 
        /* the source will have an iterator into the model, and that iterator will have been set up
-          for a given set of filtered_paramters, so now that we've changed that list we must invalidate
+          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;
+}