Respond to MMC even when synced to JACK. Fixes #3700.
[ardour.git] / libs / ardour / midi_track.cc
index 9dced1e0b95cae96b3ab81ca6234a58d18ad48ca..cdd4633448e7590138d83354a153b61319c88920 100644 (file)
@@ -49,8 +49,6 @@ using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
 
-PBD::Signal1<void,bool> MidiTrack::StepEditStatusChange;
-
 MidiTrack::MidiTrack (Session& sess, string name, Route::Flag flag, TrackMode mode)
        : Track (sess, name, flag, mode, DataType::MIDI)
        , _immediate_events(1024) // FIXME: size?
@@ -107,6 +105,10 @@ MidiTrack::set_diskstream (boost::shared_ptr<Diskstream> ds)
        _diskstream->set_record_enabled (false);
        //_diskstream->monitor_input (false);
 
+       _diskstream_data_recorded_connection.disconnect ();
+       boost::shared_ptr<MidiDiskstream> mds = boost::dynamic_pointer_cast<MidiDiskstream> (ds);
+       mds->DataRecorded.connect_same_thread (_diskstream_data_recorded_connection, boost::bind (&MidiTrack::diskstream_data_recorded, this, _1, _2));
+
        DiskstreamChanged (); /* EMIT SIGNAL */
 }
 
@@ -154,14 +156,6 @@ MidiTrack::_set_state (const XMLNode& node, int version, bool call_base)
        XMLNode *child;
 
        nlist = node.children();
-       for (niter = nlist.begin(); niter != nlist.end(); ++niter){
-               child = *niter;
-
-               if (child->name() == X_("recenable")) {
-                       _rec_enable_control->set_state (*child, version);
-                       _session.add_controllable (_rec_enable_control);
-               }
-       }
 
        if (version >= 3000) {
                if ((child = find_named_node (node, X_("Diskstream"))) != 0) {
@@ -171,6 +165,20 @@ MidiTrack::_set_state (const XMLNode& node, int version, bool call_base)
                }
        }
 
+        /* set rec-enable control *AFTER* setting up diskstream, because it may want to operate
+           on the diskstream as it sets its own state
+        */
+
+       for (niter = nlist.begin(); niter != nlist.end(); ++niter){
+               child = *niter;
+
+                if (child->name() == Controllable::xml_node_name && (prop = child->property ("name")) != 0) {
+                        if (prop->value() == X_("recenable")) {
+                                _rec_enable_control->set_state (*child, version);
+                        }
+                }
+       }
+
        pending_state = const_cast<XMLNode*> (&node);
 
        if (_session.state_of_the_state() & Session::Loading) {
@@ -290,7 +298,7 @@ MidiTrack::set_state_part_two ()
 }
 
 int
