core: factor out device_is_used_by_selected_dive() function

We have a callback for all devices with a twist: it can loop
over those devices that are used by a selected dive. This is
used for exporting a subset of the dive log.

Factor out the "is device used by selected dive" part of the
function and make it available to C. The goal is to make
the whole callback thing unnecessary and let C code loop
directly over the device list.

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

1 files changed, 15 insertions, 18 deletions

diff --git a/core/device.cpp b/core/device.cpp
index 1c0d6d5c6..63e2eea1d 100644
--- a/core/device.cpp
+++ b/core/device.cpp
@@ -285,27 +285,24 @@ extern "C" void clear_device_nodes()
 	device_table.devices.clear();
 }
 
+/* Returns whether the given device is used by a selected dive. */
+extern "C" bool device_used_by_selected_dive(const struct device *dev)
+{
+	for (dive *d: getDiveSelection()) {
+		struct divecomputer *dc;
+		for_each_dc (d, dc) {
+			if (dc->deviceid == dev->deviceId)
+				return true;
+		}
+	}
+	return false;
+}
+
 extern "C" void call_for_each_dc (void *f, void (*callback)(void *, const char *, uint32_t, const char *, const char *, const char *),
 				  bool select_only)
 {
-	for (const device &node : device_table.devices) {
-		bool found = false;
-		if (select_only) {
-			for (dive *d: getDiveSelection()) {
-				struct divecomputer *dc;
-				for_each_dc (d, dc) {
-					if (dc->deviceid == node.deviceId) {
-						found = true;
-						break;
-					}
-				}
-				if (found)
-					break;
-			}
-		} else {
-			found = true;
-		}
-		if (found)
+	for (const device &node: device_table.devices) {
+		if (!select_only || device_used_by_selected_dive(&node))
 			callback(f, node.model.c_str(), node.deviceId, node.nickName.c_str(),
 						 node.serialNumber.c_str(), node.firmware.c_str());
 	}