comparison layer/Colour3DPlotLayer.cpp @ 1123:343887ac6766 spectrogram-minor-refactor

Some tidying

                                      QPainter &paint, QRect rect) const
 {
     Colour3DPlotRenderer *renderer = getRenderer(v);
 
     Colour3DPlotRenderer::RenderResult result;
+    MagnitudeRange magRange;
+    int viewId = v->getId();
+
+    if (!renderer->geometryChanged(v)) {
+        magRange = m_viewMags[viewId];
+    }
     
     if (m_synchronous) {
 
         result = renderer->render(v, paint, rect);
 
     } else {
 
         result = renderer->renderTimeConstrained(v, paint, rect);
-
-        //!!! + mag range
 
         QRect uncached = renderer->getLargestUncachedRect(v);
         if (uncached.width() > 0) {
-            cerr << "updating rect at " << uncached.x() << " width "
-                 << uncached.width() << endl;
             v->updatePaintRect(uncached);
         }
     }
     
-    //!!! at the mo this measures the range of the whole thing, not
-    //!!! just the view - need to reset it when view extents change
-
-    m_viewMags[v->getId()].sample(result.range);
+    magRange.sample(result.range);
+
+    if (magRange.isSet()) {
+        if (!(m_viewMags[viewId] == magRange)) {
+            m_viewMags[viewId] = magRange;
+    //!!! now need to do the normalise-visible thing
+        }
+    }
     
     cerr << "mag range in this view: "
          << m_viewMags[v->getId()].getMin()
          << " -> "
          << m_viewMags[v->getId()].getMax()