X-Git-Url: http://git.code-monkey.de/?a=blobdiff_plain;f=pv%2Fbinding%2Fdevice.cpp;h=df83f082e355d1c1ecc3c106d4055075cd9123b4;hb=9ba13f5e933f55f015e7c592980dfad14b4c66a8;hp=521177ad4bfd3f7fb64a70d9c5f66fd726050aec;hpb=49c62417b6b306607048b07f86b73f127e3084a3;p=pulseview.git
diff --git a/pv/binding/device.cpp b/pv/binding/device.cpp
index 521177a..df83f08 100644
--- a/pv/binding/device.cpp
+++ b/pv/binding/device.cpp
@@ -14,27 +14,26 @@
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+ * along with this program; if not, see .
*/
-#include
+#include
#include
#include "device.hpp"
#include
-#include
#include
#include
#include
using boost::optional;
+
using std::function;
-using std::make_pair;
using std::pair;
+using std::set;
using std::shared_ptr;
using std::string;
using std::vector;
@@ -45,7 +44,6 @@ using sigrok::ConfigKey;
using sigrok::Error;
using pv::prop::Bool;
-using pv::prop::Double;
using pv::prop::Enum;
using pv::prop::Int;
using pv::prop::Property;
@@ -70,7 +68,7 @@ Device::Device(shared_ptr configurable) :
string name_str;
try {
name_str = key->description();
- } catch (Error e) {
+ } catch (Error& e) {
name_str = key->name();
}
@@ -90,8 +88,7 @@ Device::Device(shared_ptr configurable) :
break;
case SR_CONF_CAPTURE_RATIO:
- bind_int(name, "%", pair(0, 100),
- get, set);
+ bind_int(name, "", "%", pair(0, 100), get, set);
break;
case SR_CONF_PATTERN_MODE:
@@ -100,33 +97,33 @@ Device::Device(shared_ptr configurable) :
case SR_CONF_TRIGGER_SLOPE:
case SR_CONF_COUPLING:
case SR_CONF_CLOCK_EDGE:
- bind_enum(name, key, capabilities, get, set);
+ bind_enum(name, "", key, capabilities, get, set);
break;
case SR_CONF_FILTER:
case SR_CONF_EXTERNAL_CLOCK:
case SR_CONF_RLE:
case SR_CONF_POWER_OFF:
- bind_bool(name, get, set);
+ bind_bool(name, "", get, set);
break;
case SR_CONF_TIMEBASE:
- bind_enum(name, key, capabilities, get, set, print_timebase);
+ bind_enum(name, "", key, capabilities, get, set, print_timebase);
break;
case SR_CONF_VDIV:
- bind_enum(name, key, capabilities, get, set, print_vdiv);
+ bind_enum(name, "", key, capabilities, get, set, print_vdiv);
break;
case SR_CONF_VOLTAGE_THRESHOLD:
- bind_enum(name, key, capabilities, get, set, print_voltage_threshold);
+ bind_enum(name, "", key, capabilities, get, set, print_voltage_threshold);
break;
case SR_CONF_PROBE_FACTOR:
if (capabilities.count(Capability::LIST))
- bind_enum(name, key, capabilities, get, set, print_probe_factor);
+ bind_enum(name, "", key, capabilities, get, set, print_probe_factor);
else
- bind_int(name, "", pair(1, 500), get, set);
+ bind_int(name, "", "", pair(1, 500), get, set);
break;
default:
@@ -135,50 +132,56 @@ Device::Device(shared_ptr configurable) :
}
}
-void Device::bind_bool(const QString &name,
+void Device::bind_bool(const QString &name, const QString &desc,
Property::Getter getter, Property::Setter setter)
{
assert(configurable_);
properties_.push_back(shared_ptr(new Bool(
- name, getter, setter)));
+ name, desc, getter, setter)));
}
-void Device::bind_enum(const QString &name,
- const ConfigKey *key, std::set capabilities,
+void Device::bind_enum(const QString &name, const QString &desc,
+ const ConfigKey *key, set capabilities,
Property::Getter getter,
Property::Setter setter, function printer)
{
- Glib::VariantBase gvar;
- vector< pair > values;
-
assert(configurable_);
if (!capabilities.count(Capability::LIST))
return;
- Glib::VariantIter iter(configurable_->config_list(key));
- while ((iter.next_value(gvar)))
- values.push_back(make_pair(gvar, printer(gvar)));
+ try {
+ Glib::VariantContainerBase gvar = configurable_->config_list(key);
+ Glib::VariantIter iter(gvar);
+
+ vector< pair > values;
+ while ((iter.next_value(gvar)))
+ values.emplace_back(gvar, printer(gvar));
- properties_.push_back(shared_ptr(new Enum(name, values,
- getter, setter)));
+ properties_.push_back(shared_ptr(new Enum(name, desc, values,
+ getter, setter)));
+
+ } catch (sigrok::Error& e) {
+ qDebug() << "Error: Listing device key" << name << "failed!";
+ return;
+ }
}
-void Device::bind_int(const QString &name, QString suffix,
- optional< std::pair > range,
+void Device::bind_int(const QString &name, const QString &desc, QString suffix,
+ optional< pair > range,
Property::Getter getter, Property::Setter setter)
{
assert(configurable_);
- properties_.push_back(shared_ptr(new Int(name, suffix, range,
- getter, setter)));
+ properties_.push_back(shared_ptr(new Int(name, desc, suffix,
+ range, getter, setter)));
}
QString Device::print_timebase(Glib::VariantBase gvar)
{
uint64_t p, q;
g_variant_get(gvar.gobj(), "(tt)", &p, &q);
- return QString::fromUtf8(sr_period_string(p * q));
+ return QString::fromUtf8(sr_period_string(p, q));
}
QString Device::print_vdiv(Glib::VariantBase gvar)
@@ -202,5 +205,5 @@ QString Device::print_probe_factor(Glib::VariantBase gvar)
return QString("%1x").arg(factor);
}
-} // binding
-} // pv
+} // namespace binding
+} // namespace pv