From 9d582c5512c261bbea5d6257375c291a06721950 Mon Sep 17 00:00:00 2001 From: Dirk Hohndel Date: Wed, 13 Feb 2019 22:48:50 -0800 Subject: Mobile: only show dive computers in the Bluetooth connection list And offer an option to show all devices in the settings. This is intentionally not stored in the preferences as this should never be needed. We don't support BT or BLE dive computers that we don't recognize. This is a last resort in case a new firmware were to change the name or some other weird issue causes us not to recognize a dive computer - and that should be fixed instead of worked around. Signed-off-by: Dirk Hohndel --- core/btdiscovery.cpp | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) (limited to 'core/btdiscovery.cpp') diff --git a/core/btdiscovery.cpp b/core/btdiscovery.cpp index 51cf61b20..88d1cd588 100644 --- a/core/btdiscovery.cpp +++ b/core/btdiscovery.cpp @@ -91,6 +91,7 @@ bool matchesKnownDiveComputerNames(QString btName) } BTDiscovery::BTDiscovery(QObject*) : m_btValid(false), + m_showNonDiveComputers(false), discoveryAgent(nullptr) { if (m_instance) { @@ -104,6 +105,11 @@ BTDiscovery::BTDiscovery(QObject*) : m_btValid(false), #endif } +void BTDiscovery::showNonDiveComputers(bool show) +{ + m_showNonDiveComputers = show; +} + void BTDiscovery::BTDiscoveryReDiscover() { #if !defined(Q_OS_IOS) @@ -232,7 +238,9 @@ void BTDiscovery::btDeviceDiscoveredMain(const btPairedDevice &device) connectionListModel.addAddress(newDevice + " " + device.address); return; } - connectionListModel.addAddress(device.address); + // Do we want only devices we recognize as dive computers? + if (m_showNonDiveComputers) + connectionListModel.addAddress(device.address); qDebug() << "Not recognized as dive computer"; } -- cgit v1.2.3-70-g09d2