X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fprop%2Fint.cpp;h=ce77b9592d3f66778ab6247b88b20cca32b9ca74;hp=31f518a8cbd481467d0e5195861058989574ae28;hb=c063290ac7189bdd15221450f598504f43286b43;hpb=4c60462b00cc329e61daedd1c2e66724077bd412

diff --git a/pv/prop/int.cpp b/pv/prop/int.cpp
index 31f518a8..ce77b959 100644
--- a/pv/prop/int.cpp
+++ b/pv/prop/int.cpp
@@ -14,12 +14,11 @@
  * 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 <http://www.gnu.org/licenses/>.
  */
 
-#include <stdint.h>
-#include <assert.h>
+#include <cstdint>
+#include <cassert>
 
 #include <QSpinBox>
 
@@ -45,13 +44,10 @@ Int::Int(QString name,
 {
 }
 
-Int::~Int()
-{
-}
-
 QWidget* Int::get_widget(QWidget *parent, bool auto_commit)
 {
-	int64_t int_val = 0, range_min = 0, range_max = 0;
+	int64_t int_val = 0, range_min = 0;
+	uint64_t range_max = 0;
 
 	if (spin_box_)
 		return spin_box_;
@@ -71,45 +67,30 @@ QWidget* Int::get_widget(QWidget *parent, bool auto_commit)
 	const GVariantType *const type = g_variant_get_type(value);
 	assert(type);
 
-	if (g_variant_type_equal(type, G_VARIANT_TYPE_BYTE))
-	{
+	if (g_variant_type_equal(type, G_VARIANT_TYPE_BYTE)) {
 		int_val = g_variant_get_byte(value);
 		range_min = 0, range_max = UINT8_MAX;
-	}
-	else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT16))
-	{
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT16)) {
 		int_val = g_variant_get_int16(value);
 		range_min = INT16_MIN, range_max = INT16_MAX;
-	}
-	else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT16))
-	{
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT16)) {
 		int_val = g_variant_get_uint16(value);
 		range_min = 0, range_max = UINT16_MAX;
-	}
-	else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT32))
-	{
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT32)) {
 		int_val = g_variant_get_int32(value);
 		range_min = INT32_MIN, range_max = INT32_MAX;
-	}
-	else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT32))
-	{
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT32)) {
 		int_val = g_variant_get_uint32(value);
 		range_min = 0, range_max = UINT32_MAX;
-	}
-	else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT64))
-	{
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_INT64)) {
 		int_val = g_variant_get_int64(value);
 		range_min = INT64_MIN, range_max = INT64_MAX;
-	}
-	else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT64))
-	{
+	} else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT64)) {
 		int_val = g_variant_get_uint64(value);
 		range_min = 0, range_max = UINT64_MAX;
-	}
-	else
-	{
+	} else {
 		// Unexpected value type.
-		assert(0);
+		assert(false);
 	}
 
 	// @todo Sigrok supports 64-bit quantities, but Qt does not have a
@@ -118,7 +99,7 @@ QWidget* Int::get_widget(QWidget *parent, bool auto_commit)
 	// custom widget.
 
 	range_min = max(range_min, (int64_t)INT_MIN);
-	range_max = min(range_max, (int64_t)INT_MAX);
+	range_max = min(range_max, (uint64_t)INT_MAX);
 
 	if (range_)
 		spin_box_->setRange((int)range_->first, (int)range_->second);
@@ -159,10 +140,9 @@ void Int::commit()
 		new_value = g_variant_new_int64(spin_box_->value());
 	else if (g_variant_type_equal(type, G_VARIANT_TYPE_UINT64))
 		new_value = g_variant_new_uint64(spin_box_->value());
-	else
-	{
+	else {
 		// Unexpected value type.
-		assert(0);
+		assert(false);
 	}
 
 	assert(new_value);