Bug 203 - Pre-trigger sample device option does not function
Summary: Pre-trigger sample device option does not function
Status: RESOLVED FIXED
Alias: None
Product: PulseView
Classification: Unclassified
Component: Acquisition (show other bugs)
Version: unreleased development snapshot
Hardware: All All
: Normal normal
Target Milestone: PulseView 0.2.0
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-02 23:21 CET by russ.dill
Modified: 2014-01-06 18:12 CET (History)
1 user (show)


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description russ.dill 2013-12-02 23:21:55 CET 
diff --git a/pv/prop/int.cpp b/pv/prop/int.cpp
index 6a2cb7d..068cdf2 100644
--- a/pv/prop/int.cpp
+++ b/pv/prop/int.cpp
@@ -78,7 +78,7 @@ void Int::commit()
        if (!_spin_box)
                return;
 
-       _setter(g_variant_new_int64(_spin_box->value()));
+       _setter(g_variant_new_uint64(_spin_box->value()));
 }
 
 void Int::on_value_changed(int)
Comment 1 Joel Holdsworth 2013-12-29 19:09:27 CET 
russ.dill: Can you retest this? Is should have been fixed with ac223c1.
Comment 2 Joel Holdsworth 2014-01-06 18:12:39 CET 
Russ confirmes it's fixed