planner: split addStop() into external and internal versions

The DivePlannerPointsModel::addStop() function is called by
the profile to add a planner-stop. It is also used internally
to create profiles.

If we ever want to include this in the undo system, we have
to split these into to versions. One will ultimately place
an undo command and update the profile, the other one doesn't.

For now, this makes the external interface simpler, as some
parameters are redundant.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

1 files changed, 2 insertions, 1 deletions

diff --git a/qt-models/diveplannermodel.h b/qt-models/diveplannermodel.h
index 17e1fb75e..0b13f7247 100644
--- a/qt-models/diveplannermodel.h
+++ b/qt-models/diveplannermodel.h
@@ -61,7 +61,7 @@ public:
 	struct deco_state final_deco_state;
 
 	void loadFromDive(dive *d);
-	int addStop(int millimeters, int seconds, int cylinderid_in, int ccpoint, bool entered, enum divemode_t);
+	void addStop(int millimeters, int seconds);
 public
 slots:
 	void addDefaultStop();
@@ -117,6 +117,7 @@ signals:
 private:
 	explicit DivePlannerPointsModel(QObject *parent = 0);
 	void clear();
+	int addStop(int millimeters, int seconds, int cylinderid_in, int ccpoint, bool entered, enum divemode_t);
 	void setupStartTime();
 	void setupCylinders();
 	int lastEnteredPoint() const;