X-Git-Url: http://sigrok.org/gitweb/?a=blobdiff_plain;f=pv%2Fprop%2Fint.cpp;fp=pv%2Fprop%2Fint.cpp;h=3f29951b3d997b7b5b8a299fd34c3a0bf0d6db40;hb=3782d8609d2f4bd66855dc4f72c0f74d9bc11c23;hp=2930a3ea1e8c96b5d0091e6c2b640050ecb5d09f;hpb=4deee4de1660cd5cdc100f3130a3e68af97212d9;p=pulseview.git

diff --git a/pv/prop/int.cpp b/pv/prop/int.cpp
index 2930a3ea..3f29951b 100644
--- a/pv/prop/int.cpp
+++ b/pv/prop/int.cpp
@@ -20,8 +20,11 @@
 #include <cassert>
 #include <cstdint>
 
+#include <QDebug>
 #include <QSpinBox>
 
+#include <libsigrokcxx/libsigrokcxx.hpp>
+
 #include "int.hpp"
 
 using boost::optional;
@@ -47,7 +50,7 @@ Int::Int(QString name,
 
 QWidget* Int::get_widget(QWidget *parent, bool auto_commit)
 {
-	int64_t int_val = 0, range_min = 0;
+	int64_t range_min = 0;
 	uint64_t range_max = 0;
 
 	if (spin_box_)
@@ -56,7 +59,12 @@ QWidget* Int::get_widget(QWidget *parent, bool auto_commit)
 	if (!getter_)
 		return nullptr;
 
-	value_ = getter_();
+	try {
+		value_ = getter_();
+	} catch (const sigrok::Error &e) {
+		qWarning() << tr("Querying config key %1 resulted in %2").arg(name_, e.what());
+		return nullptr;
+	}
 
 	GVariant *value = value_.gobj();
 	if (!value)
@@ -69,25 +77,18 @@ QWidget* Int::get_widget(QWidget *parent, bool auto_commit)
 	assert(type);
 
 	if (g_variant_type_equal(type, G_VARIANT_TYPE_BYTE)) {
-		int_val = g_variant_get_byte(value);
 		range_min = 0, range_max = UINT8_MAX;
 	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT16)) {
-		int_val = g_variant_get_int16(value);
 		range_min = INT16_MIN, range_max = INT16_MAX;
 	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT16)) {
-		int_val = g_variant_get_uint16(value);
 		range_min = 0, range_max = UINT16_MAX;
 	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT32)) {
-		int_val = g_variant_get_int32(value);
 		range_min = INT32_MIN, range_max = INT32_MAX;
 	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT32)) {
-		int_val = g_variant_get_uint32(value);
 		range_min = 0, range_max = UINT32_MAX;
 	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT64)) {
-		int_val = g_variant_get_int64(value);
 		range_min = INT64_MIN, range_max = INT64_MAX;
 	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT64)) {
-		int_val = g_variant_get_uint64(value);
 		range_min = 0, range_max = UINT64_MAX;
 	} else {
 		// Unexpected value type.
@@ -107,7 +108,7 @@ QWidget* Int::get_widget(QWidget *parent, bool auto_commit)
 	else
 		spin_box_->setRange((int)range_min, (int)range_max);
 
-	spin_box_->setValue((int)int_val);
+	update_widget();
 
 	if (auto_commit)
 		connect(spin_box_, SIGNAL(valueChanged(int)),
@@ -116,6 +117,48 @@ QWidget* Int::get_widget(QWidget *parent, bool auto_commit)
 	return spin_box_;
 }
 
+void Int::update_widget()
+{
+	if (!spin_box_)
+		return;
+
+	try {
+		value_ = getter_();
+	} catch (const sigrok::Error &e) {
+		qWarning() << tr("Querying config key %1 resulted in %2").arg(name_, e.what());
+		return;
+	}
+
+	GVariant *value = value_.gobj();
+	assert(value);
+
+	const GVariantType *const type = g_variant_get_type(value);
+	assert(type);
+
+	int64_t int_val = 0;
+
+	if (g_variant_type_equal(type, G_VARIANT_TYPE_BYTE)) {
+		int_val = g_variant_get_byte(value);
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT16)) {
+		int_val = g_variant_get_int16(value);
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT16)) {
+		int_val = g_variant_get_uint16(value);
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT32)) {
+		int_val = g_variant_get_int32(value);
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT32)) {
+		int_val = g_variant_get_uint32(value);
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT64)) {
+		int_val = g_variant_get_int64(value);
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT64)) {
+		int_val = g_variant_get_uint64(value);
+	} else {
+		// Unexpected value type.
+		assert(false);
+	}
+
+	spin_box_->setValue((int)int_val);
+}
+
 void Int::commit()
 {
 	assert(setter_);