diff options
author | Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2018-02-25 00:17:33 +0100 |
---|---|---|
committer | Jan Mulder <jlmulder@xs4all.nl> | 2018-02-26 08:52:34 +0100 |
commit | 7bd860f2b768ac9f14fea7a55ad23092a9f4bba2 (patch) | |
tree | 0df83dd12c2d9a8ea03de6f9e53932d56d642ae5 /desktop-widgets/btdeviceselectiondialog.cpp | |
parent | d9c0df01424268ca7ba568279f08683f2c98aa44 (diff) | |
download | subsurface-7bd860f2b768ac9f14fea7a55ad23092a9f4bba2.tar.gz |
Cleanup: Use QString as parameter for [get|save]BtDeviceInfo()
Each callsite of saveBtDeviceInfo() has a QString, which is converted
to a C-string, passed and immediately converted back. Remove these
conversions by taking a reference to QString directly.
getBtDeviceInfo() is not as clear. Here, the callsite has a C-string
handed down from libdivecomputer. Nevertheless, pass a reference of
QString here as well. Firstly, for reasons of symmetry. Secondly,
to avoid multiple conversions in the getBtDeviceInfo() functions.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'desktop-widgets/btdeviceselectiondialog.cpp')
-rw-r--r-- | desktop-widgets/btdeviceselectiondialog.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/desktop-widgets/btdeviceselectiondialog.cpp b/desktop-widgets/btdeviceselectiondialog.cpp index 89fd34b53..8077298f6 100644 --- a/desktop-widgets/btdeviceselectiondialog.cpp +++ b/desktop-widgets/btdeviceselectiondialog.cpp @@ -155,7 +155,7 @@ void BtDeviceSelectionDialog::on_save_clicked() selectedRemoteDeviceInfo.reset(new QBluetoothDeviceInfo(remoteDeviceInfo)); QString address = remoteDeviceInfo.address().isNull() ? remoteDeviceInfo.deviceUuid().toString() : remoteDeviceInfo.address().toString(); - saveBtDeviceInfo(address.toUtf8().constData(), remoteDeviceInfo); + saveBtDeviceInfo(address, remoteDeviceInfo); if (remoteDeviceDiscoveryAgent->isActive()) { // Stop the SDP agent if the clear button is pressed and enable the Scan button remoteDeviceDiscoveryAgent->stop(); |