Device binding: Notify user when a config option is ignored

[pulseview.git] / pv / binding / device.cpp

diff --git a/pv/binding/device.cpp b/pv/binding/device.cpp
index 20082db2f7a949666151fc01e006d7408f7fb3ab..96217051f8433819cb9e0b575d8be28cc5a7e242 100644 (file)
--- a/pv/binding/device.cpp
+++ b/pv/binding/device.cpp
@@ -32,7 +32,6 @@
 using boost::optional;
 
 using std::function;
-using std::make_pair;
 using std::pair;
 using std::set;
 using std::shared_ptr;
@@ -62,19 +61,22 @@ Device::Device(shared_ptr<sigrok::Configurable> configurable) :
 
                auto capabilities = configurable->config_capabilities(key);
 
-               if (!capabilities.count(Capability::GET) ||
-                       !capabilities.count(Capability::SET))
-                       continue;
-
                string name_str;
                try {
                        name_str = key->description();
-               } catch (Error e) {
+               } catch (Error& e) {
                        name_str = key->name();
                }
 
                const QString name = QString::fromStdString(name_str);
 
+               if (!capabilities.count(Capability::GET) ||
+                       !capabilities.count(Capability::SET)) {
+                       qDebug() << QString(tr("Note for device developers: Ignoring device configuration capability '%1' " \
+                               "as it is missing GET and/or SET")).arg(name);
+                       continue;
+               }
+
                const Property::Getter get = [&, key]() {
                        return configurable_->config_get(key); };
                const Property::Setter set = [&, key](Glib::VariantBase value) {
@@ -92,12 +94,20 @@ Device::Device(shared_ptr<sigrok::Configurable> configurable) :
                        bind_int(name, "", "%", pair<int64_t, int64_t>(0, 100), get, set);
                        break;
 
+               case SR_CONF_LIMIT_FRAMES:
+                       // Value 0 means that there is no limit
+                       bind_int(name, "", "", pair<int64_t, int64_t>(0, 1000000), get, set,
+                               tr("No Limit"));
+                       break;
+
                case SR_CONF_PATTERN_MODE:
                case SR_CONF_BUFFERSIZE:
                case SR_CONF_TRIGGER_SOURCE:
                case SR_CONF_TRIGGER_SLOPE:
                case SR_CONF_COUPLING:
                case SR_CONF_CLOCK_EDGE:
+               case SR_CONF_DATA_SOURCE:
+               case SR_CONF_EXTERNAL_CLOCK_SOURCE:
                        bind_enum(name, "", key, capabilities, get, set);
                        break;
 
@@ -105,6 +115,7 @@ Device::Device(shared_ptr<sigrok::Configurable> configurable) :
                case SR_CONF_EXTERNAL_CLOCK:
                case SR_CONF_RLE:
                case SR_CONF_POWER_OFF:
+               case SR_CONF_AVERAGING:
                        bind_bool(name, "", get, set);
                        break;
 
@@ -127,6 +138,13 @@ Device::Device(shared_ptr<sigrok::Configurable> configurable) :
                                bind_int(name, "", "", pair<int64_t, int64_t>(1, 500), get, set);
                        break;
 
+               case SR_CONF_AVG_SAMPLES:
+                       if (capabilities.count(Capability::LIST))
+                               bind_enum(name, "", key, capabilities, get, set, print_averages);
+                       else
+                               bind_int(name, "", "", pair<int64_t, int64_t>(0, INT32_MAX), get, set);
+                       break;
+
                default:
                        break;
                }
@@ -157,7 +175,7 @@ void Device::bind_enum(const QString &name, const QString &desc,
 
                vector< pair<Glib::VariantBase, QString> > values;
                while ((iter.next_value(gvar)))
-                       values.push_back(make_pair(gvar, printer(gvar)));
+                       values.emplace_back(gvar, printer(gvar));
 
                properties_.push_back(shared_ptr<Property>(new Enum(name, desc, values,
                        getter, setter)));
@@ -169,13 +187,13 @@ void Device::bind_enum(const QString &name, const QString &desc,
 }
 
 void Device::bind_int(const QString &name, const QString &desc, QString suffix,
-       optional< pair<int64_t, int64_t> > range,
-       Property::Getter getter, Property::Setter setter)
+       optional< pair<int64_t, int64_t> > range, Property::Getter getter,
+       Property::Setter setter, QString special_value_text)
 {
        assert(configurable_);
 
        properties_.push_back(shared_ptr<Property>(new Int(name, desc, suffix,
-               range, getter, setter)));
+               range, getter, setter, special_value_text)));
 }
 
 QString Device::print_timebase(Glib::VariantBase gvar)
@@ -206,5 +224,12 @@ QString Device::print_probe_factor(Glib::VariantBase gvar)
        return QString("%1x").arg(factor);
 }
 
+QString Device::print_averages(Glib::VariantBase gvar)
+{
+       uint64_t avg;
+       avg = g_variant_get_uint64(gvar.gobj());
+       return QString("%1").arg(avg);
+}
+
 }  // namespace binding
 }  // namespace pv