diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2013-01-23 12:33:26 -0800 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2013-01-23 12:55:39 -0800 |
commit | 413b9026dd01e81cf804db27f8b0c06c48e833e2 (patch) | |
tree | fd1a82aa57a0f2b376399a682071a30a54c99415 /profile.c | |
parent | aac44f9b0740726dd9d8cc9cce80b2e99626d0e1 (diff) | |
download | subsurface-413b9026dd01e81cf804db27f8b0c06c48e833e2.tar.gz |
Fix temperature rounding issues
Temperatures can actually be negative, which means that rounding by
adding 0.5 and casting to 'int' is not correct.
We could use '(int)(rint(val))' instead, but the only place we care
about might as well just print out the floating point representation
with a precision of two digits instead. So if you have a dive computer
that gives you the precision, you might see '3.5˚C' as the temperature.
Remove the helper functions that nobody uses and that get the rounding
wrong anyway.
Reported-by: Henrik Brautaset Aronsen <subsurface@henrik.synth.no>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'profile.c')
-rw-r--r-- | profile.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -894,7 +894,7 @@ static void plot_single_temp_text(struct graphics_context *gc, int sec, int mkel deg = get_temp_units(mkelvin, &unit); - plot_text(gc, &tro, sec, mkelvin, "%d%s", (int)(deg + 0.5), unit); + plot_text(gc, &tro, sec, mkelvin, "%.2g%s", deg, unit); } static void plot_temperature_text(struct graphics_context *gc, struct plot_info *pi) |