comparison layer/RegionLayer.cpp @ 1410:d057dcc827e7 fix-static-analysis

Merge from default branch

 
     int x0 = rect.left() - 40, x1 = rect.right();
     sv_frame_t frame0 = v->getFrameForX(x0);
     sv_frame_t frame1 = v->getFrameForX(x1);
 
-    RegionModel::PointList points(m_model->getPoints(frame0, frame1));
+    sv_frame_t wholeFrame0 = v->getFrameForX(0);
+    sv_frame_t wholeFrame1 = v->getFrameForX(v->getPaintWidth());
+
+    RegionModel::PointList points(m_model->getPoints(wholeFrame0, wholeFrame1));
     if (points.empty()) return;
 
     paint.setPen(getBaseQColor());
 
     QColor brushColour(getBaseQColor());

          i != points.end(); ++i) {
 
         const RegionModel::Point &p(*i);
 
         int x = v->getXForFrame(p.frame);
+        int w = v->getXForFrame(p.frame + p.duration) - x;
         int y = getYForValue(v, p.value);
-        int w = v->getXForFrame(p.frame + p.duration) - x;
         int h = 9;
         int ex = x + w;
 
+        int gap = v->scalePixelSize(2);
+        
         RegionModel::PointList::const_iterator j = i;
         ++j;
 
         if (j != points.end()) {
             const RegionModel::Point &q(*j);

 
             } else {
                 paint.setPen(QPen(getForegroundQColor(v->getView()), 2));
             }
 
-            paint.drawRect(x, -1, ex - x, v->getPaintHeight() + 2);
+            paint.drawRect(x, -1, ex - x, v->getPaintHeight() + gap);
 
         } else {
 
             if (shouldIlluminate &&
                 // "illuminatePoint == p"

                 paint.setPen(v->getForeground());
                 paint.setBrush(v->getForeground());
 
                 QString vlabel = QString("%1%2").arg(p.value).arg(getScaleUnits());
                 PaintAssistant::drawVisibleText(v, paint, 
-                                   x - paint.fontMetrics().width(vlabel) - 2,
+                                   x - paint.fontMetrics().width(vlabel) - gap,
                                    y + paint.fontMetrics().height()/2
                                    - paint.fontMetrics().descent(), 
                                    vlabel, PaintAssistant::OutlinedText);
                 
                 QString hlabel = RealTime::frame2RealTime
                     (p.frame, m_model->getSampleRate()).toText(true).c_str();
                 PaintAssistant::drawVisibleText(v, paint, 
                                    x,
-                                   y - h/2 - paint.fontMetrics().descent() - 2,
+                                   y - h/2 - paint.fontMetrics().descent() - gap,
                                    hlabel, PaintAssistant::OutlinedText);
             }
             
             paint.drawLine(x, y-1, x + w, y-1);
             paint.drawLine(x, y+1, x + w, y+1);

          i != points.end(); ++i) {
 
         const RegionModel::Point &p(*i);
 
         int x = v->getXForFrame(p.frame);
+        int w = v->getXForFrame(p.frame + p.duration) - x;
         int y = getYForValue(v, p.value);
 
+        QString label = p.label;
+        if (label == "") {
+            label = QString("%1%2").arg(p.value).arg(getScaleUnits());
+        }
+        int labelWidth = paint.fontMetrics().width(label);
+
+        int gap = v->scalePixelSize(2);
+
+        if (m_plotStyle == PlotSegmentation) {
+            if ((x + w < x0 && x + labelWidth + gap < x0) || x > x1) {
+                continue;
+            }
+        } else {
+            if (x + w < x0 || x - labelWidth - gap > x1) {
+                continue;
+            }
+        }
+        
         bool illuminated = false;
 
         if (m_plotStyle != PlotSegmentation) {
 
             if (shouldIlluminate &&

                 illuminated = true;
             }
         }
 
         if (!illuminated) {
-            QString label = p.label;
-            if (label == "") {
-                label = QString("%1%2").arg(p.value).arg(getScaleUnits());
-            }
 
             int labelX, labelY;
 
             if (m_plotStyle != PlotSegmentation) {
-                labelX = x - paint.fontMetrics().width(label) - 2;
+                labelX = x - labelWidth - gap;
                 labelY = y + paint.fontMetrics().height()/2 
                     - paint.fontMetrics().descent();
             } else {
                 labelX = x + 5;
                 labelY = v->getTextLabelHeight(this, paint);

                     if (lastLabelY < v->getPaintHeight()/2) {
                         labelY = lastLabelY + fontHeight;
                     }
                 }
                 lastLabelY = labelY;
-                nextLabelMinX = labelX + paint.fontMetrics().width(label);
-            }
-
-            PaintAssistant::drawVisibleText(v, paint, labelX, labelY, label, PaintAssistant::OutlinedText);
+                nextLabelMinX = labelX + labelWidth;
+            }
+
+            PaintAssistant::drawVisibleText(v, paint, labelX, labelY, label,
+                                            PaintAssistant::OutlinedText);
         }
     }
 
     paint.restore();
 }