X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=pv%2Fviews%2Ftrace%2Fruler.cpp;h=bebf529147da67f5bc4d591255bce144d5d1674b;hb=945238ca07e313568f1df5be1857f5f7db33d700;hp=49cff2f14ca14633f982f266c10c3dfec29f35af;hpb=710c2a1896fbac968c82f2d1257aaabd10a48cc8;p=pulseview.git

diff --git a/pv/views/trace/ruler.cpp b/pv/views/trace/ruler.cpp
index 49cff2f1..bebf5291 100644
--- a/pv/views/trace/ruler.cpp
+++ b/pv/views/trace/ruler.cpp
@@ -19,7 +19,6 @@
 
 #include <extdef.h>
 
-#include <QApplication>
 #include <QFontMetrics>
 #include <QMenu>
 #include <QMouseEvent>
@@ -188,38 +187,50 @@ vector< shared_ptr<ViewItem> > Ruler::items()
 
 void Ruler::item_hover(const shared_ptr<ViewItem> &item, QPoint pos)
 {
-	Q_UNUSED(pos);
+	(void)pos;
+
 	hover_item_ = dynamic_pointer_cast<TimeItem>(item);
 }
 
-shared_ptr<TimeItem> Ruler::get_reference_item()
+shared_ptr<TimeItem> Ruler::get_reference_item() const
 {
+	// Note: time() returns 0 if item returns no valid time
+
 	if (mouse_modifiers_ & Qt::ShiftModifier)
 		return nullptr;
 
-	if (hover_item_ != nullptr)
+	if (hover_item_ && (hover_item_->time() != 0))
 		return hover_item_;
 
-	shared_ptr<TimeItem> found(nullptr);
+	shared_ptr<TimeItem> ref_item;
 	const vector< shared_ptr<TimeItem> > items(view_.time_items());
+
 	for (auto i = items.rbegin(); i != items.rend(); i++) {
 		if ((*i)->enabled() && (*i)->selected()) {
-			if (found == nullptr)
-				found = *i;
-			else
-				return nullptr; // Return null if multiple items are selected
+			if (!ref_item)
+				ref_item = *i;
+			else {
+				// Return nothing if multiple items are selected
+				ref_item.reset();
+				break;
+			}
 		}
 	}
 
-	return found;
+	if (ref_item && (ref_item->time() == 0))
+		ref_item.reset();
+
+	return ref_item;
 }
 
 shared_ptr<ViewItem> Ruler::get_mouse_over_item(const QPoint &pt)
 {
 	const vector< shared_ptr<TimeItem> > items(view_.time_items());
+
 	for (auto i = items.rbegin(); i != items.rend(); i++)
 		if ((*i)->enabled() && (*i)->label_rect(rect()).contains(pt))
 			return *i;
+
 	return nullptr;
 }