-MidiTrack::roll (nframes_t nframes, framepos_t start_frame, framepos_t end_frame, int declick,
+MidiTrack::roll (pframes_t nframes, framepos_t start_frame, framepos_t end_frame, int declick,
                 bool can_record, bool rec_monitors_input, bool& needs_butler)
 {
        Glib::RWLock::ReaderLock lm (_processor_lock, Glib::TRY_LOCK);
@@ -312,8 +320,7 @@ MidiTrack::roll (nframes_t nframes, framepos_t start_frame, framepos_t end_frame
                return 0;
        }
 
-       nframes_t transport_frame = _session.transport_frame();
-
+       framepos_t transport_frame = _session.transport_frame();
 
        if ((nframes = check_initial_delay (nframes, transport_frame)) == 0) {
                /* need to do this so that the diskstream sets its
@@ -372,18 +379,20 @@ MidiTrack::roll (nframes_t nframes, framepos_t start_frame, framepos_t end_frame
 
                write_out_of_band_data (bufs, start_frame, end_frame, nframes);
 
-               process_output_buffers (bufs, start_frame, end_frame, nframes,
-                                       (!_session.get_record_enabled() || !Config->get_do_not_record_plugins()), declick);
+               /* final argument: don't waste time with automation if we're recording or we've just stopped (yes it can happen) */
 
+               process_output_buffers (bufs, start_frame, end_frame, nframes,
+                                       (!_session.get_record_enabled() || !Config->get_do_not_record_plugins()), declick,
+                                        (!diskstream->record_enabled() && !_session.transport_stopped()));
        }
 
-       _main_outs->flush (nframes, end_frame - start_frame - 1);
+       _main_outs->flush_buffers (nframes, end_frame - start_frame - 1);
 
        return 0;
 }
 
 int
-MidiTrack::no_roll (nframes_t nframes, sframes_t start_frame, sframes_t end_frame,
+MidiTrack::no_roll (pframes_t nframes, framepos_t start_frame, framepos_t end_frame,
                    bool state_changing, bool can_record, bool rec_monitors_input)
 {
        int ret = Track::no_roll (nframes, start_frame, end_frame, state_changing, can_record, rec_monitors_input);
@@ -396,16 +405,20 @@ MidiTrack::no_roll (nframes_t nframes, sframes_t start_frame, sframes_t end_fram
 }
 
 void
-MidiTrack::handle_transport_stopped (bool abort, bool did_locate, bool flush_processors)
+MidiTrack::realtime_handle_transport_stopped ()
 {
+       Glib::RWLock::ReaderLock lm (_processor_lock, Glib::TRY_LOCK);
+       if (!lm.locked ()) {
+               return;
+       }
 
-       _main_outs->transport_stopped ();
-       Route::handle_transport_stopped (abort, did_locate, flush_processors);
+       for (ProcessorList::iterator i = _processors.begin(); i != _processors.end(); ++i) {
+               (*i)->realtime_handle_transport_stopped ();
+       }
 }
 
-
 void
-MidiTrack::push_midi_input_to_step_edit_ringbuffer (nframes_t nframes)
+MidiTrack::push_midi_input_to_step_edit_ringbuffer (framecnt_t nframes)
 {
        PortSet& ports (_input->ports());
 
@@ -417,17 +430,22 @@ MidiTrack::push_midi_input_to_step_edit_ringbuffer (nframes_t nframes)
 
                for (MidiBuffer::const_iterator e = mb->begin(); e != mb->end(); ++e) {
 
-                       const Evoral::MIDIEvent<nframes_t> ev(*e, false);
-
-                       /* we don't care about the time for this purpose */
+                       const Evoral::MIDIEvent<framepos_t> ev(*e, false);
 
-                       _step_edit_ring_buffer.write (0, ev.type(), ev.size(), ev.buffer());
+                        /* note on, since for step edit, note length is determined
+                           elsewhere 
+                        */
+                        
+                        if (ev.is_note_on()) {
+                                /* we don't care about the time for this purpose */
+                                _step_edit_ring_buffer.write (0, ev.type(), ev.size(), ev.buffer());
+                        }
                }
        }
 }
 
 void
-MidiTrack::write_out_of_band_data (BufferSet& bufs, sframes_t /*start*/, sframes_t /*end*/, nframes_t nframes)
+MidiTrack::write_out_of_band_data (BufferSet& bufs, framepos_t /*start*/, framepos_t /*end*/, framecnt_t nframes)
 {
        // Append immediate events
        MidiBuffer& buf (bufs.get_midi (0));
@@ -444,13 +462,13 @@ MidiTrack::write_out_of_band_data (BufferSet& bufs, sframes_t /*start*/, sframes
 }
 
 int
-MidiTrack::export_stuff (BufferSet& /*bufs*/, nframes_t /*nframes*/, sframes_t /*end_frame*/)
+MidiTrack::export_stuff (BufferSet& /*bufs*/, framecnt_t /*nframes*/, framepos_t /*end_frame*/)
 {
        return -1;
 }
 
 void
-MidiTrack::set_latency_delay (nframes_t longest_session_latency)
+MidiTrack::set_latency_delay (framecnt_t longest_session_latency)
 {
        Route::set_latency_delay (longest_session_latency);
        _diskstream->set_roll_delay (_roll_delay);
@@ -467,7 +485,7 @@ MidiTrack::bounce (InterThreadInfo& /*itt*/)
 
 
 boost::shared_ptr<Region>
-MidiTrack::bounce_range (nframes_t /*start*/, nframes_t /*end*/, InterThreadInfo& /*itt*/, bool /*enable_processing*/)
+MidiTrack::bounce_range (framepos_t /*start*/, framepos_t /*end*/, InterThreadInfo& /*itt*/, bool /*enable_processing*/)
 {
        throw;
        //vector<MidiSource*> srcs;
@@ -522,7 +540,7 @@ MidiTrack::write_immediate_event(size_t size, const uint8_t* buf)
 }
 
 void
-MidiTrack::MidiControl::set_value(float val)
+MidiTrack::MidiControl::set_value(double val)
 {
        bool valid = false;
        if (isinf(val)) {
@@ -605,7 +623,7 @@ MidiTrack::set_midi_thru (bool yn)
 }
 
 boost::shared_ptr<SMFSource>
-MidiTrack::write_source (uint32_t n)
+MidiTrack::write_source (uint32_t)
 {
        return midi_diskstream()->write_source ();
 }
@@ -633,3 +651,10 @@ MidiTrack::midi_playlist ()
 {
        return midi_diskstream()->midi_playlist ();
 }
+
+void
+MidiTrack::diskstream_data_recorded (boost::shared_ptr<MidiBuffer> buf, boost::weak_ptr<MidiSource> src)
+{
+       DataRecorded (buf, src); /* EMIT SIGNAL */
+}
+