[PATCH] Save settings in closeEvent, not destructor

Anton Lundin glance at acc.umu.se
Wed Sep 9 22:30:39 PDT 2015


This switches to use the closeEvent to save settings and cleanup instead
of the destructor.

Signed-off-by: Anton Lundin <glance at acc.umu.se>
---
 qt-ui/configuredivecomputerdialog.cpp | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/qt-ui/configuredivecomputerdialog.cpp b/qt-ui/configuredivecomputerdialog.cpp
index 04c15b1..e04dcc3 100644
--- a/qt-ui/configuredivecomputerdialog.cpp
+++ b/qt-ui/configuredivecomputerdialog.cpp
@@ -299,6 +299,10 @@ void OstcFirmwareCheck::saveOstcFirmware(QNetworkReply *reply)
 
 ConfigureDiveComputerDialog::~ConfigureDiveComputerDialog()
 {
+}
+
+void ConfigureDiveComputerDialog::closeEvent(QCloseEvent *event)
+{
 	QSettings settings;
 	settings.beginGroup("ConfigureDiveComputerDialog");
 	settings.beginGroup("ostc3GasTable");
-- 
2.1.4



More information about the subsurface mailing list