aboutsummaryrefslogtreecommitdiffstats
path: root/profile.c
diff options
context:
space:
mode:
authorGravatar Linus Torvalds <torvalds@linux-foundation.org>2014-04-26 10:55:17 -0700
committerGravatar Dirk Hohndel <dirk@hohndel.org>2014-04-26 17:24:12 -0700
commit0c5ec1d6dbf386e063a11717c7bfdbec5c57fcf0 (patch)
tree48952fba2899178633cb15ae3fa92f7b8ea1f73d /profile.c
parentcd98e4f6dbe0a9d8731bfd5c7bf506ae73f07877 (diff)
downloadsubsurface-0c5ec1d6dbf386e063a11717c7bfdbec5c57fcf0.tar.gz
Don't calculate SAC-rates for negative pressure changes
They happen - maybe the cylinder actually warmed up, or maybe the user entered just a ending pressure without a starting pressure. Regardless, just ignore cylinder pressure changes that go up. Also ignore cylinders with a zero ending pressure: that's really a *missing* pressure rather than an actual zero pressure. As Dirk says, the scuba regulators don't even work without a healthy positive pressure differential, so even when you breathe down a tank to "empty", it won't be at zero pressure (this is true even with gauge pressure, where zero means "atmospheric pressure"). 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.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/profile.c b/profile.c
index 4d5e05aa4..f5d287bfd 100644
--- a/profile.c
+++ b/profile.c
@@ -157,7 +157,7 @@ static int get_local_sac(struct plot_data *entry1, struct plot_data *entry2, str
return 0;
a.mbar = GET_PRESSURE(entry1);
b.mbar = GET_PRESSURE(entry2);
- if (!a.mbar || !b.mbar)
+ if (!b.mbar || a.mbar <= b.mbar)
return 0;
/* Mean pressure in ATM */