[PATCH] planner: Correct pO2 comparison

Anton Lundin glance at acc.umu.se
Fri Jul 11 15:25:21 PDT 2014


Don't compare pO2 with a bool, compare it with the result of the ?:
expression instead.

Signed-off-by: Anton Lundin <glance at acc.umu.se>
---
 planner.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/planner.c b/planner.c
index 4620301..630a554 100644
--- a/planner.c
+++ b/planner.c
@@ -674,7 +674,7 @@ static void add_plan_to_notes(struct diveplan *diveplan, struct dive *dive, bool
 		if (dp->time != 0) {
 			int pO2 = depth_to_atm(dp->depth, dive) * get_o2(&dp->gasmix);
 
-			if (pO2 > dp->entered ? prefs.bottompo2 : prefs.decopo2) {
+			if (pO2 > (dp->entered ? prefs.bottompo2 : prefs.decopo2)) {
 				const char *depth_unit;
 				int decimals;
 				double depth_value = get_depth_units(dp->depth, &decimals, &depth_unit);
-- 
1.9.1



More information about the subsurface mailing list