global: Do not assert >= 0 for unsigned values

[pulseview.git] / pv / view / ruler.cpp

diff --git a/pv/view/ruler.cpp b/pv/view/ruler.cpp
index 83863e61ea203e3a3efb08f772425a8510da029d..a442901c3e9c76645e44da9a5a9bd5dd18ca8df9 100644 (file)
--- a/pv/view/ruler.cpp
+++ b/pv/view/ruler.cpp
@@ -19,6 +19,8 @@
  */
 
 #include "ruler.h"
+
+#include "cursor.h"
 #include "view.h"
 #include "viewport.h"
 
@@ -47,7 +49,8 @@ const int Ruler::HoverArrowSize = 5;
 
 Ruler::Ruler(View &parent) :
        QWidget(&parent),
-       _view(parent)
+       _view(parent),
+       _grabbed_marker(NULL)
 {
        setMouseTracking(true);
 
@@ -73,10 +76,9 @@ void Ruler::paintEvent(QPaintEvent *event)
        do
        {
                tick_period = order_decimal * ScaleUnits[unit++];
-       } while(tick_period < min_period && unit < countof(ScaleUnits));
+       } while (tick_period < min_period && unit < countof(ScaleUnits));
 
        const unsigned int prefix = (order - FirstSIPrefixPower) / 3;
-       assert(prefix >= 0);
        assert(prefix < countof(SIPrefixes));
 
        const double multiplier = pow(10.0, - prefix * 3 - FirstSIPrefixPower);
@@ -96,15 +98,15 @@ void Ruler::paintEvent(QPaintEvent *event)
 
        int division = (int)round(first_minor_division -
                first_major_division * MinorTickSubdivision);
-       while(1)
+       while (1)
        {
                const double t = t0 + division * minor_tick_period;
                const double x = (t - _view.offset()) / _view.scale();
 
-               if(x >= width())
+               if (x >= width())
                        break;
 
-               if(division % MinorTickSubdivision == 0)
+               if (division % MinorTickSubdivision == 0)
                {
                        // Draw a major tick
                        QString s;
@@ -134,9 +136,41 @@ void Ruler::paintEvent(QPaintEvent *event)
        p.end();
 }
 
+void Ruler::mouseMoveEvent(QMouseEvent *e)
+{
+       if (!_grabbed_marker)
+               return;
+
+       _grabbed_marker->set_time(_view.offset() +
+               ((double)e->x() + 0.5) * _view.scale());
+}
+
+void Ruler::mousePressEvent(QMouseEvent *e)
+{
+       if (e->buttons() & Qt::LeftButton) {
+               _grabbed_marker = NULL;
+
+               if (_view.cursors_shown()) {
+                       std::pair<Cursor, Cursor> &cursors =
+                               _view.cursors();
+                       if (cursors.first.get_label_rect(
+                               rect()).contains(e->pos()))
+                               _grabbed_marker = &cursors.first;
+                       else if (cursors.second.get_label_rect(
+                               rect()).contains(e->pos()))
+                               _grabbed_marker = &cursors.second;
+               }
+       }
+}
+
+void Ruler::mouseReleaseEvent(QMouseEvent *)
+{
+       _grabbed_marker = NULL;
+}
+
 void Ruler::draw_cursors(QPainter &p)
 {
-       if(!_view.cursors_shown())
+       if (!_view.cursors_shown())
                return;
 
        const QRect r = rect();
@@ -148,7 +182,8 @@ void Ruler::draw_cursors(QPainter &p)
 void Ruler::draw_hover_mark(QPainter &p)
 {
        const int x = _view.hover_point().x();
-       if(x == -1)
+
+       if (x == -1 || _grabbed_marker)
                return;
 
        p.setPen(QPen(Qt::NoPen));