From 56c206d19fba315a633404e54c16891db49a97ac Mon Sep 17 00:00:00 2001 From: Linus Torvalds Date: Sun, 30 Jul 2017 21:05:48 -0700 Subject: For more manual gas pressure details So the manual gas pressure case keeps showing issues, and in many ways it really is a fairly complex thing, since it needs interpolation of the intermediate pressures - possibly over several gas changes. So you might have beginning and ending pressures for one cylinder, but then use another cylinder in between. We've historically got all the code to do this, but the big rewrite for multiple cylinder pressures didn't get all the details right, and so here's a few more fixes for the case that was shown by a dive by Robert Helling. Hopefully we're approaching the old code situation, except now with concurrent gas pressure handling support. Reported-by: Robert Helling Signed-off-by: Linus Torvalds Signed-off-by: Dirk Hohndel --- core/gaspressures.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'core/gaspressures.c') diff --git a/core/gaspressures.c b/core/gaspressures.c index 13196b61f..ce88df047 100644 --- a/core/gaspressures.c +++ b/core/gaspressures.c @@ -409,16 +409,6 @@ void populate_pressure_information(struct dive *dive, struct divecomputer *dc, s current->end = pressure; } - // If we have no pressure information, we will need to - // continue with or without a tracking entry. Mark any - // existing tracking entry as non-dense, and remember - // to fill in interpolated data. - if (!pressure) { - missing_pr = 1; - dense = 0; - continue; - } - // We have a final pressure for 'current' // If a gas switch has occurred, finish the // current pressure track entry and continue @@ -429,6 +419,16 @@ void populate_pressure_information(struct dive *dive, struct divecomputer *dc, s continue; } + // If we have no pressure information, we will need to + // continue with or without a tracking entry. Mark any + // existing tracking entry as non-dense, and remember + // to fill in interpolated data. + if (current && !pressure) { + missing_pr = 1; + dense = 0; + continue; + } + // If we already have a pressure tracking entry, and // it has not had any missing samples, just continue // using it - there's nothing to interpolate yet. -- cgit v1.2.3-70-g09d2