smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-12 09:36:33 +01:00
Code Issues Projects Releases Wiki Activity

Properly remove-time automation

Browse source 
Previously "remove time" was able to produce overlapping, not
ordered automation.
This commit is contained in:
Robin Gareus 2018-12-22 01:09:24 +01:00
parent 4b9d96ff8e
commit 616f7f40b6
No known key found for this signature in database
GPG key ID: A090BCE02CF57F04
1 changed files with 8 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

8
libs/evoral/src/ControlList.cpp
View file

@ -990,6 +990,14 @@ ControlList::shift (double pos, double frames)
{ {
{ {
Glib::Threads::RWLock::WriterLock lm (_lock); Glib::Threads::RWLock::WriterLock lm (_lock);
if (frames < 0) {
/* Route::shift () with negative shift is used
* for "remove time". The time [pos.. pos-frames] is removed.
* and everyhing after, moved backwards.
* TODO: consider adding guard-points (need special-casing)
*/
erase_range_internal (pos, pos - frames, _events);
}
for (iterator i = _events.begin(); i != _events.end(); ++i) { for (iterator i = _events.begin(); i != _events.end(); ++i) {
if ((*i)->when >= pos) { if ((*i)->when >= pos) {