From 04e1acc23e835018c3c13d8f29115cb285035151 Mon Sep 17 00:00:00 2001
From: Soeren Apel <soeren@apelpie.net>
Date: Thu, 16 Mar 2017 22:24:17 +0100
Subject: [PATCH] LogicSegment: Remove constructor requiring sigrok::Logic

---
 pv/data/logicsegment.cpp | 11 -----------
 pv/data/logicsegment.hpp |  1 -
 pv/session.cpp           |  7 +++----
 3 files changed, 3 insertions(+), 16 deletions(-)

diff --git a/pv/data/logicsegment.cpp b/pv/data/logicsegment.cpp
index fcf5722..25aa2d0 100644
--- a/pv/data/logicsegment.cpp
+++ b/pv/data/logicsegment.cpp
@@ -47,17 +47,6 @@ const int LogicSegment::MipMapScaleFactor = 1 << MipMapScalePower;
 const float LogicSegment::LogMipMapScaleFactor = logf(MipMapScaleFactor);
 const uint64_t LogicSegment::MipMapDataUnit = 64*1024;	// bytes
 
-LogicSegment::LogicSegment(pv::data::Logic& owner, shared_ptr<sigrok::Logic> data,
-	uint64_t samplerate) :
-	Segment(samplerate, data->unit_size()),
-	owner_(owner),
-	last_append_sample_(0)
-{
-	lock_guard<recursive_mutex> lock(mutex_);
-	memset(mip_map_, 0, sizeof(mip_map_));
-	append_payload(data);
-}
-
 LogicSegment::LogicSegment(pv::data::Logic& owner, unsigned int unit_size,
 	uint64_t samplerate) :
 	Segment(samplerate, unit_size),
diff --git a/pv/data/logicsegment.hpp b/pv/data/logicsegment.hpp
index c797172..bc9c77d 100644
--- a/pv/data/logicsegment.hpp
+++ b/pv/data/logicsegment.hpp
@@ -77,7 +77,6 @@ public:
 	typedef pair<int64_t, bool> EdgePair;
 
 public:
-	LogicSegment(pv::data::Logic& owner, shared_ptr<sigrok::Logic> data, uint64_t samplerate);
 	LogicSegment(pv::data::Logic& owner, unsigned int unit_size, uint64_t samplerate);
 
 	virtual ~LogicSegment();
diff --git a/pv/session.cpp b/pv/session.cpp
index f5f1e49..fb7d637 100644
--- a/pv/session.cpp
+++ b/pv/session.cpp
@@ -962,7 +962,7 @@ void Session::feed_in_logic(shared_ptr<Logic> logic)
 
 		// Create a new data segment
 		cur_logic_segment_ = make_shared<data::LogicSegment>(
-			*logic_data_, logic, cur_samplerate_);
+			*logic_data_, logic->unit_size(), cur_samplerate_);
 		logic_data_->push_segment(cur_logic_segment_);
 
 		// @todo Putting this here means that only listeners querying
@@ -970,11 +970,10 @@ void Session::feed_in_logic(shared_ptr<Logic> logic)
 		// frame_began is DecoderStack, but in future we need to signal
 		// this after both analog and logic sweeps have begun.
 		frame_began();
-	} else {
-		// Append to the existing data segment
-		cur_logic_segment_->append_payload(logic);
 	}
 
+	cur_logic_segment_->append_payload(logic);
+
 	data_received();
 }
 
-- 
2.30.2