X-Git-Url: http://git.code-monkey.de/?a=blobdiff_plain;f=pv%2Fprop%2Fbool.cpp;h=0cd1604612661730348acd099580f7ac02947725;hb=97378470ded88af84edaa0f1063d10d834475665;hp=9df12a0ff48208f734c78d38314d3a5bcc28acd9;hpb=eb8269e3b5eebdd77e6a82d42bcfdfbc3f7613a9;p=pulseview.git

diff --git a/pv/prop/bool.cpp b/pv/prop/bool.cpp
index 9df12a0..0cd1604 100644
--- a/pv/prop/bool.cpp
+++ b/pv/prop/bool.cpp
@@ -20,14 +20,17 @@
 #include <cassert>
 
 #include <QCheckBox>
+#include <QDebug>
+
+#include <libsigrokcxx/libsigrokcxx.hpp>
 
 #include "bool.hpp"
 
 namespace pv {
 namespace prop {
 
-Bool::Bool(QString name, Getter getter, Setter setter) :
-	Property(name, getter, setter),
+Bool::Bool(QString name, QString desc, Getter getter, Setter setter) :
+	Property(name, desc, getter, setter),
 	check_box_(nullptr)
 {
 }
@@ -40,15 +43,19 @@ QWidget* Bool::get_widget(QWidget *parent, bool auto_commit)
 	if (!getter_)
 		return nullptr;
 
-	Glib::VariantBase variant = getter_();
-	if (!variant.gobj())
+	try {
+		Glib::VariantBase variant = getter_();
+		if (!variant.gobj())
+			return nullptr;
+	} catch (const sigrok::Error &e) {
+		qWarning() << tr("Querying config key %1 resulted in %2").arg(name_, e.what());
 		return nullptr;
-
-	bool value = Glib::VariantBase::cast_dynamic<Glib::Variant<bool>>(
-		variant).get();
+	}
 
 	check_box_ = new QCheckBox(name(), parent);
-	check_box_->setCheckState(value ? Qt::Checked : Qt::Unchecked);
+	check_box_->setToolTip(desc());
+
+	update_widget();
 
 	if (auto_commit)
 		connect(check_box_, SIGNAL(stateChanged(int)),
@@ -62,6 +69,27 @@ bool Bool::labeled_widget() const
 	return true;
 }
 
+void Bool::update_widget()
+{
+	if (!check_box_)
+		return;
+
+	Glib::VariantBase variant;
+
+	try {
+		variant = getter_();
+	} catch (const sigrok::Error &e) {
+		qWarning() << tr("Querying config key %1 resulted in %2").arg(name_, e.what());
+		return;
+	}
+
+	assert(variant.gobj());
+	bool value = Glib::VariantBase::cast_dynamic<Glib::Variant<bool>>(
+		variant).get();
+
+	check_box_->setCheckState(value ? Qt::Checked : Qt::Unchecked);
+}
+
 void Bool::commit()
 {
 	assert(setter_);
@@ -69,8 +97,7 @@ void Bool::commit()
 	if (!check_box_)
 		return;
 
-	setter_(Glib::Variant<bool>::create(
-		check_box_->checkState() == Qt::Checked));
+	setter_(Glib::Variant<bool>::create(check_box_->checkState() == Qt::Checked));
 }
 
 void Bool::on_state_changed(int)
@@ -78,5 +105,5 @@ void Bool::on_state_changed(int)
 	commit();
 }
 
-} // prop
-} // pv
+}  // namespace prop
+}  // namespace pv