X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=libs%2Fardour%2Fsmf_source.cc;h=52377ee9154c74a5af959af2499ef4443b7201f4;hb=bc3e1a212250a67e53e86c972287a275834a967a;hp=5706b6a5d7a6b07bedbde4935b23be3505c8c3a5;hpb=01b761bbaa1dd7af210b20f47530efe31e9edc76;p=ardour.git diff --git a/libs/ardour/smf_source.cc b/libs/ardour/smf_source.cc index 5706b6a5d7..52377ee915 100644 --- a/libs/ardour/smf_source.cc +++ b/libs/ardour/smf_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 @@ -32,7 +32,6 @@ #include -#include "evoral/SMFReader.hpp" #include "evoral/Control.hpp" #include "ardour/audioengine.h" @@ -42,31 +41,32 @@ #include "ardour/midi_state_tracker.h" #include "ardour/session.h" #include "ardour/smf_source.h" +#include "ardour/debug.h" #include "i18n.h" using namespace ARDOUR; using namespace Glib; +using namespace PBD; /** Constructor used for new internal-to-session files. File cannot exist. */ -SMFSource::SMFSource (Session& s, const ustring& path, bool embedded, Source::Flag flags) +SMFSource::SMFSource (Session& s, const string& path, Source::Flag flags) : Source(s, DataType::MIDI, path, flags) - , MidiSource(s, path) - , FileSource(s, DataType::MIDI, path, embedded, flags) + , MidiSource(s, path, flags) + , FileSource(s, DataType::MIDI, path, string(), flags) , Evoral::SMF() , _last_ev_time_beats(0.0) , _last_ev_time_frames(0) , _smf_last_read_end (0) + , _smf_last_read_time (0) { - if (init(_name, false)) { - throw failed_constructor (); - } + /* note that origin remains empty */ - if (create(path)) { + if (init(_path, false)) { throw failed_constructor (); } - load_model(true, true); // FIXME + /* file is not opened until write */ } /** Constructor used for existing internal-to-session files. */ @@ -77,12 +77,13 @@ SMFSource::SMFSource (Session& s, const XMLNode& node, bool must_exist) , _last_ev_time_beats(0.0) , _last_ev_time_frames(0) , _smf_last_read_end (0) + , _smf_last_read_time (0) { if (set_state(node, Stateful::loading_state_version)) { throw failed_constructor (); } - if (init(_name, true)) { + if (init(_path, true)) { throw failed_constructor (); } @@ -90,7 +91,7 @@ SMFSource::SMFSource (Session& s, const XMLNode& node, bool must_exist) throw failed_constructor (); } - load_model(true, true); // FIXME + _open = true; } SMFSource::~SMFSource () @@ -100,17 +101,31 @@ SMFSource::~SMFSource () } } +int +SMFSource::open_for_write () +{ + if (create (_path)) { + return -1; + } + _open = true; + return 0; +} + /** All stamps in audio frames */ -nframes_t -SMFSource::read_unlocked (Evoral::EventSink& destination, sframes_t source_start, - sframes_t start, nframes_t duration, - sframes_t stamp_offset, sframes_t negative_stamp_offset, +framecnt_t +SMFSource::read_unlocked (Evoral::EventSink& destination, framepos_t const source_start, + framepos_t start, framecnt_t duration, MidiStateTracker* tracker) const { int ret = 0; uint64_t time = 0; // in SMF ticks, 1 tick per _ppqn - _read_data_count = 0; + if (writable() && !_open) { + /* nothing to read since nothing has ben written */ + return duration; + } + + DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked: start %1 duration %2\n", start, duration)); // Output parameters for read_event (which will allocate scratch in buffer as needed) uint32_t ev_delta_t = 0; @@ -123,29 +138,38 @@ SMFSource::read_unlocked (Evoral::EventSink& destination, sframes_t s BeatsFramesConverter converter(_session.tempo_map(), source_start); const uint64_t start_ticks = (uint64_t)(converter.from(start) * ppqn()); + DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked: start in ticks %1\n", start_ticks)); if (_smf_last_read_end == 0 || start != _smf_last_read_end) { - //cerr << "SMFSource::read_unlocked seeking to " << start << endl; + DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked: seek to %1\n", start)); Evoral::SMF::seek_to_start(); while (time < start_ticks) { - ret = read_event(&ev_delta_t, &ev_size, &ev_buffer); + gint ignored; + + ret = read_event(&ev_delta_t, &ev_size, &ev_buffer, &ignored); if (ret == -1) { // EOF _smf_last_read_end = start + duration; return duration; } time += ev_delta_t; // accumulate delta time } + } else { + DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked: set time to %1\n", _smf_last_read_time)); + time = _smf_last_read_time; } _smf_last_read_end = start + duration; while (true) { - ret = read_event(&ev_delta_t, &ev_size, &ev_buffer); + gint ignored; /* XXX don't ignore note id's ??*/ + + ret = read_event(&ev_delta_t, &ev_size, &ev_buffer, &ignored); if (ret == -1) { // EOF break; } time += ev_delta_t; // accumulate delta time + _smf_last_read_time = time; if (ret == 0) { // meta-event (skipped, just accumulate time) continue; @@ -153,25 +177,18 @@ SMFSource::read_unlocked (Evoral::EventSink& destination, sframes_t s ev_type = EventTypeMap::instance().midi_event_type(ev_buffer[0]); -#if 0 - cerr << "+++ SMF source read " - << " delta = " << ev_delta_t - << " time = " << time - << " buf[0] " << hex << (int) ev_buffer[0] << dec - << " type = " << ev_type; -#endif + DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF read_unlocked delta %1, time %2, buf[0] %3, type %4\n", + ev_delta_t, time, ev_buffer[0], ev_type)); assert(time >= start_ticks); - const sframes_t ev_frame_time = converter.to(time / (double)ppqn()) + stamp_offset; -#if 0 - cerr << " frames = " << ev_frame_time - << " w/offset = " << ev_frame_time - negative_stamp_offset - << endl; -#endif + /* Note that we add on the source start time (in session frames) here so that ev_frame_time + is in session frames. + */ + const framepos_t ev_frame_time = converter.to(time / (double)ppqn()) + source_start; if (ev_frame_time < start + duration) { - destination.write(ev_frame_time - negative_stamp_offset, ev_type, ev_size, ev_buffer); + destination.write (ev_frame_time, ev_type, ev_size, ev_buffer); if (tracker) { if (ev_buffer[0] & MIDI_CMD_NOTE_ON) { @@ -184,8 +201,6 @@ SMFSource::read_unlocked (Evoral::EventSink& destination, sframes_t s break; } - _read_data_count += ev_size; - if (ev_size > scratch_size) { scratch_size = ev_size; } @@ -195,13 +210,18 @@ SMFSource::read_unlocked (Evoral::EventSink& destination, sframes_t s return duration; } -/** All stamps in audio frames */ -nframes_t -SMFSource::write_unlocked (MidiRingBuffer& source, sframes_t position, nframes_t duration) +/** Write data to this source from a MidiRingBuffer. + * @param source Buffer to read from. + * @param position This source's start position in session frames. + */ +framecnt_t +SMFSource::write_unlocked (MidiRingBuffer& source, framepos_t position, framecnt_t duration) { - _write_data_count = 0; + if (!_writing) { + mark_streaming_write_started (); + } - nframes_t time; + framepos_t time; Evoral::EventType type; uint32_t size; @@ -212,17 +232,26 @@ SMFSource::write_unlocked (MidiRingBuffer& source, sframes_t position _model->start_write(); } - Evoral::MIDIEvent ev; + Evoral::MIDIEvent ev; + + // cerr << "SMFSource::write unlocked, begins writing from src buffer with _last_write_end = " << _last_write_end << " dur = " << duration << endl; while (true) { - bool ret = source.peek_time(&time); - if (!ret || time > _last_write_end + duration) { + bool ret; + + if (!(ret = source.peek ((uint8_t*)&time, sizeof (time)))) { + /* no more events to consider */ break; } - ret = source.read_prefix(&time, &type, &size); - if (!ret) { - cerr << "ERROR: Unable to read event prefix, corrupt MIDI ring buffer" << endl; + if ((duration != max_framecnt) && (time > (_last_write_end + duration))) { + DEBUG_TRACE (DEBUG::MidiIO, string_compose ("SMFSource::write_unlocked: dropping event @ %1 because it is later than %2 + %3\n", + time, _last_write_end, duration)); + break; + } + + if (!(ret = source.read_prefix (&time, &type, &size))) { + error << _("Unable to read event prefix, corrupt MIDI ring buffer") << endmsg; break; } @@ -233,15 +262,18 @@ SMFSource::write_unlocked (MidiRingBuffer& source, sframes_t position ret = source.read_contents(size, buf); if (!ret) { - cerr << "ERROR: Read time/size but not buffer, corrupt MIDI ring buffer" << endl; + error << _("Read time/size but not buffer, corrupt MIDI ring buffer") << endmsg; break; } + /* convert from session time to time relative to the source start */ assert(time >= position); time -= position; ev.set(buf, size, time); ev.set_event_type(EventTypeMap::instance().midi_event_type(ev.buffer()[0])); + ev.set_id (Evoral::next_event_id()); + if (!(ev.is_channel_event() || ev.is_smf_meta_event() || ev.is_sysex())) { /*cerr << "SMFSource: WARNING: caller tried to write non SMF-Event of type " << std::hex << int(ev.buffer()[0]) << endl;*/ @@ -251,14 +283,8 @@ SMFSource::write_unlocked (MidiRingBuffer& source, sframes_t position append_event_unlocked_frames(ev, position); } - if (_model) { - set_default_controls_interpolation(); - } - Evoral::SMF::flush(); - free(buf); - - ViewDataRangeReady(position + _last_write_end, duration); /* EMIT SIGNAL */ + free (buf); return duration; } @@ -273,9 +299,9 @@ SMFSource::append_event_unlocked_beats (const Evoral::Event& ev) return; } - /*printf("SMFSource: %s - append_event_unlocked_beats time = %lf, size = %u, data = ", - name().c_str(), ev.time(), ev.size()); - for (size_t i = 0; i < ev.size(); ++i) printf("%X ", ev.buffer()[i]); printf("\n");*/ + /*printf("SMFSource: %s - append_event_unlocked_beats ID = %d time = %lf, size = %u, data = ", + name().c_str(), ev.id(), ev.time(), ev.size()); + for (size_t i = 0; i < ev.size(); ++i) printf("%X ", ev.buffer()[i]); printf("\n");*/ assert(ev.time() >= 0); if (ev.time() < _last_ev_time_beats) { @@ -283,33 +309,40 @@ SMFSource::append_event_unlocked_beats (const Evoral::Event& ev) return; } + Evoral::event_id_t event_id; + + if (ev.id() < 0) { + event_id = Evoral::next_event_id(); + } else { + event_id = ev.id(); + } + + if (_model) { + _model->append (ev, event_id); + } + _length_beats = max(_length_beats, ev.time()); const double delta_time_beats = ev.time() - _last_ev_time_beats; const uint32_t delta_time_ticks = (uint32_t)lrint(delta_time_beats * (double)ppqn()); - Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer()); + Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer(), event_id); _last_ev_time_beats = ev.time(); - - _write_data_count += ev.size(); - - if (_model) { - _model->append(ev); - } } -/** Append an event with a timestamp in frames (nframes_t) */ +/** Append an event with a timestamp in frames (framepos_t) */ void -SMFSource::append_event_unlocked_frames (const Evoral::Event& ev, sframes_t position) +SMFSource::append_event_unlocked_frames (const Evoral::Event& ev, framepos_t position) { assert(_writing); if (ev.size() == 0) { + cerr << "SMFSource: asked to append zero-size event\n"; return; } - /*printf("SMFSource: %s - append_event_unlocked_frames time = %u, size = %u, data = ", - name().c_str(), ev.time(), ev.size()); - for (size_t i=0; i < ev.size(); ++i) printf("%X ", ev.buffer()[i]); printf("\n");*/ + /* printf("SMFSource: %s - append_event_unlocked_frames ID = %d time = %u, size = %u, data = ", + name().c_str(), ev.id(), ev.time(), ev.size()); + for (size_t i=0; i < ev.size(); ++i) printf("%X ", ev.buffer()[i]); printf("\n");*/ if (ev.time() < _last_ev_time_frames) { cerr << "SMFSource: Warning: Skipping event with non-monotonic time" << endl; @@ -317,30 +350,39 @@ SMFSource::append_event_unlocked_frames (const Evoral::Event& ev, sfr } BeatsFramesConverter converter(_session.tempo_map(), position); + const double ev_time_beats = converter.from(ev.time()); + Evoral::event_id_t event_id; - _length_beats = max(_length_beats, converter.from(ev.time())); + if (ev.id() < 0) { + event_id = Evoral::next_event_id(); + } else { + event_id = ev.id(); + } - const sframes_t delta_time_frames = ev.time() - _last_ev_time_frames; - const double delta_time_beats = converter.from(delta_time_frames); - const uint32_t delta_time_ticks = (uint32_t)(lrint(delta_time_beats * (double)ppqn())); + if (_model) { + const Evoral::Event beat_ev (ev.event_type(), + ev_time_beats, + ev.size(), + (uint8_t*)ev.buffer()); + _model->append (beat_ev, event_id); + } - Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer()); - _last_ev_time_frames = ev.time(); + _length_beats = max(_length_beats, ev_time_beats); - _write_data_count += ev.size(); + const Evoral::MusicalTime last_time_beats = converter.from (_last_ev_time_frames); + const Evoral::MusicalTime delta_time_beats = ev_time_beats - last_time_beats; + const uint32_t delta_time_ticks = (uint32_t)(lrint(delta_time_beats * (double)ppqn())); - if (_model) { - const double ev_time_beats = converter.from(ev.time()); - const Evoral::Event beat_ev( - ev.event_type(), ev_time_beats, ev.size(), (uint8_t*)ev.buffer()); - _model->append(beat_ev); - } + Evoral::SMF::append_event_delta(delta_time_ticks, ev.size(), ev.buffer(), event_id); + _last_ev_time_frames = ev.time(); } XMLNode& SMFSource::get_state () { - return MidiSource::get_state(); + XMLNode& node = MidiSource::get_state(); + node.add_property (X_("origin"), _origin); + return node; } int @@ -362,10 +404,17 @@ SMFSource::set_state (const XMLNode& node, int version) } void -SMFSource::mark_streaming_midi_write_started (NoteMode mode, sframes_t start_frame) +SMFSource::mark_streaming_midi_write_started (NoteMode mode) { - Glib::Mutex::Lock lm (_lock); - MidiSource::mark_streaming_midi_write_started (mode, start_frame); + /* CALLER MUST HOLD LOCK */ + + if (!_open && open_for_write()) { + error << string_compose (_("cannot open MIDI file %1 for write"), _path) << endmsg; + /* XXX should probably throw or return something */ + return; + } + + MidiSource::mark_streaming_midi_write_started (mode); Evoral::SMF::begin_write (); _last_ev_time_beats = 0.0; _last_ev_time_frames = 0; @@ -373,22 +422,35 @@ SMFSource::mark_streaming_midi_write_started (NoteMode mode, sframes_t start_fra void SMFSource::mark_streaming_write_completed () +{ + mark_midi_streaming_write_completed (Evoral::Sequence::DeleteStuckNotes); +} + +void +SMFSource::mark_midi_streaming_write_completed (Evoral::Sequence::StuckNoteOption stuck_notes_option, Evoral::MusicalTime when) { Glib::Mutex::Lock lm (_lock); - MidiSource::mark_streaming_write_completed(); + MidiSource::mark_midi_streaming_write_completed (stuck_notes_option, when); if (!writable()) { return; } - _model->set_edited(false); + if (_model) { + _model->set_edited(false); + } + Evoral::SMF::end_write (); + + /* data in the file now, not removable */ + + mark_nonremovable (); } bool -SMFSource::safe_midi_file_extension (const Glib::ustring& file) +SMFSource::safe_midi_file_extension (const string& file) { - return (file.rfind(".mid") != Glib::ustring::npos); + return (file.rfind(".mid") != string::npos) || (file.rfind (".MID") != string::npos); } void @@ -406,83 +468,141 @@ SMFSource::load_model (bool lock, bool force_reload) return; } - if (! _model) { - _model = boost::shared_ptr(new MidiModel(this)); - //cerr << _name << " loaded new model " << _model.get() << endl; + if (!_model) { + _model = boost::shared_ptr (new MidiModel (shared_from_this ())); } else { - /*cerr << _name << " reloading model " << _model.get() - << " (" << _model->n_notes() << " notes)" << endl;*/ _model->clear(); } + if (writable() && !_open) { + return; + } + _model->start_write(); Evoral::SMF::seek_to_start(); uint64_t time = 0; /* in SMF ticks */ Evoral::Event ev; - size_t scratch_size = 0; // keep track of scratch and minimize reallocs + uint32_t scratch_size = 0; // keep track of scratch and minimize reallocs uint32_t delta_t = 0; uint32_t size = 0; uint8_t* buf = NULL; int ret; - while ((ret = read_event(&delta_t, &size, &buf)) >= 0) { + gint event_id; + bool have_event_id = false; + + while ((ret = read_event (&delta_t, &size, &buf, &event_id)) >= 0) { + time += delta_t; - ev.set(buf, size, time / (double)ppqn()); - if (ret > 0) { // didn't skip (meta) event - ev.set_event_type(EventTypeMap::instance().midi_event_type(buf[0])); - _model->append(ev); + if (ret == 0) { + + /* meta-event : did we get an event ID ? + */ + + if (event_id >= 0) { + have_event_id = true; + } + + continue; } - if (ev.size() > scratch_size) { - scratch_size = ev.size(); + if (ret > 0) { + + /* not a meta-event */ + + ev.set (buf, size, time / (double)ppqn()); + ev.set_event_type(EventTypeMap::instance().midi_event_type(buf[0])); + + if (!have_event_id) { + event_id = Evoral::next_event_id(); + } +#ifndef NDEBUG + std::string ss; + + for (uint32_t xx = 0; xx < size; ++xx) { + char b[8]; + snprintf (b, sizeof (b), "0x%x ", buf[xx]); + ss += b; + } + + DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("SMF %6 load model delta %1, time %2, size %3 buf %4, type %5\n", + delta_t, time, size, ss , ev.event_type(), name())); +#endif + + _model->append (ev, event_id); + + // Set size to max capacity to minimize allocs in read_event + scratch_size = std::max(size, scratch_size); + size = scratch_size; + + _length_beats = max(_length_beats, ev.time()); } - ev.size() = scratch_size; // ensure read_event only allocates if necessary - _length_beats = max(_length_beats, ev.time()); - } + /* event ID's must immediately precede the event they are for + */ - set_default_controls_interpolation(); + have_event_id = false; + } - _model->end_write(false); - _model->set_edited(false); + _model->end_write (Evoral::Sequence::ResolveStuckNotes, _length_beats); + _model->set_edited (false); _model_iter = _model->begin(); free(buf); } -#define LINEAR_INTERPOLATION_MODE_WORKS_PROPERLY 0 +void +SMFSource::destroy_model () +{ + //cerr << _name << " destroying model " << _model.get() << endl; + _model.reset(); +} void -SMFSource::set_default_controls_interpolation () +SMFSource::flush_midi () { - // set interpolation style to defaults, can be changed by the GUI later - Evoral::ControlSet::Controls controls = _model->controls(); - for (Evoral::ControlSet::Controls::iterator c = controls.begin(); c != controls.end(); ++c) { - (*c).second->list()->set_interpolation( - // to be enabled when ControlList::rt_safe_earliest_event_linear_unlocked works properly - #if LINEAR_INTERPOLATION_MODE_WORKS_PROPERLY - EventTypeMap::instance().interpolation_of((*c).first)); - #else - Evoral::ControlList::Discrete); - #endif + if (!writable() || (writable() && !_open)) { + return; } -} + Evoral::SMF::end_write (); + /* data in the file means its no longer removable */ + mark_nonremovable (); +} void -SMFSource::destroy_model () +SMFSource::set_path (const string& p) { - //cerr << _name << " destroying model " << _model.get() << endl; - _model.reset(); + FileSource::set_path (p); + SMF::set_path (_path); } +/** Ensure that this source has some file on disk, even if it's just a SMF header */ void -SMFSource::flush_midi () +SMFSource::ensure_disk_file () { - Evoral::SMF::end_write(); + if (_model) { + /* We have a model, so write it to disk; see MidiSource::session_saved + for an explanation of what we are doing here. + */ + boost::shared_ptr mm = _model; + _model.reset (); + mm->sync_to_source (); + _model = mm; + } else { + /* No model; if it's not already open, it's an empty source, so create + and open it for writing. + */ + if (!_open) { + open_for_write (); + } + + /* Flush, which will definitely put something on disk */ + flush_midi (); + } }