From 2fca186def3f2bdd563cb12600348a91f98b9c81 Mon Sep 17 00:00:00 2001
From: Mark Watkins <jedimark@users.sourceforge.net>
Date: Mon, 28 Apr 2014 13:40:50 +1000
Subject: [PATCH] Mainwindow destructor cleanup

---
 sleepyhead/mainwindow.cpp | 35 ++++++++++++++++++++---------------
 sleepyhead/mainwindow.h   |  2 ++
 2 files changed, 22 insertions(+), 15 deletions(-)

diff --git a/sleepyhead/mainwindow.cpp b/sleepyhead/mainwindow.cpp
index 935b23a2..1ba09399 100644
--- a/sleepyhead/mainwindow.cpp
+++ b/sleepyhead/mainwindow.cpp
@@ -153,8 +153,8 @@ MainWindow::MainWindow(QWidget *parent) :
     // Using the dirty registry here. :(
     QSettings settings(getDeveloperName(), getAppName());
 
-    // Load previous Window geometry
-    this->restoreGeometry(settings.value("MainWindow/geometry").toByteArray());
+    // Load previous Window geometry (this is currently broken on Mac as of Qt5.2.1)
+    restoreGeometry(settings.value("MainWindow/geometry").toByteArray());
 
     daily = new Daily(ui->tabWidget, nullptr);
     ui->tabWidget->insertTab(1, daily, STR_TR_Daily);
@@ -206,17 +206,9 @@ MainWindow::MainWindow(QWidget *parent) :
 
     on_homeButton_clicked();
 }
-extern MainWindow *mainwin;
-MainWindow::~MainWindow()
+
+void MainWindow::closeEvent(QCloseEvent * event)
 {
-    if (systraymenu) { delete systraymenu; }
-
-    if (systray) { delete systray; }
-
-    // Save current window position
-    QSettings settings(getDeveloperName(), getAppName());
-    settings.setValue("MainWindow/geometry", saveGeometry());
-
     if (daily) {
         daily->close();
         delete daily;
@@ -232,12 +224,25 @@ MainWindow::~MainWindow()
         delete oximetry;
     }
 
-    // Trash anything allocated by the Graph objects
-    DestroyGraphGlobals();
-
     // Shutdown and Save the current User profile
     Profiles::Done();
 
+    // Save current window position
+    QSettings settings(getDeveloperName(), getAppName());
+    settings.setValue("MainWindow/geometry", saveGeometry());
+    QMainWindow::closeEvent(event);
+}
+
+extern MainWindow *mainwin;
+MainWindow::~MainWindow()
+{
+    if (systraymenu) { delete systraymenu; }
+
+    if (systray) { delete systray; }
+
+    // Trash anything allocated by the Graph objects
+    DestroyGraphGlobals();
+
     mainwin = nullptr;
     delete ui;
 }
diff --git a/sleepyhead/mainwindow.h b/sleepyhead/mainwindow.h
index af3d128e..59b072c4 100644
--- a/sleepyhead/mainwindow.h
+++ b/sleepyhead/mainwindow.h
@@ -142,6 +142,8 @@ class MainWindow : public QMainWindow
     //! \brief Internal function to set Records Box html from statistics module
     void setRecBoxHTML(QString html);
 
+    void closeEvent(QCloseEvent *);
+
   public slots:
     //! \brief Recalculate all event summaries and flags
     void doReprocessEvents();