Add automatable click-free bypass/enable feature to a-eq
[ardour.git] / libs / ardour / midi_source.cc
index ab972fabd872222b329ae762aa92044b1dc168b1..97bce4b1abc9f9c3b92c3a9f6d3bb9e842b9bf98 100644 (file)
 #include "ardour/midi_source.h"
 #include "ardour/midi_state_tracker.h"
 #include "ardour/session.h"
+#include "ardour/tempo.h"
 #include "ardour/session_directory.h"
 #include "ardour/source_factory.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 namespace ARDOUR { template <typename T> class MidiRingBuffer; }
 
@@ -113,7 +114,7 @@ MidiSource::get_state ()
 int
 MidiSource::set_state (const XMLNode& node, int /*version*/)
 {
-       const XMLProperty* prop;
+       XMLProperty const * prop;
        if ((prop = node.property ("captured-for")) != 0) {
                _captured_for = prop->value();
        }
@@ -192,9 +193,11 @@ MidiSource::midi_read (const Lock&                        lm,
                        framecnt_t                         cnt,
                        MidiStateTracker*                  tracker,
                        MidiChannelFilter*                 filter,
-                       const std::set<Evoral::Parameter>& filtered) const
+                       const std::set<Evoral::Parameter>& filtered,
+                      double                             beat,
+                      double                             start_beat) const
 {
-       BeatsFramesConverter converter(_session.tempo_map(), source_start);
+       //BeatsFramesConverter converter(_session.tempo_map(), source_start);
 
        DEBUG_TRACE (DEBUG::MidiSourceIO,
                     string_compose ("MidiSource::midi_read() %5 sstart %1 start %2 cnt %3 tracker %4\n",
@@ -205,6 +208,7 @@ MidiSource::midi_read (const Lock&                        lm,
                Evoral::Sequence<Evoral::Beats>::const_iterator& i = _model_iter;
                const bool linear_read = _last_read_end != 0 && start == _last_read_end;
                if (!linear_read || !_model_iter_valid) {
+#if 0
                        // Cached iterator is invalid, search for the first event past start
                        i = _model->begin(converter.from(start), false, filtered,
                                          linear_read ? &_model->active_notes() : NULL);
@@ -212,27 +216,70 @@ MidiSource::midi_read (const Lock&                        lm,
                        if (!linear_read) {
                                _model->active_notes().clear();
                        }
+#else
+                       /* hot-fix http://tracker.ardour.org/view.php?id=6541
+                        * "parallel playback of linked midi regions -> no note-offs"
+                        *
+                        * A midi source can be used by multiple tracks simultaneously,
+                        * in which case midi_read() may be called from different tracks for
+                        * overlapping time-ranges.
+                        *
+                        * However there is only a single iterator for a given midi-source.
+                        * This results in every midi_read() performing a seek.
+                        *
+                        * If seeking is performed with
+                        *    _model->begin(converter.from(start),...)
+                        * the model is used for seeking. That method seeks to the first
+                        * *note-on* event after 'start'.
+                        *
+                        * _model->begin(converter.from(  ) ,..) eventually calls
+                        * Sequence<Time>::const_iterator() in libs/evoral/src/Sequence.cpp
+                        * which looks up the note-event via seq.note_lower_bound(t);
+                        * but the sequence 'seq' only contains note-on events(!).
+                        * note-off events are implicit in Sequence<Time>::operator++()
+                        * via _active_notes.pop(); and not part of seq.
+                        *
+                        * see also http://tracker.ardour.org/view.php?id=6287#c16671
+                        *
+                        * The linear search below assures that reading starts at the first
+                        * event for the given time, regardless of its event-type.
+                        *
+                        * The performance of this approach is O(N), while the previous
+                        * implementation is O(log(N)). This needs to be optimized:
+                        * The model-iterator or event-sequence needs to be re-designed in
+                        * some way (maybe keep an iterator per playlist).
+                        */
+                       for (i = _model->begin(); i != _model->end(); ++i) {
+                               if (i->time().to_double() + (beat - start_beat) >= beat) {
+                                       break;
+                               }
+                       }
+                       _model_iter_valid = true;
+                       if (!linear_read) {
+                               _model->active_notes().clear();
+                       }
+#endif
                }
 
                _last_read_end = start + cnt;
 
                // Copy events in [start, start + cnt) into dst
                for (; i != _model->end(); ++i) {
-                       const framecnt_t time_frames = converter.to(i->time());
-                       if (time_frames < start + cnt) {
+                       const framecnt_t time_frames = _session.tempo_map().frame_at_beat (i->time().to_double() + (beat - start_beat));
+
+                       if (time_frames < start + cnt + source_start) {
                                if (filter && filter->filter(i->buffer(), i->size())) {
                                        DEBUG_TRACE (DEBUG::MidiSourceIO,
                                                     string_compose ("%1: filter event @ %2 type %3 size %4\n",
-                                                                    _name, time_frames + source_start, i->event_type(), i->size()));
+                                                                    _name, time_frames, i->event_type(), i->size()));
                                        continue;
                                }
-
                                // Offset by source start to convert event time to session time
-                               dst.write (time_frames + source_start, i->event_type(), i->size(), i->buffer());
+                               dst.write (time_frames, i->event_type(), i->size(), i->buffer());
 
                                DEBUG_TRACE (DEBUG::MidiSourceIO,
                                             string_compose ("%1: add event @ %2 type %3 size %4\n",
-                                                            _name, time_frames + source_start, i->event_type(), i->size()));
+                                                            _name, time_frames, i->event_type(), i->size()));
 
                                if (tracker) {
                                        tracker->track (*i);
@@ -337,6 +384,23 @@ MidiSource::mark_streaming_write_completed (const Lock& lock)
        mark_midi_streaming_write_completed (lock, Evoral::Sequence<Evoral::Beats>::DeleteStuckNotes);
 }
 
+int
+MidiSource::export_write_to (const Lock& lock, boost::shared_ptr<MidiSource> newsrc, Evoral::Beats begin, Evoral::Beats end)
+{
+       Lock newsrc_lock (newsrc->mutex ());
+
+       if (!_model) {
+               error << string_compose (_("programming error: %1"), X_("no model for MidiSource during export"));
+               return -1;
+       }
+
+       _model->write_section_to (newsrc, newsrc_lock, begin, end, true);
+
+       newsrc->flush_midi(newsrc_lock);
+
+       return 0;
+}
+
 int
 MidiSource::write_to (const Lock& lock, boost::shared_ptr<MidiSource> newsrc, Evoral::Beats begin, Evoral::Beats end)
 {