X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=libs%2Fardour%2Fmidi_state_tracker.cc;h=10b312ea9b53e0759aaff32d7d4410787d4e84a4;hb=3d366053e2719ca0f129c16575ce481fcd214f61;hp=24c9d2884c327e557b61a84b99e969ce966e34bb;hpb=ede4ecbb00ecc866c502454c81e711baea780ccd;p=ardour.git diff --git a/libs/ardour/midi_state_tracker.cc b/libs/ardour/midi_state_tracker.cc index 24c9d2884c..10b312ea9b 100644 --- a/libs/ardour/midi_state_tracker.cc +++ b/libs/ardour/midi_state_tracker.cc @@ -18,10 +18,16 @@ */ #include -#include "ardour/event_type_map.h" -#include "ardour/midi_ring_buffer.h" + +#include "pbd/compose.h" +#include "pbd/stacktrace.h" + +#include "evoral/EventSink.hpp" + +#include "ardour/debug.h" #include "ardour/midi_source.h" #include "ardour/midi_state_tracker.h" +#include "ardour/parameter_types.h" using namespace std; using namespace ARDOUR; @@ -35,25 +41,26 @@ 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; } -void -MidiStateTracker::track_note_onoffs (const Evoral::MIDIEvent& event) -{ - if (event.is_note_on()) { - add (event.note(), event.channel()); - } else if (event.is_note_off()){ - remove (event.note(), event.channel()); - } -} - void MidiStateTracker::add (uint8_t note, uint8_t chn) { + if (_active_notes[note+128 * chn] == 0) { + ++_on; + } ++_active_notes[note + 128 * chn]; - ++_on; + + if (_active_notes[note+128 * chn] > 1) { + //cerr << this << " note " << (int) note << '/' << (int) chn << " was already on, now at " << (int) _active_notes[note+128*chn] << endl; + } + + DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1 ON %2/%3 voices %5 total on %4\n", + this, (int) note, (int) chn, _on, + (int) _active_notes[note+128 * chn])); } void @@ -68,30 +75,45 @@ MidiStateTracker::remove (uint8_t note, uint8_t chn) break; default: --_active_notes [note + 128 * chn]; - break; + break; } + DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1 OFF %2/%3 current voices = %5 total on %4\n", + this, (int) note, (int) chn, _on, + (int) _active_notes[note+128 * chn])); } void -MidiStateTracker::track (const MidiBuffer::iterator &from, const MidiBuffer::iterator &to, bool& looped) +MidiStateTracker::track (const MidiBuffer::const_iterator &from, const MidiBuffer::const_iterator &to) { - looped = false; - - for (MidiBuffer::iterator i = from; i != to; ++i) { - const Evoral::MIDIEvent ev(*i, false); - if (ev.event_type() == LoopEventType) { - looped = true; - continue; - } + for (MidiBuffer::const_iterator i = from; i != to; ++i) { + track(*i); + } +} - track_note_onoffs (ev); +void +MidiStateTracker::track (const uint8_t* evbuf) +{ + const uint8_t type = evbuf[0] & 0xF0; + const uint8_t chan = evbuf[0] & 0x0F; + switch (type) { + case MIDI_CTL_ALL_NOTES_OFF: + reset(); + break; + case MIDI_CMD_NOTE_ON: + add(evbuf[1], chan); + break; + case MIDI_CMD_NOTE_OFF: + remove(evbuf[1], chan); + break; } } void -MidiStateTracker::resolve_notes (MidiBuffer &dst, framepos_t time) +MidiStateTracker::resolve_notes (MidiBuffer &dst, samplepos_t time) { + DEBUG_TRACE (PBD::DEBUG::MidiTrackers, string_compose ("%1 MB-resolve notes @ %2 on = %3\n", this, time, _on)); + if (!_on) { return; } @@ -99,11 +121,16 @@ MidiStateTracker::resolve_notes (MidiBuffer &dst, framepos_t time) for (int channel = 0; channel < 16; ++channel) { for (int note = 0; note < 128; ++note) { while (_active_notes[note + 128 * channel]) { - uint8_t buffer[3] = { MIDI_CMD_NOTE_OFF | channel, note, 0 }; - Evoral::MIDIEvent noteoff - (MIDI_CMD_NOTE_OFF, time, 3, buffer, false); + uint8_t buffer[3] = { ((uint8_t) (MIDI_CMD_NOTE_OFF | channel)), uint8_t (note), 0 }; + Evoral::Event noteoff + (Evoral::MIDI_EVENT, time, 3, buffer, false); + /* note that we do not care about failure from + push_back() ... should we warn someone ? + */ 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)); } } } @@ -111,10 +138,12 @@ MidiStateTracker::resolve_notes (MidiBuffer &dst, framepos_t time) } void -MidiStateTracker::resolve_notes (Evoral::EventSink &dst, framepos_t time) +MidiStateTracker::resolve_notes (Evoral::EventSink &dst, samplepos_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; } @@ -125,8 +154,13 @@ MidiStateTracker::resolve_notes (Evoral::EventSink &dst, framepos_t t buf[0] = MIDI_CMD_NOTE_OFF|channel; buf[1] = note; buf[2] = 0; - dst.write (time, EventTypeMap::instance().midi_event_type (buf[0]), 3, buf); + /* note that we do not care about failure from + write() ... should we warn someone ? + */ + dst.write (time, Evoral::MIDI_EVENT, 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)); } } } @@ -134,27 +168,30 @@ MidiStateTracker::resolve_notes (Evoral::EventSink &dst, framepos_t t } void -MidiStateTracker::resolve_notes (MidiSource& src, Evoral::MusicalTime time) +MidiStateTracker::resolve_notes (MidiSource& src, const MidiSource::Lock& lock, Temporal::Beats 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 */ + /* 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 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); + Evoral::Event ev (Evoral::MIDI_EVENT, 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_beats (lock, 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; + /* don't stack events up at the same time */ + time += Temporal::Beats::tick(); } } } @@ -169,7 +206,7 @@ MidiStateTracker::dump (ostream& o) for (int x = 0; x < 128; ++x) { if (_active_notes[c * 128 + x]) { o << "Channel " << c+1 << " Note " << x << " is on (" - << (int) _active_notes[c*128+x] << "times)\n"; + << (int) _active_notes[c*128+x] << " times)\n"; } } }