X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fprop%2Fbinding%2Fdeviceoptions.cpp;h=c995ea697c01e6980adaefd9c0efa0676b36d35d;hp=ea8f4b4218463e1c6e5ab03308ca6358d0066df2;hb=7c65709437184b09753d1bcacc02467b8f6171f1;hpb=945745012eb57cefa1ef457daf48cfffa99f9ec2 diff --git a/pv/prop/binding/deviceoptions.cpp b/pv/prop/binding/deviceoptions.cpp index ea8f4b42..c995ea69 100644 --- a/pv/prop/binding/deviceoptions.cpp +++ b/pv/prop/binding/deviceoptions.cpp @@ -18,70 +18,80 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA */ -#include - #include -#include "deviceoptions.h" +#include + +#include "deviceoptions.hpp" -#include -#include -#include -#include -#include +#include +#include +#include +#include -#include +#include -using boost::bind; -using boost::function; using boost::optional; -using boost::shared_ptr; +using std::function; using std::make_pair; using std::pair; +using std::shared_ptr; using std::string; using std::vector; +using sigrok::Capability; +using sigrok::Configurable; +using sigrok::ConfigKey; +using sigrok::Error; + namespace pv { namespace prop { namespace binding { -DeviceOptions::DeviceOptions(shared_ptr dev_inst, - const sr_probe_group *group) : - _dev_inst(dev_inst), - _group(group) +DeviceOptions::DeviceOptions(shared_ptr configurable) : + configurable_(configurable) { - assert(dev_inst); - - GVariant *gvar_opts; - gsize num_opts; + assert(configurable); - if (!(gvar_opts = dev_inst->list_config(group, SR_CONF_DEVICE_OPTIONS))) - /* Driver supports no device instance options. */ - return; + for (auto entry : configurable->config_keys(ConfigKey::DEVICE_OPTIONS)) { + auto key = entry.first; + auto capabilities = entry.second; - const int *const options = (const int32_t *)g_variant_get_fixed_array( - gvar_opts, &num_opts, sizeof(int32_t)); - for (unsigned int i = 0; i < num_opts; i++) { - const struct sr_config_info *const info = - sr_config_info_get(options[i]); + Glib::VariantContainerBase gvar_list; - if (!info) + if (!capabilities.count(Capability::GET) || + !capabilities.count(Capability::SET)) continue; - const int key = info->key; - GVariant *const gvar_list = dev_inst->list_config(group, key); + if (capabilities.count(Capability::LIST)) + gvar_list = configurable->config_list(key); - const QString name = QString::fromUtf8(info->name); + string name_str; + try { + name_str = key->description(); + } catch (Error e) { + name_str = key->name(); + } + + const QString name = QString::fromStdString(name_str); - switch(key) + const Property::Getter get = [&, key]() { + return configurable_->config_get(key); }; + const Property::Setter set = [&, key](Glib::VariantBase value) { + configurable_->config_set(key, value); + config_changed(); + }; + + switch (key->id()) { case SR_CONF_SAMPLERATE: // Sample rate values are not bound because they are shown - // in the SamplingBar + // in the MainBar break; case SR_CONF_CAPTURE_RATIO: - bind_int(name, key, "%", pair(0, 100)); + bind_int(name, "%", pair(0, 100), + get, set); break; case SR_CONF_PATTERN_MODE: @@ -91,112 +101,88 @@ DeviceOptions::DeviceOptions(shared_ptr dev_inst, case SR_CONF_FILTER: case SR_CONF_COUPLING: case SR_CONF_CLOCK_EDGE: - bind_enum(name, key, gvar_list); + bind_enum(name, gvar_list, get, set); break; case SR_CONF_EXTERNAL_CLOCK: case SR_CONF_RLE: - bind_bool(name, key); + bind_bool(name, get, set); break; case SR_CONF_TIMEBASE: - bind_enum(name, key, gvar_list, print_timebase); + bind_enum(name, gvar_list, get, set, print_timebase); break; case SR_CONF_VDIV: - bind_enum(name, key, gvar_list, print_vdiv); + bind_enum(name, gvar_list, get, set, print_vdiv); break; case SR_CONF_VOLTAGE_THRESHOLD: - bind_enum(name, key, gvar_list, print_voltage_threshold); + bind_enum(name, gvar_list, get, set, print_voltage_threshold); break; - } - if (gvar_list) - g_variant_unref(gvar_list); + default: + break; + } } - g_variant_unref(gvar_opts); } -void DeviceOptions::bind_bool(const QString &name, int key) +void DeviceOptions::bind_bool(const QString &name, + Property::Getter getter, Property::Setter setter) { - assert(_dev_inst); - _properties.push_back(shared_ptr(new Bool(name, - bind(&device::DevInst::get_config, _dev_inst, _group, key), - bind(&device::DevInst::set_config, _dev_inst, - _group, key, _1)))); + assert(configurable_); + properties_.push_back(shared_ptr(new Bool( + name, getter, setter))); } -void DeviceOptions::bind_enum(const QString &name, int key, - GVariant *const gvar_list, function printer) +void DeviceOptions::bind_enum(const QString &name, + Glib::VariantContainerBase gvar_list, Property::Getter getter, + Property::Setter setter, function printer) { - GVariant *gvar; - GVariantIter iter; - vector< pair > values; + Glib::VariantBase gvar; + vector< pair > values; - assert(_dev_inst); - assert(gvar_list); + assert(configurable_); - g_variant_iter_init (&iter, gvar_list); - while ((gvar = g_variant_iter_next_value (&iter))) + Glib::VariantIter iter(gvar_list); + while ((iter.next_value(gvar))) values.push_back(make_pair(gvar, printer(gvar))); - _properties.push_back(shared_ptr(new Enum(name, values, - bind(&device::DevInst::get_config, _dev_inst, _group, key), - bind(&device::DevInst::set_config, _dev_inst, - _group, key, _1)))); -} - -void DeviceOptions::bind_int(const QString &name, int key, QString suffix, - optional< std::pair > range) -{ - assert(_dev_inst); - - _properties.push_back(shared_ptr(new Int(name, suffix, range, - bind(&device::DevInst::get_config, _dev_inst, _group, key), - bind(&device::DevInst::set_config, _dev_inst, _group, key, _1)))); + properties_.push_back(shared_ptr(new Enum(name, values, + getter, setter))); } -QString DeviceOptions::print_gvariant(GVariant *const gvar) +void DeviceOptions::bind_int(const QString &name, QString suffix, + optional< std::pair > range, + Property::Getter getter, Property::Setter setter) { - QString s; - - if (g_variant_is_of_type(gvar, G_VARIANT_TYPE("s"))) - s = QString::fromUtf8(g_variant_get_string(gvar, NULL)); - else - { - gchar *const text = g_variant_print(gvar, FALSE); - s = QString::fromUtf8(text); - g_free(text); - } + assert(configurable_); - return s; + properties_.push_back(shared_ptr(new Int(name, suffix, range, + getter, setter))); } -QString DeviceOptions::print_timebase(GVariant *const gvar) +QString DeviceOptions::print_timebase(Glib::VariantBase gvar) { uint64_t p, q; - g_variant_get(gvar, "(tt)", &p, &q); + g_variant_get(gvar.gobj(), "(tt)", &p, &q); return QString::fromUtf8(sr_period_string(p * q)); } -QString DeviceOptions::print_vdiv(GVariant *const gvar) +QString DeviceOptions::print_vdiv(Glib::VariantBase gvar) { uint64_t p, q; - g_variant_get(gvar, "(tt)", &p, &q); + g_variant_get(gvar.gobj(), "(tt)", &p, &q); return QString::fromUtf8(sr_voltage_string(p, q)); } -QString DeviceOptions::print_voltage_threshold(GVariant *const gvar) +QString DeviceOptions::print_voltage_threshold(Glib::VariantBase gvar) { gdouble lo, hi; - char buf[64]; - g_variant_get(gvar, "(dd)", &lo, &hi); - snprintf(buf, sizeof(buf), "L<%.1fV H>%.1fV", lo, hi); - return QString::fromUtf8(buf); + g_variant_get(gvar.gobj(), "(dd)", &lo, &hi); + return QString("L<%1V H>%2V").arg(lo, 0, 'f', 1).arg(hi, 0, 'f', 1); } } // binding } // prop } // pv -