From 164f8d8d473e571086583d86bb4ff9982abe54b4 Mon Sep 17 00:00:00 2001
From: Stanislaw Halik <sthalik@misaki.pl>
Date: Mon, 26 Sep 2016 13:14:16 +0200
Subject: gui, options: prevent options reset on tracker stop

They should only be reset when closing the module dialog window.

This is a hack but otherwise we'd have to change all the
modules.

Issue: #466
Closes #466
---
 options/scoped.cpp | 44 +++++++++++++++++++++++++++++++++++++++++++-
 options/scoped.hpp |  9 +++++++++
 2 files changed, 52 insertions(+), 1 deletion(-)

(limited to 'options')

diff --git a/options/scoped.cpp b/options/scoped.cpp
index e1ddbae2..b0677ad9 100644
--- a/options/scoped.cpp
+++ b/options/scoped.cpp
@@ -1,10 +1,52 @@
 #include "scoped.hpp"
+#include <QApplication>
+#include <QThread>
+
+// for std::abort()
+#include <cstdlib>
+
+#include <QDebug>
 
 namespace options {
 
+std::atomic_bool opts::teardown_flag(false);
+
+void opts::set_teardown_flag(bool value)
+{
+    ensure_thread();
+
+    // we don't use exceptions in the whole project so no need for unwind protection
+    // also the calls aren't nested so no need for CAS either
+    teardown_flag = value;
+}
+
+void opts::ensure_thread()
+{
+    // only as a bug check
+
+    const QThread* ui_thread = qApp->thread();
+    const QThread* curthread = QThread::currentThread();
+
+    if (ui_thread == nullptr)
+        abort();
+
+    if (ui_thread != curthread)
+        abort();
+}
+
 opts::~opts()
 {
-    b->reload();
+    if (!is_tracker_teardown())
+        b->reload();
+#if 0
+    else
+        qDebug() << "in teardown, not reloading" << b->name();
+#endif
+}
+
+bool opts::is_tracker_teardown()
+{
+    return teardown_flag;
 }
 
 opts::opts(const QString &name) : b(make_bundle(name))
diff --git a/options/scoped.hpp b/options/scoped.hpp
index 52ae383d..4ad51fea 100644
--- a/options/scoped.hpp
+++ b/options/scoped.hpp
@@ -5,6 +5,8 @@
 
 #include "export.hpp"
 
+#include <atomic>
+
 namespace options {
 
 struct OPENTRACK_OPTIONS_EXPORT opts
@@ -14,6 +16,13 @@ struct OPENTRACK_OPTIONS_EXPORT opts
     opts& operator=(const opts&) = delete;
     opts(const opts&) = delete;
     ~opts();
+
+    // XXX hack: the flag shouldn't be here as action at distance -sh 20160926
+    static void set_teardown_flag(bool value);
+private:
+    static std::atomic_bool teardown_flag;
+    static bool is_tracker_teardown();
+    static void ensure_thread();
 };
 
 }
-- 
cgit v1.2.3