diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2021-08-15 00:09:00 -0700 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2021-08-15 09:31:17 -0700 |
| commit | 75fdb8676bdc24a232351c207ea4cdd2e9caeb22 (patch) | |
| tree | 38d8bb26484b7fca2df952845d07693374d9ace1 /core | |
| parent | 67769235e78e7963a093bcf84e3b27ad1a0b8d6c (diff) | |
| download | subsurface-75fdb8676bdc24a232351c207ea4cdd2e9caeb22.tar.gz | |
core: add downloaded GPS to existing dive site
If we download a first dive computer and add a dive site to the dive (by
setting a location name for example), and then download from another
dive computer that provides us with GPS data, we should keep the
existing dive site information, but add the GPS data from the freshly
downloaded dive computer.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'core')
| -rw-r--r-- | core/dive.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/core/dive.c b/core/dive.c index 571f244df..a7d202ba5 100644 --- a/core/dive.c +++ b/core/dive.c @@ -2639,6 +2639,12 @@ struct dive *merge_dives(const struct dive *a, const struct dive *b, int offset, /* we take the first dive site, unless it's empty */ *site = a->dive_site && !dive_site_is_empty(a->dive_site) ? a->dive_site : b->dive_site; + if (!dive_site_has_gps_location(*site) && dive_site_has_gps_location(b->dive_site)) { + /* we picked the first dive site and that didn't have GPS data, but the new dive has + * GPS data (that could be a download from a GPS enabled dive computer). + * Keep the dive site, but add the GPS data */ + (*site)->location = b->dive_site->location; + } fixup_dive(res); free(cylinders_map_a); free(cylinders_map_b); |