Respond to MMC even when synced to JACK. Fixes #3700.
[ardour.git] / libs / ardour / midi_source.cc
index 4a0f20cbdba5e272535635f65d65d46e136c3550..d330eb2bcd766f67f0c334f7adc8b63c577c00d2 100644 (file)
@@ -168,22 +168,22 @@ MidiSource::set_state (const XMLNode& node, int /*version*/)
 bool
 MidiSource::empty () const
 {
-        return _length_beats == 0;
+       return _length_beats == 0;
 }
 
 framecnt_t
 MidiSource::length (framepos_t pos) const
 {
-        if (_length_beats == 0) {
-                return 0;
-        }
+       if (_length_beats == 0) {
+               return 0;
+       }
 
        BeatsFramesConverter converter(_session.tempo_map(), pos);
        return converter.to(_length_beats);
 }
 
 void
-MidiSource::update_length (sframes_t /*pos*/, sframes_t /*cnt*/)
+MidiSource::update_length (framepos_t /*pos*/, framecnt_t /*cnt*/)
 {
        // You're not the boss of me!
 }
@@ -196,11 +196,11 @@ MidiSource::invalidate ()
 }
 
 /** @param filtered A set of parameters whose MIDI messages will not be returned */
-nframes_t
-MidiSource::midi_read (Evoral::EventSink<nframes_t>& dst, sframes_t source_start,
-                       sframes_t start, nframes_t cnt,
+framecnt_t
+MidiSource::midi_read (Evoral::EventSink<framepos_t>& dst, framepos_t source_start,
+                       framepos_t start, framecnt_t cnt,
                        MidiStateTracker* tracker,
-                      std::set<Evoral::Parameter> const & filtered) const
+                       std::set<Evoral::Parameter> const & filtered) const
 {
        Glib::Mutex::Lock lm (_lock);
 
@@ -226,7 +226,7 @@ MidiSource::midi_read (Evoral::EventSink<nframes_t>& dst, sframes_t source_start
 
                // Read events up to end
                for (; i != _model->end(); ++i) {
-                       const sframes_t time_frames = converter.to(i->time());
+                       const framecnt_t time_frames = converter.to(i->time());
                        if (time_frames < start + cnt) {
                                /* convert event times to session frames by adding on the source start position in session frames */
                                dst.write (time_frames + source_start, i->event_type(), i->size(), i->buffer());
@@ -251,17 +251,21 @@ MidiSource::midi_read (Evoral::EventSink<nframes_t>& dst, sframes_t source_start
        }
 }
 
-nframes_t
-MidiSource::midi_write (MidiRingBuffer<nframes_t>& source, sframes_t source_start, nframes_t duration)
+/** Write data from a MidiRingBuffer to this source.
+ *  @param source Source to read from.
+ *  @param source_start This source's start position in session frames.
+ */
+framecnt_t
+MidiSource::midi_write (MidiRingBuffer<framepos_t>& source, framepos_t source_start, framecnt_t duration)
 {
        Glib::Mutex::Lock lm (_lock);
-       const nframes_t ret = write_unlocked (source, source_start, duration);
+       const framecnt_t ret = write_unlocked (source, source_start, duration);
        _last_write_end += duration;
        return ret;
 }
 
 void
-MidiSource::mark_streaming_midi_write_started (NoteMode mode, sframes_t start_frame)
+MidiSource::mark_streaming_midi_write_started (NoteMode mode, framepos_t start_frame)
 {
        set_timeline_position(start_frame);
 
@@ -294,86 +298,84 @@ MidiSource::mark_streaming_write_completed ()
 boost::shared_ptr<MidiSource>
 MidiSource::clone (Evoral::MusicalTime begin, Evoral::MusicalTime end)
 {
-        string newname = PBD::basename_nosuffix(_name.val());
-        string newpath;
+       string newname = PBD::basename_nosuffix(_name.val());
+       string newpath;
 
-        /* get a new name for the MIDI file we're going to write to
-         */
+       /* get a new name for the MIDI file we're going to write to
+        */
 
-        do { 
+       do { 
 
-                newname = bump_name_once (newname, '-');
-                /* XXX build path safely */
-                newpath = _session.session_directory().midi_path().to_string() +"/"+ newname + ".mid";
+               newname = bump_name_once (newname, '-');
+               /* XXX build path safely */
+               newpath = _session.session_directory().midi_path().to_string() +"/"+ newname + ".mid";
 
-        } while (Glib::file_test (newpath, Glib::FILE_TEST_EXISTS));
+       } while (Glib::file_test (newpath, Glib::FILE_TEST_EXISTS));
         
-        boost::shared_ptr<MidiSource> newsrc = boost::dynamic_pointer_cast<MidiSource>(
-                SourceFactory::createWritable(DataType::MIDI, _session,
-                                              newpath, false, _session.frame_rate()));
-        
-        newsrc->set_timeline_position(_timeline_position);
+       boost::shared_ptr<MidiSource> newsrc = boost::dynamic_pointer_cast<MidiSource>(
+               SourceFactory::createWritable(DataType::MIDI, _session,
+                                             newpath, string(), false, _session.frame_rate()));
+
+       newsrc->set_timeline_position(_timeline_position);
        newsrc->copy_interpolation_from (this);
        newsrc->copy_automation_state_from (this);
 
-        if (_model) {
-                if (begin == Evoral::MinMusicalTime && end == Evoral::MaxMusicalTime) {
-                        _model->write_to (newsrc);
-                } else {
-                        _model->write_section_to (newsrc, begin, end);
-                }
-        } else {
-                error << string_compose (_("programming error: %1"), X_("no model for MidiSource during ::clone()"));
-                return boost::shared_ptr<MidiSource>();
-        }
+       if (_model) {
+               if (begin == Evoral::MinMusicalTime && end == Evoral::MaxMusicalTime) {
+                       _model->write_to (newsrc);
+               } else {
+                       _model->write_section_to (newsrc, begin, end);
+               }
+       } else {
+               error << string_compose (_("programming error: %1"), X_("no model for MidiSource during ::clone()"));
+               return boost::shared_ptr<MidiSource>();
+       }
 
-        newsrc->flush_midi();
+       newsrc->flush_midi();
 
-        /* force a reload of the model if the range is partial */
+       /* force a reload of the model if the range is partial */
         
-        if (begin != Evoral::MinMusicalTime || end != Evoral::MaxMusicalTime) {
-                newsrc->load_model (true, true);
-        } else {
+       if (begin != Evoral::MinMusicalTime || end != Evoral::MaxMusicalTime) {
+               newsrc->load_model (true, true);
+       } else {
                newsrc->set_model (_model);
        }
         
-        return newsrc;
+       return newsrc;
 }
 
 void
 MidiSource::session_saved()
 {
-        /* this writes a copy of the data to disk. 
-           XXX do we need to do this every time?
-        */
+       /* this writes a copy of the data to disk. 
+          XXX do we need to do this every time?
+       */
 
        if (_model && _model->edited()) {
 
+               // if the model is edited, write its contents into
+               // the current source file (overwiting previous contents.
 
+               /* temporarily drop our reference to the model so that
+                  as the model pushes its current state to us, we don't
+                  try to update it.
+               */
 
-                // if the model is edited, write its contents into
-                // the current source file (overwiting previous contents.
+               boost::shared_ptr<MidiModel> mm = _model ;
+               _model.reset ();   
 
-                /* temporarily drop our reference to the model so that
-                   as the model pushes its current state to us, we don't
-                   try to update it.
-                */
+               /* flush model contents to disk
+                */
 
-                boost::shared_ptr<MidiModel> mm = _model ;
-                _model.reset ();   
+               mm->sync_to_source ();
 
-                /* flush model contents to disk
-                 */
+               /* reacquire model */
 
-                mm->sync_to_source ();
+               _model = mm;
 
-                /* reacquire model */
-
-                _model = mm;
-
-        } else {
-                flush_midi();
-        }
+       } else {
+               flush_midi();
+       }
 }
 
 void
@@ -387,7 +389,7 @@ MidiSource::set_note_mode(NoteMode mode)
 void
 MidiSource::drop_model ()
 {
-        _model.reset(); 
+       _model.reset(); 
        ModelChanged (); /* EMIT SIGNAL */
 }
 
@@ -415,7 +417,11 @@ MidiSource::automation_state_of (Evoral::Parameter p) const
 {
        AutomationStateMap::const_iterator i = _automation_state.find (p);
        if (i == _automation_state.end()) {
-               return Off;
+               /* default to `play', otherwise if MIDI is recorded /
+                  imported with controllers etc. they are by default
+                  not played back, which is a little surprising.
+               */
+               return Play;
        }
 
        return i->second;
@@ -448,7 +454,7 @@ MidiSource::set_automation_state_of (Evoral::Parameter p, AutoState s)
                return;
        }
        
-       if (s == Off) {
+       if (s == Play) {
                /* automation state is being set to the default, so we don't need a note in our map */
                _automation_state.erase (p);
        } else {