summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rw-r--r--core/datatrak.c1
-rw-r--r--core/device.cpp142
-rw-r--r--core/device.h14
-rw-r--r--core/divecomputer.c3
-rw-r--r--core/divecomputer.h1
-rw-r--r--core/divelist.c18
-rw-r--r--core/libdivecomputer.c17
-rw-r--r--core/load-git.c19
-rw-r--r--core/parse-xml.c5
-rw-r--r--core/parse.c6
-rw-r--r--core/save-git.c10
-rw-r--r--core/save-xml.c12
-rw-r--r--dives/DL7.xml6
-rw-r--r--dives/TestDiveDM4.xml1
-rw-r--r--dives/TestDiveDM5.xml1
-rw-r--r--dives/TestDiveSeabearHUDC.xml2
-rw-r--r--dives/TestDiveSeabearNewFormat.xml18
-rw-r--r--dives/mergedVyperOstc.xml10
-rw-r--r--dives/test40-42.xml12
-rw-r--r--smtk-import/smartrak.c1
20 files changed, 93 insertions, 206 deletions
diff --git a/core/datatrak.c b/core/datatrak.c
index 2fa89010e..c19c243a5 100644
--- a/core/datatrak.c
+++ b/core/datatrak.c
@@ -566,7 +566,6 @@ static unsigned char *dt_dive_parser(unsigned char *runner, struct dive *dt_dive
dt_dive->dc.deviceid = 0;
else
dt_dive->dc.deviceid = 0xffffffff;
- create_device_node(devices, dt_dive->dc.model, dt_dive->dc.deviceid, "", "", dt_dive->dc.model);
dt_dive->dc.next = NULL;
if (!is_SCR && dt_dive->cylinders.nr > 0) {
get_cylinder(dt_dive, 0)->end.mbar = get_cylinder(dt_dive, 0)->start.mbar -
diff --git a/core/device.cpp b/core/device.cpp
index c251144a2..42c2be004 100644
--- a/core/device.cpp
+++ b/core/device.cpp
@@ -13,31 +13,33 @@ struct device_table device_table;
bool device::operator==(const device &a) const
{
return model == a.model &&
- deviceId == a.deviceId &&
- firmware == a.firmware &&
- serialNumber == a.serialNumber &&
- nickName == a.nickName;
+ serialNumber == a.serialNumber;
}
static bool same_device(const device &dev1, const device &dev2)
{
- return dev1.deviceId == dev2.deviceId && strcoll(dev1.model.c_str(), dev2.model.c_str()) == 0;
+ return strcmp(dev1.model.c_str(), dev2.model.c_str()) == 0 &&
+ strcmp(dev1.serialNumber.c_str(), dev2.serialNumber.c_str()) == 0;
}
bool device::operator<(const device &a) const
{
- if (deviceId != a.deviceId)
- return deviceId < a.deviceId;
+ int diff;
- // Use strcoll to compare model-strings, since these might be unicode
- // and therefore locale dependent? Let's hope that not, but who knows?
- return strcoll(model.c_str(), a.model.c_str()) < 0;
+ diff = strcmp(model.c_str(), a.model.c_str());
+ if (diff)
+ return diff < 0;
+
+ return strcmp(serialNumber.c_str(), a.serialNumber.c_str()) < 0;
}
extern "C" const struct device *get_device_for_dc(const struct device_table *table, const struct divecomputer *dc)
{
+ if (!dc->model || !dc->serial)
+ return NULL;
+
const std::vector<device> &dcs = table->devices;
- device dev { dc->model ?: "", dc->deviceid, {}, {}, {} };
+ device dev { dc->model, dc->serial };
auto it = std::lower_bound(dcs.begin(), dcs.end(), dev);
return it != dcs.end() && same_device(*it, dev) ? &*it : NULL;
}
@@ -48,79 +50,51 @@ extern "C" bool device_exists(const struct device_table *device_table, const str
return it != device_table->devices.end() && same_device(*it, *dev);
}
-/*
- * When setting the device ID, we also fill in the
- * serial number and firmware version data
- */
-extern "C" void set_dc_deviceid(struct divecomputer *dc, unsigned int deviceid, const struct device_table *device_table)
+void device::showchanges(const std::string &n) const
{
- if (!deviceid)
- return;
-
- dc->deviceid = deviceid;
-
- // Serial and firmware can only be deduced if we know the model
- if (empty_string(dc->model))
- return;
-
- const device *node = get_device_for_dc(device_table, dc);
- if (!node)
- return;
-
- if (!node->serialNumber.empty() && empty_string(dc->serial)) {
- free((void *)dc->serial);
- dc->serial = strdup(node->serialNumber.c_str());
- }
- if (!node->firmware.empty() && empty_string(dc->fw_version)) {
- free((void *)dc->fw_version);
- dc->fw_version = strdup(node->firmware.c_str());
+ if (nickName != n) {
+ if (!n.empty())
+ qDebug("new nickname %s for DC model %s serial %s", n.c_str(), model.c_str(), serialNumber.c_str());
+ else
+ qDebug("deleted nickname %s for DC model %s serial %s", nickName.c_str(), model.c_str(), serialNumber.c_str());
}
}
-void device::showchanges(const std::string &n, const std::string &s, const std::string &f) const
+static int addDC(std::vector<device> &dcs, const std::string &m, const std::string &s, const std::string &n)
{
- if (nickName != n && !n.empty())
- qDebug("new nickname %s for DC model %s deviceId 0x%x", n.c_str(), model.c_str(), deviceId);
- if (serialNumber != s && !s.empty())
- qDebug("new serial number %s for DC model %s deviceId 0x%x", s.c_str(), model.c_str(), deviceId);
- if (firmware != f && !f.empty())
- qDebug("new firmware version %s for DC model %s deviceId 0x%x", f.c_str(), model.c_str(), deviceId);
-}
-
-static void addDC(std::vector<device> &dcs, const std::string &m, uint32_t d, const std::string &n, const std::string &s, const std::string &f)
-{
- if (m.empty() || d == 0)
- return;
- device dev { m, d, {}, {}, {} };
+ if (m.empty() || s.empty())
+ return -1;
+ device dev { m, s, n };
auto it = std::lower_bound(dcs.begin(), dcs.end(), dev);
if (it != dcs.end() && same_device(*it, dev)) {
// debugging: show changes
if (verbose)
- it->showchanges(n, s, f);
+ it->showchanges(n);
// Update any non-existent fields from the old entry
- if (!n.empty())
- it->nickName = n;
- if (!s.empty())
- it->serialNumber = s;
- if (!f.empty())
- it->firmware = f;
+ if (n.empty()) {
+ dcs.erase(it);
+ return -1;
+ }
+ it->nickName = n;
+ return it - dcs.begin();
} else {
- dcs.insert(it, device{m, d, s, f, n});
+ if (n.empty())
+ return -1;
+
+ dev.deviceId = calculate_string_hash(s.c_str());
+ dcs.insert(it, dev);
+ return it - dcs.begin();
}
}
-extern "C" void create_device_node(struct device_table *device_table, const char *model, uint32_t deviceid, const char *serial, const char *firmware, const char *nickname)
+extern "C" int create_device_node(struct device_table *device_table, const char *model, const char *serial, const char *nickname)
{
- addDC(device_table->devices, model ?: "", deviceid, nickname ?: "", serial ?: "", firmware ?: "");
+ return addDC(device_table->devices, model ?: "", serial ?: "", nickname ?: "");
}
-/* Does not check for duplicates! */
extern "C" int add_to_device_table(struct device_table *device_table, const struct device *dev)
{
- auto it = std::lower_bound(device_table->devices.begin(), device_table->devices.end(), *dev);
- int idx = it - device_table->devices.begin();
- device_table->devices.insert(it, *dev);
- return idx;
+ return create_device_node(device_table, dev->model.c_str(), dev->serialNumber.c_str(), dev->nickName.c_str());
}
extern "C" int remove_device(struct device_table *device_table, const struct device *dev)
@@ -165,34 +139,6 @@ extern "C" int is_default_dive_computer_device(const char *name)
return qPrefDiveComputer::device() == name;
}
-extern "C" void add_devices_of_dive(const struct dive *dive, struct device_table *device_table)
-{
- if (!dive)
- return;
-
- const struct divecomputer *dc;
-
- for_each_dc (dive, dc) {
- if (!empty_string(dc->model) && dc->deviceid &&
- !get_device_for_dc(device_table, dc)) {
- // we don't have this one, yet
- if (std::any_of(device_table->devices.begin(), device_table->devices.end(),
- [dc] (const device &dev)
- { return !strcasecmp(dev.model.c_str(), dc->model); })) {
- // we already have this model but a different deviceid
- std::string simpleNick(dc->model);
- if (dc->deviceid == 0)
- simpleNick += " (unknown deviceid)";
- else
- simpleNick += " (" + QString::number(dc->deviceid, 16).toStdString() + ")";
- addDC(device_table->devices, dc->model, dc->deviceid, simpleNick, {}, {});
- } else {
- addDC(device_table->devices, dc->model, dc->deviceid, {}, {}, {});
- }
- }
- }
-}
-
const char *get_dc_nickname(const struct divecomputer *dc)
{
const device *existNode = get_device_for_dc(&device_table, dc);
@@ -227,21 +173,11 @@ extern "C" const char *device_get_model(const struct device *dev)
return dev ? dev->model.c_str() : NULL;
}
-extern "C" const uint32_t device_get_id(const struct device *dev)
-{
- return dev ? dev->deviceId : -1;
-}
-
extern "C" const char *device_get_serial(const struct device *dev)
{
return dev ? dev->serialNumber.c_str() : NULL;
}
-extern "C" const char *device_get_firmware(const struct device *dev)
-{
- return dev ? dev->firmware.c_str() : NULL;
-}
-
extern "C" const char *device_get_nickname(const struct device *dev)
{
return dev ? dev->nickName.c_str() : NULL;
diff --git a/core/device.h b/core/device.h
index 1de436e33..6bfb85e3c 100644
--- a/core/device.h
+++ b/core/device.h
@@ -16,10 +16,7 @@ struct dive_table;
// global device table
extern struct device_table device_table;
-extern void set_dc_deviceid(struct divecomputer *dc, unsigned int deviceid, const struct device_table *table);
-
-extern void add_devices_of_dive(const struct dive *dive, struct device_table *table);
-extern void create_device_node(struct device_table *table, const char *model, uint32_t deviceid, const char *serial, const char *firmware, const char *nickname);
+extern int create_device_node(struct device_table *table, const char *model, const char *serial, const char *nickname);
extern int nr_devices(const struct device_table *table);
extern const struct device *get_device(const struct device_table *table, int i);
extern struct device *get_device_mutable(struct device_table *table, int i);
@@ -35,9 +32,7 @@ extern void remove_from_device_table(struct device_table *table, int idx);
// struct device accessors for C-code. The returned strings are not stable!
const char *device_get_model(const struct device *dev);
-const uint32_t device_get_id(const struct device *dev);
const char *device_get_serial(const struct device *dev);
-const char *device_get_firmware(const struct device *dev);
const char *device_get_nickname(const struct device *dev);
// for C code that needs to alloc/free a device table. (Let's try to get rid of those)
@@ -56,16 +51,15 @@ extern void free_device_table(struct device_table *devices);
struct device {
bool operator==(const device &a) const; // TODO: remove, once devices are integrated in the undo system
bool operator<(const device &a) const;
- void showchanges(const std::string &n, const std::string &s, const std::string &f) const;
+ void showchanges(const std::string &n) const;
std::string model;
- uint32_t deviceId;
std::string serialNumber;
- std::string firmware;
std::string nickName;
+ uint32_t deviceId; // Always the string hash of the serialNumber
};
struct device_table {
- // Keep the dive computers in a vector sorted by (model, deviceId)
+ // Keep the dive computers in a vector sorted by (model, serial)
std::vector<device> devices;
};
diff --git a/core/divecomputer.c b/core/divecomputer.c
index fa836f7ca..200094588 100644
--- a/core/divecomputer.c
+++ b/core/divecomputer.c
@@ -475,6 +475,9 @@ void add_extra_data(struct divecomputer *dc, const char *key, const char *value)
{
struct extra_data **ed = &dc->extra_data;
+ if (!strcasecmp(key, "Serial"))
+ dc->deviceid = calculate_string_hash(value);
+
while (*ed)
ed = &(*ed)->next;
*ed = malloc(sizeof(struct extra_data));
diff --git a/core/divecomputer.h b/core/divecomputer.h
index eb48e9b86..8bb571c44 100644
--- a/core/divecomputer.h
+++ b/core/divecomputer.h
@@ -68,6 +68,7 @@ extern struct event *add_event(struct divecomputer *dc, unsigned int time, int t
extern void remove_event_from_dc(struct divecomputer *dc, struct event *event);
extern void add_extra_data(struct divecomputer *dc, const char *key, const char *value);
extern bool is_dc_planner(const struct divecomputer *dc);
+extern uint32_t calculate_string_hash(const char *str);
/* Check if two dive computer entries are the exact same dive (-1=no/0=maybe/1=yes) */
extern int match_one_dc(const struct divecomputer *a, const struct divecomputer *b);
diff --git a/core/divelist.c b/core/divelist.c
index a1420e407..9beea4d7c 100644
--- a/core/divelist.c
+++ b/core/divelist.c
@@ -807,13 +807,6 @@ void delete_single_dive(int idx)
void process_loaded_dives()
{
- int i;
- struct dive *dive;
-
- /* Register dive computer nick names. */
- for_each_dive(i, dive)
- add_devices_of_dive(dive, &device_table);
-
sort_dive_table(&dive_table);
sort_trip_table(&trip_table);
@@ -1172,17 +1165,6 @@ void process_imported_dives(struct dive_table *import_table, struct trip_table *
add_to_device_table(devices_to_add, dev);
}
- /* check if we need a nickname for the divecomputer for newly downloaded dives;
- * since we know they all came from the same divecomputer we just check for the
- * first one */
- if (flags & IMPORT_IS_DOWNLOADED) {
- add_devices_of_dive(import_table->dives[0], devices_to_add);
- } else {
- /* they aren't downloaded, so record / check all new ones */
- for (i = 0; i < import_table->nr; i++)
- add_devices_of_dive(import_table->dives[i], devices_to_add);
- }
-
/* Sort the table of dives to be imported and combine mergable dives */
sort_dive_table(import_table);
merge_imported_dives(import_table);
diff --git a/core/libdivecomputer.c b/core/libdivecomputer.c
index dff44e287..d2d25306d 100644
--- a/core/libdivecomputer.c
+++ b/core/libdivecomputer.c
@@ -548,7 +548,7 @@ static uint32_t calculate_diveid(const unsigned char *fingerprint, unsigned int
return csum[0];
}
-static uint32_t calculate_string_hash(const char *str)
+uint32_t calculate_string_hash(const char *str)
{
return calculate_diveid((const unsigned char *)str, strlen(str));
}
@@ -556,10 +556,7 @@ static uint32_t calculate_string_hash(const char *str)
/*
* Set the serial number.
*
- * This also sets the device ID by looking for existing devices that
- * have that serial number.
- *
- * If no existing device ID exists, create a new by hashing the serial
+ * This also sets the device ID by hashing the serial
* number string.
*/
static void set_dc_serial(struct divecomputer *dc, const char *serial, const device_data_t *devdata)
@@ -567,14 +564,7 @@ static void set_dc_serial(struct divecomputer *dc, const char *serial, const dev
const struct device *device;
dc->serial = strdup(serial);
- if ((device = get_device_for_dc(&device_table, dc)) != NULL) // prefer already known ID over downloaded ID.
- dc->deviceid = device_get_id(device);
-
- if (!dc->deviceid && (device = get_device_for_dc(devdata->devices, dc)) != NULL)
- dc->deviceid = device_get_id(device);
-
- if (!dc->deviceid)
- dc->deviceid = calculate_string_hash(serial);
+ dc->deviceid = calculate_string_hash(serial);
}
static void parse_string_field(device_data_t *devdata, struct dive *dive, dc_field_string_t *str)
@@ -937,7 +927,6 @@ static unsigned int fixup_suunto_versions(device_data_t *devdata, const dc_event
(devinfo->firmware >> 8) & 0xff,
(devinfo->firmware >> 0) & 0xff);
}
- create_device_node(devdata->devices, devdata->model, devdata->deviceid, serial_nr, firmware, "");
return serial;
}
diff --git a/core/load-git.c b/core/load-git.c
index c44a880ae..3826099e4 100644
--- a/core/load-git.c
+++ b/core/load-git.c
@@ -725,8 +725,7 @@ static void parse_dc_deviceid(char *line, struct membuffer *str, struct git_pars
{
UNUSED(str);
int id = get_hex(line);
- set_dc_deviceid(state->active_dc, id, &device_table); // prefer already known serial/firmware over those from the loaded log
- set_dc_deviceid(state->active_dc, id, state->devices);
+ UNUSED(id); // legacy
}
static void parse_dc_diveid(char *line, struct membuffer *str, struct git_parser_state *state)
@@ -974,16 +973,16 @@ static void parse_divecomputerid_keyvalue(void *_cid, const char *key, const cha
{
struct divecomputerid *cid = _cid;
- if (!strcmp(key, "deviceid")) {
- cid->deviceid = get_hex(value);
+ // Ignored legacy fields
+ if (!strcmp(key, "firmware"))
return;
- }
+ if (!strcmp(key, "deviceid"))
+ return;
+
+ // Serial number and nickname matter
if (!strcmp(key, "serial")) {
cid->serial = value;
- return;
- }
- if (!strcmp(key, "firmware")) {
- cid->firmware = value;
+ cid->deviceid = calculate_string_hash(value);
return;
}
if (!strcmp(key, "nickname")) {
@@ -1014,7 +1013,7 @@ static void parse_settings_divecomputerid(char *line, struct membuffer *str, str
break;
line = parse_keyvalue_entry(parse_divecomputerid_keyvalue, &id, line, str);
}
- create_device_node(state->devices, id.model, id.deviceid, id.serial, id.firmware, id.nickname);
+ create_device_node(state->devices, id.model, id.serial, id.nickname);
}
static void parse_picture_filename(char *line, struct membuffer *str, struct git_parser_state *state)
diff --git a/core/parse-xml.c b/core/parse-xml.c
index c5864d6f5..5b9abc341 100644
--- a/core/parse-xml.c
+++ b/core/parse-xml.c
@@ -830,11 +830,8 @@ static void try_to_fill_dc(struct divecomputer *dc, const char *name, char *buf,
return;
if (MATCH("model", utf8_string, &dc->model))
return;
- if (MATCH("deviceid", hex_value, &deviceid)) {
- set_dc_deviceid(dc, deviceid, &device_table); // prefer already known serial/firmware over those from the loaded log
- set_dc_deviceid(dc, deviceid, state->devices);
+ if (MATCH("deviceid", hex_value, &deviceid))
return;
- }
if (MATCH("diveid", hex_value, &dc->diveid))
return;
if (MATCH("dctype", get_dc_type, &dc->divemode))
diff --git a/core/parse.c b/core/parse.c
index 241d5763f..fdc73fc9f 100644
--- a/core/parse.c
+++ b/core/parse.c
@@ -203,8 +203,10 @@ void dc_settings_start(struct parser_state *state)
void dc_settings_end(struct parser_state *state)
{
- create_device_node(state->devices, state->cur_settings.dc.model, state->cur_settings.dc.deviceid, state->cur_settings.dc.serial_nr,
- state->cur_settings.dc.firmware, state->cur_settings.dc.nickname);
+ create_device_node(state->devices,
+ state->cur_settings.dc.model,
+ state->cur_settings.dc.serial_nr,
+ state->cur_settings.dc.nickname);
reset_dc_settings(state);
}
diff --git a/core/save-git.c b/core/save-git.c
index cea69974b..6bd237a8b 100644
--- a/core/save-git.c
+++ b/core/save-git.c
@@ -853,19 +853,15 @@ static void save_one_device(struct membuffer *b, const struct device *d)
const char *model = device_get_model(d);
const char *nickname = device_get_nickname(d);
const char *serial = device_get_serial(d);
- const char *firmware = device_get_firmware(d);
- if (!empty_string(nickname) && !strcmp(model, nickname))
- nickname = NULL;
+
if (empty_string(serial)) serial = NULL;
- if (empty_string(firmware)) firmware = NULL;
if (empty_string(nickname)) nickname = NULL;
- if (!nickname && !serial && !firmware)
+ if (!nickname || !serial)
return;
show_utf8(b, "divecomputerid ", model, "");
- put_format(b, " deviceid=%08x", device_get_id(d));
+ put_format(b, " deviceid=%08x", calculate_string_hash(serial));
show_utf8(b, " serial=", serial, "");
- show_utf8(b, " firmware=", firmware, "");
show_utf8(b, " nickname=", nickname, "");
put_string(b, "\n");
}
diff --git a/core/save-xml.c b/core/save-xml.c
index 4c7c2ee04..3836ea35c 100644
--- a/core/save-xml.c
+++ b/core/save-xml.c
@@ -583,29 +583,23 @@ static void save_one_device(struct membuffer *b, const struct device *d)
const char *model = device_get_model(d);
const char *nickname = device_get_nickname(d);
const char *serial_nr = device_get_serial(d);
- const char *firmware = device_get_firmware(d);
/* Nicknames that are empty or the same as the device model are not interesting */
if (empty_string(nickname) || !strcmp(model, nickname))
- nickname = NULL;
+ nickname = NULL;
/* Serial numbers that are empty are not interesting */
if (empty_string(serial_nr))
serial_nr = NULL;
- /* Firmware strings that are empty are not interesting */
- if (empty_string(firmware))
- firmware = NULL;
-
/* Do we have anything interesting about this dive computer to save? */
- if (!serial_nr && !nickname && !firmware)
+ if (!serial_nr || !nickname)
return;
put_format(b, "<divecomputerid");
show_utf8(b, model, " model='", "'", 1);
- put_format(b, " deviceid='%08x'", device_get_id(d));
+ put_format(b, " deviceid='%08x'", calculate_string_hash(serial_nr));
show_utf8(b, serial_nr, " serial='", "'", 1);
- show_utf8(b, firmware, " firmware='", "'", 1);
show_utf8(b, nickname, " nickname='", "'", 1);
put_format(b, "/>\n");
}
diff --git a/dives/DL7.xml b/dives/DL7.xml
index 869da0022..f538c3634 100644
--- a/dives/DL7.xml
+++ b/dives/DL7.xml
@@ -5,12 +5,12 @@
</divesites>
<dives>
<dive number='1' date='2018-01-01' time='10:10:00'>
- <divecomputer model='Imported from CSV' deviceid='ffffffff'>
+ <divecomputer model='Imported from CSV'>
<temperature air='27.0 C' water='25.0 C' />
</divecomputer>
</dive>
<dive number='2' date='2018-01-02' time='10:10:00' duration='60:00 min'>
- <divecomputer model='Imported from CSV' deviceid='ffffffff'>
+ <divecomputer model='Imported from CSV'>
<depth max='10.0 m' mean='9.508 m' />
<temperature air='27.0 C' water='25.0 C' />
<sample time='0:00 min' depth='1.0 m' />
@@ -20,7 +20,7 @@
</divecomputer>
</dive>
<dive number='3' date='2018-01-03' time='10:10:00'>
- <divecomputer model='Imported from CSV' deviceid='ffffffff'>
+ <divecomputer model='Imported from CSV'>
<temperature air='28.0 C' water='26.0 C' />
</divecomputer>
</dive>
diff --git a/dives/TestDiveDM4.xml b/dives/TestDiveDM4.xml
index fd34100b2..246a0b209 100644
--- a/dives/TestDiveDM4.xml
+++ b/dives/TestDiveDM4.xml
@@ -1,6 +1,5 @@
<divelog program='subsurface' version='3'>
<settings>
-<divecomputerid model='Vyper Air' deviceid='ffffffff' serial='20400612'/>
</settings>
<divesites>
</divesites>
diff --git a/dives/TestDiveDM5.xml b/dives/TestDiveDM5.xml
index fdbcd6393..1cbd58256 100644
--- a/dives/TestDiveDM5.xml
+++ b/dives/TestDiveDM5.xml
@@ -1,6 +1,5 @@
<divelog program='subsurface' version='3'>
<settings>
-<divecomputerid model='Vyper Air' deviceid='013749e4' serial='20400612'/>
</settings>
<divesites>
</divesites>
diff --git a/dives/TestDiveSeabearHUDC.xml b/dives/TestDiveSeabearHUDC.xml
index c0ccce23e..4b94cacef 100644
--- a/dives/TestDiveSeabearHUDC.xml
+++ b/dives/TestDiveSeabearHUDC.xml
@@ -5,7 +5,7 @@
</divesites>
<dives>
<dive otu='4' cns='1%' date='2009-10-10' time='05:32:41' duration='7:32 min'>
- <divecomputer model='DC text' deviceid='ffffffff'>
+ <divecomputer model='DC text'>
<depth max='40.0 m' mean='22.32 m' />
<temperature water='1.0 C' />
<sample time='0:01 min' depth='1.0 m' temp='1.0 C' ndl='0:01 min' />
diff --git a/dives/TestDiveSeabearNewFormat.xml b/dives/TestDiveSeabearNewFormat.xml
index 36643540a..3a9a58a61 100644
--- a/dives/TestDiveSeabearNewFormat.xml
+++ b/dives/TestDiveSeabearNewFormat.xml
@@ -5,7 +5,7 @@
</divesites>
<dives>
<dive number='2' date='2012-10-01' time='06:22:00' duration='16:25 min'>
- <divecomputer model='Seabear H3' deviceid='ffffffff' dctype='CCR'>
+ <divecomputer model='Seabear H3' dctype='CCR'>
<depth max='69.9 m' mean='32.928 m' />
<temperature water='25.0 C' />
<extradata key='Firmware version' value='1.31' />
@@ -212,7 +212,7 @@
</divecomputer>
</dive>
<dive number='3' otu='14' cns='8%' date='2012-10-01' time='06:49:00' duration='16:25 min'>
- <divecomputer model='Seabear H3' deviceid='ffffffff'>
+ <divecomputer model='Seabear H3'>
<depth max='69.9 m' mean='32.928 m' />
<temperature water='25.0 C' />
<extradata key='Firmware version' value='1.31' />
@@ -419,7 +419,7 @@
</divecomputer>
</dive>
<dive number='4' otu='14' cns='16%' date='2012-10-01' time='07:07:00' duration='16:17 min'>
- <divecomputer model='Seabear H3' deviceid='ffffffff' dctype='Freedive'>
+ <divecomputer model='Seabear H3' dctype='Freedive'>
<depth max='70.1 m' mean='33.197 m' />
<temperature water='25.0 C' />
<extradata key='Firmware version' value='1.31' />
@@ -1352,7 +1352,7 @@
</divecomputer>
</dive>
<dive number='5' otu='14' cns='24%' date='2012-10-01' time='07:24:00' duration='16:25 min'>
- <divecomputer model='Seabear H3' deviceid='ffffffff'>
+ <divecomputer model='Seabear H3'>
<depth max='69.9 m' mean='32.928 m' />
<temperature water='25.0 C' />
<extradata key='Firmware version' value='1.31' />
@@ -1560,7 +1560,7 @@
<dive number='1' otu='16' cns='5%' date='2012-10-01' time='06:02:00' duration='16:25 min'>
<cylinder description='oxygen' o2='100.0%' use='oxygen' />
<cylinder description='diluent' use='diluent' />
- <divecomputer model='Seabear T1' deviceid='ffffffff' dctype='CCR' no_o2sensors='3'>
+ <divecomputer model='Seabear T1' dctype='CCR' no_o2sensors='3'>
<depth max='69.9 m' mean='32.928 m' />
<temperature water='25.0 C' />
<extradata key='Firmware version' value='1.31' />
@@ -1767,7 +1767,7 @@
</divecomputer>
</dive>
<dive number='2' cns='5%' date='2012-10-01' time='06:22:00' duration='16:25 min'>
- <divecomputer model='Seabear T1' deviceid='ffffffff' dctype='CCR'>
+ <divecomputer model='Seabear T1' dctype='CCR'>
<depth max='69.9 m' mean='32.928 m' />
<temperature water='25.0 C' />
<extradata key='Firmware version' value='1.31' />
@@ -1974,7 +1974,7 @@
</divecomputer>
</dive>
<dive number='3' otu='14' cns='12%' date='2012-10-01' time='06:49:00' duration='16:25 min'>
- <divecomputer model='Seabear T1' deviceid='ffffffff'>
+ <divecomputer model='Seabear T1'>
<depth max='69.9 m' mean='32.928 m' />
<temperature water='25.0 C' />
<extradata key='Firmware version' value='1.31' />
@@ -2181,7 +2181,7 @@
</divecomputer>
</dive>
<dive number='4' otu='14' cns='20%' date='2012-10-01' time='07:07:00' duration='16:17 min'>
- <divecomputer model='Seabear T1' deviceid='ffffffff' dctype='Freedive'>
+ <divecomputer model='Seabear T1' dctype='Freedive'>
<depth max='70.1 m' mean='33.197 m' />
<temperature water='24.0 C' />
<extradata key='Firmware version' value='1.31' />
@@ -3114,7 +3114,7 @@
</divecomputer>
</dive>
<dive number='5' otu='14' cns='28%' date='2012-10-01' time='07:24:00' duration='16:25 min'>
- <divecomputer model='Seabear T1' deviceid='ffffffff'>
+ <divecomputer model='Seabear T1'>
<depth max='69.9 m' mean='32.928 m' />
<temperature water='25.0 C' />
<extradata key='Firmware version' value='1.31' />
diff --git a/dives/mergedVyperOstc.xml b/dives/mergedVyperOstc.xml
index 98f059c3a..99a6d7807 100644
--- a/dives/mergedVyperOstc.xml
+++ b/dives/mergedVyperOstc.xml
@@ -1,7 +1,5 @@
<divelog program='subsurface' version='3'>
<settings>
-<divecomputerid model='Suunto Vyper Air' deviceid='11223344' serial='99999999'/>
-<divecomputerid model='Heinrichs Weikamp OSTC Sport' deviceid='abcdef00' serial='10000' firmware='10.31'/>
</settings>
<divesites>
</divesites>
@@ -12,7 +10,7 @@
<cylinder size='11.094 l' workpressure='206.843 bar' description='AL80' o2='31.0%' />
<cylinder size='11.094 l' workpressure='206.843 bar' description='AL80' />
<cylinder size='11.094 l' workpressure='206.843 bar' description='AL80' />
- <divecomputer model='Heinrichs Weikamp OSTC Sport' deviceid='abcdef00' diveid='b4ffa0ce'>
+ <divecomputer model='Heinrichs Weikamp OSTC Sport' deviceid='8e8f3f68' diveid='b4ffa0ce'>
<depth max='26.32 m' mean='16.594 m' />
<temperature water='26.0 C' />
<surface pressure='1.008 bar' />
@@ -109,7 +107,7 @@
<cylinder size='11.094 l' workpressure='206.843 bar' description='AL80' o2='31.0%' />
<cylinder size='11.094 l' workpressure='206.843 bar' description='AL80' />
<cylinder size='11.094 l' workpressure='206.843 bar' description='AL80' />
- <divecomputer model='Heinrichs Weikamp OSTC Sport' deviceid='ae480b5b' diveid='7ab00781'>
+ <divecomputer model='Heinrichs Weikamp OSTC Sport' deviceid='8e8f3f68' diveid='7ab00781'>
<depth max='27.44 m' mean='14.427 m' />
<temperature water='26.2 C' />
<surface pressure='1.009 bar' />
@@ -196,7 +194,7 @@
</dive>
<dive number='1' sac='9.026 l/min' otu='54' cns='17%' date='2017-02-03' time='06:59:41' duration='71:40 min'>
<cylinder size='11.094 l' workpressure='206.843 bar' description='AL80' o2='32.0%' />
- <divecomputer model='Suunto Vyper Air' deviceid='11223344' diveid='c51cb31b'>
+ <divecomputer model='Suunto Vyper Air' deviceid='f95a40f1' diveid='c51cb31b'>
<depth max='26.72 m' mean='16.837 m' />
<temperature air='28.0 C' water='27.0 C' />
<extradata key='Serial' value='99312861' />
@@ -422,7 +420,7 @@
</dive>
<dive number='2' sac='9.173 l/min' otu='41' cns='17%' date='2017-02-03' time='10:25:16' duration='65:40 min'>
<cylinder size='11.094 l' workpressure='206.843 bar' description='AL80' o2='32.0%' />
- <divecomputer model='Suunto Vyper Air' deviceid='20993833' diveid='5372ae57'>
+ <divecomputer model='Suunto Vyper Air' deviceid='f95a40f1' diveid='5372ae57'>
<depth max='27.8 m' mean='14.594 m' />
<temperature air='29.0 C' water='27.0 C' />
<extradata key='Serial' value='99312861' />
diff --git a/dives/test40-42.xml b/dives/test40-42.xml
index 727aa5475..29f31f8c3 100644
--- a/dives/test40-42.xml
+++ b/dives/test40-42.xml
@@ -9,7 +9,7 @@
<dive number='1' otu='9' cns='5%' date='2013-10-01' time='10:34:00' duration='45:00 min'>
<buddy>Dirk</buddy>
<suit>wet, 5mm</suit>
- <divecomputer model='csv' last-manual-time='45:00 min' deviceid='ffffffff'>
+ <divecomputer model='csv' last-manual-time='45:00 min'>
<depth max='18.0 m' mean='16.0 m' />
<sample time='0:00 min' depth='0.0 m' />
<sample time='2:00 min' depth='18.0 m' />
@@ -22,7 +22,7 @@
<dive number='2' otu='5' cns='7%' date='2013-10-01' time='12:13:00' duration='41:00 min'>
<buddy>Linus</buddy>
<suit>wet, shorty, 3mm</suit>
- <divecomputer model='csv' last-manual-time='41:00 min' deviceid='ffffffff'>
+ <divecomputer model='csv' last-manual-time='41:00 min'>
<depth max='16.2 m' mean='13.5 m' />
<sample time='0:00 min' depth='0.0 m' />
<sample time='1:48 min' depth='16.2 m' />
@@ -35,7 +35,7 @@
<dive number='3' date='2014-10-01' time='10:02:00' duration='48:00 min'>
<buddy>Tomaz</buddy>
<suit>none</suit>
- <divecomputer model='csv' last-manual-time='48:00 min' deviceid='ffffffff'>
+ <divecomputer model='csv' last-manual-time='48:00 min'>
<depth max='13.3 m' mean='11.5 m' />
<sample time='0:00 min' depth='0.0 m' />
<sample time='1:29 min' depth='13.3 m' />
@@ -48,7 +48,7 @@
<dive number='4' otu='13' cns='5%' date='2014-10-01' time='14:19:00' duration='34:00 min'>
<buddy>Don</buddy>
<suit>dry, Whites Fusion</suit>
- <divecomputer model='csv' last-manual-time='34:00 min' deviceid='ffffffff'>
+ <divecomputer model='csv' last-manual-time='34:00 min'>
<depth max='24.9 m' mean='20.1 m' />
<sample time='0:00 min' depth='0.0 m' />
<sample time='2:46 min' depth='24.9 m' />
@@ -775,7 +775,7 @@
<notes>CCR dive</notes>
<cylinder size='2.0 l' workpressure='232.0 bar' description='Oxy2' o2='100.0%' start='190.0 bar' end='130.0 bar' use='oxygen' />
<cylinder size='2.0 l' workpressure='232.0 bar' description='Dil2' start='185.0 bar' end='130.0 bar' use='diluent' />
- <divecomputer model='Heinrichs Weikamp OSTC 3' deviceid='01234567' diveid='76543210' dctype='CCR'>
+ <divecomputer model='Heinrichs Weikamp OSTC 3' diveid='76543210' dctype='CCR'>
<depth max='38.99 m' mean='17.72 m' />
<temperature water='4.3 C' />
<surface pressure='1.02 bar' />
@@ -3272,7 +3272,7 @@
<notes>CCR dive</notes>
<cylinder size='2.0 l' workpressure='232.0 bar' description='Oxy2' o2='100.0%' start='190.0 bar' end='130.0 bar' use='oxygen' />
<cylinder size='2.0 l' workpressure='232.0 bar' description='Dil2' start='185.0 bar' end='130.0 bar' use='diluent' />
- <divecomputer model='Heinrichs Weikamp OSTC 3' deviceid='01234567' diveid='76543210' dctype='CCR'>
+ <divecomputer model='Heinrichs Weikamp OSTC 3' diveid='76543210' dctype='CCR'>
<depth max='38.99 m' mean='17.72 m' />
<temperature water='4.3 C' />
<surface pressure='1.02 bar' />
diff --git a/smtk-import/smartrak.c b/smtk-import/smartrak.c
index ffc62a9c7..257cc143f 100644
--- a/smtk-import/smartrak.c
+++ b/smtk-import/smartrak.c
@@ -1112,7 +1112,6 @@ void smartrak_import(const char *file, struct dive_table *divetable)
smtkdive->notes = smtk_concat_str(smtkdive->notes, "\n", "%s", col[coln(REMARKS)]->bind_ptr);
record_dive_to_table(smtkdive, divetable);
- add_devices_of_dive(smtkdive, devices);
device_data_free(devdata);
}
mdb_free_tabledef(mdb_table);
generated by cgit v1.2.3-70-g09d2 (git 2.47.0) at 2024-12-26 16:47:50 +0000