From 364d2155b342d3ef8f5f2c22be54bb35d0882b76 Mon Sep 17 00:00:00 2001
From: Joel Holdsworth <joel@airwebreathe.org.uk>
Date: Thu, 15 Oct 2015 21:07:38 -0600
Subject: [PATCH] TraceTreeItemOwner: Make list_by_type return a vector instead
 of a set

---
 pv/view/tracetreeitemowner.hpp | 6 +++---
 pv/view/view.cpp               | 4 +++-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/pv/view/tracetreeitemowner.hpp b/pv/view/tracetreeitemowner.hpp
index 4c0043f6..c1776995 100644
--- a/pv/view/tracetreeitemowner.hpp
+++ b/pv/view/tracetreeitemowner.hpp
@@ -128,12 +128,12 @@ public:
 	 * Creates a list of decendant signals filtered by type.
 	 */
 	template<class T>
-	std::set< std::shared_ptr<T> > list_by_type() {
-		std::set< std::shared_ptr<T> > items;
+	std::vector< std::shared_ptr<T> > list_by_type() {
+		std::vector< std::shared_ptr<T> > items;
 		for (const auto &r : *this) {
 			std::shared_ptr<T> p = std::dynamic_pointer_cast<T>(r);
 			if (p)
-				items.insert(p);
+				items.push_back(p);
 		}
 
 		return items;
diff --git a/pv/view/view.cpp b/pv/view/view.cpp
index 540b90e1..c0b3e251 100644
--- a/pv/view/view.cpp
+++ b/pv/view/view.cpp
@@ -875,7 +875,9 @@ void View::signals_changed()
 
 	// Make a list of traces that are being added, and a list of traces
 	// that are being removed
-	const set<shared_ptr<Trace>> prev_traces = list_by_type<Trace>();
+	const vector<shared_ptr<Trace>> prev_trace_list = list_by_type<Trace>();
+	const set<shared_ptr<Trace>> prev_traces(
+		prev_trace_list.begin(), prev_trace_list.end());
 
 	shared_lock<shared_mutex> lock(session_.signals_mutex());
 	const unordered_set< shared_ptr<Signal> > &sigs(session_.signals());
-- 
2.30.2