Connect preferences to the rest of the code

The biggest problem here was that bool has different sizes in C and C++
code. So using this in a structure shared between the two sides wasn't a
smart idea.

Instead I went with 'short', but that caused problems with Qt being to
smart for its own good and not doing the right thing when dealing with
'boolean' settings and a short value. This may be something in the way I
implemented things (as I doubt that something this fundamental would be
broken) but the workaround implemented here (explicitly using 0 or 1
depending on the value of the boolean) seems to work.

I also decided to get rid of the confusion of where gflow/gfhigh are
floating point (0..1) and when they are integers (0..100). We now use
integers anywhere outside of deco.c.

I also applied some serious spelling corrections to the preferences
dialog's ui file.

Finally, this enables the code that selects which partial pressure graph
to show.

Still to do: font size, metric/imperial logic

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>

5 files changed, 79 insertions, 58 deletions

diff --git a/qt-ui/mainwindow.cpp b/qt-ui/mainwindow.cpp
index 69ffdb224..7ed4f8aeb 100644
--- a/qt-ui/mainwindow.cpp
+++ b/qt-ui/mainwindow.cpp
@@ -372,11 +372,11 @@ void MainWindow::readSettings()
 	GET_BOOL(v, "OTU", prefs.visible_cols.otu);
 	GET_BOOL(v, "MAXCNS", prefs.visible_cols.maxcns);
 	GET_BOOL(v, "SAC", prefs.visible_cols.sac);
+	settings.endGroup();
+	settings.beginGroup("TecDetails");
 	GET_BOOL(v, "po2graph", prefs.pp_graphs.po2);
 	GET_BOOL(v, "pn2graph", prefs.pp_graphs.pn2);
 	GET_BOOL(v, "phegraph", prefs.pp_graphs.phe);
-	settings.endGroup();
-	settings.beginGroup("TecDetails");
 	v = settings.value(QString("po2threshold"));
 	if (v.isValid())
 		prefs.pp_graphs.po2_threshold = v.toDouble();
@@ -392,14 +392,15 @@ void MainWindow::readSettings()
 		prefs.mod_ppO2 = v.toDouble();
 	GET_BOOL(v, "ead", prefs.ead);
 	GET_BOOL(v, "redceiling", prefs.profile_red_ceiling);
+	GET_BOOL(v, "show_dc_reported_ceiling", prefs.profile_dc_ceiling);
 	GET_BOOL(v, "calcceiling", prefs.profile_calc_ceiling);
 	GET_BOOL(v, "calcceiling3m", prefs.calc_ceiling_3m_incr);
 	v = settings.value(QString("gflow"));
 	if (v.isValid())
-		prefs.gflow = v.toInt() / 100.0;
+		prefs.gflow = v.toInt();
 	v = settings.value(QString("gfhigh"));
 	if (v.isValid())
-		prefs.gfhigh = v.toInt() / 100.0;
+		prefs.gfhigh = v.toInt();
 	set_gf(prefs.gflow, prefs.gfhigh);
 	settings.endGroup();
 
@@ -420,7 +421,9 @@ void MainWindow::readSettings()
 
 #define SAVE_VALUE(name, field)				\
 	if (prefs.field != default_prefs.field)		\
-		settings.setValue(name, prefs.field)
+		settings.setValue(name, prefs.field);	\
+	else						\
+		settings.remove(name)
 
 void MainWindow::writeSettings()
 {
diff --git a/qt-ui/preferences.cpp b/qt-ui/preferences.cpp
index 3e684d88a..a63894777 100644
--- a/qt-ui/preferences.cpp
+++ b/qt-ui/preferences.cpp
@@ -1,6 +1,5 @@
 #include "preferences.h"
 #include "ui_preferences.h"
-#include "../dive.h"
 #include <QSettings>
 
 PreferencesDialog* PreferencesDialog::instance()
@@ -9,32 +8,36 @@ PreferencesDialog* PreferencesDialog::instance()
 	return dialog;
 }
 
