diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2014-07-18 18:37:28 -0700 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2014-07-18 18:46:51 -0700 |
| commit | 17bcd1bc6085013877888792754f52ce6b375aea (patch) | |
| tree | bb4005fc606644336fd03d436fd1d5f7215e4928 /divelist.c | |
| parent | 359613210bf03d47fec5d19420f537aa5128f5cc (diff) | |
| download | subsurface-17bcd1bc6085013877888792754f52ce6b375aea.tar.gz | |
get_gas_at_time needs to always give us a valid gasv4.1.91
Before this function was changed it was really supposed to just change a
gas that was passed in in case there was an event that changed the mix -
but with the new name the caller will assume that they get a valid gasmix.
And promptly we had one caller that didn't initialize gas to be based on
the first cylinder before calling get_gas_at_time().
Instead of adding yet one more spot that knows about the oddity of the old
API I simply changed get_gas_at_time() to do what it name appears to imply
and fixed the other callers not to bother to initialize the gasmix.
Fixes #647
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'divelist.c')
| -rw-r--r-- | divelist.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/divelist.c b/divelist.c index 2abeac9ca..1e279efca 100644 --- a/divelist.c +++ b/divelist.c @@ -153,7 +153,7 @@ int total_weight(struct dive *dive) static int active_o2(struct dive *dive, struct divecomputer *dc, duration_t time) { - struct gasmix gas = dive->cylinder[0].gasmix; + struct gasmix gas; get_gas_at_time(dive, dc, time, &gas); return get_o2(&gas); } |