summaryrefslogtreecommitdiffstats
path: root/profile.c
diff options
context:
space:
mode:
authorGravatar Dirk Hohndel <dirk@hohndel.org>2012-11-11 17:56:32 +0100
committerGravatar Dirk Hohndel <dirk@hohndel.org>2012-11-11 17:56:32 +0100
commitec042222f0827c7394458a4a0aa4dcbce0c390fb (patch)
treea1398046278b8c91e20c96c7f7e01158700b8976 /profile.c
parent7fc1213aab23d50395c322d4f590f5cf18d43d3b (diff)
downloadsubsurface-ec042222f0827c7394458a4a0aa4dcbce0c390fb.tar.gz
Handle dives without samples correctly
The code in commit f99e1b476b18 "Trim the dive to exclude surface time at beginning and end" failed rather badly if a dive has no samples at all - which is true for many of our test dives. This makes sure that we don't exclude data points if we never set up start and end times. Reported-by: Lubomir I. Ivanov <neolit123@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'profile.c')
-rw-r--r--profile.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/profile.c b/profile.c
index d4369d97f..a8bda4672 100644
--- a/profile.c
+++ b/profile.c
@@ -1706,7 +1706,8 @@ static struct plot_info *create_plot_info(struct dive *dive, int nr_samples, str
int delay = 0;
struct sample *sample = dive_sample+i;
- if (sample->time.seconds < dive->start || sample->time.seconds > dive->end) {
+ if ((dive->start > -1 && sample->time.seconds < dive->start) ||
+ (dive->end > -1 && sample->time.seconds > dive->end)) {
pi_idx--;
continue;
}