+#define B(V, P) s.value(#V, default_prefs.P).toBool()
+#define D(V, P) s.value(#V, default_prefs.P).toDouble()
+#define I(V, P) s.value(#V, default_prefs.P).toInt()
+
 PreferencesDialog::PreferencesDialog(QWidget* parent, Qt::WindowFlags f) : QDialog(parent, f)
 , ui(new Ui::PreferencesDialog())
 {
 	ui->setupUi(this);
 	connect(ui->buttonBox, SIGNAL(accepted()), this, SLOT(syncSettings()));
+	connect(ui->buttonBox, SIGNAL(rejected()), this, SLOT(resetSettings()));
 
-	#define B(X) s.value(#X, false).toBool()
-	#define D(X) s.value(#X, 0.0).toDouble()
+	oldPrefs = prefs;
 
 	QSettings s;
 
 	// Graph
 	s.beginGroup("TecDetails");
-	ui->calculated_ceiling->setChecked(B(show_calculated_ceiling));
-	ui->phe->setChecked(B(show_phe));
-	ui->po2->setChecked(B(show_po2));
-	ui->pn2->setChecked(B(show_pn2));
-	ui->pheThreshould->setValue(D(phe_threshould));
-	ui->po2Threashould->setValue(D(po2_threshould));
-	ui->pn2Threshould->setValue(D(pn2_threshould));
-	ui->ead_end_eadd->setChecked(B(show_ead_end_eadd));
-	ui->dc_reported_ceiling->setChecked(B(show_dc_reported_ceiling));
-	ui->calculated_ceiling->setChecked(B(show_calculated_ceiling));
-	ui->increment_3m->setChecked(B(show_3m_increments));
-	ui->gflow->setValue(D(gflow));
-	ui->gfhigh->setValue(D(gfhigh));
+	ui->calculated_ceiling->setChecked(B(calcceiling, profile_calc_ceiling));
+	ui->phe->setChecked(B(phegraph, pp_graphs.phe));
+	ui->po2->setChecked(B(po2graph, pp_graphs.po2));
+	ui->pn2->setChecked(B(pn2graph, pp_graphs.pn2));
+	ui->pheThreshold->setValue(D(phethreshold, pp_graphs.phe_threshold));
+	ui->po2Threshold->setValue(D(po2threshold, pp_graphs.po2_threshold));
+	ui->pn2Threshold->setValue(D(pn2threshold, pp_graphs.pn2_threshold));
+	ui->ead_end_eadd->setChecked(B(ead, ead));
+	ui->dc_reported_ceiling->setChecked(B(dcceiling, profile_dc_ceiling));
+	ui->calculated_ceiling->setChecked(B(calceiling, profile_calc_ceiling));
+	ui->increment_3m->setChecked(B(calcceiling3m, calc_ceiling_3m_incr));
+	ui->gflow->setValue((int)(I(gflow, gflow)));
+	ui->gfhigh->setValue((int)(I(gfhigh, gfhigh)));
 	s.endGroup();
 
 	// Units
@@ -68,36 +71,44 @@ PreferencesDialog::PreferencesDialog(QWidget* parent, Qt::WindowFlags f) : QDial
 	// Defaults
 	s.beginGroup("GeneralSettings");
 	ui->font->setFont( QFont(s.value("table_fonts").toString()));
-	ui->fontsize->setValue(D(font_size));
+	ui->fontsize->setValue(D(font_size, font_size));
 
-	ui->defaultfilename->setText(s.value("default_file").toString());
-	ui->displayinvalid->setChecked(B(show_invalid));
+	ui->defaultfilename->setText(s.value("default_filename").toString());
+	ui->displayinvalid->setChecked(B(show_invalid, show_invalid));
 	s.endGroup();
+}
+
 #undef B
 #undef D
+
+void PreferencesDialog::resetSettings()
+{
+	prefs = oldPrefs;
 }
 
