X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=libs%2Fardour%2Fmidi_region.cc;h=eef2811c5b963c7b438862cb02486ba86f0c783c;hb=c8c6bca6587450ff64303dbc994a4cd28d6ce7aa;hp=f22a1994bf65c70b2fcabc2f1b905d93c5ddec86;hpb=79f91c7a205d981d2b8cc15e32a6da02d8423065;p=ardour.git diff --git a/libs/ardour/midi_region.cc b/libs/ardour/midi_region.cc index f22a1994bf..eef2811c5b 100644 --- a/libs/ardour/midi_region.cc +++ b/libs/ardour/midi_region.cc @@ -24,141 +24,310 @@ #include -#include -#include +#include +#include +#include -#include +#include "evoral/Beats.hpp" -#include "pbd/basename.h" #include "pbd/xml++.h" -#include "pbd/enumwriter.h" +#include "pbd/basename.h" +#include "ardour/automation_control.h" +#include "ardour/midi_model.h" #include "ardour/midi_region.h" -#include "ardour/session.h" -#include "ardour/gain.h" -#include "ardour/dB.h" -#include "ardour/playlist.h" +#include "ardour/midi_ring_buffer.h" #include "ardour/midi_source.h" +#include "ardour/region_factory.h" +#include "ardour/session.h" +#include "ardour/source_factory.h" +#include "ardour/tempo.h" #include "ardour/types.h" -#include "ardour/midi_ring_buffer.h" -#include "i18n.h" +#include "pbd/i18n.h" #include using namespace std; using namespace ARDOUR; +using namespace PBD; -/** Basic MidiRegion constructor (one channel) */ -MidiRegion::MidiRegion (boost::shared_ptr src, nframes_t start, nframes_t length) - : Region (src, start, length, PBD::basename_nosuffix(src->name()), DataType::MIDI, 0, Region::Flag(Region::DefaultFlags|Region::External)) +namespace ARDOUR { + namespace Properties { + PBD::PropertyDescriptor start_beats; + PBD::PropertyDescriptor length_beats; + } +} + +void +MidiRegion::make_property_quarks () { - assert(_name.find("/") == string::npos); - midi_source(0)->Switched.connect(sigc::mem_fun(this, &MidiRegion::switch_source)); + Properties::start_beats.property_id = g_quark_from_static_string (X_("start-beats")); + DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for start-beats = %1\n", Properties::start_beats.property_id)); + Properties::length_beats.property_id = g_quark_from_static_string (X_("length-beats")); + DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for length-beats = %1\n", Properties::length_beats.property_id)); } -/* Basic MidiRegion constructor (one channel) */ -MidiRegion::MidiRegion (boost::shared_ptr src, nframes_t start, nframes_t length, const string& name, layer_t layer, Flag flags) - : Region (src, start, length, name, DataType::MIDI, layer, flags) +void +MidiRegion::register_properties () { - assert(_name.find("/") == string::npos); - midi_source(0)->Switched.connect(sigc::mem_fun(this, &MidiRegion::switch_source)); + add_property (_start_beats); + add_property (_length_beats); } /* Basic MidiRegion constructor (many channels) */ -MidiRegion::MidiRegion (const SourceList& srcs, nframes_t start, nframes_t length, const string& name, layer_t layer, Flag flags) - : Region (srcs, start, length, name, DataType::MIDI, layer, flags) +MidiRegion::MidiRegion (const SourceList& srcs) + : Region (srcs) + , _start_beats (Properties::start_beats, Evoral::Beats()) + , _length_beats (Properties::length_beats, midi_source(0)->length_beats()) { - assert(_name.find("/") == string::npos); - midi_source(0)->Switched.connect(sigc::mem_fun(this, &MidiRegion::switch_source)); + register_properties (); + + midi_source(0)->ModelChanged.connect_same_thread (_source_connection, boost::bind (&MidiRegion::model_changed, this)); + model_changed (); + assert(_name.val().find("/") == string::npos); + assert(_type == DataType::MIDI); } +MidiRegion::MidiRegion (boost::shared_ptr other) + : Region (other) + , _start_beats (Properties::start_beats, other->_start_beats) + , _length_beats (Properties::length_beats, other->_length_beats) +{ + //update_length_beats (); + register_properties (); -/** Create a new MidiRegion, that is part of an existing one */ -MidiRegion::MidiRegion (boost::shared_ptr other, nframes_t offset, nframes_t length, const string& name, layer_t layer, Flag flags) - : Region (other, offset, length, name, layer, flags) + assert(_name.val().find("/") == string::npos); + midi_source(0)->ModelChanged.connect_same_thread (_source_connection, boost::bind (&MidiRegion::model_changed, this)); + model_changed (); +} + +/** Create a new MidiRegion that is part of an existing one */ +MidiRegion::MidiRegion (boost::shared_ptr other, frameoffset_t offset, const int32_t sub_num) + : Region (other, offset, sub_num) + , _start_beats (Properties::start_beats, Evoral::Beats()) + , _length_beats (Properties::length_beats, other->_length_beats) { - assert(_name.find("/") == string::npos); - midi_source(0)->Switched.connect(sigc::mem_fun(this, &MidiRegion::switch_source)); + _start_beats = Evoral::Beats (_session.tempo_map().exact_beat_at_frame ((other->_position + offset), sub_num) - other->beat()) + other->_start_beats; + update_length_beats (sub_num); + register_properties (); + + assert(_name.val().find("/") == string::npos); + midi_source(0)->ModelChanged.connect_same_thread (_source_connection, boost::bind (&MidiRegion::model_changed, this)); + model_changed (); } -MidiRegion::MidiRegion (boost::shared_ptr other) - : Region (other) +MidiRegion::~MidiRegion () { - assert(_name.find("/") == string::npos); - midi_source(0)->Switched.connect(sigc::mem_fun(this, &MidiRegion::switch_source)); } -MidiRegion::MidiRegion (boost::shared_ptr src, const XMLNode& node) - : Region (src, node) +/** Export the MIDI data of the MidiRegion to a new MIDI file (SMF). + */ +bool +MidiRegion::do_export (string path) const { - if (set_state (node)) { - throw failed_constructor(); + boost::shared_ptr newsrc; + + /* caller must check for pre-existing file */ + assert (!path.empty()); + assert (!Glib::file_test (path, Glib::FILE_TEST_EXISTS)); + newsrc = boost::dynamic_pointer_cast( + SourceFactory::createWritable(DataType::MIDI, _session, + path, false, _session.frame_rate())); + + BeatsFramesConverter bfc (_session.tempo_map(), _position); + Evoral::Beats const bbegin = bfc.from (_start); + Evoral::Beats const bend = bfc.from (_start + _length); + + { + /* Lock our source since we'll be reading from it. write_to() will + take a lock on newsrc. */ + Source::Lock lm (midi_source(0)->mutex()); + if (midi_source(0)->export_write_to (lm, newsrc, bbegin, bend)) { + return false; + } } - midi_source(0)->Switched.connect(sigc::mem_fun(this, &MidiRegion::switch_source)); - assert(_name.find("/") == string::npos); - assert(_type == DataType::MIDI); + return true; +} + + +/** Create a new MidiRegion that has its own version of some/all of the Source used by another. + */ +boost::shared_ptr +MidiRegion::clone (string path) const +{ + boost::shared_ptr newsrc; + + /* caller must check for pre-existing file */ + assert (!path.empty()); + assert (!Glib::file_test (path, Glib::FILE_TEST_EXISTS)); + newsrc = boost::dynamic_pointer_cast( + SourceFactory::createWritable(DataType::MIDI, _session, + path, false, _session.frame_rate())); + return clone (newsrc); } -MidiRegion::MidiRegion (const SourceList& srcs, const XMLNode& node) - : Region (srcs, node) +boost::shared_ptr +MidiRegion::clone (boost::shared_ptr newsrc) const { - if (set_state (node)) { - throw failed_constructor(); + BeatsFramesConverter bfc (_session.tempo_map(), _position); + Evoral::Beats const bbegin = bfc.from (_start); + Evoral::Beats const bend = bfc.from (_start + _length); + + { + /* Lock our source since we'll be reading from it. write_to() will + take a lock on newsrc. */ + Source::Lock lm (midi_source(0)->mutex()); + if (midi_source(0)->write_to (lm, newsrc, bbegin, bend)) { + return boost::shared_ptr (); + } } - midi_source(0)->Switched.connect(sigc::mem_fun(this, &MidiRegion::switch_source)); - assert(_name.find("/") == string::npos); - assert(_type == DataType::MIDI); + PropertyList plist; + + plist.add (Properties::name, PBD::basename_nosuffix (newsrc->name())); + plist.add (Properties::whole_file, true); + plist.add (Properties::start, _start); + plist.add (Properties::start_beats, _start_beats); + plist.add (Properties::length, _length); + plist.add (Properties::length_beats, _length_beats); + plist.add (Properties::layer, 0); + + boost::shared_ptr ret (boost::dynamic_pointer_cast (RegionFactory::create (newsrc, plist, true))); + ret->set_beat (beat()); + + return ret; } -MidiRegion::~MidiRegion () +void +MidiRegion::post_set (const PropertyChange& pc) +{ + Region::post_set (pc); + + if (pc.contains (Properties::length) && !pc.contains (Properties::length_beats)) { + /* update non-musically */ + update_length_beats (0); + } else if (pc.contains (Properties::start) && !pc.contains (Properties::start_beats)) { + set_start_beats_from_start_frames (); + } +} + +void +MidiRegion::set_start_beats_from_start_frames () +{ + _start_beats = Evoral::Beats (beat() - _session.tempo_map().beat_at_frame (_position - _start)); +} + +void +MidiRegion::set_length_internal (framecnt_t len, const int32_t sub_num) { + Region::set_length_internal (len, sub_num); + update_length_beats (sub_num); } void -MidiRegion::set_position_internal (nframes_t pos, bool allow_bbt_recompute) +MidiRegion::update_after_tempo_map_change (bool /* send */) { - BeatsFramesConverter old_converter(_session, _position - _start); - double length_beats = old_converter.from(_length); + boost::shared_ptr pl (playlist()); - Region::set_position_internal(pos, allow_bbt_recompute); + if (!pl || position_lock_style() != MusicTime) { + return; + } + + const framepos_t old_pos = _position; + const framepos_t old_length = _length; + const framepos_t old_start = _start; + + Region::update_after_tempo_map_change (false); + + /* _start has now been updated. */ + _length = _session.tempo_map().frame_at_beat (beat() + _length_beats.val().to_double()) - _position; + + PropertyChange s_and_l; + if (old_start != _start) { + s_and_l.add (Properties::start); + } + if (old_length != _length) { + s_and_l.add (Properties::length); + } + if (old_pos != _position) { + s_and_l.add (Properties::position); + } + + send_change (s_and_l); +} - BeatsFramesConverter new_converter(_session, pos - _start); +void +MidiRegion::update_length_beats (const int32_t sub_num) +{ + _length_beats = Evoral::Beats (_session.tempo_map().exact_beat_at_frame (_position + _length, sub_num) - beat()); +} + +void +MidiRegion::set_position_internal (framepos_t pos, bool allow_bbt_recompute, const int32_t sub_num) +{ + Region::set_position_internal (pos, allow_bbt_recompute, sub_num); - set_length(new_converter.to(length_beats), 0); + /* set _start to new position in tempo map */ + _start = _position - _session.tempo_map().frame_at_beat (beat() - _start_beats.val().to_double()); + + /* in construction from src */ + if (_length_beats == Evoral::Beats()) { + update_length_beats (sub_num); + } + + /* leave _length_beats alone, and change _length to reflect the state of things + at the new position (tempo map may dictate a different number of frames). + */ + Region::set_length_internal (_session.tempo_map().frame_at_beat (beat() + _length_beats.val().to_double()) - _position, sub_num); } -nframes_t -MidiRegion::read_at (MidiRingBuffer& out, sframes_t position, nframes_t dur, uint32_t chan_n, NoteMode mode) const +framecnt_t +MidiRegion::read_at (Evoral::EventSink& out, + framepos_t position, + framecnt_t dur, + uint32_t chan_n, + NoteMode mode, + MidiStateTracker* tracker, + MidiChannelFilter* filter) const { - return _read_at (_sources, out, position, dur, chan_n, mode); + return _read_at (_sources, out, position, dur, chan_n, mode, tracker, filter); } -nframes_t -MidiRegion::master_read_at (MidiRingBuffer& out, sframes_t position, nframes_t dur, uint32_t chan_n, NoteMode mode) const +framecnt_t +MidiRegion::master_read_at (MidiRingBuffer& out, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode) const { - return _read_at (_master_sources, out, position, dur, chan_n, mode); + return _read_at (_master_sources, out, position, dur, chan_n, mode); /* no tracker */ } -nframes_t -MidiRegion::_read_at (const SourceList& /*srcs*/, MidiRingBuffer& dst, nframes_t position, nframes_t dur, uint32_t chan_n, NoteMode mode) const +framecnt_t +MidiRegion::_read_at (const SourceList& /*srcs*/, + Evoral::EventSink& dst, + framepos_t position, + framecnt_t dur, + uint32_t chan_n, + NoteMode mode, + MidiStateTracker* tracker, + MidiChannelFilter* filter) const { - nframes_t internal_offset = 0; - nframes_t src_offset = 0; - nframes_t to_read = 0; + frameoffset_t internal_offset = 0; + framecnt_t to_read = 0; /* precondition: caller has verified that we cover the desired section */ assert(chan_n == 0); + if (muted()) { + return 0; /* read nothing */ + } + if (position < _position) { + /* we are starting the read from before the start of the region */ internal_offset = 0; - src_offset = _position - position; - dur -= src_offset; + dur -= _position - position; } else { + /* we are starting the read from after the start of the region */ internal_offset = position - _position; - src_offset = 0; } if (internal_offset >= _length) { @@ -169,138 +338,66 @@ MidiRegion::_read_at (const SourceList& /*srcs*/, MidiRingBuffer& dst return 0; /* read nothing */ } - // FIXME: non-opaque MIDI regions not yet supported - assert(opaque()); - - if (muted()) { - return 0; /* read nothing */ - } + boost::shared_ptr src = midi_source(chan_n); - _read_data_count = 0; + Glib::Threads::Mutex::Lock lm(src->mutex()); - boost::shared_ptr src = midi_source(chan_n); - src->set_note_mode(mode); + src->set_note_mode(lm, mode); - nframes_t output_buffer_position = 0; - nframes_t negative_output_buffer_position = 0; - if (_position >= _start) { - // handle resizing of beginnings of regions correctly - output_buffer_position = _position - _start; - } else { - // when _start is greater than _position, we have to subtract - // _start from the note times in the midi source - negative_output_buffer_position = _start; - } + /* + cerr << "MR " << name () << " read @ " << position << " * " << to_read + << " _position = " << _position + << " _start = " << _start + << " intoffset = " << internal_offset + << endl; + */ -#if 0 - cerr << "\t\tsource read from " << _position << " - " << _start << " (" << _position - _start << ") " - << " start in source " << _start << " + " << internal_offset << " (" << _start + internal_offset << ") " - << " dur = " << to_read - << " offset = " << output_buffer_position - << " negoffset = " << negative_output_buffer_position - << endl; -#endif + /* This call reads events from a source and writes them to `dst' timed in session frames */ if (src->midi_read ( + lm, // source lock dst, // destination buffer - _position - _start, // start position of the source in this read context + _position - _start, // start position of the source in session frames _start + internal_offset, // where to start reading in the source to_read, // read duration in frames - output_buffer_position, // the offset in the output buffer - negative_output_buffer_position // amount to substract from note times - ) != to_read) { + tracker, + filter, + _filtered_parameters, + beat(), + _start_beats.val().to_double() + ) != to_read) { return 0; /* "read nothing" */ } - _read_data_count += src->read_data_count(); - return to_read; } XMLNode& -MidiRegion::state (bool full) +MidiRegion::state () { - XMLNode& node (Region::state (full)); - char buf[64]; - char buf2[64]; - LocaleGuard lg (X_("POSIX")); - - node.add_property ("flags", enum_2_string (_flags)); - - // XXX these should move into Region - - for (uint32_t n=0; n < _sources.size(); ++n) { - snprintf (buf2, sizeof(buf2), "source-%d", n); - _sources[n]->id().print (buf, sizeof(buf)); - node.add_property (buf2, buf); - } - - for (uint32_t n=0; n < _master_sources.size(); ++n) { - snprintf (buf2, sizeof(buf2), "master-source-%d", n); - _master_sources[n]->id().print (buf, sizeof (buf)); - node.add_property (buf2, buf); - } - - if (full && _extra_xml) { - node.add_child_copy (*_extra_xml); - } - - return node; + return Region::state (); } int -MidiRegion::set_live_state (const XMLNode& node, Change& what_changed, bool send) +MidiRegion::set_state (const XMLNode& node, int version) { - const XMLProperty *prop; - LocaleGuard lg (X_("POSIX")); - - Region::set_live_state (node, what_changed, false); - - uint32_t old_flags = _flags; - - if ((prop = node.property ("flags")) != 0) { - _flags = Flag (string_2_enum (prop->value(), _flags)); - - //_flags = Flag (strtol (prop->value().c_str(), (char **) 0, 16)); - - _flags = Flag (_flags & ~Region::LeftOfSplit); - _flags = Flag (_flags & ~Region::RightOfSplit); - } - - if ((old_flags ^ _flags) & Muted) { - what_changed = Change (what_changed|MuteChanged); - } - if ((old_flags ^ _flags) & Opaque) { - what_changed = Change (what_changed|OpacityChanged); - } - if ((old_flags ^ _flags) & Locked) { - what_changed = Change (what_changed|LockChanged); - } + int ret = Region::set_state (node, version); - if (send) { - send_change (what_changed); + if (ret == 0) { + /* set length beats to the frame (non-musical) */ + if (position_lock_style() == AudioTime) { + update_length_beats (0); + } } - return 0; -} - -int -MidiRegion::set_state (const XMLNode& node, int version) -{ - /* Region::set_state() calls the virtual set_live_state(), - which will get us back to AudioRegion::set_live_state() - to handle the relevant stuff. - */ - - return Region::set_state (node); + return ret; } void MidiRegion::recompute_at_end () { /* our length has changed - * (non destructively) "chop" notes that pass the end boundary, to - * prevent stuck notes. + * so what? stuck notes are dealt with via a note state tracker */ } @@ -323,10 +420,28 @@ MidiRegion::separate_by_channel (ARDOUR::Session&, vector< boost::shared_ptr +MidiRegion::control (const Evoral::Parameter& id, bool create) { - return -1; + return model()->control(id, create); +} + +boost::shared_ptr +MidiRegion::control (const Evoral::Parameter& id) const +{ + return model()->control(id); +} + +boost::shared_ptr +MidiRegion::model() +{ + return midi_source()->model(); +} + +boost::shared_ptr +MidiRegion::model() const +{ + return midi_source()->model(); } boost::shared_ptr @@ -336,18 +451,145 @@ MidiRegion::midi_source (uint32_t n) const return boost::dynamic_pointer_cast(source(n)); } - void -MidiRegion::switch_source(boost::shared_ptr src) +MidiRegion::model_changed () { - boost::shared_ptr msrc = boost::dynamic_pointer_cast(src); - if (!msrc) + if (!model()) { return; + } + + /* build list of filtered Parameters, being those whose automation state is not `Play' */ + + _filtered_parameters.clear (); + + Automatable::Controls const & c = model()->controls(); + + for (Automatable::Controls::const_iterator i = c.begin(); i != c.end(); ++i) { + boost::shared_ptr ac = boost::dynamic_pointer_cast (i->second); + assert (ac); + if (ac->alist()->automation_state() != Play) { + _filtered_parameters.insert (ac->parameter ()); + } + } + + /* watch for changes to controls' AutoState */ + midi_source()->AutomationStateChanged.connect_same_thread ( + _model_connection, boost::bind (&MidiRegion::model_automation_state_changed, this, _1) + ); +} + +void +MidiRegion::model_automation_state_changed (Evoral::Parameter const & p) +{ + /* Update our filtered parameters list after a change to a parameter's AutoState */ - // MIDI regions have only one source - _sources.clear(); - _sources.push_back(msrc); + boost::shared_ptr ac = model()->automation_control (p); + if (!ac || ac->alist()->automation_state() == Play) { + /* It should be "impossible" for ac to be NULL, but if it is, don't + filter the parameter so events aren't lost. */ + _filtered_parameters.erase (p); + } else { + _filtered_parameters.insert (p); + } + + /* the source will have an iterator into the model, and that iterator will have been set up + for a given set of filtered_parameters, so now that we've changed that list we must invalidate + the iterator. + */ + Glib::Threads::Mutex::Lock lm (midi_source(0)->mutex(), Glib::Threads::TRY_LOCK); + if (lm.locked()) { + /* TODO: This is too aggressive, we need more fine-grained invalidation. */ + midi_source(0)->invalidate (lm); + } +} + +/** This is called when a trim drag has resulted in a -ve _start time for this region. + * Fix it up by adding some empty space to the source. + */ +void +MidiRegion::fix_negative_start () +{ + BeatsFramesConverter c (_session.tempo_map(), _position); + + model()->insert_silence_at_start (c.from (-_start)); + _start = 0; + _start_beats = Evoral::Beats(); +} - set_name(msrc->name()); +void +MidiRegion::set_start_internal (framecnt_t s, const int32_t sub_num) +{ + Region::set_start_internal (s, sub_num); + + if (position_lock_style() == AudioTime) { + set_start_beats_from_start_frames (); + } } +void +MidiRegion::trim_to_internal (framepos_t position, framecnt_t length, const int32_t sub_num) +{ + framepos_t new_start; + + if (locked()) { + return; + } + + PropertyChange what_changed; + + /* beat has been set exactly by set_position_internal, but the source starts on a frame. + working in beats seems the correct thing to do, but reports of a missing first note + on playback suggest otherwise. for now, we work in exact beats. + */ + const double pos_beat = _session.tempo_map().exact_beat_at_frame (position, sub_num); + const double beat_delta = pos_beat - beat(); + + /* Set position before length, otherwise for MIDI regions this bad thing happens: + * 1. we call set_length_internal; length in beats is computed using the region's current + * (soon-to-be old) position + * 2. we call set_position_internal; position is set and length in frames re-computed using + * length in beats from (1) but at the new position, which is wrong if the region + * straddles a tempo/meter change. + */ + + if (_position != position) { + set_position_internal (position, true, sub_num); + what_changed.add (Properties::position); + } + + const double new_start_beat = _start_beats.val().to_double() + beat_delta; + new_start = _position - _session.tempo_map().frame_at_beat (beat() - new_start_beat); + + if (!verify_start_and_length (new_start, length)) { + return; + } + + if (_start != new_start) { + _start_beats = Evoral::Beats (new_start_beat); + what_changed.add (Properties::start_beats); + + set_start_internal (new_start, sub_num); + what_changed.add (Properties::start); + } + + if (_length != length) { + set_length_internal (length, sub_num); + what_changed.add (Properties::length); + what_changed.add (Properties::length_beats); + } + + set_whole_file (false); + + PropertyChange start_and_length; + + start_and_length.add (Properties::start); + start_and_length.add (Properties::length); + + if (what_changed.contains (start_and_length)) { + first_edit (); + } + + if (!what_changed.empty()) { + send_change (what_changed); + } +}