From 3657d2c0d9807d6f88378981513a944cce4a1a16 Mon Sep 17 00:00:00 2001
From: Stanislaw Halik <sthalik@misaki.pl>
Date: Wed, 29 Jun 2016 10:37:15 +0200
Subject: logic, gui: no need to reference "struct main_settings"

Now that "struct main_settings" doesn't implicitly reload in dtor, we no
longer have to pass it around from the user interface.

Only reload it where it's modified, i.e. in the options dialog. Changing
the filter/dialog/mapping comboboxen implicitly saves the main options
however.
---
 gui/curve-config.cpp |  4 +++-
 gui/curve-config.h   |  2 +-
 gui/ui.cpp           | 12 +++++++-----
 gui/ui.h             |  2 +-
 4 files changed, 12 insertions(+), 8 deletions(-)

(limited to 'gui')

diff --git a/gui/curve-config.cpp b/gui/curve-config.cpp
index 61459c1c..3b56dbb8 100644
--- a/gui/curve-config.cpp
+++ b/gui/curve-config.cpp
@@ -8,7 +8,7 @@
 
 #include "curve-config.h"
 #include "opentrack-logic/main-settings.hpp"
-MapWidget::MapWidget(Mappings& m, main_settings& s) :
+MapWidget::MapWidget(Mappings& m) :
     m(m)
 {
     ui.setupUi( this );
@@ -68,6 +68,8 @@ MapWidget::MapWidget(Mappings& m, main_settings& s) :
     connect(ui.buttonBox, SIGNAL(accepted()), this, SLOT(doOK()));
     connect(ui.buttonBox, SIGNAL(rejected()), this, SLOT(doCancel()));
 
+    main_settings s;
+
     tie_setting(s.a_x.altp, ui.tx_altp);
     tie_setting(s.a_y.altp, ui.ty_altp);
     tie_setting(s.a_z.altp, ui.tz_altp);
diff --git a/gui/curve-config.h b/gui/curve-config.h
index 9a9475b9..04a25ab4 100644
--- a/gui/curve-config.h
+++ b/gui/curve-config.h
@@ -7,7 +7,7 @@ class MapWidget: public QWidget
 {
     Q_OBJECT
 public:
-    MapWidget(Mappings &m, main_settings &s);
+    MapWidget(Mappings &m);
 private:
     Ui::UICCurveConfigurationDialog ui;
     Mappings& m;
diff --git a/gui/ui.cpp b/gui/ui.cpp
index cda7d94b..c82d2e52 100644
--- a/gui/ui.cpp
+++ b/gui/ui.cpp
@@ -69,7 +69,7 @@ MainWindow::MainWindow() :
 
     connect(ui.btnStartTracker, SIGNAL(clicked()), this, SLOT(startTracker()));
     connect(ui.btnStopTracker, SIGNAL(clicked()), this, SLOT(stopTracker()));
-    connect(ui.iconcomboProfile, SIGNAL(currentTextChanged(QString)), this, SLOT(profileSelected(QString)));
+    connect(ui.iconcomboProfile, SIGNAL(currentTextChanged(QString)), this, SLOT(profile_selected(QString)));
 
     connect(&pose_update_timer, SIGNAL(timeout()), this, SLOT(showHeadPose()));
     connect(&kbd_quit, SIGNAL(activated()), this, SLOT(exit()));
@@ -172,7 +172,7 @@ void MainWindow::save_mappings()
 void MainWindow::save()
 {
     s.b->save();
-    save_mappings();
+    //save_mappings();
 
 #if defined(__unix) || defined(__linux)
     QString currentFile = group::ini_pathname();
@@ -189,6 +189,8 @@ void MainWindow::save()
 void MainWindow::load_mappings()
 {
     pose.load_mappings();
+    if (mapping_widget)
+        mapping_widget->reload();
 }
 
 void MainWindow::load_settings()
@@ -312,7 +314,7 @@ void MainWindow::startTracker()
 
     save();
 
-    work = std::make_shared<Work>(s, pose, libs, winId());
+    work = std::make_shared<Work>(pose, libs, winId());
 
     reload_options();
 
@@ -500,7 +502,7 @@ void MainWindow::show_options_dialog()
 
 void MainWindow::showCurveConfiguration()
 {
-    mk_window(&mapping_widget, pose, s);
+    mk_window(&mapping_widget, pose);
 }
 
 void MainWindow::exit()
@@ -508,7 +510,7 @@ void MainWindow::exit()
     QCoreApplication::exit(0);
 }
 
-void MainWindow::profileSelected(QString name)
+void MainWindow::profile_selected(const QString& name)
 {
     if (name == "" || is_refreshing_profiles)
         return;
diff --git a/gui/ui.h b/gui/ui.h
index 626b1bb3..8fab64ed 100644
--- a/gui/ui.h
+++ b/gui/ui.h
@@ -82,7 +82,7 @@ class MainWindow : public QMainWindow, private State
 private slots:
     void save();
     void exit();
-    void profileSelected(QString name);
+    void profile_selected(const QString& name);
 
     void showTrackerSettings();
     void showProtocolSettings();
-- 
cgit v1.2.3