Dive sites: don't delete unused dive sites on save

Unused dive sites were deleted on save. This clashed with the undo
system in the following scenario:

1) Delete single-use dive site.
2) Save (dive site deleted)
3) Undo (reference to freed dive site)

Therefore, as a quick-fix, keep the referenced dive site around.
Note that this also means that empty dive sites must not be
deleted, as it might refer to a dive in the undo system. Instead
only clear references to empty dive sites in the global dive
table. Factor this functionality out, as it was common to the
XML and git savers.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

1 files changed, 1 insertions, 0 deletions

diff --git a/core/divesite.h b/core/divesite.h
index 3ce87128a..1cb0cbeff 100644
--- a/core/divesite.h
+++ b/core/divesite.h
@@ -73,6 +73,7 @@ void clear_dive_site(struct dive_site *ds);
 unsigned int get_distance(const location_t *loc1, const location_t *loc2);
 struct dive_site *find_or_create_dive_site_with_name(const char *name, timestamp_t divetime);
 void merge_dive_sites(struct dive_site *ref, struct dive_site *dive_sites[], int count);
+void purge_empty_dive_sites();
 
 #ifdef __cplusplus
 }