X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=libs%2Fardour%2Fsmf_source.cc;h=812e06c27b08d30252630d1bb776fc4178567a8c;hb=cd8aa4e3044bfa2f6956e02ccc5458b3c9fbadc4;hp=e0462efb08137f9cb14e8004e32f3aed4e4b2d1d;hpb=760ccbabfbec67d80c3bdac9c0803ecf7d1742b6;p=ardour.git diff --git a/libs/ardour/smf_source.cc b/libs/ardour/smf_source.cc index e0462efb08..812e06c27b 100644 --- a/libs/ardour/smf_source.cc +++ b/libs/ardour/smf_source.cc @@ -24,12 +24,15 @@ #include #include #include +#include #include "pbd/pathscanner.h" #include "pbd/stl_delete.h" #include "pbd/strsplit.h" +#include #include +#include #include "evoral/Control.hpp" @@ -60,11 +63,24 @@ SMFSource::SMFSource (Session& s, const string& path, Source::Flag flags) { /* note that origin remains empty */ - if (init(_path, false)) { + if (init (_path, false)) { throw failed_constructor (); } + + assert (!Glib::file_test (_path, Glib::FILE_TEST_EXISTS)); + existence_check (); - /* file is not opened until write */ + /* file is not opened until write */ + + if (flags & Writable) { + return; + } + + if (open (_path)) { + throw failed_constructor (); + } + + _open = true; } /** Constructor used for existing internal-to-session files. */ @@ -81,21 +97,24 @@ SMFSource::SMFSource (Session& s, const XMLNode& node, bool must_exist) throw failed_constructor (); } - if (init(_path, true)) { + if (init (_path, true)) { throw failed_constructor (); } + assert (Glib::file_test (_path, Glib::FILE_TEST_EXISTS)); + existence_check (); + if (open(_path)) { throw failed_constructor (); } - _open = true; + _open = true; } SMFSource::~SMFSource () { if (removable()) { - unlink (_path.c_str()); + ::g_unlink (_path.c_str()); } } @@ -103,25 +122,27 @@ int SMFSource::open_for_write () { if (create (_path)) { - return -1; - } - _open = true; - return 0; + return -1; + } + _open = true; + return 0; } /** All stamps in audio frames */ framecnt_t -SMFSource::read_unlocked (Evoral::EventSink& destination, framepos_t const source_start, - framepos_t start, framecnt_t duration, - MidiStateTracker* tracker) const +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 - if (writable() && !_open) { - /* nothing to read since nothing has ben written */ - return duration; - } + 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)); @@ -208,16 +229,14 @@ SMFSource::read_unlocked (Evoral::EventSink& destination, framepos_t return 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) +SMFSource::write_unlocked (MidiRingBuffer& source, + framepos_t position, + framecnt_t cnt) { - if (!_writing) { - mark_streaming_write_started (); - } + if (!_writing) { + mark_streaming_write_started (); + } framepos_t time; Evoral::EventType type; @@ -226,55 +245,57 @@ SMFSource::write_unlocked (MidiRingBuffer& source, framepos_t positi size_t buf_capacity = 4; uint8_t* buf = (uint8_t*)malloc(buf_capacity); - if (_model && ! _model->writing()) { + if (_model && !_model->writing()) { _model->start_write(); } Evoral::MIDIEvent ev; - - // cerr << "SMFSource::write unlocked, begins writing from src buffer with _last_write_end = " << _last_write_end << " dur = " << duration << endl; - while (true) { + /* Get the event time, in frames since session start but ignoring looping. */ bool ret; - if (!(ret = source.peek ((uint8_t*)&time, sizeof (time)))) { - /* no more events to consider */ + /* Ring is empty, no more events. */ break; } - 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)); + if ((cnt != max_framecnt) && + (time > position + _capture_length + cnt)) { + /* The diskstream doesn't want us to write everything, and this + event is past the end of this block, so we're done for now. */ break; } + /* Read the time, type, and size of the event. */ if (!(ret = source.read_prefix (&time, &type, &size))) { - error << _("Unable to read event prefix, corrupt MIDI ring buffer") << endmsg; + error << _("Unable to read event prefix, corrupt MIDI ring") << endmsg; break; } + /* Enlarge body buffer if necessary now that we know the size. */ if (size > buf_capacity) { buf_capacity = size; buf = (uint8_t*)realloc(buf, size); } + /* Read the event body into buffer. */ ret = source.read_contents(size, buf); if (!ret) { - error << _("Read time/size but not buffer, corrupt MIDI ring buffer") << endmsg; + error << _("Event has time and size but no body, corrupt MIDI ring") << endmsg; break; } - /* convert from session time to time relative to the source start */ - assert(time >= position); + /* Convert event time from absolute to source relative. */ + if (time < position) { + error << _("Event time is before MIDI source position") << endmsg; + break; + } 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()); + 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;*/ continue; } @@ -284,16 +305,14 @@ SMFSource::write_unlocked (MidiRingBuffer& source, framepos_t positi Evoral::SMF::flush (); free (buf); - return duration; + return cnt; } - /** Append an event with a timestamp in beats (double) */ void SMFSource::append_event_unlocked_beats (const Evoral::Event& ev) { - assert(_writing); - if (ev.size() == 0) { + if (!_writing || ev.size() == 0) { return; } @@ -301,9 +320,9 @@ SMFSource::append_event_unlocked_beats (const Evoral::Event& ev) 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) { - cerr << "SMFSource: Warning: Skipping event with non-monotonic time" << endl; + warning << string_compose(_("Skipping event with unordered time %1"), ev.time()) + << endmsg; return; } @@ -332,9 +351,7 @@ SMFSource::append_event_unlocked_beats (const Evoral::Event& ev) void 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"; + if (!_writing || ev.size() == 0) { return; } @@ -343,7 +360,8 @@ SMFSource::append_event_unlocked_frames (const Evoral::Event& ev, fr // 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; + warning << string_compose(_("Skipping event with unordered time %1"), ev.time()) + << endmsg; return; } @@ -404,13 +422,13 @@ SMFSource::set_state (const XMLNode& node, int version) void SMFSource::mark_streaming_midi_write_started (NoteMode mode) { - /* CALLER MUST HOLD LOCK */ + /* 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; - } + 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 (); @@ -449,7 +467,35 @@ SMFSource::mark_midi_streaming_write_completed (Evoral::Sequence*, gint >& a, + const std::pair< Evoral::Event*, gint >& b) { + return ( a.first->time() < b.first->time() ); } void @@ -473,9 +519,9 @@ SMFSource::load_model (bool lock, bool force_reload) _model->clear(); } - if (writable() && !_open) { - return; - } + if (writable() && !_open) { + return; + } _model->start_write(); Evoral::SMF::seek_to_start(); @@ -490,60 +536,74 @@ SMFSource::load_model (bool lock, bool force_reload) uint8_t* buf = NULL; int ret; gint event_id; - bool have_event_id = false; - - while ((ret = read_event (&delta_t, &size, &buf, &event_id)) >= 0) { + bool have_event_id; - time += delta_t; + // TODO simplify event allocation + std::list< std::pair< Evoral::Event*, gint > > eventlist; - if (ret == 0) { + for (unsigned i = 1; i <= num_tracks(); ++i) { + if (seek_to_track(i)) continue; - /* meta-event : did we get an event ID ? - */ - - if (event_id >= 0) { - have_event_id = true; - } + time = 0; + have_event_id = false; - continue; - } + while ((ret = read_event (&delta_t, &size, &buf, &event_id)) >= 0) { - if (ret > 0) { + time += delta_t; - /* not a meta-event */ + if (ret == 0) { + /* meta-event : did we get an event ID ? */ + if (event_id >= 0) { + have_event_id = true; + } + continue; + } - ev.set (buf, size, time / (double)ppqn()); - ev.set_event_type(EventTypeMap::instance().midi_event_type(buf[0])); + if (ret > 0) { + /* not a meta-event */ - if (!have_event_id) { - event_id = Evoral::next_event_id(); - } + if (!have_event_id) { + event_id = Evoral::next_event_id(); + } + uint32_t event_type = EventTypeMap::instance().midi_event_type(buf[0]); + double event_time = time / (double) ppqn(); #ifndef NDEBUG - std::string ss; + std::string ss; - for (uint32_t xx = 0; xx < size; ++xx) { - char b[8]; - snprintf (b, sizeof (b), "0x%x ", buf[xx]); - ss += b; - } + 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())); + 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 , event_type, name())); #endif - _model->append (ev, event_id); + eventlist.push_back(make_pair ( + new Evoral::Event ( + event_type, event_time, + size, buf, true) + , event_id)); - // Set size to max capacity to minimize allocs in read_event - scratch_size = std::max(size, scratch_size); - size = scratch_size; + // 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()); + _length_beats = max(_length_beats, event_time); + } + + /* event ID's must immediately precede the event they are for */ + have_event_id = false; } + } - /* event ID's must immediately precede the event they are for - */ + eventlist.sort(compare_eventlist); - have_event_id = false; + std::list< std::pair< Evoral::Event*, gint > >::iterator it; + for (it=eventlist.begin(); it!=eventlist.end(); ++it) { + _model->append (*it->first, it->second); + delete it->first; } _model->end_write (Evoral::Sequence::ResolveStuckNotes, _length_beats); @@ -605,3 +665,44 @@ SMFSource::ensure_disk_file () } } +void +SMFSource::prevent_deletion () +{ + /* Unlike the audio case, the MIDI file remains mutable (because we can + edit MIDI data) + */ + + _flags = Flag (_flags & ~(Removable|RemovableIfEmpty|RemoveAtDestroy)); +} + +int +SMFSource::rename (const string& newname) +{ + Glib::Threads::Mutex::Lock lm (_lock); + string oldpath = _path; + string newpath = _session.new_source_path_from_name (DataType::MIDI, newname); + + if (newpath.empty()) { + error << string_compose (_("programming error: %1"), "cannot generate a changed file path") << endmsg; + return -1; + } + + // Test whether newpath exists, if yes notify the user but continue. + if (Glib::file_test (newpath, Glib::FILE_TEST_EXISTS)) { + error << string_compose (_("Programming error! %1 tried to rename a file over another file! It's safe to continue working, but please report this to the developers."), PROGRAM_NAME) << endmsg; + return -1; + } + + if (Glib::file_test (oldpath.c_str(), Glib::FILE_TEST_EXISTS)) { + /* rename only needed if file exists on disk */ + if (::rename (oldpath.c_str(), newpath.c_str()) != 0) { + error << string_compose (_("cannot rename file %1 to %2 (%3)"), oldpath, newpath, strerror(errno)) << endmsg; + return -1; + } + } + + _name = Glib::path_get_basename (newpath); + _path = newpath; + + return 0; +}