From fca81c9a6a591b6f94223454a981d919c9ad51a5 Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Sun, 21 Apr 2013 13:10:27 -0400 Subject: add Group::clear(), do not clear _canvas member of Item when unparented (only the parent is changed) --- gtk2_ardour/tempo_lines.cc | 182 +++++++-------------------------------------- 1 file changed, 28 insertions(+), 154 deletions(-) (limited to 'gtk2_ardour/tempo_lines.cc') diff --git a/gtk2_ardour/tempo_lines.cc b/gtk2_ardour/tempo_lines.cc index dddb1ee426..27b5aac974 100644 --- a/gtk2_ardour/tempo_lines.cc +++ b/gtk2_ardour/tempo_lines.cc @@ -25,42 +25,30 @@ using namespace std; -#define MAX_CACHED_LINES 128 - TempoLines::TempoLines (ArdourCanvas::Canvas& canvas, ArdourCanvas::Group* group, double screen_height) : _canvas (canvas) - , _group(group) - , _clean_left(DBL_MAX) - , _clean_right(0.0) - , _height(screen_height) + , _group (group) + , _height (screen_height) { } void TempoLines::tempo_map_changed() { - _clean_left = DBL_MAX; - _clean_right = 0.0; - - double_t d = 1.0; - // TODO: Dirty/slow, but 'needed' for zoom :( - for (Lines::iterator i = _lines.begin(); i != _lines.end(); d += 1.0) { - Lines::iterator next = i; - ++next; - i->second->set_x0 (-d); - i->second->set_x1 (-d); - ArdourCanvas::Line* f = i->second; - _lines.erase(i); - _lines.insert(make_pair(- d, f)); - i = next; - } + /* remove all lines from the group, put them in the cache (to avoid + * unnecessary object destruction+construction later), and clear _lines + */ + + _group->clear (); + _cache.insert (_cache.end(), _lines.begin(), _lines.end()); + _lines.clear (); } void TempoLines::show () { for (Lines::iterator i = _lines.begin(); i != _lines.end(); ++i) { - i->second->show(); + (*i)->show(); } } @@ -68,7 +56,7 @@ void TempoLines::hide () { for (Lines::iterator i = _lines.begin(); i != _lines.end(); ++i) { - i->second->hide(); + (*i)->hide(); } } @@ -78,18 +66,13 @@ TempoLines::draw (const ARDOUR::TempoMap::BBTPointList::const_iterator& begin, double samples_per_pixel) { ARDOUR::TempoMap::BBTPointList::const_iterator i; - ArdourCanvas::Line *line = 0; - gdouble xpos; + ArdourCanvas::Rect const visible = _canvas.visible_area (); double beat_density; uint32_t beats = 0; uint32_t bars = 0; uint32_t color; - const size_t needed = distance (begin, end); - - ArdourCanvas::Rect const visible = _canvas.visible_area (); - /* get the first bar spacing */ i = end; @@ -105,31 +88,7 @@ TempoLines::draw (const ARDOUR::TempoMap::BBTPointList::const_iterator& begin, return; } - xpos = rint(((framepos_t)(*i).frame) / (double)samples_per_pixel); - const double needed_right = xpos; - - i = begin; - - xpos = rint(((framepos_t)(*i).frame) / (double)samples_per_pixel); - const double needed_left = xpos; - - Lines::iterator left = _lines.lower_bound(xpos); // first line >= xpos - - bool exhausted = (left == _lines.end()); - Lines::iterator li = left; - if (li != _lines.end()) - line = li->second; - - // Tempo map hasn't changed and we're entirely within a clean - // range, don't need to do anything. Yay. - if (needed_left >= _clean_left && needed_right <= _clean_right) { - // cerr << endl << "*** LINE CACHE PERFECT HIT" << endl; - return; - } - - // cerr << endl << "*** LINE CACHE MISS" << endl; - - bool invalidated = false; + tempo_map_changed (); for (i = begin; i != end; ++i) { @@ -142,109 +101,24 @@ TempoLines::draw (const ARDOUR::TempoMap::BBTPointList::const_iterator& begin, color = ARDOUR_UI::config()->get_canvasvar_MeasureLineBeat(); } - xpos = rint(((framepos_t)(*i).frame) / (double)samples_per_pixel); - - li = _lines.lower_bound(xpos); // first line >= xpos - - line = (li != _lines.end()) ? li->second : 0; - assert(!line || line->x0() == li->first); - - Lines::iterator next = li; - if (next != _lines.end()) - ++next; - - exhausted = (next == _lines.end()); - - // Hooray, line is perfect - if (line && line->x0() == xpos) { - if (li != _lines.end()) - ++li; - - line->set_outline_color (color); - - // Use existing line, moving if necessary - } else if (!exhausted) { - Lines::iterator steal = _lines.end(); - --steal; - - // Steal from the right - if (left->first > needed_left && li != steal && steal->first > needed_right) { - // cerr << "*** STEALING FROM RIGHT" << endl; - double const x = steal->first; - line = steal->second; - _lines.erase(steal); - line->set_x0 (xpos); - line->set_x1 (xpos); - line->set_outline_color (color); - _lines.insert(make_pair(xpos, line)); - invalidated = true; - - // Shift clean range left - _clean_left = min(_clean_left, xpos); - _clean_right = min(_clean_right, x); - - // Move this line to where we need it - } else { - Lines::iterator existing = _lines.find(xpos); - if (existing != _lines.end()) { - //cout << "*** EXISTING LINE" << endl; - li = existing; - li->second->set_outline_color (color); - } else { - //cout << "*** MOVING LINE" << endl; - const double x1 = line->x0(); - const bool was_clean = x1 >= _clean_left && x1 <= _clean_right; - invalidated = invalidated || was_clean; - // Invalidate clean portion (XXX: too harsh?) - _clean_left = needed_left; - _clean_right = needed_right; - _lines.erase(li); - line->set_outline_color (color); - line->set_x0 (xpos); - line->set_x1 (xpos); - _lines.insert(make_pair(xpos, line)); - } - } - - // Create a new line - } else if (_lines.size() < needed || _lines.size() < MAX_CACHED_LINES) { - // cerr << "*** CREATING LINE" << endl; - /* if we already have a line there ... don't sweat it */ - if (_lines.find (xpos) == _lines.end()) { - line = new ArdourCanvas::Line (_group); - line->set_x0 (xpos); - line->set_x1 (xpos); - line->set_y0 (0.0); - line->set_y1 (ArdourCanvas::COORD_MAX); - line->set_outline_color (color); - _lines.insert(make_pair(xpos, line)); - } - - // Steal from the left + ArdourCanvas::Coord xpos = rint(((framepos_t)(*i).frame) / (double)samples_per_pixel); + + ArdourCanvas::Line* line; + + if (!_cache.empty()) { + line = _cache.back (); + _cache.pop_back (); + line->reparent (_group); } else { - // cerr << "*** STEALING FROM LEFT" << endl; - if (_lines.find (xpos) == _lines.end()) { - Lines::iterator steal = _lines.begin(); - double const x = steal->first; - line = steal->second; - _lines.erase(steal); - line->set_outline_color (color); - line->set_x0 (xpos); - line->set_x1 (xpos); - _lines.insert(make_pair(xpos, line)); - invalidated = true; - - // Shift clean range right - _clean_left = max(_clean_left, x); - _clean_right = max(_clean_right, xpos); - } + line = new ArdourCanvas::Line (_group); } - } - // Extend range to what we've 'fixed' - if (!invalidated) { - _clean_left = min(_clean_left, needed_left); - _clean_right = max(_clean_right, needed_right); + line->set_x0 (xpos); + line->set_x1 (xpos); + line->set_y0 (0.0); + line->set_y1 (_height); + line->set_outline_color (color); + line->show (); } } -- cgit v1.2.3