fix last commit

[ardour.git] / libs / ardour / midi_source.cc

diff --git a/libs/ardour/midi_source.cc b/libs/ardour/midi_source.cc
index 9f7c1e3baa2145f4e8d0dc547f0ab88de885705d..ce58d4f585a5319ab5402ce27a17f70f23ba8ef2 100644 (file)
--- a/libs/ardour/midi_source.cc
+++ b/libs/ardour/midi_source.cc
@@ -33,6 +33,7 @@
 #include "pbd/basename.h"
 
 #include "ardour/audioengine.h"
+#include "ardour/debug.h"
 #include "ardour/midi_model.h"
 #include "ardour/midi_ring_buffer.h"
 #include "ardour/midi_state_tracker.h"
@@ -48,7 +49,7 @@ using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
 
-sigc::signal<void,MidiSource *> MidiSource::MidiSourceCreated;
+PBD::Signal1<void,MidiSource*> MidiSource::MidiSourceCreated;
 
 MidiSource::MidiSource (Session& s, string name, Source::Flag flags)
        : Source(s, DataType::MIDI, name, flags)
@@ -111,7 +112,7 @@ MidiSource::set_state (const XMLNode& node, int /*version*/)
 sframes_t
 MidiSource::length (sframes_t pos) const
 {
-       BeatsFramesConverter converter(_session, pos);
+       BeatsFramesConverter converter(_session.tempo_map(), pos);
        return converter.to(_length_beats);
 }
 
@@ -129,28 +130,29 @@ MidiSource::invalidate ()
 }
 
 nframes_t
-MidiSource::midi_read (MidiRingBuffer<nframes_t>& dst, sframes_t source_start,
-                      sframes_t start, nframes_t cnt,
-                      sframes_t stamp_offset, sframes_t negative_stamp_offset,
-                      MidiStateTracker* tracker) const
+MidiSource::midi_read (Evoral::EventSink<nframes_t>& dst, sframes_t source_start,
+                       sframes_t start, nframes_t cnt,
+                       sframes_t stamp_offset, sframes_t negative_stamp_offset,
+                       MidiStateTracker* tracker) const
 {
        Glib::Mutex::Lock lm (_lock);
 
-       BeatsFramesConverter converter(_session, source_start);
+       BeatsFramesConverter converter(_session.tempo_map(), source_start);
 
        if (_model) {
-#define BEATS_TO_FRAMES(t) (converter.to(t) + stamp_offset - negative_stamp_offset)
-
                Evoral::Sequence<double>::const_iterator& i = _model_iter;
 
                // If the cached iterator is invalid, search for the first event past start
                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(); i != _model->end(); ++i) {
                                if (converter.to(i->time()) >= start) {
                                        break;
                                }
                        }
                        _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;
@@ -164,8 +166,10 @@ MidiSource::midi_read (MidiRingBuffer<nframes_t>& dst, sframes_t source_start,
                                if (tracker) {
                                        Evoral::MIDIEvent<Evoral::MusicalTime>& ev (*(Evoral::MIDIEvent<Evoral::MusicalTime>*) (&(*i)));
                                        if (ev.is_note_on()) {
+                                               DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("\t%1 add note on %2 @ %3\n", _name, ev.note(), time_frames));
                                                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());
                                        }
                                }
@@ -188,16 +192,6 @@ MidiSource::midi_write (MidiRingBuffer<nframes_t>& source, sframes_t source_star
        return ret;
 }
 
-bool
-MidiSource::file_changed (string path)
-{
-       struct stat stat_file;
-
-       int e1 = stat (path.c_str(), &stat_file);
-
-       return !e1;
-}
-
 void
 MidiSource::mark_streaming_midi_write_started (NoteMode mode, sframes_t start_frame)
 {
@@ -236,7 +230,7 @@ MidiSource::session_saved()
 
        if (_model && _model->edited()) {
                string newname;
-               const string basename = PBD::basename_nosuffix(_name);
+               const string basename = PBD::basename_nosuffix(_name.get());
                string::size_type last_dash = basename.find_last_of("-");
                if (last_dash == string::npos || last_dash == basename.find_first_of("-")) {
                        newname = basename + "-1";
@@ -255,7 +249,7 @@ MidiSource::session_saved()
 
                boost::shared_ptr<MidiSource> newsrc = boost::dynamic_pointer_cast<MidiSource>(
                                SourceFactory::createWritable(DataType::MIDI, _session,
-                                               newpath, true, false, _session.frame_rate()));
+                                               newpath, false, _session.frame_rate()));
 
                newsrc->set_timeline_position(_timeline_position);
                _model->write_to(newsrc);
@@ -266,7 +260,7 @@ MidiSource::session_saved()
 
                newsrc->flush_midi();
 
-               Switched.emit(newsrc);
+               Switched (newsrc); /* EMIT SIGNAL */
        }
 }