summaryrefslogtreecommitdiff
path: root/libs/evoral
diff options
context:
space:
mode:
Diffstat (limited to 'libs/evoral')
-rw-r--r--libs/evoral/evoral/Control.hpp5
-rw-r--r--libs/evoral/evoral/ControlList.hpp7
-rw-r--r--libs/evoral/evoral/ControlSet.hpp29
-rw-r--r--libs/evoral/evoral/Sequence.hpp42
-rw-r--r--libs/evoral/src/Control.cpp15
-rw-r--r--libs/evoral/src/ControlList.cpp4
-rw-r--r--libs/evoral/src/ControlSet.cpp42
-rw-r--r--libs/evoral/src/Sequence.cpp167
8 files changed, 155 insertions, 156 deletions
diff --git a/libs/evoral/evoral/Control.hpp b/libs/evoral/evoral/Control.hpp
index 15d50fcdca..01dc2eebed 100644
--- a/libs/evoral/evoral/Control.hpp
+++ b/libs/evoral/evoral/Control.hpp
@@ -34,7 +34,7 @@ class Transport;
class Control
{
public:
- Control(boost::shared_ptr<ControlList>);
+ Control(const Parameter& parameter, boost::shared_ptr<ControlList>);
virtual ~Control() {}
void set_value(float val, bool to_list=false, nframes_t frame=0);
@@ -46,9 +46,10 @@ public:
boost::shared_ptr<ControlList> list() { return _list; }
boost::shared_ptr<const ControlList> list() const { return _list; }
- const Parameter& parameter() const;
+ inline const Parameter& parameter() const { return _parameter; }
protected:
+ Parameter _parameter;
boost::shared_ptr<ControlList> _list;
float _user_value;
};
diff --git a/libs/evoral/evoral/ControlList.hpp b/libs/evoral/evoral/ControlList.hpp
index 1b2b46b10b..36799300fe 100644
--- a/libs/evoral/evoral/ControlList.hpp
+++ b/libs/evoral/evoral/ControlList.hpp
@@ -84,13 +84,12 @@ public:
typedef EventList::const_iterator const_iterator;
ControlList (const Parameter& id);
- //ControlList (const XMLNode&, Parameter id);
+ ControlList (const ControlList&);
+ ControlList (const ControlList&, double start, double end);
virtual ~ControlList();
virtual boost::shared_ptr<ControlList> create(Parameter id);
-
- ControlList (const ControlList&);
- ControlList (const ControlList&, double start, double end);
+
ControlList& operator= (const ControlList&);
bool operator== (const ControlList&);
diff --git a/libs/evoral/evoral/ControlSet.hpp b/libs/evoral/evoral/ControlSet.hpp
index 73fa5554e6..1b5edb92ac 100644
--- a/libs/evoral/evoral/ControlSet.hpp
+++ b/libs/evoral/evoral/ControlSet.hpp
@@ -22,6 +22,7 @@
#include <set>
#include <map>
#include <boost/shared_ptr.hpp>
+#include <boost/utility.hpp>
#include <glibmm/thread.h>
#include <evoral/types.hpp>
#include <evoral/Parameter.hpp>
@@ -32,27 +33,32 @@ class Control;
class ControlList;
class ControlEvent;
-class ControlSet {
+class ControlSet : public boost::noncopyable {
public:
ControlSet();
virtual ~ControlSet() {}
-
- virtual boost::shared_ptr<Control> control(const Parameter& id, bool create_if_missing=false);
- virtual boost::shared_ptr<const Control> control(const Parameter& id) const;
- virtual boost::shared_ptr<Control> control_factory(boost::shared_ptr<ControlList> list) const;
- virtual boost::shared_ptr<ControlList> control_list_factory(const Parameter& param) const;
+ virtual boost::shared_ptr<Evoral::Control>
+ control_factory(const Evoral::Parameter& id) = 0;
+ boost::shared_ptr<Control>
+ control (const Parameter& id, bool create_if_missing=false);
+
+ inline boost::shared_ptr<const Control>
+ control (const Parameter& id) const {
+ const Controls::const_iterator i = _controls.find(id);
+ return (i != _controls.end() ? i->second : boost::shared_ptr<Control>());
+ }
+
typedef std::map< Parameter, boost::shared_ptr<Control> > Controls;
- Controls& controls() { return _controls; }
- const Controls& controls() const { return _controls; }
+ inline Controls& controls() { return _controls; }
+ inline const Controls& controls() const { return _controls; }
virtual void add_control(boost::shared_ptr<Control>);
- virtual bool find_next_event(nframes_t start, nframes_t end, ControlEvent& ev) const;
+ bool find_next_event(nframes_t start, nframes_t end, ControlEvent& ev) const;
- virtual float default_parameter_value(const Parameter& param) { return 1.0f; }
-
+ virtual bool empty() const { return _controls.size() == 0; }
virtual void clear();
void what_has_data(std::set<Parameter>&) const;
@@ -64,6 +70,7 @@ protected:
Controls _controls;
};
+
} // namespace Evoral
#endif // EVORAL_CONTROLLABLE_HPP
diff --git a/libs/evoral/evoral/Sequence.hpp b/libs/evoral/evoral/Sequence.hpp
index 10a61704ed..73ddaca21b 100644
--- a/libs/evoral/evoral/Sequence.hpp
+++ b/libs/evoral/evoral/Sequence.hpp
@@ -25,7 +25,6 @@
#include <map>
#include <utility>
#include <boost/shared_ptr.hpp>
-#include <boost/utility.hpp>
#include <glibmm/thread.h>
#include <evoral/types.hpp>
#include <evoral/Note.hpp>
@@ -39,6 +38,7 @@ class Note;
class Event;
class ControlList;
+
/** This class keeps track of the current x and y for a control
*/
class ControlIterator {
@@ -59,12 +59,13 @@ public:
/** This is a higher level view of events, with separate representations for
* notes (instead of just unassociated note on/off events) and controller data.
- * Controller data is represented as a list of time-stamped float values.
- */
-class Sequence : public boost::noncopyable, virtual public ControlSet {
+ * Controller data is represented as a list of time-stamped float values. */
+class Sequence : virtual public ControlSet {
public:
- Sequence(size_t size);
+ Sequence(size_t size=0);
+ bool read_locked() { return _read_iter.locked(); }
+
void write_lock();
void write_unlock();
@@ -92,10 +93,10 @@ public:
inline const boost::shared_ptr<Note> note_at(unsigned i) { return _notes[i]; }
inline size_t n_notes() const { return _notes.size(); }
- inline bool empty() const { return _notes.size() == 0 && _controls.size() == 0; }
+ inline bool empty() const { return _notes.size() == 0 && ControlSet::empty(); }
- inline static bool note_time_comparator(const boost::shared_ptr<const Note> a,
- const boost::shared_ptr<const Note> b) {
+ inline static bool note_time_comparator(const boost::shared_ptr<const Note>& a,
+ const boost::shared_ptr<const Note>& b) {
return a->time() < b->time();
}
@@ -117,6 +118,7 @@ public:
const_iterator(const Sequence& seq, double t);
~const_iterator();
+ inline bool valid() const { return !_is_end && _event; }
inline bool locked() const { return _locked; }
const Event& operator*() const { return *_event; }
@@ -149,28 +151,28 @@ public:
std::vector<ControlIterator>::iterator _control_iter;
};
- const_iterator begin() const { return const_iterator(*this, 0); }
- const const_iterator& end() const { return _end_iter; }
+ const_iterator begin(double t=0) const { return const_iterator(*this, t); }
+ const const_iterator& end() const { return _end_iter; }
+ void read_seek(double t) { _read_iter = begin(t); }
+ double read_time() const { return _read_iter.valid() ? _read_iter->time() : 0.0; }
+
bool control_to_midi_event(boost::shared_ptr<Event>& ev,
const ControlIterator& iter) const;
- typedef std::map< Parameter, boost::shared_ptr<Control> > Controls;
- Controls& controls() { return _controls; }
- const Controls& controls() const { return _controls; }
-
bool edited() const { return _edited; }
void set_edited(bool yn) { _edited = yn; }
+
+#ifndef NDEBUG
+ bool is_sorted() const;
+#endif
-protected:
void add_note_unlocked(const boost::shared_ptr<Note> note);
void remove_note_unlocked(const boost::shared_ptr<const Note> note);
+protected:
mutable const_iterator _read_iter;
bool _edited;
-#ifndef NDEBUG
- bool is_sorted() const;
-#endif
private:
friend class const_iterator;
@@ -182,7 +184,6 @@ private:
mutable Glib::RWLock _lock;
Notes _notes;
- Controls _controls;
typedef std::vector<size_t> WriteNotes;
WriteNotes _write_notes[16];
@@ -197,7 +198,7 @@ private:
/** FIXME: Make fully dynamic, map to URIs */
enum EventTypes {
- midi_cc_type=1,
+ midi_cc_type=0x20, // FIXME FIXME FIXME eeww
midi_pc_type,
midi_pb_type,
midi_ca_type
@@ -209,6 +210,7 @@ private:
ActiveNotes;
};
+
} // namespace Evoral
#endif // EVORAL_SEQUENCE_HPP
diff --git a/libs/evoral/src/Control.cpp b/libs/evoral/src/Control.cpp
index d23f6c3c9a..75b038f1d4 100644
--- a/libs/evoral/src/Control.cpp
+++ b/libs/evoral/src/Control.cpp
@@ -24,9 +24,10 @@ namespace Evoral {
Parameter::TypeMetadata Parameter::_type_metadata;
-Control::Control(boost::shared_ptr<ControlList> list)
- : _list(list)
- , _user_value(list->default_value())
+Control::Control(const Parameter& parameter, boost::shared_ptr<ControlList> list)
+ : _parameter(parameter)
+ , _list(list)
+ , _user_value(list ? list->default_value() : parameter.normal())
{
}
@@ -70,14 +71,6 @@ void
Control::set_list(boost::shared_ptr<ControlList> list)
{
_list = list;
- _user_value = list->default_value();
-}
-
-
-const Parameter&
-Control::parameter() const
-{
- return _list->parameter();
}
} // namespace Evoral
diff --git a/libs/evoral/src/ControlList.cpp b/libs/evoral/src/ControlList.cpp
index 62c49a97e6..fd0a2e52bd 100644
--- a/libs/evoral/src/ControlList.cpp
+++ b/libs/evoral/src/ControlList.cpp
@@ -109,7 +109,6 @@ ControlList::~ControlList()
delete (*x);
}
}
-
boost::shared_ptr<ControlList>
ControlList::create(Parameter id)
@@ -117,7 +116,6 @@ ControlList::create(Parameter id)
return boost::shared_ptr<ControlList>(new ControlList(id));
}
-
bool
ControlList::operator== (const ControlList& other)
{
@@ -205,7 +203,7 @@ ControlList::reposition_for_rt_add (double when)
void
ControlList::rt_add (double when, double value)
{
- // cerr << "RT: alist @ " << this << " add " << value << " @ " << when << endl;
+ cerr << "RT: alist " << this << " add " << value << " @ " << when << endl;
{
Glib::Mutex::Lock lm (_lock);
diff --git a/libs/evoral/src/ControlSet.cpp b/libs/evoral/src/ControlSet.cpp
index 5aacff598d..837810c727 100644
--- a/libs/evoral/src/ControlSet.cpp
+++ b/libs/evoral/src/ControlSet.cpp
@@ -26,6 +26,7 @@ using namespace std;
namespace Evoral {
+
ControlSet::ControlSet()
{
}
@@ -40,17 +41,14 @@ void
ControlSet::what_has_data (set<Parameter>& s) const
{
Glib::Mutex::Lock lm (_control_lock);
- Controls::const_iterator li;
-
- // FIXME: correct semantics?
- for (li = _controls.begin(); li != _controls.end(); ++li) {
- s.insert ((*li).first);
+ for (Controls::const_iterator li = _controls.begin(); li != _controls.end(); ++li) {
+ s.insert(li->first);
}
}
-/** If \a create_if_missing is true, a control list will be created and returned
- * if one does not already exists. Otherwise NULL will be returned if a control list
- * for \a parameter does not exist.
+/** If a control for the given parameter does not exist and \a create_if_missing is true,
+ * a control will be created, added to this set, and returned.
+ * If \a create_if_missing is false this function may return null.
*/
boost::shared_ptr<Control>
ControlSet::control (const Parameter& parameter, bool create_if_missing)
@@ -61,8 +59,7 @@ ControlSet::control (const Parameter& parameter, bool create_if_missing)
return i->second;
} else if (create_if_missing) {
- boost::shared_ptr<ControlList> al (control_list_factory(parameter));
- boost::shared_ptr<Control> ac(control_factory(al));
+ boost::shared_ptr<Control> ac(control_factory(parameter));
add_control(ac);
return ac;
@@ -72,19 +69,6 @@ ControlSet::control (const Parameter& parameter, bool create_if_missing)
}
}
-boost::shared_ptr<const Control>
-ControlSet::control (const Parameter& parameter) const
-{
- Controls::const_iterator i = _controls.find(parameter);
-
- if (i != _controls.end()) {
- return i->second;
- } else {
- //warning << "ControlList " << parameter.to_string() << " not found for " << _name << endmsg;
- return boost::shared_ptr<Control>();
- }
-}
-
bool
ControlSet::find_next_event (nframes_t now, nframes_t end, ControlEvent& next_event) const
{
@@ -122,18 +106,6 @@ ControlSet::clear ()
for (Controls::iterator li = _controls.begin(); li != _controls.end(); ++li)
li->second->list()->clear();
}
-
-boost::shared_ptr<Control>
-ControlSet::control_factory(boost::shared_ptr<ControlList> list) const
-{
- return boost::shared_ptr<Control>(new Control(list));
-}
-
-boost::shared_ptr<ControlList>
-ControlSet::control_list_factory(const Parameter& param) const
-{
- return boost::shared_ptr<ControlList>(new ControlList(param));
-}
} // namespace Evoral
diff --git a/libs/evoral/src/Sequence.cpp b/libs/evoral/src/Sequence.cpp
index fecc37be29..9e073699d9 100644
--- a/libs/evoral/src/Sequence.cpp
+++ b/libs/evoral/src/Sequence.cpp
@@ -52,6 +52,16 @@ void Sequence::read_unlock() const {
_lock.reader_unlock();
}
+struct null_ostream : public std::ostream {
+ null_ostream(): std::ios(0), std::ostream(0) {}
+};
+static null_ostream nullout;
+
+//static ostream& debugout = cout;
+//static ostream& errorout = cerr;
+static ostream& debugout = nullout;
+static ostream& errorout = nullout;
+
// Read iterator (const_iterator)
Sequence::const_iterator::const_iterator(const Sequence& seq, double t)
@@ -59,7 +69,7 @@ Sequence::const_iterator::const_iterator(const Sequence& seq, double t)
, _is_end( (t == DBL_MAX) || seq.empty() )
, _locked( !_is_end )
{
- //cerr << "Created MIDI iterator @ " << t << " (is end: " << _is_end << ")" << endl;
+ debugout << "Created Iterator @ " << t << " (is end: " << _is_end << ")" << endl;
if (_is_end) {
return;
@@ -78,30 +88,31 @@ Sequence::const_iterator::const_iterator(const Sequence& seq, double t)
ControlIterator earliest_control(boost::shared_ptr<ControlList>(), DBL_MAX, 0.0);
- _control_iters.reserve(seq.controls().size());
+ _control_iters.reserve(seq._controls.size());
// find the earliest control event available
- for (Controls::const_iterator i = seq.controls().begin(); i != seq.controls().end(); ++i) {
+ for (Controls::const_iterator i = seq._controls.begin(); i != seq._controls.end(); ++i) {
+ debugout << "Iterator: control: " << i->first.symbol() << endl;
double x, y;
bool ret = i->second->list()->rt_safe_earliest_event_unlocked(t, DBL_MAX, x, y);
if (!ret) {
- //cerr << "MIDI Iterator: CC " << i->first.id() << " (size " << i->second->list()->size()
- // << ") has no events past " << t << endl;
+ debugout << "Iterator: CC " << i->first.id() << " (size " << i->second->list()->size()
+ << ") has no events past " << t << endl;
continue;
}
assert(x >= 0);
- if (y < i->first.min() || y > i->first.max()) {
- cerr << "ERROR: Controller (" << i->first.type() << ") value '" << y
- << "' out of range [" << i->first.min() << "," << i->first.max()
+ /*if (y < i->first.min() || y > i->first.max()) {
+ errorout << "ERROR: Controller " << i->first.symbol() << " value " << y
+ << " out of range [" << i->first.min() << "," << i->first.max()
<< "], event ignored" << endl;
continue;
- }
+ }*/
const ControlIterator new_iter(i->second->list(), x, y);
- //cerr << "MIDI Iterator: CC " << i->first.id() << " added (" << x << ", " << y << ")" << endl;
+ debugout << "Iterator: CC " << i->first.id() << " added (" << x << ", " << y << ")" << endl;
_control_iters.push_back(new_iter);
// if the x of the current control is less than earliest_control
@@ -137,7 +148,7 @@ Sequence::const_iterator::const_iterator(const Sequence& seq, double t)
}
if ( (! _event.get()) || _event->size() == 0) {
- //cerr << "Created MIDI iterator @ " << t << " is at end." << endl;
+ debugout << "New iterator @ " << t << " is at end." << endl;
_is_end = true;
// eliminate possible race condition here (ugly)
@@ -148,7 +159,8 @@ Sequence::const_iterator::const_iterator(const Sequence& seq, double t)
_locked = false;
}
} else {
- //printf("New MIDI Iterator = %X @ %lf\n", _event->type(), _event->time());
+ debugout << "New Iterator = " << hex << _event->type();
+ debugout << " @ " << _event->time() << endl;
}
assert(_is_end || (_event->buffer() && _event->buffer()[0] != '\0'));
@@ -161,7 +173,8 @@ Sequence::const_iterator::~const_iterator()
}
}
-const Sequence::const_iterator& Sequence::const_iterator::operator++()
+const
+Sequence::const_iterator& Sequence::const_iterator::operator++()
{
if (_is_end) {
throw std::logic_error("Attempt to iterate past end of Sequence");
@@ -169,10 +182,12 @@ const Sequence::const_iterator& Sequence::const_iterator::operator++()
assert(_event->buffer() && _event->buffer()[0] != '\0');
- /*cerr << "const_iterator::operator++: " << _event->to_string() << endl;*/
+ //debugout << "const_iterator::operator++: " << _event->to_string() << endl;
- if (! (_event->is_note() || _event->is_cc() || _event->is_pgm_change() || _event->is_pitch_bender() || _event->is_channel_aftertouch()) ) {
- cerr << "FAILED event buffer: " << hex << int(_event->buffer()[0]) << int(_event->buffer()[1]) << int(_event->buffer()[2]) << endl;
+ if (! (_event->is_note() || _event->is_cc() || _event->is_pgm_change()
+ || _event->is_pitch_bender() || _event->is_channel_aftertouch()) ) {
+ errorout << "Unknown event type: " << hex << int(_event->buffer()[0])
+ << int(_event->buffer()[1]) << int(_event->buffer()[2]) << endl;
}
assert((_event->is_note() || _event->is_cc() || _event->is_pgm_change() || _event->is_pitch_bender() || _event->is_channel_aftertouch()));
@@ -202,7 +217,7 @@ const Sequence::const_iterator& Sequence::const_iterator::operator++()
}
}
- enum Type {NIL, NOTE_ON, NOTE_OFF, AUTOMATION};
+ enum Type {NIL, NOTE_ON, NOTE_OFF, CONTROL};
Type type = NIL;
double t = 0;
@@ -222,26 +237,27 @@ const Sequence::const_iterator& Sequence::const_iterator::operator++()
}
// Use the next earliest controller iff it's earlier than the note event
- if (_control_iter != _control_iters.end() && _control_iter->x != DBL_MAX /*&& _control_iter != old_control_iter */) {
+ if (_control_iter != _control_iters.end() && _control_iter->x != DBL_MAX
+ && _control_iter != old_control_iter) {
if (type == NIL || _control_iter->x < t) {
- type = AUTOMATION;
+ type = CONTROL;
}
}
if (type == NOTE_ON) {
- //cerr << "********** MIDI Iterator = note on" << endl;
+ debugout << "Iterator = note on" << endl;
*_event = (*_note_iter)->on_event();
_active_notes.push(*_note_iter);
++_note_iter;
} else if (type == NOTE_OFF) {
- //cerr << "********** MIDI Iterator = note off" << endl;
+ debugout << "Iterator = note off" << endl;
*_event = _active_notes.top()->off_event();
_active_notes.pop();
- } else if (type == AUTOMATION) {
- //cerr << "********** MIDI Iterator = Automation" << endl;
+ } else if (type == CONTROL) {
+ debugout << "Iterator = control" << endl;
_seq->control_to_midi_event(_event, *_control_iter);
} else {
- //cerr << "********** MIDI Iterator = End" << endl;
+ debugout << "Iterator = End" << endl;
_is_end = true;
}
@@ -250,7 +266,8 @@ const Sequence::const_iterator& Sequence::const_iterator::operator++()
return *this;
}
-bool Sequence::const_iterator::operator==(const const_iterator& other) const
+bool
+Sequence::const_iterator::operator==(const const_iterator& other) const
{
if (_is_end || other._is_end) {
return (_is_end == other._is_end);
@@ -259,7 +276,8 @@ bool Sequence::const_iterator::operator==(const const_iterator& other) const
}
}
-Sequence::const_iterator& Sequence::const_iterator::operator=(const const_iterator& other)
+Sequence::const_iterator&
+Sequence::const_iterator::operator=(const const_iterator& other)
{
if (_locked && _seq != other._seq) {
_seq->read_unlock();
@@ -292,6 +310,7 @@ Sequence::Sequence(size_t size)
, _next_read(UINT32_MAX)
, _percussive(false)
{
+ debugout << "Sequence (size " << size << ") constructed: " << this << endl;
assert(_end_iter._is_end);
assert( ! _end_iter._locked);
}
@@ -300,18 +319,21 @@ Sequence::Sequence(size_t size)
* adding \a offset to each event's timestamp.
* \return number of events written to \a dst
*/
-size_t Sequence::read(EventSink& dst, timestamp_t start, timestamp_t nframes, timedur_t offset) const
+size_t
+Sequence::read(EventSink& dst, timestamp_t start, timedur_t nframes, timestamp_t offset) const
{
- //cerr << this << " MM::read @ " << start << " frames: " << nframes << " -> " << stamp_offset << endl;
- //cerr << this << " MM # notes: " << n_notes() << endl;
+ debugout << this << " read ev @ " << start << " * " << nframes << " + " << offset << endl;
+ debugout << this << " # notes: " << n_notes() << endl;
+ debugout << this << " controls: " << &_controls << endl;
+ debugout << this << " # controls: " << _controls.size() << endl;
size_t read_events = 0;
if (start != _next_read) {
_read_iter = const_iterator(*this, (double)start);
- //cerr << "Repositioning iterator from " << _next_read << " to " << start << endl;
+ debugout << "Repositioning iterator from " << _next_read << " to " << start << endl;
} else {
- //cerr << "Using cached iterator at " << _next_read << endl;
+ debugout << "Using cached iterator at " << _next_read << endl;
}
_next_read = (nframes_t) floor (start + nframes);
@@ -323,11 +345,11 @@ size_t Sequence::read(EventSink& dst, timestamp_t start, timestamp_t nframes, ti
_read_iter->size(),
_read_iter->buffer());
- /*cerr << this << " Sequence::read event @ " << _read_iter->time()
- << " type: " << hex << int(_read_iter->type()) << dec
- << " note: " << int(_read_iter->note())
- << " velocity: " << int(_read_iter->velocity())
- << endl;*/
+ debugout << this << " read event @ " << _read_iter->time()
+ << " type: " << hex << int(_read_iter->type()) << dec
+ << " note: " << int(_read_iter->note())
+ << " velocity: " << int(_read_iter->velocity())
+ << endl;
++_read_iter;
++read_events;
@@ -406,10 +428,10 @@ Sequence::control_to_midi_event(boost::shared_ptr<Event>& ev, const ControlItera
return true;
}
-
/** Clear all events from the model.
*/
-void Sequence::clear()
+void
+Sequence::clear()
{
_lock.writer_lock();
_notes.clear();
@@ -420,7 +442,6 @@ void Sequence::clear()
_lock.writer_unlock();
}
-
/** Begin a write of events to the model.
*
* If \a mode is Sustained, complete notes with duration are constructed as note
@@ -428,9 +449,10 @@ void Sequence::clear()
* stored; note off events are discarded entirely and all contained notes will
* have duration 0.
*/
-void Sequence::start_write()
+void
+Sequence::start_write()
{
- //cerr << "MM " << this << " START WRITE, PERCUSSIVE = " << _percussive << endl;
+ debugout << this << " START WRITE, PERCUSSIVE = " << _percussive << endl;
write_lock();
_writing = true;
for (int i = 0; i < 16; ++i)
@@ -446,17 +468,18 @@ void Sequence::start_write()
* that were never resolved with a corresonding note off will be deleted.
* Otherwise they will remain as notes with duration 0.
*/
-void Sequence::end_write(bool delete_stuck)
+void
+Sequence::end_write(bool delete_stuck)
{
write_lock();
assert(_writing);
- //cerr << "MM " << this << " END WRITE: " << _notes.size() << " NOTES\n";
+ debugout << this << " END WRITE: " << _notes.size() << " NOTES\n";
if (!_percussive && delete_stuck) {
for (Notes::iterator n = _notes.begin(); n != _notes.end() ;) {
if ((*n)->duration() == 0) {
- cerr << "WARNING: Stuck note lost: " << (*n)->note() << endl;
+ errorout << "WARNING: Stuck note lost: " << (*n)->note() << endl;
n = _notes.erase(n);
// we have to break here because erase invalidates the iterator
break;
@@ -468,7 +491,7 @@ void Sequence::end_write(bool delete_stuck)
for (int i = 0; i < 16; ++i) {
if (!_write_notes[i].empty()) {
- cerr << "WARNING: Sequence::end_write: Channel " << i << " has "
+ errorout << "WARNING: Sequence::end_write: Channel " << i << " has "
<< _write_notes[i].size() << " stuck notes" << endl;
}
_write_notes[i].clear();
@@ -488,7 +511,8 @@ void Sequence::end_write(bool delete_stuck)
* the start of this model (t=0) and MUST be monotonically increasing
* and MUST be >= the latest event currently in the model.
*/
-void Sequence::append(const Event& ev)
+void
+Sequence::append(const Event& ev)
{
write_lock();
_edited = true;
@@ -503,7 +527,7 @@ void Sequence::append(const Event& ev)
append_note_off_unlocked(ev.channel(), ev.time(), ev.note());
} else if (ev.is_cc()) {
append_control_unlocked(
- Evoral::MIDI::ContinuousController(midi_cc_type, ev.cc_number(), ev.channel()),
+ Evoral::MIDI::ContinuousController(midi_cc_type, ev.channel(), ev.cc_number()),
ev.time(), ev.cc_value());
} else if (ev.is_pgm_change()) {
append_control_unlocked(
@@ -525,12 +549,10 @@ void Sequence::append(const Event& ev)
write_unlock();
}
-void Sequence::append_note_on_unlocked(uint8_t chan, double time,
- uint8_t note_num, uint8_t velocity)
+void
+Sequence::append_note_on_unlocked(uint8_t chan, double time, uint8_t note_num, uint8_t velocity)
{
- /*cerr << "Sequence " << this << " chan " << (int)chan <<
- " note " << (int)note_num << " on @ " << time << endl;*/
-
+ debugout << this << " c" << (int)chan << " note " << (int)note_num << " off @ " << time << endl;
assert(note_num <= 127);
assert(chan < 16);
assert(_writing);
@@ -539,26 +561,24 @@ void Sequence::append_note_on_unlocked(uint8_t chan, double time,
boost::shared_ptr<Note> new_note(new Note(chan, time, 0, note_num, velocity));
_notes.push_back(new_note);
if (!_percussive) {
- //cerr << "MM Sustained: Appending active note on " << (unsigned)(uint8_t)note_num << endl;
+ debugout << "Sustained: Appending active note on " << (unsigned)(uint8_t)note_num << endl;
_write_notes[chan].push_back(_notes.size() - 1);
- }/* else {
- cerr << "MM Percussive: NOT appending active note on" << endl;
- }*/
+ } else {
+ debugout << "Percussive: NOT appending active note on" << endl;
+ }
}
-void Sequence::append_note_off_unlocked(uint8_t chan, double time,
- uint8_t note_num)
+void
+Sequence::append_note_off_unlocked(uint8_t chan, double time, uint8_t note_num)
{
- /*cerr << "Sequence " << this << " chan " << (int)chan <<
- " note " << (int)note_num << " off @ " << time << endl;*/
-
+ debugout << this << " c" << (int)chan << " note " << (int)note_num << " off @ " << time << endl;
assert(note_num <= 127);
assert(chan < 16);
assert(_writing);
_edited = true;
if (_percussive) {
- cerr << "Sequence Ignoring note off (percussive mode)" << endl;
+ debugout << "Sequence Ignoring note off (percussive mode)" << endl;
return;
}
@@ -576,37 +596,43 @@ void Sequence::append_note_off_unlocked(uint8_t chan, double time,
assert(time >= note.time());
note.set_duration(time - note.time());
_write_notes[chan].erase(n);
- //cerr << "MM resolved note, duration: " << note.duration() << endl;
+ debugout << "resolved note, duration: " << note.duration() << endl;
resolved = true;
break;
}
}
if (!resolved) {
- cerr << "Sequence " << this << " spurious note off chan " << (int)chan
+ errorout << this << " spurious note off chan " << (int)chan
<< ", note " << (int)note_num << " @ " << time << endl;
}
}
-void Sequence::append_control_unlocked(const Parameter& param, double time, double value)
+void
+Sequence::append_control_unlocked(const Parameter& param, double time, double value)
{
+ debugout << this << " " << param.symbol() << " @ " << time << " = " << value
+ << " controls: " << &_controls
+ << " # controls: " << _controls.size() << endl;
control(param, true)->list()->rt_add(time, value);
}
-void Sequence::add_note_unlocked(const boost::shared_ptr<Note> note)
+void
+Sequence::add_note_unlocked(const boost::shared_ptr<Note> note)
{
- //cerr << "Sequence " << this << " add note " << (int)note.note() << " @ " << note.time() << endl;
+ debugout << this << " add note " << (int)note->note() << " @ " << note->time() << endl;
_edited = true;
Notes::iterator i = upper_bound(_notes.begin(), _notes.end(), note,
note_time_comparator);
_notes.insert(i, note);
}
-void Sequence::remove_note_unlocked(const boost::shared_ptr<const Note> note)
+void
+Sequence::remove_note_unlocked(const boost::shared_ptr<const Note> note)
{
_edited = true;
- //cerr << "Sequence " << this << " remove note " << (int)note.note() << " @ " << note.time() << endl;
+ debugout << this << " remove note " << (int)note->note() << " @ " << note->time() << endl;
for (Notes::iterator n = _notes.begin(); n != _notes.end(); ++n) {
Note& _n = *(*n);
const Note& _note = *note;
@@ -628,7 +654,8 @@ void Sequence::remove_note_unlocked(const boost::shared_ptr<const Note> note)
/** Slow! for debugging only. */
#ifndef NDEBUG
-bool Sequence::is_sorted() const {
+bool
+Sequence::is_sorted() const {
bool t = 0;
for (Notes::const_iterator n = _notes.begin(); n != _notes.end(); ++n)
if ((*n)->time() < t)