X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=pv%2Fview%2Fview.cpp;h=615802239e2486d1a37d44705f0c98c98265ba80;hb=f0c0b38f812bfbb2bb1978d40e2ce93f1da3f89d;hp=2cb3f3ed9b6f3dd1f2591b94911b150004f4ba94;hpb=1ef49ddd04f44a4369403fbab19ed235483e2975;p=pulseview.git

diff --git a/pv/view/view.cpp b/pv/view/view.cpp
index 2cb3f3ed..61580223 100644
--- a/pv/view/view.cpp
+++ b/pv/view/view.cpp
@@ -22,9 +22,10 @@
 #include <libsigrokdecode/libsigrokdecode.h>
 #endif
 
-#include <assert.h>
-#include <limits.h>
-#include <math.h>
+#include <cassert>
+#include <climits>
+#include <cmath>
+#include <mutex>
 
 #include <QEvent>
 #include <QMouseEvent>
@@ -42,10 +43,13 @@
 #include "pv/data/logic.h"
 #include "pv/data/logicsnapshot.h"
 
+using boost::shared_lock;
+using boost::shared_mutex;
 using pv::data::SignalData;
 using std::back_inserter;
 using std::deque;
 using std::list;
+using std::lock_guard;
 using std::max;
 using std::make_pair;
 using std::min;
@@ -65,7 +69,6 @@ const int View::MaxScrollValue = INT_MAX / 2;
 
 const int View::SignalHeight = 30;
 const int View::SignalMargin = 10;
-const int View::SignalSnapGridSize = 10;
 
 const QColor View::CursorAreaColour(220, 231, 243);
 
@@ -149,6 +152,16 @@ const SigSession& View::session() const
 	return _session;
 }
 
+View* View::view()
+{
+	return this;
+}
+
+const View* View::view() const
+{
+	return this;
+}
+
 Viewport* View::viewport()
 {
 	return _viewport;
@@ -169,7 +182,7 @@ double View::offset() const
 	return _offset;
 }
 
-int View::v_offset() const
+int View::owner_v_offset() const
 {
 	return -_v_offset;
 }
@@ -204,9 +217,6 @@ void View::zoom_one_to_one()
 {
 	using pv::data::SignalData;
 
-	const vector< shared_ptr<Signal> > sigs(
-		session().get_signals());
-
 	// Make a set of all the visible data objects
 	set< shared_ptr<SignalData> > visible_data = get_visible_data();
 	if (visible_data.empty())
@@ -241,52 +251,10 @@ void View::set_scale_offset(double scale, double offset)
 	scale_offset_changed();
 }
 
-vector< shared_ptr<Trace> > View::get_traces() const
-{
-	vector< shared_ptr<Trace> > traces;
-
-	const vector< shared_ptr<Signal> > sigs(
-		session().get_signals());
-	copy(sigs.begin(), sigs.end(), back_inserter(traces));
-
-#ifdef ENABLE_DECODE
-	const vector< shared_ptr<DecodeTrace> > decode_sigs(
-		session().get_decode_signals());
-	copy(decode_sigs.begin(), decode_sigs.end(), back_inserter(traces));
-#endif
-
-	stable_sort(traces.begin(), traces.end(),
-		[](const shared_ptr<Trace> &a, const shared_ptr<Trace> &b) {
-			return a->v_offset() < b->v_offset(); });
-
-	return traces;
-}
-
-list<weak_ptr<SelectableItem> > View::selected_items() const
-{
-	list<weak_ptr<SelectableItem> > items;
-
-	// List the selected signals
-	const vector< shared_ptr<Trace> > traces(get_traces());
-	for (shared_ptr<Trace> t : traces) {
-		assert(t);
-		if (t->selected())
-			items.push_back(t);
-	}
-
-	// List the selected cursors
-	if (_cursors.first()->selected())
-		items.push_back(_cursors.first());
-	if (_cursors.second()->selected())
-		items.push_back(_cursors.second());
-
-	return items;
-}
-
 set< shared_ptr<SignalData> > View::get_visible_data() const
 {
-	const vector< shared_ptr<Signal> > sigs(
-		session().get_signals());
+	shared_lock<shared_mutex> lock(session().signals_mutex());
+	const vector< shared_ptr<Signal> > &sigs(session().signals());
 
 	// Make a set of all the visible data objects
 	set< shared_ptr<SignalData> > visible_data;
@@ -357,15 +325,13 @@ const QPoint& View::hover_point() const
 
 void View::normalize_layout()
 {
-	const vector< shared_ptr<Trace> > traces(get_traces());
-
 	int v_min = INT_MAX;
-	for (const shared_ptr<Trace> t : traces)
-		v_min = min(t->v_offset(), v_min);
+	for (const shared_ptr<RowItem> r : *this)
+		v_min = min(r->v_offset(), v_min);
 
 	const int delta = -min(v_min, 0);
-	for (shared_ptr<Trace> t : traces)
-		t->set_v_offset(t->v_offset() + delta);
+	for (shared_ptr<RowItem> r : *this)
+		r->set_v_offset(r->v_offset() + delta);
 
 	verticalScrollBar()->setSliderPosition(_v_offset + delta);
 	v_scroll_value_changed(verticalScrollBar()->sliderPosition());
@@ -441,6 +407,19 @@ void View::update_layout()
 	update_scroll();
 }
 
+void View::paint_label(QPainter &p, int right, bool hover)
+{
+	(void)p;
+	(void)right;
+	(void)hover;
+}
+
+QRectF View::label_rect(int right)
+{
+	(void)right;
+	return QRectF();
+}
+
 bool View::eventFilter(QObject *object, QEvent *event)
 {
 	const QEvent::Type type = event->type();
@@ -518,16 +497,34 @@ void View::v_scroll_value_changed(int value)
 
 void View::signals_changed()
 {
+	// Populate the traces
+	clear_child_items();
+
+	shared_lock<shared_mutex> lock(session().signals_mutex());
+	const vector< shared_ptr<Signal> > &sigs(session().signals());
+	for (auto s : sigs)
+		add_child_item(s);
+
+#ifdef ENABLE_DECODE
+	const vector< shared_ptr<DecodeTrace> > decode_sigs(
+		session().get_decode_signals());
+	for (auto s : decode_sigs)
+		add_child_item(s);
+#endif
+
+	// Create the initial layout
 	int offset = SignalMargin + SignalHeight;
-	const vector< shared_ptr<Trace> > traces(get_traces());
-	for (shared_ptr<Trace> t : traces) {
-		t->set_view(this);
-		t->set_v_offset(offset);
+	for (shared_ptr<RowItem> r : *this) {
+		r->set_v_offset(offset);
 		offset += SignalHeight + 2 * SignalMargin;
 	}
 
 	update_layout();
 	normalize_layout();
+
+	// Update the child widgets
+	_header->signals_updated();
+	_viewport->signals_updated();
 }
 
 void View::data_updated()
@@ -558,9 +555,8 @@ void View::on_geometry_updated()
 
 void View::on_hover_point_changed()
 {
-	const vector< shared_ptr<Trace> > traces(get_traces());
-	for (shared_ptr<Trace> t : traces)
-		t->hover_point_changed();
+	for (shared_ptr<RowItem> r : *this)
+		r->hover_point_changed();
 }
 
 } // namespace view