[PATCH] Copy all cylinders when (re)planning a dive

Anton Lundin glance at acc.umu.se
Thu Dec 4 11:56:18 PST 2014


This changes the code so we include all the cylinders from a previous
dive when we re-plain a dive.

Otherwise we would have a hole in the cylinders table when we plan a
dive based on a dive where we have unused cylinders, and it makes more
sense to copy all of them rather than not.

Signed-off-by: Anton Lundin <glance at acc.umu.se>
---
 qt-ui/diveplanner.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/qt-ui/diveplanner.cpp b/qt-ui/diveplanner.cpp
index 84dec0c..fabc11e 100644
--- a/qt-ui/diveplanner.cpp
+++ b/qt-ui/diveplanner.cpp
@@ -138,7 +138,7 @@ void DivePlannerPointsModel::setupCylinders()
 	if (mode == PLAN && current_dive) {
 		// take the used cylinders from the selected dive as starting point
 		CylindersModel::instance()->copyFromDive(current_dive);
-		copy_cylinders(current_dive, &displayed_dive, true);
+		copy_cylinders(current_dive, &displayed_dive, false);
 		reset_cylinders(&displayed_dive, true);
 		return;
 	}
-- 
2.1.0



More information about the subsurface mailing list