[2015-09-23 20:10:35 -1000] Gaetan Bisson:
> So I do not change the dive site at all (just fiddle with the notes),
> and when I click "Apply Changes", in MainTab::updateDiveSite(), the
> pickedUuid variable is 16777215 == 2^24-1. I'm not sure how that
> happens, but obviously it should be zero in order to be caught by the
> test you added.

Ha! The currUuid variable was never initialized...

Patch attached.

-- 
Gaetan
>From d5fd492439ada38d795b703d78f8a4384ace3a08 Mon Sep 17 00:00:00 2001
From: Gaetan Bisson <[email protected]>
Date: Wed, 23 Sep 2015 21:12:18 -1000
Subject: [PATCH 1/1] Initialize currUuid

Signed-off-by: Gaetan Bisson <[email protected]>
---
 qt-ui/locationinformation.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/qt-ui/locationinformation.cpp b/qt-ui/locationinformation.cpp
index 6e6db0b..213d63b 100644
--- a/qt-ui/locationinformation.cpp
+++ b/qt-ui/locationinformation.cpp
@@ -360,6 +360,7 @@ bool DiveLocationModel::setData(const QModelIndex& index, const QVariant& value,
 DiveLocationLineEdit::DiveLocationLineEdit(QWidget *parent) : QLineEdit(parent),
 	proxy(new DiveLocationFilterProxyModel()), model(new DiveLocationModel()), view(new DiveLocationListView())
 {
+	currUuid = 0;
 	location_line_edit = this;
 
 	proxy->setSourceModel(model);
-- 
2.5.3

_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to