Fix uninitialised variable causing garbage output from panners in some cases.
[ardour.git] / libs / ardour / midi_source.cc
index 719c2a3affb07148dc8d408f61a8b23661e3d9da..48d9cdf877b68142710c6bf02c9eb94aa5eeaa83 100644 (file)
@@ -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
 #include "pbd/pthread_utils.h"
 #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"
 #include "ardour/midi_source.h"
 #include "ardour/session.h"
 #include "ardour/session_directory.h"
 #include "ardour/source_factory.h"
-#include "ardour/tempo.h"
 
 #include "i18n.h"
 
+namespace ARDOUR { template <typename T> class MidiRingBuffer; }
+
 using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
@@ -55,8 +54,6 @@ PBD::Signal1<void,MidiSource*> MidiSource::MidiSourceCreated;
 
 MidiSource::MidiSource (Session& s, string name, Source::Flag flags)
        : Source(s, DataType::MIDI, name, flags)
-       , _read_data_count(0)
-       , _write_data_count(0)
        , _writing(false)
        , _model_iter_valid(false)
        , _length_beats(0.0)
@@ -67,17 +64,12 @@ MidiSource::MidiSource (Session& s, string name, Source::Flag flags)
 
 MidiSource::MidiSource (Session& s, const XMLNode& node)
        : Source(s, node)
-       , _read_data_count(0)
-       , _write_data_count(0)
        , _writing(false)
        , _model_iter_valid(false)
        , _length_beats(0.0)
        , _last_read_end(0)
        , _last_write_end(0)
 {
-       _read_data_count = 0;
-       _write_data_count = 0;
-
        if (set_state (node, Stateful::loading_state_version)) {
                throw failed_constructor();
        }
@@ -130,7 +122,7 @@ MidiSource::set_state (const XMLNode& node, int /*version*/)
                                error << _("Missing parameter property on InterpolationStyle") << endmsg;
                                return -1;
                        }
-                       
+
                        Evoral::Parameter p = EventTypeMap::instance().new_parameter (prop->value());
 
                        if ((prop = (*i)->property (X_("style"))) == 0) {
@@ -140,16 +132,16 @@ MidiSource::set_state (const XMLNode& node, int /*version*/)
 
                        Evoral::ControlList::InterpolationStyle s = static_cast<Evoral::ControlList::InterpolationStyle> (string_2_enum (prop->value(), s));
                        set_interpolation_of (p, s);
-                       
+
                } else if ((*i)->name() == X_("AutomationState")) {
-                       
+
                        XMLProperty* prop;
 
                        if ((prop = (*i)->property (X_("parameter"))) == 0) {
                                error << _("Missing parameter property on AutomationState") << endmsg;
                                return -1;
                        }
-                       
+
                        Evoral::Parameter p = EventTypeMap::instance().new_parameter (prop->value());
 
                        if ((prop = (*i)->property (X_("state"))) == 0) {
@@ -183,7 +175,7 @@ MidiSource::length (framepos_t pos) const
 }
 
 void
-MidiSource::update_length (framepos_t /*pos*/, framecnt_t /*cnt*/)
+MidiSource::update_length (framecnt_t)
 {
        // You're not the boss of me!
 }
@@ -206,6 +198,9 @@ MidiSource::midi_read (Evoral::EventSink<framepos_t>& dst, framepos_t source_sta
 
        BeatsFramesConverter converter(_session.tempo_map(), source_start);
 
+       DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("MidiSource::midi-read() %5 sstart %1 start %2 cnt %3 tracker %4\n",
+                                                         source_start, start, cnt, tracker, name()));
+
        if (_model) {
                Evoral::Sequence<double>::const_iterator& i = _model_iter;
 
@@ -214,12 +209,13 @@ MidiSource::midi_read (Evoral::EventSink<framepos_t>& dst, framepos_t source_sta
                        DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("*** %1 search for relevant iterator for %1 / %2\n", _name, source_start, start));
                        for (i = _model->begin(0, false, filtered); i != _model->end(); ++i) {
                                if (converter.to(i->time()) >= start) {
+                                       DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("***\tstop iterator search @ %1\n", i->time()));
                                        break;
                                }
                        }
                        _model_iter_valid = true;
                } else {
-                       DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("*** %1 use cached iterator for %1 / %2\n", _name, source_start, start));
+                       DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("*** %1 use cachediterator for %1 / %2\n", _name, source_start, start));
                }
 
                _last_read_end = start + cnt;
@@ -231,17 +227,22 @@ MidiSource::midi_read (Evoral::EventSink<framepos_t>& dst, framepos_t source_sta
                                /* convert event times to session frames by adding on the source start position in session frames */
                                dst.write (time_frames + source_start, i->event_type(), i->size(), i->buffer());
 
+                                DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("%1: add event @ %2 type %3 size = %4\n",
+                                                                                  _name, time_frames + source_start, i->event_type(), i->size()));
+
                                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 velocity %4\n", _name, ev.note(), time_frames, (int) ev.velocity()));
+                                               DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("\t%1 track note on %2 @ %3 velocity %4\n", _name, (int) ev.note(), time_frames, (int) ev.velocity()));
                                                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));
