libs/evoral/src/Sequence.cpp

   1 /* This file is part of Evoral.
   2  * Copyright (C) 2008 Dave Robillard <http://drobilla.net>
   3  * Copyright (C) 2000-2008 Paul Davis
   4  *
   5  * Evoral is free software; you can redistribute it and/or modify it under the
   6  * terms of the GNU General Public License as published by the Free Software
   7  * Foundation; either version 2 of the License, or (at your option) any later
   8  * version.
   9  *
  10  * Evoral is distributed in the hope that it will be useful, but WITHOUT ANY
  11  * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
  12  * FOR A PARTICULAR PURPOSE.  See the GNU General Public License for details.
  13  *
  14  * You should have received a copy of the GNU General Public License along
  15  * with this program; if not, write to the Free Software Foundation, Inc.,
  16  * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
  17  */
  18
  19 #define __STDC_LIMIT_MACROS 1
  20
  21 #include <iostream>
  22 #include <cmath>
  23 #include <algorithm>
  24 #include <stdexcept>
  25 #include <stdint.h>
  26 #include <evoral/Sequence.hpp>
  27 #include <evoral/ControlList.hpp>
  28 #include <evoral/Control.hpp>
  29 #include <evoral/ControlSet.hpp>
  30 #include <evoral/EventSink.hpp>
  31 #include <evoral/MIDIParameters.hpp>
  32 #include <evoral/TypeMap.hpp>
  33
  34 using namespace std;
  35
  36 namespace Evoral {
  37
  38 void Sequence::write_lock() {
  39         _lock.writer_lock();
  40         _control_lock.lock();
  41 }
  42
  43 void Sequence::write_unlock() {
  44         _lock.writer_unlock();
  45         _control_lock.unlock();
  46 }
  47
  48 void Sequence::read_lock() const {
  49         _lock.reader_lock();
  50 }
  51
  52 void Sequence::read_unlock() const {
  53         _lock.reader_unlock();
  54 }
  55
  56 struct null_ostream : public std::ostream {
  57         null_ostream(): std::ios(0), std::ostream(0) {}
  58 };
  59 static null_ostream nullout;
  60
  61 //static ostream& debugout = cout;
  62 static ostream& debugout = nullout;
  63 static ostream& errorout = cerr;
  64
  65 // Read iterator (const_iterator)
  66
  67 Sequence::const_iterator::const_iterator(const Sequence& seq, EventTime t)
  68         : _seq(&seq)
  69         , _is_end( (t == DBL_MAX) || seq.empty() )
  70         , _locked( !_is_end )
  71 {
  72         debugout << "Created Iterator @ " << t << " (is end: " << _is_end << ")" << endl;
  73
  74         if (_is_end) {
  75                 return;
  76         }
  77
  78         seq.read_lock();
  79
  80         // find first note which begins after t
  81         _note_iter = seq.notes().end();
  82         for (Sequence::Notes::const_iterator i = seq.notes().begin(); i != seq.notes().end(); ++i) {
  83                 if ((*i)->time() >= t) {
  84                         _note_iter = i;
  85                         break;
  86                 }
  87         }
  88
  89         ControlIterator earliest_control(boost::shared_ptr<ControlList>(), DBL_MAX, 0.0);
  90
  91         _control_iters.reserve(seq._controls.size());
  92
  93         // find the earliest control event available
  94         for (Controls::const_iterator i = seq._controls.begin(); i != seq._controls.end(); ++i) {
  95                 debugout << "Iterator: control: " << i->first.symbol() << endl;
  96                 double x, y;
  97                 bool ret = i->second->list()->rt_safe_earliest_event_unlocked(t, DBL_MAX, x, y);
  98                 if (!ret) {
  99                         debugout << "Iterator: CC " << i->first.id() << " (size " << i->second->list()->size()
 100                                 << ") has no events past " << t << endl;
 101                         continue;
 102                 }
 103
 104                 assert(x >= 0);
 105
 106                 /*if (y < i->first.min() || y > i->first.max()) {
 107                         errorout << "ERROR: Controller " << i->first.symbol() << " value " << y
 108                                 << " out of range [" << i->first.min() << "," << i->first.max()
 109                                 << "], event ignored" << endl;
 110                         continue;
 111                 }*/
 112
 113                 const ControlIterator new_iter(i->second->list(), x, y);
 114
 115                 debugout << "Iterator: CC " << i->first.id() << " added (" << x << ", " << y << ")" << endl;
 116                 _control_iters.push_back(new_iter);
 117
 118                 // if the x of the current control is less than earliest_control
 119                 // we have a new earliest_control
 120                 if (x < earliest_control.x) {
 121                         earliest_control = new_iter;
 122                         _control_iter = _control_iters.end();
 123                         --_control_iter;
 124                         // now _control_iter points to the last Element in _control_iters
 125                 }
 126         }
 127
 128         if (_note_iter != seq.notes().end()
 129                         && (*_note_iter)->on_event().time() >= t
 130                         && (!earliest_control.list
 131                                 || (*_note_iter)->on_event().time() < earliest_control.x)) {
 132                 debugout << "Reading note on event @ " << (*_note_iter)->on_event().time() << endl;
 133                 _event = boost::shared_ptr<Event>(new Event((*_note_iter)->on_event(), true));
 134                 _active_notes.push(*_note_iter);
 135                 ++_note_iter;
 136                 _control_iter = _control_iters.end();
 137         } else if (earliest_control.list) {
 138                 debugout << "Reading control event @ " << earliest_control.x << endl;
 139                 seq.control_to_midi_event(_event, earliest_control);
 140         }
 141
 142         if ( (! _event.get()) || _event->size() == 0) {
 143                 debugout << "New iterator @ " << t << " is at end." << endl;
 144                 _is_end = true;
 145
 146                 // eliminate possible race condition here (ugly)
 147                 static Glib::Mutex mutex;
 148                 Glib::Mutex::Lock lock(mutex);
 149                 if (_locked) {
 150                         _seq->read_unlock();
 151                         _locked = false;
 152                 }
 153         } else {
 154                 debugout << "New Iterator = " << _event->event_type();
 155                 debugout << " : " << hex << (int)((MIDIEvent*)_event.get())->type();
 156                 debugout << " @ " <<  _event->time() << endl;
 157         }
 158
 159         //assert(_is_end || (_event->buffer() && _event->buffer()[0] != '\0'));
 160 }
 161
 162 Sequence::const_iterator::~const_iterator()
 163 {
 164         if (_locked) {
 165                 _seq->read_unlock();
 166         }
 167 }
 168
 169 const
 170 Sequence::const_iterator& Sequence::const_iterator::operator++()
 171 {
 172         if (_is_end) {
 173                 throw std::logic_error("Attempt to iterate past end of Sequence");
 174         }
 175
 176         debugout << "Iterator ++" << endl;
 177         assert(_event->buffer() && _event->size() > 0);
 178
 179         const MIDIEvent& ev = *((MIDIEvent*)_event.get());
 180
 181         //debugout << "const_iterator::operator++: " << _event->to_string() << endl;
 182
 183         if (! (ev.is_note() || ev.is_cc() || ev.is_pgm_change()
 184                                 || ev.is_pitch_bender() || ev.is_channel_pressure()) ) {
 185                 errorout << "Unknown event type: " << hex << int(ev.buffer()[0])
 186                         << int(ev.buffer()[1]) << int(ev.buffer()[2]) << endl;
 187         }
 188         assert((ev.is_note() || ev.is_cc() || ev.is_pgm_change() || ev.is_pitch_bender() || ev.is_channel_pressure()));
 189
 190         // Increment past current control event
 191         if (!ev.is_note() && _control_iter != _control_iters.end() && _control_iter->list.get()) {
 192                 double x = 0.0, y = 0.0;
 193                 const bool ret = _control_iter->list->rt_safe_earliest_event_unlocked(
 194                                 _control_iter->x, DBL_MAX, x, y, false);
 195
 196                 assert(!ret || x > _control_iter->x);
 197
 198                 if (ret) {
 199                         _control_iter->x = x;
 200                         _control_iter->y = y;
 201                 } else {
 202                         _control_iter->list.reset();
 203                         _control_iter->x = DBL_MAX;
 204                         _control_iter->y = DBL_MAX;
 205                 }
 206         }
 207
 208         _control_iter = _control_iters.begin();
 209
 210         // find the _control_iter with the earliest event time
 211         for (ControlIterators::iterator i = _control_iters.begin(); i != _control_iters.end(); ++i) {
 212                 if (i->x < _control_iter->x) {
 213                         _control_iter = i;
 214                 }
 215         }
 216
 217         enum Type {NIL, NOTE_ON, NOTE_OFF, CONTROL};
 218
 219         Type type = NIL;
 220         EventTime t = 0;
 221
 222         // Next earliest note on
 223         if (_note_iter != _seq->notes().end()) {
 224                 type = NOTE_ON;
 225                 t = (*_note_iter)->time();
 226         }
 227
 228         // Use the next earliest note off iff it's earlier than the note on
 229         if (!_seq->percussive() && (! _active_notes.empty())) {
 230                 if (type == NIL || _active_notes.top()->end_time() <= t) {
 231                         type = NOTE_OFF;
 232                         t = _active_notes.top()->end_time();
 233                 }
 234         }
 235
 236         // Use the next earliest controller iff it's earlier than the note event
 237         if (_control_iter != _control_iters.end() && _control_iter->x != DBL_MAX) {
 238                 if (type == NIL || _control_iter->x < t) {
 239                         type = CONTROL;
 240                 }
 241         }
 242
 243         if (type == NOTE_ON) {
 244                 debugout << "Iterator = note on" << endl;
 245                 *_event = (*_note_iter)->on_event();
 246                 _active_notes.push(*_note_iter);
 247                 ++_note_iter;
 248         } else if (type == NOTE_OFF) {
 249                 debugout << "Iterator = note off" << endl;
 250                 *_event = _active_notes.top()->off_event();
 251                 _active_notes.pop();
 252         } else if (type == CONTROL) {
 253                 debugout << "Iterator = control" << endl;
 254                 _seq->control_to_midi_event(_event, *_control_iter);
 255         } else {
 256                 debugout << "Iterator = End" << endl;
 257                 _is_end = true;
 258         }
 259
 260         assert(_is_end || (_event->size() > 0 && _event->buffer() && _event->buffer()[0] != '\0'));
 261
 262         return *this;
 263 }
 264
 265 bool
 266 Sequence::const_iterator::operator==(const const_iterator& other) const
 267 {
 268         if (_is_end || other._is_end) {
 269                 return (_is_end == other._is_end);
 270         } else {
 271                 return (_event == other._event);
 272         }
 273 }
 274
 275 Sequence::const_iterator&
 276 Sequence::const_iterator::operator=(const const_iterator& other)
 277 {
 278         if (_locked && _seq != other._seq) {
 279                 _seq->read_unlock();
 280         }
 281
 282         _seq           = other._seq;
 283         _active_notes  = other._active_notes;
 284         _is_end        = other._is_end;
 285         _locked        = other._locked;
 286         _note_iter     = other._note_iter;
 287         _control_iters = other._control_iters;
 288         size_t index   = other._control_iter - other._control_iters.begin();
 289         _control_iter  = _control_iters.begin() + index;
 290
 291         if (!_is_end && other._event) {
 292                 if (_event) {
 293                         *_event = *other._event.get();
 294                 } else {
 295                         _event = boost::shared_ptr<Event>(new Event(*other._event, true));
 296                 }
 297         } else {
 298                 if (_event) {
 299                         _event->clear();
 300                 }
 301         }
 302
 303         return *this;
 304 }
 305
 306 // Sequence
 307
 308 Sequence::Sequence(const TypeMap& type_map, size_t size)
 309         : _read_iter(*this, DBL_MAX)
 310         , _edited(false)
 311         , _type_map(type_map)
 312         , _notes(size)
 313         , _writing(false)
 314         , _end_iter(*this, DBL_MAX)
 315         , _next_read(UINT32_MAX)
 316         , _percussive(false)
 317 {
 318         debugout << "Sequence (size " << size << ") constructed: " << this << endl;
 319         assert(_end_iter._is_end);
 320         assert( ! _end_iter._locked);
 321 }
 322
 323 /** Read events in frame range \a start .. \a start+cnt into \a dst,
 324  * adding \a offset to each event's timestamp.
 325  * \return number of events written to \a dst
 326  */
 327 size_t
 328 Sequence::read(EventSink& dst, timestamp_t start, timedur_t nframes, timestamp_t offset) const
 329 {
 330         debugout << this << " read @ " << start << " * " << nframes << " + " << offset << endl;
 331         debugout << this << " # notes: " << n_notes() << endl;
 332         debugout << this << " # controls: " << _controls.size() << endl;
 333
 334         size_t read_events = 0;
 335
 336         if (start != _next_read) {
 337                 debugout << "Repositioning iterator from " << _next_read << " to " << start << endl;
 338                 _read_iter = const_iterator(*this, (double)start);
 339         } else {
 340                 debugout << "Using cached iterator at " << _next_read << endl;
 341         }
 342
 343         _next_read = (nframes_t) floor (start + nframes);
 344
 345         while (_read_iter != end() && _read_iter->time() < start + nframes) {
 346                 assert(_read_iter->size() > 0);
 347                 assert(_read_iter->buffer());
 348                 dst.write(_read_iter->time() + offset,
 349                           _read_iter->event_type(),
 350                           _read_iter->size(),
 351                           _read_iter->buffer());
 352
 353                  debugout << this << " read event type " << _read_iter->event_type()
 354                          << " @ " << _read_iter->time() << " : ";
 355                  for (size_t i = 0; i < _read_iter->size(); ++i)
 356                          debugout << hex << (int)_read_iter->buffer()[i];
 357                  debugout << endl;
 358
 359                 ++_read_iter;
 360                 ++read_events;
 361         }
 362
 363         return read_events;
 364 }
 365
 366 /** Write the controller event pointed to by \a iter to \a ev.
 367  * The buffer of \a ev will be allocated or resized as necessary.
 368  * The event_type of \a ev should be set to the expected output type.
 369  * \return true on success
 370  */
 371 bool
 372 Sequence::control_to_midi_event(boost::shared_ptr<Event>& ev, const ControlIterator& iter) const
 373 {
 374         assert(iter.list.get());
 375         const uint32_t event_type = iter.list->parameter().type();
 376         if (!ev) {
 377                 ev = boost::shared_ptr<Event>(new Event(event_type, 0, 3, NULL, true));
 378         }
 379
 380         uint8_t midi_type = _type_map.parameter_midi_type(iter.list->parameter());
 381         ev->set_event_type(_type_map.midi_event_type(midi_type));
 382         switch (midi_type) {
 383         case MIDI_CMD_CONTROL:
 384                 assert(iter.list.get());
 385                 assert(iter.list->parameter().channel() < 16);
 386                 assert(iter.list->parameter().id() <= INT8_MAX);
 387                 assert(iter.y <= INT8_MAX);
 388
 389                 ev->time() = iter.x;
 390                 ev->realloc(3);
 391                 ev->buffer()[0] = MIDI_CMD_CONTROL + iter.list->parameter().channel();
 392                 ev->buffer()[1] = (uint8_t)iter.list->parameter().id();
 393                 ev->buffer()[2] = (uint8_t)iter.y;
 394                 break;
 395
 396         case MIDI_CMD_PGM_CHANGE:
 397                 assert(iter.list.get());
 398                 assert(iter.list->parameter().channel() < 16);
 399                 assert(iter.y <= INT8_MAX);
 400
 401                 ev->time() = iter.x;
 402                 ev->realloc(2);
 403                 ev->buffer()[0] = MIDI_CMD_PGM_CHANGE + iter.list->parameter().channel();
 404                 ev->buffer()[1] = (uint8_t)iter.y;
 405                 break;
 406
 407         case MIDI_CMD_BENDER:
 408                 assert(iter.list.get());
 409                 assert(iter.list->parameter().channel() < 16);
 410                 assert(iter.y < (1<<14));
 411
 412                 ev->time() = iter.x;
 413                 ev->realloc(3);
 414                 ev->buffer()[0] = MIDI_CMD_BENDER + iter.list->parameter().channel();
 415                 ev->buffer()[1] = uint16_t(iter.y) & 0x7F; // LSB
 416                 ev->buffer()[2] = (uint16_t(iter.y) >> 7) & 0x7F; // MSB
 417                 break;
 418
 419         case MIDI_CMD_CHANNEL_PRESSURE:
 420                 assert(iter.list.get());
 421                 assert(iter.list->parameter().channel() < 16);
 422                 assert(iter.y <= INT8_MAX);
 423
 424                 ev->time() = iter.x;
 425                 ev->realloc(2);
 426                 ev->buffer()[0] = MIDI_CMD_CHANNEL_PRESSURE + iter.list->parameter().channel();
 427                 ev->buffer()[1] = (uint8_t)iter.y;
 428                 break;
 429
 430         default:
 431                 return false;
 432         }
 433
 434         return true;
 435 }
 436
 437 /** Clear all events from the model.
 438  */
 439 void
 440 Sequence::clear()
 441 {
 442         _lock.writer_lock();
 443         _notes.clear();
 444         for (Controls::iterator li = _controls.begin(); li != _controls.end(); ++li)
 445                 li->second->list()->clear();
 446         _next_read = 0;
 447         _read_iter = end();
 448         _lock.writer_unlock();
 449 }
 450
 451 /** Begin a write of events to the model.
 452  *
 453  * If \a mode is Sustained, complete notes with length are constructed as note
 454  * on/off events are received.  Otherwise (Percussive), only note on events are
 455  * stored; note off events are discarded entirely and all contained notes will
 456  * have length 0.
 457  */
 458 void
 459 Sequence::start_write()
 460 {
 461         debugout << this << " START WRITE, PERCUSSIVE = " << _percussive << endl;
 462         write_lock();
 463         _writing = true;
 464         for (int i = 0; i < 16; ++i)
 465                 _write_notes[i].clear();
 466
 467         _dirty_controls.clear();
 468         write_unlock();
 469 }
 470
 471 /** Finish a write of events to the model.
 472  *
 473  * If \a delete_stuck is true and the current mode is Sustained, note on events
 474  * that were never resolved with a corresonding note off will be deleted.
 475  * Otherwise they will remain as notes with length 0.
 476  */
 477 void
 478 Sequence::end_write(bool delete_stuck)
 479 {
 480         write_lock();
 481         assert(_writing);
 482
 483         debugout << this << " END WRITE: " << _notes.size() << " NOTES\n";
 484
 485         if (!_percussive && delete_stuck) {
 486                 for (Notes::iterator n = _notes.begin(); n != _notes.end() ;) {
 487                         if ((*n)->length() == 0) {
 488                                 errorout << "WARNING: Stuck note lost: " << (*n)->note() << endl;
 489                                 n = _notes.erase(n);
 490                                 // we have to break here because erase invalidates the iterator
 491                                 break;
 492                         } else {
 493                                 ++n;
 494                         }
 495                 }
 496         }
 497
 498         for (int i = 0; i < 16; ++i) {
 499                 if (!_write_notes[i].empty()) {
 500                         errorout << "WARNING: Sequence::end_write: Channel " << i << " has "
 501                                         << _write_notes[i].size() << " stuck notes" << endl;
 502                 }
 503                 _write_notes[i].clear();
 504         }
 505
 506         for (ControlLists::const_iterator i = _dirty_controls.begin(); i != _dirty_controls.end(); ++i) {
 507                 (*i)->mark_dirty();
 508         }
 509
 510         _writing = false;
 511         write_unlock();
 512 }
 513
 514 /** Append \a ev to model.  NOT realtime safe.
 515  *
 516  * Timestamps of events in \a buf are expected to be relative to
 517  * the start of this model (t=0) and MUST be monotonically increasing
 518  * and MUST be >= the latest event currently in the model.
 519  */
 520 void
 521 Sequence::append(const Event& event)
 522 {
 523         write_lock();
 524         _edited = true;
 525
 526         const MIDIEvent& ev = (const MIDIEvent&)event;
 527
 528         assert(_notes.empty() || ev.time() >= _notes.back()->time());
 529         assert(_writing);
 530
 531         if (ev.is_note_on()) {
 532                 append_note_on_unlocked(ev.channel(), ev.time(), ev.note(),
 533                                 ev.velocity());
 534         } else if (ev.is_note_off()) {
 535                 append_note_off_unlocked(ev.channel(), ev.time(), ev.note());
 536         } else if (!_type_map.type_is_midi(ev.event_type())) {
 537                 printf("WARNING: Sequence: Unknown event type %X\n", ev.event_type());
 538         } else if (ev.is_cc()) {
 539                 append_control_unlocked(
 540                                 Evoral::MIDI::ContinuousController(ev.event_type(), ev.channel(), ev.cc_number()),
 541                                 ev.time(), ev.cc_value());
 542         } else if (ev.is_pgm_change()) {
 543                 append_control_unlocked(
 544                                 Evoral::MIDI::ProgramChange(ev.event_type(), ev.channel()),
 545                                 ev.time(), ev.pgm_number());
 546         } else if (ev.is_pitch_bender()) {
 547                 append_control_unlocked(
 548                                 Evoral::MIDI::PitchBender(ev.event_type(), ev.channel()),
 549                                 ev.time(), double(  (0x7F & ev.pitch_bender_msb()) << 7
 550                                         | (0x7F & ev.pitch_bender_lsb()) ));
 551         } else if (ev.is_channel_pressure()) {
 552                 append_control_unlocked(
 553                                 Evoral::MIDI::ChannelPressure(ev.event_type(), ev.channel()),
 554                                 ev.time(), ev.channel_pressure());
 555         } else {
 556                 printf("WARNING: Sequence: Unknown MIDI event type %X\n", ev.type());
 557         }
 558
 559         write_unlock();
 560 }
 561
 562 void
 563 Sequence::append_note_on_unlocked(uint8_t chan, EventTime time, uint8_t note_num, uint8_t velocity)
 564 {
 565         debugout << this << " c" << (int)chan << " note " << (int)note_num << " off @ " << time << endl;
 566         assert(note_num <= 127);
 567         assert(chan < 16);
 568         assert(_writing);
 569         _edited = true;
 570
 571         boost::shared_ptr<Note> new_note(new Note(chan, time, 0, note_num, velocity));
 572         _notes.push_back(new_note);
 573         if (!_percussive) {
 574                 debugout << "Sustained: Appending active note on " << (unsigned)(uint8_t)note_num << endl;
 575                 _write_notes[chan].push_back(_notes.size() - 1);
 576         } else {
 577                 debugout << "Percussive: NOT appending active note on" << endl;
 578          }
 579 }
 580
 581 void
 582 Sequence::append_note_off_unlocked(uint8_t chan, EventTime time, uint8_t note_num)
 583 {
 584         debugout << this << " c" << (int)chan << " note " << (int)note_num << " off @ " << time << endl;
 585         assert(note_num <= 127);
 586         assert(chan < 16);
 587         assert(_writing);
 588         _edited = true;
 589
 590         if (_percussive) {
 591                 debugout << "Sequence Ignoring note off (percussive mode)" << endl;
 592                 return;
 593         }
 594
 595         /* FIXME: make _write_notes fixed size (127 noted) for speed */
 596
 597         /* FIXME: note off velocity for that one guy out there who actually has
 598          * keys that send it */
 599
 600         bool resolved = false;
 601
 602         for (WriteNotes::iterator n = _write_notes[chan].begin(); n
 603                         != _write_notes[chan].end(); ++n) {
 604                 Note& note = *_notes[*n].get();
 605                 if (note.note() == note_num) {
 606                         assert(time >= note.time());
 607                         note.set_length(time - note.time());
 608                         _write_notes[chan].erase(n);
 609                         debugout << "resolved note, length: " << note.length() << endl;
 610                         resolved = true;
 611                         break;
 612                 }
 613         }
 614
 615         if (!resolved) {
 616                 errorout << this << " spurious note off chan " << (int)chan
 617                                 << ", note " << (int)note_num << " @ " << time << endl;
 618         }
 619 }
 620
 621 void
 622 Sequence::append_control_unlocked(const Parameter& param, EventTime time, double value)
 623 {
 624         debugout << this << " " << param.symbol() << " @ " << time << " \t= \t" << value
 625                         << " # controls: " << _controls.size() << endl;
 626         control(param, true)->list()->rt_add(time, value);
 627 }
 628
 629
 630 void
 631 Sequence::add_note_unlocked(const boost::shared_ptr<Note> note)
 632 {
 633         debugout << this << " add note " << (int)note->note() << " @ " << note->time() << endl;
 634         _edited = true;
 635         Notes::iterator i = upper_bound(_notes.begin(), _notes.end(), note,
 636                         note_time_comparator);
 637         _notes.insert(i, note);
 638 }
 639
 640 void
 641 Sequence::remove_note_unlocked(const boost::shared_ptr<const Note> note)
 642 {
 643         _edited = true;
 644         debugout << this << " remove note " << (int)note->note() << " @ " << note->time() << endl;
 645         for (Notes::iterator n = _notes.begin(); n != _notes.end(); ++n) {
 646                 Note& _n = *(*n);
 647                 const Note& _note = *note;
 648                 // TODO: There is still the issue, that after restarting ardour
 649                 // persisted undo does not work, because of rounding errors in the
 650                 // event times after saving/restoring to/from MIDI files
 651                 /*cerr << "======================================= " << endl;
 652                 cerr << int(_n.note()) << "@" << int(_n.time()) << "[" << int(_n.channel()) << "] --" << int(_n.length()) << "-- #" << int(_n.velocity()) << endl;
 653                 cerr << int(_note.note()) << "@" << int(_note.time()) << "[" << int(_note.channel()) << "] --" << int(_note.length()) << "-- #" << int(_note.velocity()) << endl;
 654                 cerr << "Equal: " << bool(_n == _note) << endl;
 655                 cerr << endl << endl;*/
 656                 if (_n == _note) {
 657                         _notes.erase(n);
 658                         // we have to break here, because erase invalidates all iterators, ie. n itself
 659                         break;
 660                 }
 661         }
 662 }
 663
 664 /** Slow!  for debugging only. */
 665 #ifndef NDEBUG
 666 bool
 667 Sequence::is_sorted() const {
 668         bool t = 0;
 669         for (Notes::const_iterator n = _notes.begin(); n != _notes.end(); ++n)
 670                 if ((*n)->time() < t)
 671                         return false;
 672                 else
 673                         t = (*n)->time();
 674
 675         return true;
 676 }
 677 #endif
 678
 679 } // namespace Evoral
 680