Enum: Use an iterator instead of iterating by indexes

[pulseview.git] / pv / prop / bool.cpp

diff --git a/pv/prop/bool.cpp b/pv/prop/bool.cpp
index 179208a5e1c927b74de3469b3aca4a8d3634d453..e0e052fc6b2b377c3d580192beda10793d331884 100644 (file)
--- a/pv/prop/bool.cpp
+++ b/pv/prop/bool.cpp
@@ -24,9 +24,6 @@
 
 #include "bool.h"
 
-using namespace std;
-using namespace boost;
-
 namespace pv {
 namespace prop {
 
@@ -45,15 +42,14 @@ QWidget* Bool::get_widget(QWidget *parent, bool auto_commit)
        if (_check_box)
                return _check_box;
 
-       _check_box = new QCheckBox(name(), parent);
-
        GVariant *const value = _getter ? _getter() : NULL;
+       if (!value)
+               return NULL;
 
-       if (value) {
-               _check_box->setCheckState(g_variant_get_boolean(value) ?
-                       Qt::Checked : Qt::Unchecked);
-               g_variant_unref(value);
-       }
+       _check_box = new QCheckBox(name(), parent);
+       _check_box->setCheckState(g_variant_get_boolean(value) ?
+               Qt::Checked : Qt::Unchecked);
+       g_variant_unref(value);
 
        if (auto_commit)
                connect(_check_box, SIGNAL(stateChanged(int)),