X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=pv%2Fview%2Fview.cpp;h=615802239e2486d1a37d44705f0c98c98265ba80;hb=ee5f2a5445a6cdda7e764ab5fb1defe7fd1bbb9c;hp=079d8d89b38ffaec8e42b1c6a7f173e56a46bb30;hpb=c3a740dd5d095eb1cdf42e00df4d5a5c480ac5b3;p=pulseview.git

diff --git a/pv/view/view.cpp b/pv/view/view.cpp
index 079d8d89..61580223 100644
--- a/pv/view/view.cpp
+++ b/pv/view/view.cpp
@@ -43,6 +43,8 @@
 #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;
@@ -51,7 +53,6 @@ using std::lock_guard;
 using std::max;
 using std::make_pair;
 using std::min;
-using std::mutex;
 using std::pair;
 using std::set;
 using std::shared_ptr;
@@ -68,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);
 
@@ -251,29 +251,9 @@ void View::set_scale_offset(double scale, double offset)
 	scale_offset_changed();
 }
 
-list<weak_ptr<SelectableItem> > View::selected_items() const
-{
-	list<weak_ptr<SelectableItem> > items;
-
-	// List the selected signals
-	const vector< shared_ptr<RowItem> > row_items(child_items());
-	for (shared_ptr<RowItem> r : row_items) {
-		if (r && r->selected())
-			items.push_back(r);
-	}
-
-	// 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
 {
-	lock_guard<mutex> lock(session().signals_mutex());
+	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
@@ -345,14 +325,12 @@ const QPoint& View::hover_point() const
 
 void View::normalize_layout()
 {
-	const vector< shared_ptr<RowItem> > row_items(child_items());
-
 	int v_min = INT_MAX;
-	for (const shared_ptr<RowItem> r : row_items)
+	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<RowItem> r : row_items)
+	for (shared_ptr<RowItem> r : *this)
 		r->set_v_offset(r->v_offset() + delta);
 
 	verticalScrollBar()->setSliderPosition(_v_offset + delta);
@@ -522,7 +500,7 @@ void View::signals_changed()
 	// Populate the traces
 	clear_child_items();
 
-	lock_guard<mutex> lock(session().signals_mutex());
+	shared_lock<shared_mutex> lock(session().signals_mutex());
 	const vector< shared_ptr<Signal> > &sigs(session().signals());
 	for (auto s : sigs)
 		add_child_item(s);
@@ -536,13 +514,17 @@ void View::signals_changed()
 
 	// Create the initial layout
 	int offset = SignalMargin + SignalHeight;
-	for (shared_ptr<RowItem> r : child_items()) {
+	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()
@@ -573,8 +555,7 @@ void View::on_geometry_updated()
 
 void View::on_hover_point_changed()
 {
-	const vector< shared_ptr<RowItem> > row_items(child_items());
-	for (shared_ptr<RowItem> r : row_items)
+	for (shared_ptr<RowItem> r : *this)
 		r->hover_point_changed();
 }