mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

Core: remove dive->downloaded flag

Browse source 
This flag had two distinct uses:
 - signal that dives were downloaded, not imported
 - use to mark imported dives

Both are not used anymore, therefore remove the flag.
The uemis downloaded misused the flag to mark deleted
dives. Instead misuse the "hidden_by_filter" flag.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger 2018-10-03 22:03:39 +02:00 • committed by Dirk Hohndel
parent 6dc1dcaea5
commit 90b019bc8d
5 changed files with 5 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
core/cochran.c
View file

@ -785,7 +785,6 @@ static void cochran_parse_dive(const unsigned char *decode, unsigned mod,
dc->duration.seconds = duration;
}
dive->downloaded = true;
record_dive_to_table(dive, table);
mark_divelist_changed(true);

1
core/dive.h
View file

@ -309,7 +309,6 @@ struct dive {
struct dive *next, **pprev;
bool selected;
bool hidden_by_filter;
bool downloaded;
timestamp_t when;
uint32_t dive_site_uuid;
char *notes;

1
core/libdivecomputer.c
View file

@ -861,7 +861,6 @@ static int dive_cb(const unsigned char *data, unsigned int size,
add_dive_to_trip(dive, devdata->trip);
}
dive->downloaded = true;
record_dive_to_table(dive, devdata->download_table);
mark_divelist_changed(true);
return true;

1
core/liquivision.c
View file

@ -427,7 +427,6 @@ static void parse_dives (int log_version, const unsigned char *buf, unsigned int
}
// End dive
dive->downloaded = true;
record_dive_to_table(dive, table);
dive = NULL;
mark_divelist_changed(true);

11
core/uemis-downloader.c
View file

@ -197,7 +197,6 @@ static void uemis_get_weight(char *buffer, weightsystem_t *weight, int diveid)
static struct dive *uemis_start_dive(uint32_t deviceid)
{
struct dive *dive = alloc_dive();
dive->downloaded = true;
dive->dc.model = strdup("Uemis Zurich");
dive->dc.deviceid = deviceid;
return dive;
@ -1147,12 +1146,12 @@ static int get_memory(struct dive_table *td, int checkpoint)
return UEMIS_MEM_OK;
}
/* mark a dive as deleted by setting download to false
* this will be picked up by some cleaning statement later */
/* we misuse the hidden_by_filter flag to mark a dive as deleted.
* this will be picked up by some cleaning statement later. */
static void do_delete_dives(struct dive_table *td, int idx)
{
for (int x = idx; x < td->nr; x++)
td->dives[x]->downloaded = false;
td->dives[x]->hidden_by_filter = true;
}
static bool load_uemis_divespot(const char *mountpath, int divespot_id)
@ -1265,7 +1264,7 @@ static bool get_matching_dive(int idx, char *newmax, int *uemis_mem_status, devi
#endif
deleted_files++;
/* mark this log entry as deleted and cleanup later, otherwise we mess up our array */
dive->downloaded = false;
dive->hidden_by_filter = true;
#if UEMIS_DEBUG & 2
fprintf(debugfile, "Deleted dive from %s, with id %d from table -- newmax is %s\n", d_time, dive->dc.diveid, newmax);
#endif
@ -1471,7 +1470,7 @@ const char *do_uemis_import(device_data_t *data)
* to see if we have to clean some dead bodies from our download table */
next_table_index = 0;
while (next_table_index < data->download_table->nr) {
if (!data->download_table->dives[next_table_index]->downloaded)
if (data->download_table->dives[next_table_index]->hidden_by_filter)
uemis_delete_dive(data, data->download_table->dives[next_table_index]->dc.diveid);
else
next_table_index++;