comparison layer/Colour3DPlotRenderer.cpp @ 1494:78d9282519b0

Ensure rightCropFrame doesn't end up rounded to the left of the desired end pixel

HG Enter commit message.  Lines beginning with 'HG:' are removed.

     QRect pr = rect & m_cache.getValidArea();
     paint.drawImage(pr.x(), pr.y(), m_cache.getImage(),
                     pr.x(), pr.y(), pr.width(), pr.height());
 
     if (!timeConstrained && (pr != rect)) {
+        QRect cva = m_cache.getValidArea();
         SVCERR << "WARNING: failed to render entire requested rect "
                << "even when not time-constrained: wanted "
                << rect.x() << "," << rect.y() << " "
                << rect.width() << "x" << rect.height() << ", got "
                << pr.x() << "," << pr.y() << " "
                << pr.width() << "x" << pr.height()
                << ", after request of width " << (x1 - x0)
+               << endl
+               << "(cache valid area is "
+               << cva.x() << "," << cva.y() << " "
+               << cva.width() << "x" << cva.height() << ")"
                << endl;
     }
 
     MagnitudeRange range = m_magCache.getRange(reqx0, reqx1 - reqx0);
 

     }
     
     for (int x = x0 + repaintWidth; ; ++x) {
         sv_frame_t f = v->getFrameForX(x);
         if ((f / binResolution) * binResolution == f) {
+            if (v->getXForFrame(f) < x0 + repaintWidth) {
+                continue;
+            }
             if (rightCropFrame == -1) rightCropFrame = f;
             else if (x > x0 + repaintWidth + 2) {
                 rightBoundaryFrame = f;
                 break;
             }

     int scaledLeft = v->getXForFrame(leftBoundaryFrame);
     int scaledRight = v->getXForFrame(rightBoundaryFrame);
 
 #ifdef DEBUG_COLOUR_PLOT_REPAINT
     SVDEBUG << "scaling draw buffer from width " << m_drawBuffer.width()
-            << " to " << (scaledRight - scaledLeft) << " (nb drawBufferWidth = "
-            << drawBufferWidth << ")" << endl;
+            << " to " << (scaledRight - scaledLeft)
+            << " (nb drawBufferWidth = "
+            << drawBufferWidth << ", attainedWidth = "
+            << attainedWidth << ")" << endl;
 #endif
 
     QImage scaled = scaleDrawBufferImage
         (m_drawBuffer, scaledRight - scaledLeft, h);
             

     
     int sourceLeft = targetLeft - scaledLeft;
     if (sourceLeft < 0) {
         sourceLeft = 0;
     }
-    
-#ifdef DEBUG_COLOUR_PLOT_REPAINT
-    SVDEBUG << "repaintWidth = " << repaintWidth
+
+#ifdef DEBUG_COLOUR_PLOT_REPAINT
+    SVDEBUG << "leftBoundaryFrame = " << leftBoundaryFrame
+            << ", leftCropFrame = " << leftCropFrame
+            << ", scaledLeft = " << scaledLeft
+            << ", scaledLeftCrop = " << scaledLeftCrop
+            << endl;
+    SVDEBUG << "rightBoundaryFrame = " << rightBoundaryFrame
+            << ", rightCropFrame = " << rightCropFrame
+            << ", scaledRight = " << scaledRight
+            << ", scaledRightCrop = " << scaledRightCrop
+            << endl;
+#endif
+    
+#ifdef DEBUG_COLOUR_PLOT_REPAINT
+    SVDEBUG << "x0 = " << x0
+            << ", repaintWidth = " << repaintWidth
+            << ", targetLeft = " << targetLeft 
             << ", targetWidth = " << targetWidth << endl;
 #endif
     
     if (targetWidth > 0) {
         // we are copying from an image that has already been scaled,