[PATCH] De-duplicate planner default settings

Anton Lundin glance at acc.umu.se
Sat Jul 19 04:36:57 PDT 2014


All other default settings we get from subsurfacestartup.c's
initialization of the prefs struct. The planner had its defaults there
and in the retrieve of the settings from QSettings.

This changes so the defaults for planner settings will be read from the
defaults prefs struct as all other settings.

Signed-off-by: Anton Lundin <glance at acc.umu.se>
---
 qt-ui/diveplanner.cpp | 22 +++++++++++-----------
 subsurfacestartup.c   |  4 ++--
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/qt-ui/diveplanner.cpp b/qt-ui/diveplanner.cpp
index 2853472..4ecf7a9 100644
--- a/qt-ui/diveplanner.cpp
+++ b/qt-ui/diveplanner.cpp
@@ -365,17 +365,17 @@ PlannerSettingsWidget::PlannerSettingsWidget(QWidget *parent, Qt::WindowFlags f)
 
 	QSettings s;
 	s.beginGroup("Planner");
-	prefs.ascrate75 = s.value("ascrate75", 9 * 1000.0 / 60.0).toInt();
-	prefs.ascrate50 = s.value("ascrate50", 6 * 1000.0 / 60.0).toInt();
-	prefs.ascratestops = s.value("ascratestops", 6 * 1000.0 / 60.0).toInt();
-	prefs.ascratelast6m = s.value("ascratelast6m", 1 * 1000.0 / 60.0).toInt();
-	prefs.descrate = s.value("descrate", 18 * 1000.0 / 60.0).toInt();
-	prefs.bottompo2 = s.value("bottompo2", 1400).toInt();
-	prefs.decopo2 = s.value("decopo2",1600).toInt();
-	prefs.doo2breaks = s.value("doo2breaks", false).toBool();
-	prefs.drop_stone_mode = s.value("drop_stone_mode", false).toBool();
-	prefs.bottomsac = s.value("bottomsac", 20000).toInt();
-	prefs.decosac = s.value("decosac", 17000).toInt();
+	prefs.ascrate75 = s.value("ascrate75", prefs.ascrate75).toInt();
+	prefs.ascrate50 = s.value("ascrate50", prefs.ascrate50).toInt();
+	prefs.ascratestops = s.value("ascratestops", prefs.ascratestops).toInt();
+	prefs.ascratelast6m = s.value("ascratelast6m", prefs.ascratelast6m).toInt();
+	prefs.descrate = s.value("descrate", prefs.descrate).toInt();
+	prefs.bottompo2 = s.value("bottompo2", prefs.bottompo2).toInt();
+	prefs.decopo2 = s.value("decopo2", prefs.decopo2).toInt();
+	prefs.doo2breaks = s.value("doo2breaks", prefs.doo2breaks).toBool();
+	prefs.drop_stone_mode = s.value("drop_stone_mode", prefs.drop_stone_mode).toBool();
+	prefs.bottomsac = s.value("bottomsac", prefs.bottomsac).toInt();
+	prefs.decosac = s.value("decosac", prefs.decosac).toInt();
 	s.endGroup();
 
 	updateUnitsUI();
diff --git a/subsurfacestartup.c b/subsurfacestartup.c
index 29d3ddd..3e6e928 100644
--- a/subsurfacestartup.c
+++ b/subsurfacestartup.c
@@ -42,8 +42,8 @@ struct preferences default_prefs = {
 	.decopo2 = 1600,
 	.doo2breaks = false,
 	.drop_stone_mode = false,
-	.bottomsac = 20,
-	.decosac = 17,
+	.bottomsac = 20000,
+	.decosac = 17000,
 	.show_pictures_in_profile = true
 };
 
-- 
1.9.1



More information about the subsurface mailing list