comparison base/View.cpp @ 111:1be256ac7681

* It turns out we can be more efficient in the spectrogram repaints by
  marking the spectrogram layer as non-scrollable, and using the spectrogram
  cache alone instead of both spectrogram and view caches.

 
     // We want a list of all the scrollable layers that are behind the
     // backmost non-scrollable layer.
 
     LayerList scrollables;
+    bool metUnscrollable = false;
+
     for (LayerList::const_iterator i = m_layers.begin(); i != m_layers.end(); ++i) {
 	if ((*i)->isLayerDormant(this)) continue;
 	if ((*i)->isLayerOpaque()) {
 	    // You can't see anything behind an opaque layer!
 	    scrollables.clear();
-	}
-	if ((*i)->isLayerScrollable(this)) scrollables.push_back(*i);
-	else break;
+            if (metUnscrollable) break;
+	}
+	if (!metUnscrollable && (*i)->isLayerScrollable(this)) {
+            scrollables.push_back(*i);
+        } else {
+            metUnscrollable = true;
+        }
     }
 
     if (testChanged && scrollables != m_lastScrollableBackLayers) {
 	m_lastScrollableBackLayers = scrollables;
 	changed = true;

 
 View::LayerList
 View::getNonScrollableFrontLayers(bool testChanged, bool &changed) const
 {
     changed = false;
-    LayerList scrollables = getScrollableBackLayers(testChanged, changed);
     LayerList nonScrollables;
 
     // Everything in front of the first non-scrollable from the back
     // should also be considered non-scrollable