mirror of
https://github.com/subsurface/subsurface.git
synced 2025-01-19 06:15:26 +00:00
Introduce mode field in Bluetooth device selection dialog
Some BT devices support both, classical and LE, modes. Users could
choose either by prepending or removing "LE:" in the device address
field. After commit d23bd46a1b, the
device field is always disabled in Bluetooth mode.
Therefore, add a mode combo box to the Bluetooth device selection
dialog. In the default mode (auto), the old code path (based on
the Qt device flags) is used. The two other modes (force LE, force
classical) allow the user to force the preferred behavior.
This feature is meant as a stop-gap measure until a more refined
transport choice is implemented. Therefore, the value of the new
combo box is not saved in the settings, to avoid cluttering of
the preferences with soon to be obsolete entries.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger
Lubomir I. Ivanov
parent
a95825e95c
commit
aca4a3a4fa
3 changed files with 124 additions and 80 deletions
|
|
@ -132,18 +132,20 @@ extern void addBtUuid(QBluetoothUuid uuid);
|
|||
extern QHash<QString, QStringList> productList;
|
||||
extern QStringList vendorList;
|
||||
|
||||
QString btDeviceAddress(const QBluetoothDeviceInfo *device, bool isBle)
|
||||
{
|
||||
QString address = device->address().isNull() ?
|
||||
device->deviceUuid().toString() : device->address().toString();
|
||||
const char *prefix = isBle ? "LE:" : "";
|
||||
return prefix + address;
|
||||
}
|
||||
|
||||
QString markBLEAddress(const QBluetoothDeviceInfo *device)
|
||||
{
|
||||
QBluetoothDeviceInfo::CoreConfigurations flags;
|
||||
QString prefix = "";
|
||||
QBluetoothDeviceInfo::CoreConfigurations flags = device->coreConfigurations();
|
||||
bool isBle = flags == QBluetoothDeviceInfo::LowEnergyCoreConfiguration;
|
||||
|
||||
flags = device->coreConfigurations();
|
||||
if (flags == QBluetoothDeviceInfo::LowEnergyCoreConfiguration)
|
||||
prefix = "LE:";
|
||||
if (device->address().isNull())
|
||||
return prefix + device->deviceUuid().toString();
|
||||
else
|
||||
return prefix + device->address().toString();
|
||||
return btDeviceAddress(device, isBle);
|
||||
}
|
||||
|
||||
void BTDiscovery::btDeviceDiscovered(const QBluetoothDeviceInfo &device)
|
||||
|
|
|
|||
|
|
@ -449,13 +449,25 @@ void BtDeviceSelectionDialog::deviceDiscoveryError(QBluetoothDeviceDiscoveryAgen
|
|||
}
|
||||
|
||||
extern QString markBLEAddress(const QBluetoothDeviceInfo *device);
|
||||
extern QString btDeviceAddress(const QBluetoothDeviceInfo *device, bool isBle);
|
||||
|
||||
QString BtDeviceSelectionDialog::getSelectedDeviceAddress()
|
||||
{
|
||||
if (selectedRemoteDeviceInfo)
|
||||
return markBLEAddress(selectedRemoteDeviceInfo.data());
|
||||
if (!selectedRemoteDeviceInfo)
|
||||
return QString();
|
||||
|
||||
return QString();
|
||||
int btMode = ui->btMode->currentIndex();
|
||||
QBluetoothDeviceInfo *device = selectedRemoteDeviceInfo.data();
|
||||
|
||||
switch (btMode) {
|
||||
case 0: // Auto
|
||||
default:
|
||||
return markBLEAddress(device);
|
||||
case 1: // Force LE
|
||||
return btDeviceAddress(device, true);
|
||||
case 2: // Force classical
|
||||
return btDeviceAddress(device, false);
|
||||
}
|
||||
}
|
||||
|
||||
QString BtDeviceSelectionDialog::getSelectedDeviceName()
|
||||
|
|
|
|||
|
|
@ -14,25 +14,6 @@
|
|||
<string>Remote Bluetooth device selection</string>
|
||||
</property>
|
||||
<layout class="QGridLayout" name="gridLayout">
|
||||
<item row="0" column="0">
|
||||
<widget class="QLabel" name="discoveredDevicesLabel">
|
||||
<property name="sizePolicy">
|
||||
<sizepolicy hsizetype="Fixed" vsizetype="Fixed">
|
||||
<horstretch>0</horstretch>
|
||||
<verstretch>0</verstretch>
|
||||
</sizepolicy>
|
||||
</property>
|
||||
<property name="font">
|
||||
<font>
|
||||
<weight>75</weight>
|
||||
<bold>true</bold>
|
||||
</font>
|
||||
</property>
|
||||
<property name="text">
|
||||
<string>Discovered devices</string>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item row="3" column="1">
|
||||
<layout class="QHBoxLayout" name="dialogControls">
|
||||
<item>
|
||||
|
|
@ -57,49 +38,15 @@
|
|||
</item>
|
||||
</layout>
|
||||
</item>
|
||||
<item row="1" column="0" rowspan="2">
|
||||
<layout class="QVBoxLayout" name="remoteDevicesSection">
|
||||
<item>
|
||||
<widget class="QListWidget" name="discoveredDevicesList">
|
||||
<property name="sizePolicy">
|
||||
<sizepolicy hsizetype="MinimumExpanding" vsizetype="Expanding">
|
||||
<horstretch>0</horstretch>
|
||||
<verstretch>0</verstretch>
|
||||
</sizepolicy>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item>
|
||||
<layout class="QVBoxLayout" name="scanningControls">
|
||||
<item>
|
||||
<widget class="QPushButton" name="scan">
|
||||
<property name="sizePolicy">
|
||||
<sizepolicy hsizetype="Minimum" vsizetype="Minimum">
|
||||
<horstretch>0</horstretch>
|
||||
<verstretch>0</verstretch>
|
||||
</sizepolicy>
|
||||
</property>
|
||||
<property name="text">
|
||||
<string>Scan</string>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item>
|
||||
<widget class="QPushButton" name="clear">
|
||||
<property name="sizePolicy">
|
||||
<sizepolicy hsizetype="Minimum" vsizetype="Minimum">
|
||||
<horstretch>0</horstretch>
|
||||
<verstretch>0</verstretch>
|
||||
</sizepolicy>
|
||||
</property>
|
||||
<property name="text">
|
||||
<string>Clear</string>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
</layout>
|
||||
</item>
|
||||
</layout>
|
||||
<item row="3" column="0">
|
||||
<widget class="QLabel" name="dialogStatus">
|
||||
<property name="text">
|
||||
<string/>
|
||||
</property>
|
||||
<property name="wordWrap">
|
||||
<bool>true</bool>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item row="1" column="1">
|
||||
<widget class="QGroupBox" name="localDeviceDetails">
|
||||
|
|
@ -207,16 +154,99 @@
|
|||
</layout>
|
||||
</widget>
|
||||
</item>
|
||||
<item row="3" column="0">
|
||||
<widget class="QLabel" name="dialogStatus">
|
||||
<property name="text">
|
||||
<string/>
|
||||
<item row="0" column="0">
|
||||
<widget class="QLabel" name="discoveredDevicesLabel">
|
||||
<property name="sizePolicy">
|
||||
<sizepolicy hsizetype="Fixed" vsizetype="Fixed">
|
||||
<horstretch>0</horstretch>
|
||||
<verstretch>0</verstretch>
|
||||
</sizepolicy>
|
||||
</property>
|
||||
<property name="wordWrap">
|
||||
<bool>true</bool>
|
||||
<property name="font">
|
||||
<font>
|
||||
<weight>75</weight>
|
||||
<bold>true</bold>
|
||||
</font>
|
||||
</property>
|
||||
<property name="text">
|
||||
<string>Discovered devices</string>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item row="1" column="0" rowspan="2">
|
||||
<layout class="QVBoxLayout" name="remoteDevicesSection">
|
||||
<item>
|
||||
<widget class="QListWidget" name="discoveredDevicesList">
|
||||
<property name="sizePolicy">
|
||||
<sizepolicy hsizetype="MinimumExpanding" vsizetype="Expanding">
|
||||
<horstretch>0</horstretch>
|
||||
<verstretch>0</verstretch>
|
||||
</sizepolicy>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item>
|
||||
<layout class="QHBoxLayout" name="btModeSection">
|
||||
<item>
|
||||
<widget class="QLabel" name="btModeLabel">
|
||||
<property name="text">
|
||||
<string>Bluetooth mode</string>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item>
|
||||
<widget class="QComboBox" name="btMode">
|
||||
<item>
|
||||
<property name="text">
|
||||
<string>Auto</string>
|
||||
</property>
|
||||
</item>
|
||||
<item>
|
||||
<property name="text">
|
||||
<string>Force LE</string>
|
||||
</property>
|
||||
</item>
|
||||
<item>
|
||||
<property name="text">
|
||||
<string>Force classical</string>
|
||||
</property>
|
||||
</item>
|
||||
</widget>
|
||||
</item>
|
||||
</layout>
|
||||
</item>
|
||||
<item>
|
||||
<layout class="QVBoxLayout" name="scanningControls">
|
||||
<item>
|
||||
<widget class="QPushButton" name="scan">
|
||||
<property name="sizePolicy">
|
||||
<sizepolicy hsizetype="Minimum" vsizetype="Minimum">
|
||||
<horstretch>0</horstretch>
|
||||
<verstretch>0</verstretch>
|
||||
</sizepolicy>
|
||||
</property>
|
||||
<property name="text">
|
||||
<string>Scan</string>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item>
|
||||
<widget class="QPushButton" name="clear">
|
||||
<property name="sizePolicy">
|
||||
<sizepolicy hsizetype="Minimum" vsizetype="Minimum">
|
||||
<horstretch>0</horstretch>
|
||||
<verstretch>0</verstretch>
|
||||
</sizepolicy>
|
||||
</property>
|
||||
<property name="text">
|
||||
<string>Clear</string>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
</layout>
|
||||
</item>
|
||||
</layout>
|
||||
</item>
|
||||
</layout>
|
||||
</widget>
|
||||
<resources/>
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue