From 696631f04e1a7eabc33b0c7823f64d011718e1ab Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Sun, 9 Sep 2012 20:31:43 +0000 Subject: [PATCH] comment tweaks from a second reading git-svn-id: svn://localhost/ardour2/branches/3.0@13182 d708f5d6-7413-0410-9779-e7cbd77b26cf --- libs/evoral/src/ControlList.cpp | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/libs/evoral/src/ControlList.cpp b/libs/evoral/src/ControlList.cpp index 907428af28..83222857ed 100644 --- a/libs/evoral/src/ControlList.cpp +++ b/libs/evoral/src/ControlList.cpp @@ -571,11 +571,12 @@ ControlList::add (double when, double value) if (!_events.empty()) { // avoid O(N) _events.size() here if (_events.back()->value == value) { EventList::iterator b = _events.end(); - --b; // last point, which we know exists - if (b != _events.begin()) { // step back again, which may not be possible - --b; // next-to-last-point + --b; // final point, which we know exists + if (b != _events.begin()) { // step back again, but check first that it is legal + --b; // penultimate-point if ((*b)->value == value) { - /* straight line - just move the last + /* there are at least two points with the exact same value ... + * straight line - just move the final * point to the new time */ _events.back()->when = when;