Planner: bring sanity to the SAC rate handling

The old implementation was... let's call it creative.
This tries to actually get things right instead of using magic.
Don't pretend that double values are ints.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>

1 files changed, 3 insertions, 6 deletions

diff --git a/dive.c b/dive.c
index 38599b104..22cf3ad46 100644
--- a/dive.c
+++ b/dive.c
@@ -188,15 +188,12 @@ double get_volume_units(unsigned int ml, int *frac, const char **units)
 	return vol;
 }
 
-int units_to_sac(int volume)
+int units_to_sac(double volume)
 {
 	if(get_units()->volume == CUFT)
-		if (volume < 10)
-			return cuft_to_l(volume) * 100;
-		else
-			return cuft_to_l(volume) * 10;
+		return rint(cuft_to_l(volume) * 1000.0);
 	else
-		return volume * 1000;
+		return rint(volume * 1000);
 }
 
 unsigned int units_to_depth(double depth)