summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar Dirk Hohndel <dirk@hohndel.org>2014-05-30 11:02:17 -0700
committerGravatar Dirk Hohndel <dirk@hohndel.org>2014-05-30 11:02:17 -0700
commit8cbf2071693f06a2fe25c6ae322c3131302dd2bf (patch)
tree78c60492b728d0ed217e9965f64080757efd4267
parentb36f8dd738963faae31312deca014c438e53e516 (diff)
downloadsubsurface-8cbf2071693f06a2fe25c6ae322c3131302dd2bf.tar.gz
Planner: remove pointless code
This never made any sense. If we have NO datapoints then go and cancel the plan? And anyway, we already check this higher up in the call stack. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
-rw-r--r--qt-ui/diveplanner.cpp3
1 files changed, 0 insertions, 3 deletions
diff --git a/qt-ui/diveplanner.cpp b/qt-ui/diveplanner.cpp
index 2b1662f7d..d11b89617 100644
--- a/qt-ui/diveplanner.cpp
+++ b/qt-ui/diveplanner.cpp
@@ -874,9 +874,6 @@ void DivePlannerPointsModel::createPlan()
char *cache = NULL;
tempDive = NULL;
- if (!diveplan.dp)
- return cancelPlan();
-
bool oldRecalc = plannerModel->setRecalc(false);
removeDeco();
createTemporaryPlan();