summaryrefslogtreecommitdiffstats
path: root/qt-ui/mainwindow.cpp
diff options
context:
space:
mode:
authorGravatar Dirk Hohndel <dirk@hohndel.org>2014-08-19 21:54:18 -0500
committerGravatar Dirk Hohndel <dirk@hohndel.org>2014-08-19 21:57:15 -0500
commit642ad1e0fe292895d65765dc4ac7b5df9fe42ca8 (patch)
tree8fda55a60c8ba7d28bf4b84526363084c3673fab /qt-ui/mainwindow.cpp
parentfc96daf93fa73dd4b9c125c7a3d032d3b0f0caab (diff)
downloadsubsurface-642ad1e0fe292895d65765dc4ac7b5df9fe42ca8.tar.gz
Fix crash when calling the planner twice in a row
Turns out that the fix in commit f7119bdccfb2 ("Planner: make sure no old handles are around when entering Add/Plan") was incorrect. We ONLY want to remove the existing handlers when we re-plan a dive, NOT when we call add or plan. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'qt-ui/mainwindow.cpp')
-rw-r--r--qt-ui/mainwindow.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/qt-ui/mainwindow.cpp b/qt-ui/mainwindow.cpp
index c8461aee8..17c554777 100644
--- a/qt-ui/mainwindow.cpp
+++ b/qt-ui/mainwindow.cpp
@@ -468,6 +468,7 @@ void MainWindow::on_actionReplanDive_triggered()
DivePlannerPointsModel::instance()->setPlanMode(DivePlannerPointsModel::PLAN);
ui.newProfile->setPlanState();
+ ui.newProfile->clearHandlers();
ui.infoPane->setCurrentIndex(PLANNERWIDGET);
DivePlannerPointsModel::instance()->loadFromDive(current_dive);
reset_cylinders(&displayed_dive, true);