comparison layer/WaveformLayer.cpp @ 1372:2c01be836570

Short-circuit drawing a little bit for contiguous ranges

     QPainterPath meanPath;
     QPainterPath clipPath;
     vector<QPointF> individualSamplePoints;
 
     bool firstPoint = true;
+    double prevRangeBottom = 0, prevRangeTop = 0;
     
     for (int x = x0; x <= x1; ++x) {
 
         sv_frame_t f0, f1;
         sv_frame_t i0, i1;

             if (!trivialRange) {
                 // common e.g. in "butterfly" merging mode
                 individualSamplePoints.push_back(QPointF(px, rangeBottom));
             }
         }
+
+        bool contiguous = true;
+        if (rangeTop > prevRangeBottom + 0.5 ||
+            rangeBottom < prevRangeTop - 0.5) {
+            contiguous = false;
+        }
         
-        if (firstPoint) {
-            waveformPath = QPainterPath(QPointF(px, rangeMiddle));
-            firstPoint = false;
+        if (firstPoint || (contiguous && !trivialRange)) {
+            waveformPath.moveTo(QPointF(px, rangeTop));
+            waveformPath.lineTo(QPointF(px, rangeBottom));
+            waveformPath.moveTo(QPointF(px, rangeMiddle));
         } else {
             waveformPath.lineTo(QPointF(px, rangeMiddle));
-        }
-
-        if (!trivialRange) {
-            waveformPath.lineTo(QPointF(px, rangeTop));
-            waveformPath.lineTo(QPointF(px, rangeBottom));
-            waveformPath.lineTo(QPointF(px, rangeMiddle));
-        }
-
+            if (!trivialRange) {
+                waveformPath.lineTo(QPointF(px, rangeTop));
+                waveformPath.lineTo(QPointF(px, rangeBottom));
+                waveformPath.lineTo(QPointF(px, rangeMiddle));
+            }
+        }
+
+        firstPoint = false;
+        prevRangeTop = rangeTop;
+        prevRangeBottom = rangeBottom;
+        
         if (drawMean) {
             meanPath.moveTo(QPointF(px, meanBottom));
             meanPath.lineTo(QPointF(px, meanTop));
         }