From fe7d6661d5f520279e9d54067ad096fa4a6424c5 Mon Sep 17 00:00:00 2001
From: Soeren Apel <soeren@apelpie.net>
Date: Tue, 14 Mar 2017 22:54:20 +0100
Subject: [PATCH] SignalBase: Allow checking for logic data directly

If we always return a valid pointer, we need to look
at the channel type to figure out if this data is
supposed to be valid or not. This isn't useful.
---
 pv/data/signalbase.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pv/data/signalbase.cpp b/pv/data/signalbase.cpp
index ccdc1da7..e29bbe87 100644
--- a/pv/data/signalbase.cpp
+++ b/pv/data/signalbase.cpp
@@ -122,7 +122,7 @@ shared_ptr<data::Analog> SignalBase::analog_data() const
 	if (channel_type_ == AnalogChannel)
 		return dynamic_pointer_cast<data::Analog>(data_);
 	else
-		return shared_ptr<data::Analog>();
+		return nullptr;
 }
 
 shared_ptr<data::Logic> SignalBase::logic_data() const
@@ -130,7 +130,7 @@ shared_ptr<data::Logic> SignalBase::logic_data() const
 	if (channel_type_ == LogicChannel)
 		return dynamic_pointer_cast<data::Logic>(data_);
 	else
-		return shared_ptr<data::Logic>();
+		return nullptr;
 }
 
 #ifdef ENABLE_DECODE
-- 
2.30.2