Replace a bunch of potential crashes with graceful handling of the situation.
[ardour.git] / libs / ardour / midi_port.cc
index 880c1b5474b1e3e0b9d7284e681c628f971ec3e2..cc72eabba49ed67f6e9d03ef198882da8dd0ef4a 100644 (file)
@@ -21,6 +21,7 @@
 
 #include "ardour/midi_port.h"
 #include "ardour/data_type.h"
+#include "ardour/audioengine.h"
 
 using namespace ARDOUR;
 using namespace std;
@@ -28,8 +29,10 @@ using namespace std;
 MidiPort::MidiPort (const std::string& name, Flags flags)
        : Port (name, DataType::MIDI, flags)
        , _has_been_mixed_down (false)
+       , _resolve_required (false)
+       , _input_active (true)
 {
-       _buffer = new MidiBuffer (raw_buffer_size(0));
+       _buffer = new MidiBuffer (AudioEngine::instance()->raw_buffer_size (DataType::MIDI));
 }
 
 MidiPort::~MidiPort()
@@ -38,10 +41,11 @@ MidiPort::~MidiPort()
 }
 
 void
-MidiPort::cycle_start (nframes_t nframes)
+MidiPort::cycle_start (pframes_t nframes)
 {
+       Port::cycle_start (nframes);
+
        _buffer->clear ();
-       assert (_buffer->size () == 0);
 
        if (sends_output ()) {
                jack_midi_clear_buffer (jack_port_get_buffer (_jack_port, nframes));
@@ -49,7 +53,7 @@ MidiPort::cycle_start (nframes_t nframes)
 }
 
 MidiBuffer &
-MidiPort::get_midi_buffer (nframes_t nframes, nframes_t offset)
+MidiPort::get_midi_buffer (pframes_t nframes)
 {
        if (_has_been_mixed_down) {
                return *_buffer;
@@ -57,30 +61,40 @@ MidiPort::get_midi_buffer (nframes_t nframes, nframes_t offset)
 
        if (receives_input ()) {
 
-               void* jack_buffer = jack_port_get_buffer (_jack_port, nframes);
-               const nframes_t event_count = jack_midi_get_event_count(jack_buffer);
-
-               assert (event_count < _buffer->capacity());
-
-               /* suck all relevant MIDI events from the JACK MIDI port buffer
-                  into our MidiBuffer
-               */
-
-               nframes_t off = offset + _port_offset;
-
-               for (nframes_t i = 0; i < event_count; ++i) {
-
-                       jack_midi_event_t ev;
-
-                       jack_midi_event_get (&ev, jack_buffer, i);
-
-                       if (ev.time > off && ev.time < off+nframes) {
-                               _buffer->push_back (ev);
-                       }
-               }
-
-               if (nframes) {
-                       _has_been_mixed_down = true;
+               if (_input_active) {
+
+                       void* jack_buffer = jack_port_get_buffer (_jack_port, nframes);
+                       const pframes_t event_count = jack_midi_get_event_count (jack_buffer);
+                       
+                       /* suck all relevant MIDI events from the JACK MIDI port buffer
+                          into our MidiBuffer
+                       */
+                       
+                       for (pframes_t i = 0; i < event_count; ++i) {
+                               
+                               jack_midi_event_t ev;
+                               
+                               jack_midi_event_get (&ev, jack_buffer, i);
+                               
+                               if (ev.buffer[0] == 0xfe) {
+                                       /* throw away active sensing */
+                                       continue;
+                               }
+                               
+                               /* check that the event is in the acceptable time range */
+                               
+                               if ((ev.time >= (_global_port_buffer_offset + _port_buffer_offset)) &&
+                                   (ev.time < (_global_port_buffer_offset + _port_buffer_offset + nframes))) {
+                                       _buffer->push_back (ev);
+                               } else {
+                                       cerr << "Dropping incoming MIDI at time " << ev.time << "; offset="
+                                            << _global_port_buffer_offset << " limit="
+                                            << (_global_port_buffer_offset + _port_buffer_offset + nframes) << "\n";
+                               }
+                       } 
+
+               } else {
+                       _buffer->silence (nframes);
                }
 
        } else {
@@ -96,7 +110,7 @@ MidiPort::get_midi_buffer (nframes_t nframes, nframes_t offset)
 
 
 void
-MidiPort::cycle_end (nframes_t /*nframes*/)
+MidiPort::cycle_end (pframes_t /*nframes*/)
 {
        _has_been_mixed_down = false;
 }
@@ -108,31 +122,91 @@ MidiPort::cycle_split ()
 }
 
 void
-MidiPort::flush_buffers (nframes_t nframes, nframes_t offset)
+MidiPort::resolve_notes (void* jack_buffer, MidiBuffer::TimeType when)
+{
+       for (uint8_t channel = 0; channel <= 0xF; channel++) {
+
+               uint8_t ev[3] = { ((uint8_t) (MIDI_CMD_CONTROL | channel)), MIDI_CTL_SUSTAIN, 0 };
+
+               /* we need to send all notes off AND turn the
+                * sustain/damper pedal off to handle synths
+                * that prioritize sustain over AllNotesOff
+                */
+
+               if (jack_midi_event_write (jack_buffer, when, ev, 3) != 0) {
+                       cerr << "failed to deliver sustain-zero on channel " << channel << " on port " << name() << endl;
+               }
+
+               ev[1] = MIDI_CTL_ALL_NOTES_OFF;
+
+               if (jack_midi_event_write (jack_buffer, 0, ev, 3) != 0) {
+                       cerr << "failed to deliver ALL NOTES OFF on channel " << channel << " on port " << name() << endl;
+               }
+       }
+}
+
+void
+MidiPort::flush_buffers (pframes_t nframes)
 {
        if (sends_output ()) {
 
                void* jack_buffer = jack_port_get_buffer (_jack_port, nframes);
 
+               if (_resolve_required) {
+                       /* resolve all notes at the start of the buffer */
+                       resolve_notes (jack_buffer, 0);
+                       _resolve_required = false;
+               }
+
                for (MidiBuffer::iterator i = _buffer->begin(); i != _buffer->end(); ++i) {
-                       const Evoral::Event<nframes_t>& ev = *i;
 
-                       // event times are in frames, relative to cycle start
+                       const Evoral::MIDIEvent<MidiBuffer::TimeType> ev (*i, false);
 
-                       // XXX split cycle start or cycle start?
+                       // event times are in frames, relative to cycle start
 
-                       assert(ev.time() < (nframes+offset+_port_offset));
+                       assert (ev.time() < (nframes + _global_port_buffer_offset + _port_buffer_offset));
 
-                       if (ev.time() >= offset + _port_offset) {
-                               jack_midi_event_write (jack_buffer, (jack_nframes_t) ev.time(), ev.buffer(), ev.size());
+                       if (ev.time() >= _global_port_buffer_offset + _port_buffer_offset) {
+                               if (jack_midi_event_write (jack_buffer, (jack_nframes_t) ev.time(), ev.buffer(), ev.size()) != 0) {
+                                       cerr << "write failed, drop flushed note off on the floor, time "
+                                            << ev.time() << " > " << _global_port_buffer_offset + _port_buffer_offset << endl;
+                               }
+                       } else {
+                               cerr << "drop flushed event on the floor, time " << ev
+                                    << " to early for " << _global_port_buffer_offset + _port_buffer_offset << endl;
                        }
                }
        }
 }
 
-size_t
-MidiPort::raw_buffer_size (nframes_t nframes) const
+void
+MidiPort::require_resolve ()
+{
+       _resolve_required = true;
+}
+
+void
+MidiPort::transport_stopped ()
+{
+       _resolve_required = true;
+}
+
+void
+MidiPort::realtime_locate ()
 {
-       return jack_midi_max_event_size(jack_port_get_buffer(_jack_port, nframes));
+       _resolve_required = true;
 }
 
+void
+MidiPort::reset ()
+{
+       Port::reset ();
+       delete _buffer;
+       _buffer = new MidiBuffer (AudioEngine::instance()->raw_buffer_size (DataType::MIDI));
+}
+
+void
+MidiPort::set_input_active (bool yn)
+{
+       _input_active = yn;
+}