Don't compare signed and unsigned values

This fixes a "reproducable hang when using "edit dive in planner" on
non-planned dive".
Comparing unsigned and signed integers isn't a great idea.

Fixes #880

Signed-off-by: Jan Darowski <jan.darowski@gmail.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>

1 files changed, 1 insertions, 1 deletions

diff --git a/planner.c b/planner.c
index ad855e619..c8578c320 100644
--- a/planner.c
+++ b/planner.c
@@ -1028,7 +1028,7 @@ bool plan(struct diveplan *diveplan, char **cached_datap, bool is_planner, bool
 						       TIMESTEP, po2, &displayed_dive, prefs.decosac);
 			clock += TIMESTEP;
 			depth -= deltad;
-		} while (depth > stoplevels[stopidx]);
+		} while (depth > 0 && depth > stoplevels[stopidx]);
 
 		if (depth <= 0)
 			break; /* We are at the surface */