comparison layer/Colour3DPlotRenderer.cpp @ 1080:2e5945b87aca spectrogram-minor-refactor

Minor adjustments

     if (!v) {
 	throw std::logic_error("no LayerGeometryProvider provided");
     }
 
     sv_frame_t startFrame = v->getStartFrame();
-    int zoomLevel = v->getZoomLevel();
     
     int x0 = v->getXForViewX(rect.x());
     int x1 = v->getXForViewX(rect.x() + rect.width());
     if (x0 < 0) x0 = 0;
     if (x1 > v->getPaintWidth()) x1 = v->getPaintWidth();

         // sub-regions of our target region in right-to-left order in
         // order to ensure contiguity
         rightToLeft = isLeftOfValidArea;
     }
     
-    int repaintWidth = x1 - x0;
-
-    QRect rendered = renderToCache(x0, repaintWidth, timeConstrained);
+    renderToCache(x0, x1 - x0, rightToLeft, timeConstrained);
 
     QRect pr = rect & m_cache.getValidArea();
     paint.drawImage(pr.x(), pr.y(), m_cache.getImage(),
                     pr.x(), pr.y(), pr.width(), pr.height());
 

     //!!! fft model scaling?
     
     //!!! should we own the Dense3DModelPeakCache here? or should it persist
 }
 
-QRect
-Colour3DPlotRenderer::renderToCache(int x0, int repaintWidth, bool timeConstrained)
+void
+Colour3DPlotRenderer::renderToCache(int x0, int repaintWidth,
+                                    bool rightToLeft, bool timeConstrained)
 {
     // Draw to the draw buffer, and then scale-copy from there.
 
     DenseThreeDimensionalModel *model = m_sources.source;
     if (!model || !model->isOK() || !model->isReady()) {

     
     bool usePeaksCache = false;
     int binsPerPeak = 1;
 
     if (bufferIsBinResolution) {
+
         for (int x = 0; x < drawWidth; ++x) {
             binforx[x] = int(leftBoundaryFrame / binResolution) + x;
         }
+
+        // calculating boundaries later will be too fiddly for partial
+        // paints, and painting should be fast anyway when this is the
+        // case because it means we're well zoomed in
+        timeConstrained = false;
+
     } else {
         for (int x = 0; x < drawWidth; ++x) {
             sv_frame_t f0 = v->getFrameForX(x);
             double s0 = double(f0 - model->getStartFrame()) / binResolution;
             binforx[x] = int(s0 + 0.0001);

                 ColourScale::PhaseColourScale) {
                 usePeaksCache = false;
             }
         }
     }
-
-    int attainedDrawWidth = drawWidth;
-
-    //!!! todo: all
-    
+    /*!!!
+    for (int y = 0; y < h; ++y) {
+        double q0 = 0, q1 = 0;
+        if (!getSmoothedYBinRange(v, h-y-1, q0, q1)) {
+            binfory[y] = -1;
+        } else {
+            binfory[y] = q0;
+        }
+    }
+
+    int attainedWidth = renderDrawBuffer(v,
+                                         repaintWidth,
+                                         h,
+                                         binforx,
+                                         binfory,
+                                         usePeaksCache,
+                                         rightToLeft,
+                                         timeConstrained);
+    */    
 }
  
 void
 Colour3DPlotRenderer::clearDrawBuffer(int w, int h)
 {