X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=libs%2Fardour%2Fmidi_region.cc;h=cb0d103b1e58d0f45f2674cb296d06579b6c1b26;hb=a9936205b182faf1b32b06dbf312b4c9c77f4872;hp=ffb521713743f995dc5d6828fe1f747d8da32f65;hpb=10bdce85a0e7381d1b5db38e3640600c6fd3ec79;p=ardour.git diff --git a/libs/ardour/midi_region.cc b/libs/ardour/midi_region.cc index ffb5217137..cb0d103b1e 100644 --- a/libs/ardour/midi_region.cc +++ b/libs/ardour/midi_region.cc @@ -24,21 +24,23 @@ #include -#include +#include +#include +#include -#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 @@ -49,44 +51,72 @@ using namespace PBD; namespace ARDOUR { namespace Properties { - PBD::PropertyDescriptor midi_data; - } + PBD::PropertyDescriptor midi_data; + PBD::PropertyDescriptor start_beats; + PBD::PropertyDescriptor length_beats; + } } void MidiRegion::make_property_quarks () { - Properties::midi_data.property_id = g_quark_from_static_string (X_("midi-data")); - DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for midi-data = %1\n", Properties::midi_data.property_id)); + Properties::midi_data.property_id = g_quark_from_static_string (X_("midi-data")); + DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for midi-data = %1\n", Properties::midi_data.property_id)); + 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)); } void MidiRegion::register_properties () { - /* none yet, but its only a matter of time */ + add_property (_start_beats); + add_property (_length_beats); } /* Basic MidiRegion constructor (many channels) */ MidiRegion::MidiRegion (const SourceList& srcs) : Region (srcs) + , _start_beats (Properties::start_beats, 0) + , _length_beats (Properties::length_beats, midi_source(0)->length_beats()) { - register_properties (); + register_properties (); - // midi_source(0)->Switched.connect_same_thread (*this, boost::bind (&MidiRegion::switch_source, this, _1)); 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); } -/** Create a new MidiRegion, that is part of an existing one */ -MidiRegion::MidiRegion (boost::shared_ptr other, frameoffset_t offset, bool offset_relative) - : Region (other, offset, offset_relative) +MidiRegion::MidiRegion (boost::shared_ptr other) + : Region (other) + , _start_beats (Properties::start_beats, other->_start_beats) + , _length_beats (Properties::length_beats, (Evoral::MusicalTime) 0) { - register_properties (); + update_length_beats (); + 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 (); +} + +/** Create a new MidiRegion that is part of an existing one */ +MidiRegion::MidiRegion (boost::shared_ptr other, frameoffset_t offset) + : Region (other, offset) + , _start_beats (Properties::start_beats, (Evoral::MusicalTime) 0) + , _length_beats (Properties::length_beats, (Evoral::MusicalTime) 0) +{ + BeatsFramesConverter bfc (_session.tempo_map(), _position); + Evoral::MusicalTime const offset_beats = bfc.from (offset); + + _start_beats = other->_start_beats + offset_beats; + _length_beats = other->_length_beats - offset_beats; + + register_properties (); assert(_name.val().find("/") == string::npos); - // midi_source(0)->Switched.connect_same_thread (*this, boost::bind (&MidiRegion::switch_source, this, _1)); midi_source(0)->ModelChanged.connect_same_thread (_source_connection, boost::bind (&MidiRegion::model_changed, this)); model_changed (); } @@ -95,55 +125,120 @@ MidiRegion::~MidiRegion () { } -/** Create a new MidiRegion that has its own version of some/all of the Source used by another. +/** Create a new MidiRegion that has its own version of some/all of the Source used by another. */ boost::shared_ptr -MidiRegion::clone () +MidiRegion::clone (string path) const { - BeatsFramesConverter bfc (_session.tempo_map(), _position); - double bbegin = bfc.from (_position); - double bend = bfc.from (last_frame() + 1); + 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); +} - boost::shared_ptr ms = midi_source(0)->clone (bbegin, bend); +boost::shared_ptr +MidiRegion::clone (boost::shared_ptr newsrc) const +{ + BeatsFramesConverter bfc (_session.tempo_map(), _position); + Evoral::MusicalTime const bbegin = bfc.from (_start); + Evoral::MusicalTime const bend = bfc.from (_start + _length); + + if (midi_source(0)->write_to (newsrc, bbegin, bend)) { + return boost::shared_ptr (); + } - PropertyList plist; + PropertyList plist; - plist.add (Properties::name, ms->name()); - plist.add (Properties::whole_file, true); - plist.add (Properties::start, 0); - plist.add (Properties::length, _length); - plist.add (Properties::layer, 0); + 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); - return boost::dynamic_pointer_cast (RegionFactory::create (ms, plist, true)); + return boost::dynamic_pointer_cast (RegionFactory::create (newsrc, plist, true)); } void -MidiRegion::set_position_internal (framepos_t pos, bool allow_bbt_recompute) +MidiRegion::post_set (const PropertyChange& pc) { - BeatsFramesConverter old_converter(_session.tempo_map(), _position - _start); - double length_beats = old_converter.from(_length); + Region::post_set (pc); - Region::set_position_internal(pos, allow_bbt_recompute); + if (pc.contains (Properties::length) && !pc.contains (Properties::length_beats)) { + update_length_beats (); + } 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 () +{ + BeatsFramesConverter c (_session.tempo_map(), _position - _start); + _start_beats = c.from (_start); +} + +void +MidiRegion::set_length_internal (framecnt_t len) +{ + Region::set_length_internal (len); + update_length_beats (); +} + +void +MidiRegion::update_after_tempo_map_change () +{ + Region::update_after_tempo_map_change (); + + /* _position has now been updated for the new tempo map */ + _start = _position - _session.tempo_map().framepos_minus_beats (_position, _start_beats); - BeatsFramesConverter new_converter(_session.tempo_map(), pos - _start); + send_change (Properties::start); +} - set_length(new_converter.to(length_beats), 0); +void +MidiRegion::update_length_beats () +{ + BeatsFramesConverter converter (_session.tempo_map(), _position); + _length_beats = converter.from (_length); +} + +void +MidiRegion::set_position_internal (framepos_t pos, bool allow_bbt_recompute) +{ + Region::set_position_internal (pos, allow_bbt_recompute); + /* zero length regions don't exist - so if _length_beats is zero, this object + is under construction. + */ + if (_length_beats) { + /* 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 + */ + BeatsFramesConverter converter (_session.tempo_map(), _position); + Region::set_length_internal (converter.to (_length_beats)); + } } framecnt_t -MidiRegion::read_at (Evoral::EventSink& out, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode, MidiStateTracker* tracker) const +MidiRegion::read_at (Evoral::EventSink& out, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode, MidiStateTracker* tracker) const { return _read_at (_sources, out, position, dur, chan_n, mode, tracker); } framecnt_t -MidiRegion::master_read_at (MidiRingBuffer& out, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode) const +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); /* no tracker */ } framecnt_t -MidiRegion::_read_at (const SourceList& /*srcs*/, Evoral::EventSink& dst, framepos_t position, framecnt_t dur, uint32_t chan_n, +MidiRegion::_read_at (const SourceList& /*srcs*/, Evoral::EventSink& dst, framepos_t position, framecnt_t dur, uint32_t chan_n, NoteMode mode, MidiStateTracker* tracker) const { frameoffset_t internal_offset = 0; @@ -174,18 +269,16 @@ MidiRegion::_read_at (const SourceList& /*srcs*/, Evoral::EventSink& return 0; /* read nothing */ } - _read_data_count = 0; - boost::shared_ptr src = midi_source(chan_n); src->set_note_mode(mode); - - /* - cerr << "MR read @ " << position << " * " << to_read - << " _position = " << _position - << " _start = " << _start - << " intoffset = " << internal_offset - << endl; - */ + + /* + cerr << "MR " << name () << " read @ " << position << " * " << to_read + << " _position = " << _position + << " _start = " << _start + << " intoffset = " << internal_offset + << endl; + */ /* This call reads events from a source and writes them to `dst' timed in session frames */ @@ -200,8 +293,6 @@ MidiRegion::_read_at (const SourceList& /*srcs*/, Evoral::EventSink& return 0; /* "read nothing" */ } - _read_data_count += src->read_data_count(); - return to_read; } @@ -214,14 +305,20 @@ MidiRegion::state () int MidiRegion::set_state (const XMLNode& node, int version) { - return Region::set_state (node, version); + int ret = Region::set_state (node, version); + + if (ret == 0) { + update_length_beats (); + } + + return ret; } void MidiRegion::recompute_at_end () { /* our length has changed - * so what? stuck notes are dealt with via a note state tracker + * so what? stuck notes are dealt with via a note state tracker */ } @@ -244,40 +341,35 @@ 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::midi_source (uint32_t n) const +boost::shared_ptr +MidiRegion::control (const Evoral::Parameter& id) const { - // Guaranteed to succeed (use a static cast?) - return boost::dynamic_pointer_cast(source(n)); + return model()->control(id); } - -void -MidiRegion::switch_source(boost::shared_ptr src) +boost::shared_ptr +MidiRegion::model() { - _source_connection.disconnect (); - - boost::shared_ptr msrc = boost::dynamic_pointer_cast(src); - if (!msrc) { - return; - } - - // MIDI regions have only one source - SourceList srcs; - srcs.push_back (msrc); + return midi_source()->model(); +} - drop_sources (); - use_sources (srcs); - - set_name (msrc->name()); +boost::shared_ptr +MidiRegion::model() const +{ + return midi_source()->model(); +} - msrc->ModelChanged.connect_same_thread (_source_connection, boost::bind (&MidiRegion::model_changed, this)); +boost::shared_ptr +MidiRegion::midi_source (uint32_t n) const +{ + // Guaranteed to succeed (use a static cast?) + return boost::dynamic_pointer_cast(source(n)); } void @@ -286,7 +378,7 @@ MidiRegion::model_changed () if (!model()) { return; } - + /* build list of filtered Parameters, being those whose automation state is not `Play' */ _filtered_parameters.clear (); @@ -306,34 +398,55 @@ MidiRegion::model_changed () _model_connection, boost::bind (&MidiRegion::model_automation_state_changed, this, _1) ); - model()->ContentsChanged.connect_same_thread ( - _model_contents_connection, boost::bind (&MidiRegion::model_contents_changed, this)); + model()->ContentsChanged.connect_same_thread ( + _model_contents_connection, boost::bind (&MidiRegion::model_contents_changed, this)); } void MidiRegion::model_contents_changed () { - send_change (PropertyChange (Properties::midi_data)); + send_change (PropertyChange (Properties::midi_data)); } void MidiRegion::model_automation_state_changed (Evoral::Parameter const & p) { /* Update our filtered parameters list after a change to a parameter's AutoState */ - - boost::shared_ptr ac = model()->automation_control (p); - assert (ac); - if (ac->alist()->automation_state() == Play) { + 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_paramters, so now that we've changed that list we must invalidate - the iterator. - */ - Glib::Mutex::Lock lm (midi_source(0)->mutex()); - midi_source(0)->invalidate (); +/** 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 = 0; +} + +/** Transpose the notes in this region by a given number of semitones */ +void +MidiRegion::transpose (int semitones) +{ + BeatsFramesConverter c (_session.tempo_map(), _start); + model()->transpose (c.from (_start), c.from (_start + _length), semitones); +} + +void +MidiRegion::set_start_internal (framecnt_t s) +{ + Region::set_start_internal (s); + set_start_beats_from_start_frames (); }