main fix: when transport stops, clear per-region per-playlist note trackers even...

[ardour.git] / libs / ardour / midi_state_tracker.cc

diff --git a/libs/ardour/midi_state_tracker.cc b/libs/ardour/midi_state_tracker.cc
index 03ac16ca8722af7ae7d8f2d54f99ecb8e3f25878..a8f803a8a07abc9df4a4979eedffe8b1fed43c5d 100644 (file)
--- a/libs/ardour/midi_state_tracker.cc
+++ b/libs/ardour/midi_state_tracker.cc
@@ -18,8 +18,13 @@
 */
 
 #include <iostream>
+
+#include "pbd/compose.h"
+
+#include "ardour/debug.h"
 #include "ardour/event_type_map.h"
 #include "ardour/midi_ring_buffer.h"
+#include "ardour/midi_source.h"
 #include "ardour/midi_state_tracker.h"
 
 using namespace std;
@@ -34,6 +39,7 @@ MidiStateTracker::MidiStateTracker ()
 void
 MidiStateTracker::reset ()
 {
+       DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1: reset\n", this));
        memset (_active_notes, 0, sizeof (_active_notes));
        _on = 0;
 }
@@ -53,6 +59,8 @@ MidiStateTracker::add (uint8_t note, uint8_t chn)
 {
        ++_active_notes[note + 128 * chn];
        ++_on;
+       DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("MST @ %1 ON %2/%3 total on %4\n",
+                                                              this, (int) note, (int) chn, _on));
 }
 
 void
@@ -70,6 +78,8 @@ MidiStateTracker::remove (uint8_t note, uint8_t chn)
                                break;
 
        }
+       DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("MST @ %1 OFF %2/%3 total on %4\n",
+                                                              this, (int) note, (int) chn, _on));
 }
 
 void
@@ -77,6 +87,8 @@ MidiStateTracker::track (const MidiBuffer::iterator &from, const MidiBuffer::ite
 {
        looped = false;
 
+       DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1 track notes, looped = %2\n", this, looped));
+
        for (MidiBuffer::iterator i = from; i != to; ++i) {
                const Evoral::MIDIEvent<MidiBuffer::TimeType> ev(*i, false);
                if (ev.event_type() == LoopEventType) {
@@ -84,13 +96,23 @@ MidiStateTracker::track (const MidiBuffer::iterator &from, const MidiBuffer::ite
                        continue;
                }
 
+               /* catch AllNotesOff message and turn off all notes
+                */
+               
+               if (ev.type() == MIDI_CTL_ALL_NOTES_OFF) {
+                       cerr << "State tracker sees ALL_NOTES_OFF, silenceing " << sizeof (_active_notes) << endl;
+                       memset (_active_notes, 0, sizeof (_active_notes));
+               }
+
                track_note_onoffs (ev);
        }
 }
 
 void
-MidiStateTracker::resolve_notes (MidiBuffer &dst, nframes64_t time)
+MidiStateTracker::resolve_notes (MidiBuffer &dst, framepos_t time)
 {
+       DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1 MB-resolve notes @ %2 on = %3\n", this, time, _on));
+
        if (!_on) {
                return;
        }
@@ -100,9 +122,11 @@ MidiStateTracker::resolve_notes (MidiBuffer &dst, nframes64_t time)
                        while (_active_notes[note + 128 * channel]) {
                                uint8_t buffer[3] = { MIDI_CMD_NOTE_OFF | channel, note, 0 };
                                Evoral::MIDIEvent<MidiBuffer::TimeType> noteoff
-                                       (time, MIDI_CMD_NOTE_OFF, 3, buffer, false);
+                                       (MIDI_CMD_NOTE_OFF, time, 3, buffer, false);
                                dst.push_back (noteoff);
                                _active_notes[note + 128 * channel]--;
+                               DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1: MB-resolved note %2/%3 at %4\n", 
+                                                                                      this, (int) note, (int) channel, time));
                        }
                }
        }
@@ -110,10 +134,12 @@ MidiStateTracker::resolve_notes (MidiBuffer &dst, nframes64_t time)
 }
 
 void
-MidiStateTracker::resolve_notes (Evoral::EventSink<nframes_t> &dst, nframes64_t time)
+MidiStateTracker::resolve_notes (Evoral::EventSink<framepos_t> &dst, framepos_t time)
 {
        uint8_t buf[3];
 
+       DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1 EVS-resolve notes @ %2 on = %3\n", this, time, _on));
+
        if (!_on) {
                return;
        }
@@ -126,6 +152,39 @@ MidiStateTracker::resolve_notes (Evoral::EventSink<nframes_t> &dst, nframes64_t
                                buf[2] = 0;
                                dst.write (time, EventTypeMap::instance().midi_event_type (buf[0]), 3, buf);
                                _active_notes[note + 128 * channel]--;
+                               DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1: EVS-resolved note %2/%3 at %4\n", 
+                                                                                      this, (int) note, (int) channel, time));
+                       }
+               }
+       }
+       _on = 0;
+}
+
+void
+MidiStateTracker::resolve_notes (MidiSource& src, Evoral::MusicalTime time)
+{
+       DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1 MS-resolve notes @ %2 on = %3\n", this, time, _on));
+
+       if (!_on) {
+               return;
+       }
+
+       /* NOTE: the src must be locked */
+
+       for (int channel = 0; channel < 16; ++channel) {
+               for (int note = 0; note < 128; ++note) {
+                       while (_active_notes[note + 128 * channel]) {
+                               Evoral::MIDIEvent<Evoral::MusicalTime> ev ((MIDI_CMD_NOTE_OFF|channel), time, 3, 0, true);
+                               ev.set_type (MIDI_CMD_NOTE_OFF);
+                               ev.set_channel (channel);
+                               ev.set_note (note);
+                               ev.set_velocity (0);
+                               src.append_event_unlocked_beats (ev);
+                               DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1: MS-resolved note %2/%3 at %4\n", 
+                                                                                      this, (int) note, (int) channel, time));
+                               _active_notes[note + 128 * channel]--;
+                               /* don't stack events up at the same time */
+                               time += 1.0/128.0;
                        }
                }
        }