diff options
-rw-r--r-- | core/divelist.c | 11 | ||||
-rw-r--r-- | core/gpslocation.cpp | 8 | ||||
-rw-r--r-- | desktop-widgets/subsurfacewebservices.cpp | 9 | ||||
-rw-r--r-- | desktop-widgets/tab-widgets/TabDiveInformation.cpp | 3 | ||||
-rw-r--r-- | qt-models/diveplannermodel.cpp | 2 | ||||
-rw-r--r-- | qt-models/divetripmodel.cpp | 5 |
6 files changed, 20 insertions, 18 deletions
diff --git a/core/divelist.c b/core/divelist.c index 60b398539..d079b11bf 100644 --- a/core/divelist.c +++ b/core/divelist.c @@ -229,7 +229,7 @@ static int calculate_cns(struct dive *dive) if (divenr) { prev_dive = get_dive(divenr - 1); if (prev_dive) { - endtime = prev_dive->when + prev_dive->duration.seconds; + endtime = dive_endtime(prev_dive); if (dive->when < (endtime + 3600 * 12)) { cns = calculate_cns(prev_dive); cns = cns * 1 / pow(2, (dive->when - endtime) / (90.0 * 60.0)); @@ -395,13 +395,12 @@ unsigned int init_decompression(struct dive *dive) * for how far back we need to go */ if (dive->divetrip && pdive->divetrip != dive->divetrip) continue; - if (!pdive || pdive->when >= when || pdive->when + pdive->duration.seconds + 48 * 60 * 60 < when) + if (!pdive || pdive->when >= when || dive_endtime(pdive) + 48 * 60 * 60 < when) break; /* For simultaneous dives, only consider the first */ if (pdive->when == laststart) continue; - when = pdive->when; - lasttime = when + pdive->duration.seconds; + lasttime = dive_endtime(pdive); } while (++i < (divenr >= 0 ? divenr : dive_table.nr)) { struct dive *pdive = get_dive(i); @@ -421,7 +420,7 @@ unsigned int init_decompression(struct dive *dive) } if (pdive->when > lasttime) { surface_time = pdive->when - lasttime; - lasttime = pdive->when + pdive->duration.seconds; + lasttime = dive_endtime(pdive); add_segment(surface_pressure, &air, surface_time, 0, dive, prefs.decosac); #if DECO_CALC_DEBUG & 2 printf("after surface intervall of %d:%02u\n", FRACTION(surface_time, 60)); @@ -1136,7 +1135,7 @@ void process_dives(bool is_imported, bool prefer_imported) /* only try to merge overlapping dives - or if one of the dives has * zero duration (that might be a gps marker from the webservice) */ if (prev->duration.seconds && dive->duration.seconds && - prev->when + prev->duration.seconds < dive->when) + dive_endtime(prev) < dive->when) continue; merged = try_to_merge(prev, dive, prefer_imported); diff --git a/core/gpslocation.cpp b/core/gpslocation.cpp index a823a9124..22525ca3d 100644 --- a/core/gpslocation.cpp +++ b/core/gpslocation.cpp @@ -275,7 +275,7 @@ bool GpsLocation::applyLocations() qDebug() << "processing gpsFix @" << get_dive_date_string(gpsTable[j].when) << "which is withing six hours of dive from" << get_dive_date_string(d->when) << "until" << - get_dive_date_string(d->when + d->duration.seconds); + get_dive_date_string(dive_endtime(d)); /* * If position is fixed during dive. This is the good one. * Asign and mark position, and end gps_location loop @@ -310,7 +310,7 @@ bool GpsLocation::applyLocations() if (verbose) qDebug() << "which is closer to the start of the dive, do continue with that"; continue; - } else if (gpsTable[j].when > d->when + d->duration.seconds) { + } else if (gpsTable[j].when > dive_endtime(d)) { if (verbose) qDebug() << "which is even later after the end of the dive, so pick the previous one"; copy_gps_location(gpsTable[j], d); @@ -319,7 +319,7 @@ bool GpsLocation::applyLocations() break; } else { /* ok, gpsFix is before, nextgpsFix is after */ - if (d->when - gpsTable[j].when <= gpsTable[j+1].when - (d->when + d->duration.seconds)) { + if (d->when - gpsTable[j].when <= gpsTable[j+1].when - dive_endtime(d)) { if (verbose) qDebug() << "pick the one before as it's closer to the start"; copy_gps_location(gpsTable[j], d); @@ -351,7 +351,7 @@ bool GpsLocation::applyLocations() /* If position is out of SAME_GROUP range and in the future, mark position for * next dive iteration and end the gps_location loop */ - if (gpsTable[j].when >= d->when + d->duration.seconds + SAME_GROUP) { + if (gpsTable[j].when >= dive_endtime(d) + SAME_GROUP) { last = j; break; } diff --git a/desktop-widgets/subsurfacewebservices.cpp b/desktop-widgets/subsurfacewebservices.cpp index 371a8475a..6c982d27f 100644 --- a/desktop-widgets/subsurfacewebservices.cpp +++ b/desktop-widgets/subsurfacewebservices.cpp @@ -12,6 +12,7 @@ #include "core/subsurface-qt/SettingsObjectWrapper.h" #include <errno.h> #include "core/cloudstorage.h" +#include "core/dive.h" #include <QDir> #include <QHttpMultiPart> @@ -70,7 +71,7 @@ static bool merge_locations_into_dives(void) qDebug() << "processing gpsfix @" << get_dive_date_string(gpsfix->when) << "which is withing six hours of dive from" << get_dive_date_string(dive->when) << "until" << - get_dive_date_string(dive->when + dive->duration.seconds); + get_dive_date_string(dive_endtime(dive)); /* * If position is fixed during dive. This is the good one. * Asign and mark position, and end gps_location loop @@ -106,7 +107,7 @@ static bool merge_locations_into_dives(void) if (verbose) qDebug() << "which is closer to the start of the dive, do continue with that"; continue; - } else if (gpsfix->when > dive->when + dive->duration.seconds) { + } else if (gpsfix->when > dive_endtime(dive)) { if (verbose) qDebug() << "which is even later after the end of the dive, so pick the previous one"; copy_gps_location(gpsfix, dive); @@ -115,7 +116,7 @@ static bool merge_locations_into_dives(void) break; } else { /* ok, gpsfix is before, nextgpsfix is after */ - if (dive->when - gpsfix->when <= nextgpsfix->when - (dive->when + dive->duration.seconds)) { + if (dive->when - gpsfix->when <= nextgpsfix->when - dive_endtime(dive)) { if (verbose) qDebug() << "pick the one before as it's closer to the start"; copy_gps_location(gpsfix, dive); @@ -147,7 +148,7 @@ static bool merge_locations_into_dives(void) /* If position is out of SAME_GROUP range and in the future, mark position for * next dive iteration and end the gps_location loop */ - if (gpsfix->when >= dive->when + dive->duration.seconds + SAME_GROUP) { + if (gpsfix->when >= dive_endtime(dive) + SAME_GROUP) { tracer = j; break; } diff --git a/desktop-widgets/tab-widgets/TabDiveInformation.cpp b/desktop-widgets/tab-widgets/TabDiveInformation.cpp index 419e73739..ae3998ed8 100644 --- a/desktop-widgets/tab-widgets/TabDiveInformation.cpp +++ b/desktop-widgets/tab-widgets/TabDiveInformation.cpp @@ -6,6 +6,7 @@ #include <core/helpers.h> #include <core/statistics.h> #include <core/display.h> +#include <core/dive.h> TabDiveInformation::TabDiveInformation(QWidget *parent) : TabBase(parent), ui(new Ui::TabDiveInformation()) { @@ -81,7 +82,7 @@ void TabDiveInformation::updateData() process_all_dives(&displayed_dive, &prevd); if (prevd) - ui->surfaceIntervalText->setText(get_dive_surfint_string(displayed_dive.when - (prevd->when + prevd->duration.seconds), tr("d"), tr("h"), tr("min"))); + ui->surfaceIntervalText->setText(get_dive_surfint_string(displayed_dive.when - (dive_endtime(prevd)), tr("d"), tr("h"), tr("min"))); else ui->surfaceIntervalText->clear(); diff --git a/qt-models/diveplannermodel.cpp b/qt-models/diveplannermodel.cpp index ca84fd597..4c60e6736 100644 --- a/qt-models/diveplannermodel.cpp +++ b/qt-models/diveplannermodel.cpp @@ -58,7 +58,7 @@ void DivePlannerPointsModel::setupStartTime() startTime = QDateTime::currentDateTimeUtc().addSecs(3600 + gettimezoneoffset()); if (dive_table.nr) { struct dive *d = get_dive(dive_table.nr - 1); - time_t ends = d->when + d->duration.seconds; + time_t ends = dive_endtime(d); time_t diff = ends - startTime.toTime_t(); if (diff > 0) { startTime = startTime.addSecs(diff + 3600); diff --git a/qt-models/divetripmodel.cpp b/qt-models/divetripmodel.cpp index 0293f6a45..5c6dd30ff 100644 --- a/qt-models/divetripmodel.cpp +++ b/qt-models/divetripmodel.cpp @@ -4,6 +4,7 @@ #include "core/metrics.h" #include "core/divelist.h" #include "core/helpers.h" +#include "core/dive.h" #include <QIcon> static int nitrox_sort_value(struct dive *dive) @@ -325,12 +326,12 @@ QString DiveItem::displayDepthWithUnit() const int DiveItem::countPhotos(dive *dive) const { // Determine whether dive has pictures, and whether they were taken during or before/after dive. const int bufperiod = 120; // A 2-min buffer period. Photos within 2 min of dive are assumed as - int diveDuration = dive->duration.seconds; // taken during the dive, not before/after. + int diveTotaltime = dive_endtime(dive) - dive->when; // taken during the dive, not before/after. int pic_offset, icon_index = 0; FOR_EACH_PICTURE (dive) { // Step through each of the pictures for this dive: if (!picture) break; // if there are no pictures for this dive, return 0 pic_offset = picture->offset.seconds; - if ((pic_offset < -bufperiod) | (pic_offset > diveDuration+bufperiod)) { + if ((pic_offset < -bufperiod) | (pic_offset > diveTotaltime+bufperiod)) { icon_index |= 0x02; // If picture is before/after the dive } // then set the appropriate bit ... else { |