X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=pv%2Fdata%2Fdecoderstack.cpp;h=b252877b7098940c80ca7f82ba91b4466aa3246a;hb=46bf6027eacaec6d922be0e72e8403a10bbc25be;hp=87ac32613f954e8fdbaa4333513e4b3211266dbd;hpb=f70d8673a56471c7e4b22159b99684da7d6c4be1;p=pulseview.git

diff --git a/pv/data/decoderstack.cpp b/pv/data/decoderstack.cpp
index 87ac3261..b252877b 100644
--- a/pv/data/decoderstack.cpp
+++ b/pv/data/decoderstack.cpp
@@ -193,6 +193,14 @@ void DecoderStack::begin_decode()
 
 	clear();
 
+	// Check that all decoders have the required probes
+	BOOST_FOREACH(const shared_ptr<decode::Decoder> &dec, _stack)
+		if (!dec->have_required_probes()) {
+			_error_message = tr("One or more required probes "
+				"have not been specified");
+			return;
+		}
+
 	// Add classes
 	BOOST_FOREACH (const shared_ptr<decode::Decoder> &dec, _stack)
 	{
@@ -249,8 +257,7 @@ void DecoderStack::begin_decode()
 	if (_samplerate == 0.0)
 		_samplerate = 1.0;
 
-	_decode_thread = boost::thread(&DecoderStack::decode_proc, this,
-		data);
+	_decode_thread = boost::thread(&DecoderStack::decode_proc, this);
 }
 
 uint64_t DecoderStack::get_max_sample_count() const
@@ -309,9 +316,10 @@ void DecoderStack::decode_data(
 		}
 	}
 
+	new_decode_data();
 }
 
-void DecoderStack::decode_proc(shared_ptr<data::Logic> data)
+void DecoderStack::decode_proc()
 {
 	optional<int64_t> sample_count;
 	srd_session *session;
@@ -320,11 +328,6 @@ void DecoderStack::decode_proc(shared_ptr<data::Logic> data)
 	assert(data);
 	assert(_snapshot);
 
-	// Check that all decoders have the required probes
-	BOOST_FOREACH(const shared_ptr<decode::Decoder> &dec, _stack)
-		if (!dec->have_required_probes())
-			return;
-
 	// Create the session
 	srd_session_new(&session);
 	assert(session);
@@ -413,8 +416,6 @@ void DecoderStack::annotation_callback(srd_proto_data *pdata, void *decoder)
 
 	// Add the annotation
 	(*row_iter).second.push_annotation(a);
-
-	d->new_decode_data();
 }
 
 void DecoderStack::on_new_frame()
@@ -426,7 +427,8 @@ void DecoderStack::on_data_received()
 {
 	{
 		unique_lock<mutex> lock(_input_mutex);
-		_sample_count = _snapshot->get_sample_count();
+		if (_snapshot)
+			_sample_count = _snapshot->get_sample_count();
 	}
 	_input_cond.notify_one();
 }
@@ -435,7 +437,8 @@ void DecoderStack::on_frame_ended()
 {
 	{
 		unique_lock<mutex> lock(_input_mutex);
-		_frame_complete = true;
+		if (_snapshot)
+			_frame_complete = true;
 	}
 	_input_cond.notify_one();
 }