mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2024-12-12 03:51:29 +00:00
Code Issues Projects Releases Packages Wiki Activity

uemis: unglobalize mindiveid

Browse source 
uemis_get_divenr() returns maxdiveid and passes mindiveid as a
global variable.

Make this more reasonable by returning a min, max pair.

The way mindiveid is an unsigned int and then reinterpreted as
int is very sketchy. This commit attempts to not change that
behavior.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger 2024-05-01 17:08:56 +02:00
parent f4d2159694
commit f907f97513
1 changed files with 12 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
core/uemis-downloader.cpp
View file

@ -83,7 +83,6 @@ static int number_of_files;
static int max_mem_used = -1; static int max_mem_used = -1;
static int dive_to_read = 0; static int dive_to_read = 0;
static uint32_t mindiveid;
/* Linked list to remember already executed divespot download requests */ /* Linked list to remember already executed divespot download requests */
struct divespot_mapping { struct divespot_mapping {
@ -1004,11 +1003,12 @@ static bool process_raw_buffer(device_data_t *devdata, uint32_t deviceid, std::s
return true; return true;
} }
static int uemis_get_divenr(uint32_t deviceid, struct dive_table *table, int force) // Returns (mindiveid, maxdiveid)
static std::pair<uint32_t, uint32_t> uemis_get_divenr(uint32_t deviceid, struct dive_table *table, int force)
{ {
uint32_t maxdiveid = 0; uint32_t maxdiveid = 0;
uint32_t mindiveid = 0xFFFFFFFF;
int i; int i;
mindiveid = 0xFFFFFFFF;
/* /*
* If we are are retrying after a disconnect/reconnect, we * If we are are retrying after a disconnect/reconnect, we
@ -1038,7 +1038,7 @@ static int uemis_get_divenr(uint32_t deviceid, struct dive_table *table, int for
} }
} }
} }
return maxdiveid; return std::make_pair(mindiveid, maxdiveid);
} }
#if UEMIS_DEBUG #if UEMIS_DEBUG
@ -1326,12 +1326,15 @@ std::string do_uemis_import(device_data_t *data)
goto bail; goto bail;
param_buff[1] = "notempty"; param_buff[1] = "notempty";
newmax = uemis_get_divenr(deviceidnr, data->log->dives, force_download); {
auto [mindiveid, maxdiveid] = uemis_get_divenr(deviceidnr, data->log->dives, force_download);
newmax = maxdiveid;
if (verbose) if (verbose)
report_info("Uemis downloader: start looking at dive nr %d", newmax); report_info("Uemis downloader: start looking at dive nr %d", newmax);
first = start = newmax; first = start = newmax;
dive_to_read = (int)mindiveid < first ? first - mindiveid : first; dive_to_read = (int)mindiveid < first ? first - mindiveid : first;
}
if (dive_offset > 0) if (dive_offset > 0)
start += dive_offset; start += dive_offset;
for (;;) { for (;;) {