X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=libs%2Fardour%2Fmidi_source.cc;h=6f8e0184df7d2701ac35c818500fc5200096cbdb;hb=a7067557107fc2f01586a88bb8b0a097914798ea;hp=4a0f20cbdba5e272535635f65d65d46e136c3550;hpb=f645b4119e4f2b03d5cd24618253e2f8b4140262;p=ardour.git diff --git a/libs/ardour/midi_source.cc b/libs/ardour/midi_source.cc index 4a0f20cbdb..1b1cf20c68 100644 --- a/libs/ardour/midi_source.cc +++ b/libs/ardour/midi_source.cc @@ -1,6 +1,6 @@ /* Copyright (C) 2006 Paul Davis - Written by Dave Robillard, 2006 + Author: David Robillard This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -20,7 +20,6 @@ #include #include #include -#include #include #include #include @@ -29,24 +28,28 @@ #include #include +#include #include "pbd/xml++.h" #include "pbd/pthread_utils.h" #include "pbd/basename.h" -#include "ardour/audioengine.h" +#include "evoral/Control.hpp" +#include "evoral/EventSink.hpp" + #include "ardour/debug.h" #include "ardour/midi_model.h" -#include "ardour/midi_ring_buffer.h" #include "ardour/midi_state_tracker.h" #include "ardour/midi_source.h" +#include "ardour/file_source.h" #include "ardour/session.h" #include "ardour/session_directory.h" #include "ardour/source_factory.h" -#include "ardour/tempo.h" #include "i18n.h" +namespace ARDOUR { template class MidiRingBuffer; } + using namespace std; using namespace ARDOUR; using namespace PBD; @@ -55,35 +58,29 @@ PBD::Signal1 MidiSource::MidiSourceCreated; MidiSource::MidiSource (Session& s, string name, Source::Flag flags) : Source(s, DataType::MIDI, name, flags) - , _read_data_count(0) - , _write_data_count(0) , _writing(false) , _model_iter_valid(false) , _length_beats(0.0) , _last_read_end(0) - , _last_write_end(0) + , _capture_length(0) + , _capture_loop_length(0) { } MidiSource::MidiSource (Session& s, const XMLNode& node) : Source(s, node) - , _read_data_count(0) - , _write_data_count(0) , _writing(false) , _model_iter_valid(false) , _length_beats(0.0) , _last_read_end(0) - , _last_write_end(0) + , _capture_length(0) + , _capture_loop_length(0) { - _read_data_count = 0; - _write_data_count = 0; - if (set_state (node, Stateful::loading_state_version)) { throw failed_constructor(); } } - MidiSource::~MidiSource () { } @@ -116,7 +113,6 @@ int MidiSource::set_state (const XMLNode& node, int /*version*/) { const XMLProperty* prop; - if ((prop = node.property ("captured-for")) != 0) { _captured_for = prop->value(); } @@ -124,39 +120,31 @@ MidiSource::set_state (const XMLNode& node, int /*version*/) XMLNodeList children = node.children (); for (XMLNodeConstIterator i = children.begin(); i != children.end(); ++i) { if ((*i)->name() == X_("InterpolationStyle")) { - XMLProperty* prop; - if ((prop = (*i)->property (X_("parameter"))) == 0) { error << _("Missing parameter property on InterpolationStyle") << endmsg; return -1; } - - Evoral::Parameter p = EventTypeMap::instance().new_parameter (prop->value()); + Evoral::Parameter p = EventTypeMap::instance().from_symbol (prop->value()); if ((prop = (*i)->property (X_("style"))) == 0) { error << _("Missing style property on InterpolationStyle") << endmsg; return -1; } - - Evoral::ControlList::InterpolationStyle s = static_cast (string_2_enum (prop->value(), s)); + Evoral::ControlList::InterpolationStyle s = static_cast( + string_2_enum (prop->value(), s)); set_interpolation_of (p, s); - - } else if ((*i)->name() == X_("AutomationState")) { - - XMLProperty* prop; + } else if ((*i)->name() == X_("AutomationState")) { if ((prop = (*i)->property (X_("parameter"))) == 0) { error << _("Missing parameter property on AutomationState") << endmsg; return -1; } - - Evoral::Parameter p = EventTypeMap::instance().new_parameter (prop->value()); + Evoral::Parameter p = EventTypeMap::instance().from_symbol (prop->value()); if ((prop = (*i)->property (X_("state"))) == 0) { error << _("Missing state property on AutomationState") << endmsg; return -1; } - AutoState s = static_cast (string_2_enum (prop->value(), s)); set_automation_state_of (p, s); } @@ -168,216 +156,240 @@ MidiSource::set_state (const XMLNode& node, int /*version*/) bool MidiSource::empty () const { - return _length_beats == 0; + return !_length_beats; } framecnt_t MidiSource::length (framepos_t pos) const { - if (_length_beats == 0) { - return 0; - } + if (!_length_beats) { + 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 (framecnt_t) { // You're not the boss of me! } void -MidiSource::invalidate () +MidiSource::invalidate (const Lock& lock) { _model_iter_valid = false; _model_iter.invalidate(); } -/** @param filtered A set of parameters whose MIDI messages will not be returned */ -nframes_t -MidiSource::midi_read (Evoral::EventSink& dst, sframes_t source_start, - sframes_t start, nframes_t cnt, - MidiStateTracker* tracker, - std::set const & filtered) const +framecnt_t +MidiSource::midi_read (const Lock& lm, + Evoral::EventSink& dst, + framepos_t source_start, + framepos_t start, + framecnt_t cnt, + MidiStateTracker* tracker, + const std::set& filtered) const { - Glib::Mutex::Lock lm (_lock); - BeatsFramesConverter converter(_session.tempo_map(), source_start); - if (_model) { - Evoral::Sequence::const_iterator& i = _model_iter; + DEBUG_TRACE (DEBUG::MidiSourceIO, + string_compose ("MidiSource::midi_read() %5 sstart %1 start %2 cnt %3 tracker %4\n", + source_start, start, cnt, tracker, name())); - // If the cached iterator is invalid, search for the first event past start + if (_model) { + // Find appropriate model iterator + Evoral::Sequence::const_iterator& i = _model_iter; if (_last_read_end == 0 || start != _last_read_end || !_model_iter_valid) { - DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("*** %1 search for relevant iterator for %1 / %2\n", _name, source_start, start)); - for (i = _model->begin(0, false, filtered); i != _model->end(); ++i) { - if (converter.to(i->time()) >= start) { - break; - } - } + // Cached iterator is invalid, search for the first event past start + i = _model->begin(converter.from(start), false, filtered); _model_iter_valid = true; - } else { - DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("*** %1 use cached iterator for %1 / %2\n", _name, source_start, start)); } _last_read_end = start + cnt; - // Read events up to end + // Copy events in [start, start + cnt) into dst 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 */ + // Offset by source start to convert event time to session time dst.write (time_frames + source_start, i->event_type(), i->size(), i->buffer()); + DEBUG_TRACE (DEBUG::MidiSourceIO, + string_compose ("%1: add event @ %2 type %3 size %4\n", + _name, time_frames + source_start, i->event_type(), i->size())); + if (tracker) { - Evoral::MIDIEvent& ev (*(Evoral::MIDIEvent*) (&(*i))); - if (ev.is_note_on()) { - DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("\t%1 add note on %2 @ %3 velocity %4\n", _name, ev.note(), time_frames, (int) ev.velocity())); - tracker->add (ev.note(), ev.channel()); - } else if (ev.is_note_off()) { - DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("\t%1 add note off %2 @ %3\n", _name, ev.note(), time_frames)); - tracker->remove (ev.note(), ev.channel()); - } + tracker->track (*i); } } else { + DEBUG_TRACE (DEBUG::MidiSourceIO, + string_compose ("%1: reached end with event @ %2 vs. %3\n", + _name, time_frames, start+cnt)); break; } } return cnt; } else { - return read_unlocked (dst, source_start, start, cnt, tracker); + return read_unlocked (lm, dst, source_start, start, cnt, tracker); } } -nframes_t -MidiSource::midi_write (MidiRingBuffer& source, sframes_t source_start, nframes_t duration) +framecnt_t +MidiSource::midi_write (const Lock& lm, + MidiRingBuffer& source, + framepos_t source_start, + framecnt_t cnt) { - Glib::Mutex::Lock lm (_lock); - const nframes_t ret = write_unlocked (source, source_start, duration); - _last_write_end += duration; + const framecnt_t ret = write_unlocked (lm, source, source_start, cnt); + + if (cnt == max_framecnt) { + _last_read_end = 0; + invalidate(lm); + } else { + _capture_length += cnt; + } + return ret; } void -MidiSource::mark_streaming_midi_write_started (NoteMode mode, sframes_t start_frame) +MidiSource::mark_streaming_midi_write_started (const Lock& lock, NoteMode mode) { - set_timeline_position(start_frame); - if (_model) { - _model->set_note_mode(mode); - _model->start_write(); + _model->set_note_mode (mode); + _model->start_write (); } - _last_write_end = start_frame; _writing = true; } void -MidiSource::mark_streaming_write_started () +MidiSource::mark_write_starting_now (framecnt_t position, + framecnt_t capture_length, + framecnt_t loop_length) +{ + /* I'm not sure if this is the best way to approach this, but + _capture_length needs to be set up with the transport frame + when a record actually starts, as it is used by + SMFSource::write_unlocked to decide whether incoming notes + are within the correct time range. + mark_streaming_midi_write_started (perhaps a more logical + place to do this) is not called at exactly the time when + record starts, and I don't think it necessarily can be + because it is not RT-safe. + */ + + set_timeline_position(position); + _capture_length = capture_length; + _capture_loop_length = loop_length; + + BeatsFramesConverter converter(_session.tempo_map(), position); + _length_beats = converter.from(capture_length); +} + +void +MidiSource::mark_streaming_write_started (const Lock& lock) { NoteMode note_mode = _model ? _model->note_mode() : Sustained; - mark_streaming_midi_write_started(note_mode, _session.transport_frame()); + mark_streaming_midi_write_started (lock, note_mode); } void -MidiSource::mark_streaming_write_completed () +MidiSource::mark_midi_streaming_write_completed (const Lock& lock, + Evoral::Sequence::StuckNoteOption option, + Evoral::MusicalTime end) { if (_model) { - _model->end_write(false); + _model->end_write (option, end); + + /* Make captured controls discrete to play back user input exactly. */ + for (MidiModel::Controls::iterator i = _model->controls().begin(); i != _model->controls().end(); ++i) { + if (i->second->list()) { + i->second->list()->set_interpolation(Evoral::ControlList::Discrete); + _interpolation_style.insert(std::make_pair(i->second->parameter(), Evoral::ControlList::Discrete)); + } + } } _writing = false; } -boost::shared_ptr -MidiSource::clone (Evoral::MusicalTime begin, Evoral::MusicalTime end) +void +MidiSource::mark_streaming_write_completed (const Lock& lock) { - string newname = PBD::basename_nosuffix(_name.val()); - string newpath; - - /* get a new name for the MIDI file we're going to write to - */ - - do { + mark_midi_streaming_write_completed (lock, Evoral::Sequence::DeleteStuckNotes); +} - newname = bump_name_once (newname, '-'); - /* XXX build path safely */ - newpath = _session.session_directory().midi_path().to_string() +"/"+ newname + ".mid"; +int +MidiSource::write_to (const Lock& lock, boost::shared_ptr newsrc, Evoral::MusicalTime begin, Evoral::MusicalTime end) +{ + Lock newsrc_lock (newsrc->mutex ()); - } while (Glib::file_test (newpath, Glib::FILE_TEST_EXISTS)); - - boost::shared_ptr newsrc = boost::dynamic_pointer_cast( - SourceFactory::createWritable(DataType::MIDI, _session, - newpath, false, _session.frame_rate())); - - newsrc->set_timeline_position(_timeline_position); + 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(); - } - - newsrc->flush_midi(); - - /* force a reload of the model if the range is partial */ - - if (begin != Evoral::MinMusicalTime || end != Evoral::MaxMusicalTime) { - newsrc->load_model (true, true); - } else { - newsrc->set_model (_model); + if (_model) { + if (begin == Evoral::MinMusicalTime && end == Evoral::MaxMusicalTime) { + _model->write_to (newsrc, newsrc_lock); + } else { + _model->write_section_to (newsrc, newsrc_lock, begin, end); + } + } else { + error << string_compose (_("programming error: %1"), X_("no model for MidiSource during ::clone()")); + return -1; } - - return newsrc; -} -void -MidiSource::session_saved() -{ - /* this writes a copy of the data to disk. - XXX do we need to do this every time? - */ + newsrc->flush_midi(newsrc_lock); - if (_model && _model->edited()) { + /* force a reload of the model if the range is partial */ + if (begin != Evoral::MinMusicalTime || end != Evoral::MaxMusicalTime) { + newsrc->load_model (newsrc_lock, true); + } else { + newsrc->set_model (newsrc_lock, _model); + } + /* this file is not removable (but since it is MIDI, it is mutable) */ - // if the model is edited, write its contents into - // the current source file (overwiting previous contents. + boost::dynamic_pointer_cast (newsrc)->prevent_deletion (); - /* 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. - */ + return 0; +} - boost::shared_ptr mm = _model ; - _model.reset (); +void +MidiSource::session_saved() +{ + Lock lm (_lock); + + /* this writes a copy of the data to disk. + XXX do we need to do this every time? + */ - /* flush model contents to disk - */ + if (_model && _model->edited()) { + /* The model is edited, write its contents into the current source + file (overwiting previous contents). */ - mm->sync_to_source (); + /* 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. */ + boost::shared_ptr mm = _model; + _model.reset (); - /* reacquire model */ + /* Flush model contents to disk. */ + mm->sync_to_source (lm); - _model = mm; + /* Reacquire model. */ + _model = mm; - } else { - flush_midi(); - } + } else { + flush_midi(lm); + } } void -MidiSource::set_note_mode(NoteMode mode) +MidiSource::set_note_mode(const Lock& lock, NoteMode mode) { if (_model) { _model->set_note_mode(mode); @@ -385,20 +397,21 @@ MidiSource::set_note_mode(NoteMode mode) } void -MidiSource::drop_model () +MidiSource::drop_model (const Lock& lock) { - _model.reset(); + _model.reset(); + invalidate(lock); ModelChanged (); /* EMIT SIGNAL */ } void -MidiSource::set_model (boost::shared_ptr m) +MidiSource::set_model (const Lock& lock, boost::shared_ptr m) { _model = m; + invalidate(lock); ModelChanged (); /* EMIT SIGNAL */ } -/** @return Interpolation style that should be used for control parameter \a p */ Evoral::ControlList::InterpolationStyle MidiSource::interpolation_of (Evoral::Parameter p) const { @@ -415,7 +428,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; @@ -430,7 +447,7 @@ MidiSource::set_interpolation_of (Evoral::Parameter p, Evoral::ControlList::Inte if (interpolation_of (p) == s) { return; } - + if (EventTypeMap::instance().interpolation_of (p) == s) { /* interpolation type is being set to the default, so we don't need a note in our map */ _interpolation_style.erase (p); @@ -447,8 +464,8 @@ MidiSource::set_automation_state_of (Evoral::Parameter p, AutoState s) if (automation_state_of (p) == 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 {