+#define SB(V, B) s.setValue(V, (int)(B->isChecked() ? 1 : 0))
+
 void PreferencesDialog::syncSettings()
 {
 	QSettings s;
 
 	// Graph
 	s.beginGroup("TecDetails");
-	s.setValue("show_calculated_ceiling", ui->calculated_ceiling->isChecked());
-	s.setValue("show_phe", ui->phe->isChecked());
-	s.setValue("show_po2", ui->po2->isChecked());
-	s.setValue("show_pn2", ui->pn2->isChecked());
-	s.setValue("phe_threshould", ui->pheThreshould->value());
-	s.setValue("po2_threshould", ui->po2Threashould->value());
-	s.setValue("pn2_threshould", ui->pn2Threshould->value());
-	s.setValue("show_ead_end_eadd", ui->ead_end_eadd->isChecked());
-	s.setValue("show_dc_reported_ceiling", ui->dc_reported_ceiling->isChecked());
-	s.setValue("show_calculated_ceiling", ui->calculated_ceiling->isChecked());
-
-	s.setValue("show_3m_increments", ui->increment_3m->isChecked());
+
+	SB("calcceiling", ui->calculated_ceiling);
+	SB("phegraph", ui->phe);
+	SB("po2graph", ui->po2);
+	SB("pn2graph", ui->pn2);
+	s.setValue("phethreshold", ui->pheThreshold->value());
+	s.setValue("po2threshold", ui->po2Threshold->value());
+	s.setValue("pn2threshold", ui->pn2Threshold->value());
+	SB("ead", ui->ead_end_eadd);
+	SB("dcceiling", ui->dc_reported_ceiling);
+	SB("calceiling3m", ui->increment_3m);
 	s.setValue("gflow", ui->gflow->value());
 	s.setValue("gfhigh", ui->gfhigh->value());
 	s.endGroup();
+
 	// Units
 	s.beginGroup("Units");
 	s.setValue("units_metric", ui->metric->isChecked());
@@ -111,10 +122,13 @@ void PreferencesDialog::syncSettings()
 	s.beginGroup("GeneralSettings");
 	s.value("table_fonts", ui->font->font().family());
 	s.value("font_size", ui->fontsize->value());
-	s.value("default_file", ui->defaultfilename->text());
+	s.value("default_filename", ui->defaultfilename->text());
 	s.value("displayinvalid", ui->displayinvalid->isChecked());
 	s.endGroup();
 	s.sync();
 
+	oldPrefs = prefs;
 	emit settingsChanged();
 }
+
+#undef SB
diff --git a/qt-ui/preferences.h b/qt-ui/preferences.h
index ab5a214ed..34fc26b07 100644
--- a/qt-ui/preferences.h
+++ b/qt-ui/preferences.h
@@ -2,6 +2,8 @@
 #define PREFERENCES_DIALOG_H
 
 #include <QDialog>
+#include "../dive.h"
+#include "../pref.h"
 
 namespace Ui{
 	class PreferencesDialog;
@@ -17,10 +19,12 @@ signals:
 
 public slots:
 	void syncSettings();
-	
+	void resetSettings();
+
 private:
 	explicit PreferencesDialog(QWidget* parent = 0, Qt::WindowFlags f = 0);
-    Ui::PreferencesDialog* ui;
+	Ui::PreferencesDialog* ui;
+	struct preferences oldPrefs;
 };
 
-#endif
\ No newline at end of file
+#endif
diff --git a/qt-ui/preferences.ui b/qt-ui/preferences.ui
index e36aef5b3..c442f5d2c 100644
--- a/qt-ui/preferences.ui
+++ b/qt-ui/preferences.ui
@@ -449,12 +449,12 @@
                  <bool>false</bool>
                 </property>
                 <property name="text">
-                 <string>threshould</string>
+                 <string>threshold</string>
                 </property>
                </widget>
               </item>
               <item>
-               <widget class="QSpinBox" name="po2Threashould">
+               <widget class="QDoubleSpinBox" name="po2Threshold">
                 <property name="enabled">
                  <bool>false</bool>
                 </property>
@@ -490,12 +490,12 @@
                  <bool>false</bool>
                 </property>
                 <property name="text">
-                 <string>threshould</string>
+                 <string>threshold</string>
                 </property>
                </widget>
               </item>
               <item>
-               <widget class="QSpinBox" name="pn2Threshould">
+               <widget class="QDoubleSpinBox" name="pn2Threshold">
                 <property name="enabled">
                  <bool>false</bool>
                 </property>
@@ -531,12 +531,12 @@
                  <bool>false</bool>
                 </property>
                 <property name="text">
-                 <string>threshould</string>
+                 <string>threshold</string>
                 </property>
                </widget>
               </item>
               <item>