+                                               DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("\t%1 track note off %2 @ %3\n", _name, (int) ev.note(), time_frames));
                                                tracker->remove (ev.note(), ev.channel());
                                        }
                                }
                        } else {
+                                DEBUG_TRACE (DEBUG::MidiSourceIO, string_compose ("%1: reached end with event @ %2 vs. %3\n",
+                                                                                  _name, time_frames, start+cnt));
                                break;
                        }
                }
@@ -259,8 +260,15 @@ framecnt_t
 MidiSource::midi_write (MidiRingBuffer<framepos_t>& source, framepos_t source_start, framecnt_t duration)
 {
        Glib::Mutex::Lock lm (_lock);
+
        const framecnt_t ret = write_unlocked (source, source_start, duration);
-       _last_write_end += duration;
+
+       if (duration == max_framecnt) {
+               _last_read_end = 0;
+       } else {
+               _last_write_end += duration;
+       }
+
        return ret;
 }
 
@@ -291,6 +299,8 @@ MidiSource::mark_write_starting_now ()
 
        set_timeline_position (_session.transport_frame ());
        _last_write_end = _session.transport_frame ();
+       cerr << name() << " last write set to " << _last_write_end << endl;
+
 }
 
 void
@@ -301,32 +311,43 @@ MidiSource::mark_streaming_write_started ()
 }
 
 void
-MidiSource::mark_streaming_write_completed ()
+MidiSource::mark_midi_streaming_write_completed (Evoral::Sequence<Evoral::MusicalTime>::StuckNoteOption option, Evoral::MusicalTime end)
 {
        if (_model) {
-               _model->end_write(false);
+               _model->end_write (option, end);
        }
 
        _writing = false;
 }
 
+void
+MidiSource::mark_streaming_write_completed ()
+{
+       mark_midi_streaming_write_completed (Evoral::Sequence<Evoral::MusicalTime>::DeleteStuckNotes);
+}
+
 boost::shared_ptr<MidiSource>
-MidiSource::clone (Evoral::MusicalTime begin, Evoral::MusicalTime end)
+MidiSource::clone (const string& path, Evoral::MusicalTime begin, Evoral::MusicalTime end)
 {
        string newname = PBD::basename_nosuffix(_name.val());
        string newpath;
 
-       /* get a new name for the MIDI file we're going to write to
-        */
-
-       do { 
+       if (path.empty()) {
 
-               newname = bump_name_once (newname, '-');
-               /* XXX build path safely */
-               newpath = _session.session_directory().midi_path().to_string() +"/"+ newname + ".mid";
+               /* get a new name for the MIDI file we're going to write to
+                */
+               
+               do {
+                       newname = bump_name_once (newname, '-');
+                       /* XXX build path safely */
+                       newpath = _session.session_directory().midi_path().to_string() +"/"+ newname + ".mid";
+                       
+               } while (Glib::file_test (newpath, Glib::FILE_TEST_EXISTS));
+       } else {
+               /* caller must check for pre-existing file */
+               newpath = path;
+       }
 
-       } while (Glib::file_test (newpath, Glib::FILE_TEST_EXISTS));
-        
        boost::shared_ptr<MidiSource> newsrc = boost::dynamic_pointer_cast<MidiSource>(
                SourceFactory::createWritable(DataType::MIDI, _session,
                                              newpath, string(), false, _session.frame_rate()));
@@ -349,20 +370,20 @@ MidiSource::clone (Evoral::MusicalTime begin, Evoral::MusicalTime end)
        newsrc->flush_midi();
 
        /* force a reload of the model if the range is partial */
-        
+
        if (begin != Evoral::MinMusicalTime || end != Evoral::MaxMusicalTime) {
                newsrc->load_model (true, true);
        } else {
                newsrc->set_model (_model);
        }
-        
+
        return newsrc;
 }
 
 void
 MidiSource::session_saved()
 {
-       /* this writes a copy of the data to disk. 
+       /* this writes a copy of the data to disk.
           XXX do we need to do this every time?
        */
 
@@ -376,8 +397,8 @@ MidiSource::session_saved()
                   try to update it.
                */
 
-               boost::shared_ptr<MidiModel> mm = _model ;
-               _model.reset ();   
+               boost::shared_ptr<MidiModel> mm = _model;
+               _model.reset ();
 
                /* flush model contents to disk
                 */
@@ -404,7 +425,7 @@ MidiSource::set_note_mode(NoteMode mode)
 void
 MidiSource::drop_model ()
 {
-       _model.reset(); 
+       _model.reset();
        ModelChanged (); /* EMIT SIGNAL */
 }
 
@@ -451,7 +472,7 @@ MidiSource::set_interpolation_of (Evoral::Parameter p, Evoral::ControlList::Inte
        if (interpolation_of (p) == s) {
                return;
        }
-       
+
        if (EventTypeMap::instance().interpolation_of (p) == s) {
                /* interpolation type is being set to the default, so we don't need a note in our map */
                _interpolation_style.erase (p);
@@ -468,7 +489,7 @@ MidiSource::set_automation_state_of (Evoral::Parameter p, AutoState s)
        if (automation_state_of (p) == s) {
                return;
        }
-       
+
        if (s == Play) {
                /* automation state is being set to the default, so we don't need a note in our map */
                _automation_state.erase (p);