-               <widget class="QSpinBox" name="pheThreshould">
+               <widget class="QDoubleSpinBox" name="pheThreshold">
                 <property name="enabled">
                  <bool>false</bool>
                 </property>
@@ -650,7 +650,7 @@
              </widget>
             </item>
             <item row="0" column="1">
-             <widget class="QDoubleSpinBox" name="gflow"/>
+             <widget class="QSpinBox" name="gflow"/>
             </item>
             <item row="1" column="0">
              <widget class="QLabel" name="label_20">
@@ -660,7 +660,7 @@
              </widget>
             </item>
             <item row="1" column="1">
-             <widget class="QDoubleSpinBox" name="gfhigh"/>
+             <widget class="QSpinBox" name="gfhigh"/>
             </item>
            </layout>
           </widget>
@@ -754,7 +754,7 @@
   <connection>
    <sender>po2</sender>
    <signal>clicked(bool)</signal>
-   <receiver>po2Threashould</receiver>
+   <receiver>po2Threshold</receiver>
    <slot>setEnabled(bool)</slot>
    <hints>
     <hint type="sourcelabel">
@@ -786,7 +786,7 @@
   <connection>
    <sender>pn2</sender>
    <signal>clicked(bool)</signal>
-   <receiver>pn2Threshould</receiver>
+   <receiver>pn2Threshold</receiver>
    <slot>setEnabled(bool)</slot>
    <hints>
     <hint type="sourcelabel">
@@ -818,7 +818,7 @@
   <connection>
    <sender>phe</sender>
    <signal>clicked(bool)</signal>
-   <receiver>pheThreshould</receiver>
+   <receiver>pheThreshold</receiver>
    <slot>setEnabled(bool)</slot>
    <hints>
     <hint type="sourcelabel">
diff --git a/qt-ui/profilegraphics.cpp b/qt-ui/profilegraphics.cpp
index bf244e9e9..060c36b51 100644
--- a/qt-ui/profilegraphics.cpp
+++ b/qt-ui/profilegraphics.cpp
@@ -429,7 +429,7 @@ void ProfileGraphicsView::plot_pp_gas_profile()
 	setup_pp_limits(&gc);
 	QColor c;
 	QPointF from, to;
-	//if (prefs.pp_graphs.pn2) {
+	if (prefs.pp_graphs.pn2) {
 		c = profile_color[PN2].first();
 		entry = pi->entry;
 		from = QPointF(SCALEGC(entry->sec, entry->pn2));
@@ -465,9 +465,9 @@ void ProfileGraphicsView::plot_pp_gas_profile()
 				from = QPointF(SCALEGC(entry->sec, entry->pn2));
 			}
 		}
-	//}
+	}
 
-	//if (prefs.pp_graphs.phe) {
+	if (prefs.pp_graphs.phe) {
 		c = profile_color[PHE].first();
 		entry = pi->entry;
 
@@ -504,8 +504,8 @@ void ProfileGraphicsView::plot_pp_gas_profile()
 				from = QPointF(SCALEGC(entry->sec, entry->phe));
 			}
 		}
-	//}
-	//if (prefs.pp_graphs.po2) {
+	}
+	if (prefs.pp_graphs.po2) {
 		c = profile_color[PO2].first();
 		entry = pi->entry;
 		from = QPointF(SCALEGC(entry->sec, entry->po2));
@@ -539,7 +539,7 @@ void ProfileGraphicsView::plot_pp_gas_profile()
 				from = QPointF(SCALEGC(entry->sec, entry->po2));
 			}
 		}
-	//}
+	}
 }
 
 void ProfileGraphicsView::plot_deco_text()
@@ -549,7 +549,7 @@ void ProfileGraphicsView::plot_deco_text()
 		float y = gc.topy = 1.0;
 		static text_render_options_t tro = {PRESSURE_TEXT_SIZE, PRESSURE_TEXT, CENTER, -0.2};
 		gc.bottomy = 0.0;
-		plot_text(&tro, QPointF(x, y), QString("GF %1/%2").arg(prefs.gflow * 100).arg(prefs.gfhigh * 100));
+		plot_text(&tro, QPointF(x, y), QString("GF %1/%2").arg(prefs.gflow).arg(prefs.gfhigh));
 	}
 }