summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fix various qdoc warnings in QtBluetoothAlex Blasche2018-09-183-7/+7
| | | | | Change-Id: I49653a6ac6fa004987d32365f96a6407f50e207c Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* Merge remote-tracking branch 'origin/5.11' into 5.12Qt Forward Merge Bot2018-09-151-0/+58
|\ | | | | | | Change-Id: I7c24df8729794b8241572bddd8d8cd0465993b42
| * Merge remote-tracking branch 'origin/5.11.2' into 5.11Qt Forward Merge Bot2018-09-091-0/+58
| |\ | | | | | | | | | Change-Id: I7ea913083777358265f62b4bf6beed3f2db0ebe1
| | * Add changelog for 5.11.2 releasev5.11.2Alex Blasche2018-08-311-0/+58
| | | | | | | | | | | | | | | | | | Change-Id: Ie610cc0935594bda8c05cfad41dd49218e1930f1 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | Fix wrong logging messageAlex Blasche2018-09-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | This was caused by mindless copy & paste. Change-Id: I129a5f52753e7aa825cf7f6337b5f24644d9b9a0 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io> Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* | | Remove duplicated closure code and add missing cases of itAlex Blasche2018-09-142-33/+24
| | | | | | | | | | | | | | | | | | Change-Id: I4f33ec95b2af858b8d0c3ff60882b550a91563df Reviewed-by: Oliver Wolff <oliver.wolff@qt.io> Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* | | Implement handling of BTLE Battery servicesAlex Blasche2018-09-147-16/+299
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since BlueZ 5.48 battery services are no longer exposed via the generic GATT interface but have their own dedicated Battery1 interface. This patch transforms the dedicated interface back into the previous behavior. Essentially we are emulating the old interface to ensure that the QLowEnergyService user does not have to distinguish. Fixes: QTBUG-70222 Change-Id: Ib9fef41cf16f7562f169f51ee45b19f52de6a0c0 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | | Remove dead/unused member variablev5.12.0-alpha1Alex Blasche2018-09-112-4/+1
| | | | | | | | | | | | | | | Change-Id: I8a30d5d12fad73a714159ebe3e1d54e8c0b3e407 Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* | | Merge remote-tracking branch 'origin/5.11' into 5.12Qt Forward Merge Bot2018-09-081-1/+5
|\| | | | | | | | | | | Change-Id: I576079e30f4075117ca764f45198bc1e5269d887
| * | winrt: Do proper error handling if connecting to a LE device failsOliver Wolff2018-08-311-1/+5
| |/ | | | | | | | | | | Task-number: QTBUG-70162 Change-Id: I4556cbf1965121b042a55e3e36e24383f8fa8dc9 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Doc: Remove empty section in Qt NFC example overview pagePaul Wicking2018-09-071-2/+0
| | | | | | | | | | | | | | Remove unused section "C++ Tutorial". Change-Id: I55ba7286c00ae4573d97f6244dff1f589634b949 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Fix a few clazy/clang warnings in HeartRate Game exampleAlex Blasche2018-09-058-29/+29
| | | | | | | | | | Change-Id: I2a7b8c403ca466998ddcbd8496c31feb9856f0ad Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* | Fix QNearFieldManager related override warningsAlex Blasche2018-09-035-27/+29
| | | | | | | | | | Change-Id: If7adc165e063445ee0c34291a3e3840ddae0f570 Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* | Fix bluez config test failure on clangAlex Blasche2018-08-311-5/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This compile failure used to be a warning only but these days is a failure for clang: > /home/ablasche/dev/qt/qt512/qtconnectivity/config.tests/bluez/main.cpp:36:11: error: taking the address of a temporary object of type 'bdaddr_t' [-Waddress-of-temporary] > bacmp(BDADDR_ANY, BDADDR_LOCAL); > ^~~~~~~~~~ > /usr/include/bluetooth/bluetooth.h:310:23: note: expanded from macro 'BDADDR_ANY' > #define BDADDR_ANY (&(bdaddr_t) {{0, 0, 0, 0, 0, 0}}) > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > /home/ablasche/dev/qt/qt512/qtconnectivity/config.tests/bluez/main.cpp:36:23: error: taking the address of a temporary object of type 'bdaddr_t' [-Waddress-of-temporary] > bacmp(BDADDR_ANY, BDADDR_LOCAL); > ^~~~~~~~~~~~ > /usr/include/bluetooth/bluetooth.h:312:23: note: expanded from macro 'BDADDR_LOCAL' > #define BDADDR_LOCAL (&(bdaddr_t) {{0, 0, 0, 0xff, 0xff, 0xff}}) > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 2 errors generated. The need for BDADDR_NONE is obsolete now since the test does not rely on those defines anymore. The main purpose is to check that bluetooth.h is available which is already tested by using bacmp and bdaddr_t. Change-Id: I09da4dc5cd3945ffae7819628b45477a52789006 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io> Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* | Fix failing restart of HeartRateServer exampleAlex Blasche2018-08-311-4/+7
| | | | | | | | | | | | | | | | Disconnecting from central devices invalidates the peripheral's service. The change ensures that the service is recreated upon disconnection. Change-Id: I2defc6ffb4c9f23d70b6f181c25f47746b3859a4 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Ensure that local services have state LocalServiceAlex Blasche2018-08-302-4/+6
| | | | | | | | | | | | | | | | and invalidate the state once they have been disconnected. This affects Android, Bluez and Apple code lines. Change-Id: I2b87577930b6ae78e7ef7ef061e597beba946594 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Merge remote-tracking branch 'origin/5.11' into 5.12Qt Forward Merge Bot2018-08-301-2/+6
|\| | | | | | | Change-Id: I20db491b185972148ad3fbe6953e5d6ce2889a7f
| * Provide more informative error messages in the BLE Scanner exampleShawn Rutledge2018-08-211-2/+6
| | | | | | | | | | | | | | | | | | QMetaEnum helps us avoid writing separate strings for each error description; it's not very verbose but it's way better than saying that most of the errors are "unknown". Change-Id: I61aaf3be6982a2987aabfa6be3192e1afb7df17a Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Fix LEController peripheral mode on BlueZAlex Blasche2018-08-301-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | LEController in peripheral mode uses QBluetoothSocket::setSocketDescriptor(). Since QBluetoothsocket was ported to DBus (enabled when Bluez is 5.46 or later) setSocketDescriptor()/server mode does not work. That's why f602d7fef2e2f067e123e5740d4b0bf16c4ec0e2 made sure that QBluetoothServer uses the old raw socket implementation. Unfortunately QLowEnergyController was forgotten and needs the same workaround. Change-Id: I6ba3654ee199d0ffcf4b69f16dfbc5508d6b13d4 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Normalize QObject::connect() statementsAlex Blasche2018-08-299-104/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Some cases use the Q_PRIVATE_SLOT logic which does not require QObject inheritance. Those cases were converted to lambda logic to avoid need for QObject inheritance. The Q_PRIVATE_SLOT macro was removed from qbluetoothdevicediscoveryagent.h. This is not a BC problem because the macro expands to nothing. Only moc recognizes the pattern. Change-Id: Ic7cb4cde397f9b230b6fd0b4046e59e504583e58 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* | qRegisterMetaType for QBluetoothServiceInfoAlex Blasche2018-08-282-1/+1
| | | | | | | | | | | | | | | | Android internals seem to require it. All other existing test related registrations are removed as they are obsolete after this patch. Change-Id: I49aa3b79bdc8ca5f463f4f7a88cbb612db39d15e Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Code cleanup: Use nullptr wherever possibleAlex Blasche2018-08-2450-196/+171
| | | | | | | | | | Change-Id: I7dd2d055c8d667f049d7cb2c371619137bf76030 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Cleanup lowenergyscanner example with clang tidy and clazy helpAlex Blasche2018-08-249-45/+37
| | | | | | | | | | Change-Id: Iaaf169b1ac9b3d4c348d9cabf737b395198a921f Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Add QBluetoothDeviceDiscoveryAgent::deviceUpdated signalAlex Blasche2018-08-236-10/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For iBeacons and such, it's normal to keep discovering indefinitely, while the advertisement can sometimes carry changing payload data. This signal helps keep the UI up-to-date. Changed setManufacturerData to return a bool indicating whether the data was really modified or not, to avoid sending deviceUpdated in case it was not changed. For convenience QBluetoothDeviceInfo::manufacturerData() was added. WinRT and Apple platforms do not emit the new signal yet and relevant TODO items were added. Done-with: Shawn Rutledge <shawn.rutledge@qt.io> Task-number: QTBUG-46008 Task-number: QTBUG-69747 Change-Id: I6d6b373f6cc5ee3dd6af2ad05d8640dbd25c5d6a Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io> Reviewed-by: Oliver Wolff <oliver.wolff@qt.io> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Spell QBluetoothDeviceInfo::manufacturerIds correctlyShawn Rutledge2018-08-212-2/+2
| | | | | | | | | | | | | | Amends 0f930cddefe386c6b6f65b8b11342c9e76876a9b Change-Id: Ic3769d30198d3607a1fee218d9f2e90b9addf2d3 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Remove direct usage of QLowEnergyController::ctorAlex Blasche2018-08-215-5/+5
| | | | | | | | | | | | | | | | QLowEnergyController::createCentral() is the official API to do the same. Change-Id: Idbd26d201b24add6697f6f9cdf1194511c160273 Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io> Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Add Qt 6 API todoAlex Blasche2018-08-211-0/+1
| | | | | | | | | | Change-Id: Ibc8a3e86b7671381c5a00f6421878961e8e2f4a6 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Merge remote-tracking branch 'origin/5.11' into devQt Forward Merge Bot2018-08-181-4/+10
|\| | | | | | | Change-Id: I3ea075969f9a9b8e94a34b3c7cd92aaea72175af
| * Android: Fix QBluetoothServiceDiscoveryAgent::uuidFilter() behaviorAlex Blasche2018-08-151-4/+10
| | | | | | | | | | | | Task-number: QTBUG-69700 Change-Id: Ib8082156f81949926380e4e296d91c8dfc01b231 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Add manufacturer data to QBluetoothDeviceInfoAlex Blasche2018-08-176-7/+108
| | | | | | | | | | | | | | The initial patch was contributed by Thiemo van Engelen. Change-Id: I45fad793ba092ab2820e606d8bf8807afa3e911e Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Suppress "We mean it" header warning as profilemanager1_p.h is generatedAlex Blasche2018-08-161-1/+1
| | | | | | | | | | Change-Id: I252cc1bf59550d06a403bf73d697e7297c0a6648 Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* | Enable new QBluetoothSocket backend for dbus sockets on Bluez 5.46+Alex Blasche2018-08-161-20/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The API was introduced earlier than 5.46 already but so far it has not been tested. For now we use 5.46. There is another dependency to QLowEnergyController's GATT custom implementation. Custom GATT only works with the QBluetoothSocket raw socket implementation. By setting the minimal version for QBluetoothSocket slightly higher than the 5.42 version for custom gatt, we ensure that the new dbus socket code is never run together with the GATT custom stack. Task-number: QTBUG-68550 Change-Id: I240f7fc8acb116c71e7601df8baf82f61e53c33e Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io> Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Ensure that QBluetoothServer works with new DBus based socketsAlex Blasche2018-08-164-12/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | or better ensure that the DBus based socket is not used for QBluetoothServer. The server implementation will continue to use the raw socket implementation for the foreseeable future. The reason being the DBus version not yet working. convertAddress() in the server cpp was removed because it was a duplicate implementation. Including qbluetoothsocket_bluez_p.h pulls in the qbluetoothsocketbase_p.h which provides the official convertAddress() implementations. Unfortunately this requires a new ctor for QBluetoothSocket which is only specified for BlueZ builds. It is used to permit QBluetoothServer to set the correct dptr for QBluetoothSocket. Task-number: QTBUG-68550 Change-Id: I19298f75e9bc2ab93322d9f34e1816ad733ac6d9 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | winrt: Properly handle sequences in SDP registrationOliver Wolff2018-08-161-12/+25
| | | | | | | | | | Change-Id: Ic0e9a18b73915fee1458578ca49359a3647159c1 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | winrt: Properly handle strings in SDP registrationOliver Wolff2018-08-161-11/+40
| | | | | | | | | | Change-Id: Ifb9531830ef11d0e49c205eada5884906051e467 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | winrt: Add helper functionality for more complex sdp typesOliver Wolff2018-08-161-1/+44
| | | | | | | | | | | | | | | | | | | | In preparation for following patches which will add proper support for strings, sequences, alternatives, and urls as SDP attributes, their base IDs as well 2 helper functions (typeIsOfBase and getLEngthForBaseType) are added. Change-Id: I40982c83435985a7f57d7854ac1e353350c29da5 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Clean up btchat exampleOliver Wolff2018-08-169-70/+90
| | | | | | | | | | | | | | | | | | | | | | - Replace old connect syntax - Clean up includes - Replace 0 with nullptr - Add socket error handling Change-Id: Id4c7634db29a23184d3ce2d92ffa39c71505de12 Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | winrt: Add missing error handling in connectToServiceHelperOliver Wolff2018-08-161-1/+2
| | | | | | | | | | | | | | | | | | QBluetoothSocketPrivateWinRT::connectToServiceHelper was missing one hr check. Change-Id: Iae357df99881874b2d672dde6625e36d6221de0f Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* | QBluetoothSocket: Move (dis-)connected logic into setSocketStateOliver Wolff2018-08-165-20/+16
| | | | | | | | | | | | | | | | | | | | | | | | Instead of duplicating that logic everywhere, it can be handled in one place. Additionally setOpenMode should be called before setSocketState so that every member variable is changed, before signals are emitted. Change-Id: Ic1d4317ba31046d78d97874ec00c59481a67bb50 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Replace foreach with for loop and set QT_NO_FOREACHOliver Wolff2018-08-1558-152/+205
| | | | | | | | | | | | | | | | To avoid unnecessary copies, const is used wherever possible. Change-Id: Ic743716512751cfd24fad5bd37c244b115dd26fe Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | winrt: Handle UUID cases in service registrationOliver Wolff2018-08-151-13/+11
| | | | | | | | | | | | Change-Id: I227f9bd81e666d398366fc573573a0df973a5ff8 Reviewed-by: Andre de la Rocha <andre.rocha@qt.io> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Bluez: Fix missing signalling for QBluetoothSocket::abortAlex Blasche2018-08-151-0/+7
| | | | | | | | | | | | | | | | This is a bug introduced by 45c843ceca9fac1aa1246043730eaeeb3ed23235 where this backend was simply forgotten. Change-Id: I11c9eb1476a3ded7d0c12aaf1d203ff61194f3c4 Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
* | Android: Fix missing connected() signal emission for QBluetoothSocketAlex Blasche2018-08-141-1/+1
| | | | | | | | | | | | | | | | This only happens when QBluetoothSocket was used in connection with QBluetoothServer. Change-Id: I9fcc3026415f815f9a44a10d721056437465f8b8 Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* | winrt: Support additional data types for service registrationOliver Wolff2018-08-131-18/+55
| | | | | | | | | | Change-Id: I7ec6b6ef65a8754b22d1b97e508d7231b2a410c0 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | winrt: Unify logging format and give additional informationOliver Wolff2018-08-131-8/+11
| | | | | | | | | | Change-Id: I9f28e94556e596794e52bd571b36590487bd3cc1 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | winrt: Make device radio discoverable when listening on socketOliver Wolff2018-08-131-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | Microsoft changed the default behavior so that the device is no longer radio discoverable, when advertising is started. By creating the service provider from the Xaml thread we avoid messages, that it cannot be used in the background by default (even though this does not seem to have an effect on its functionality). Change-Id: Ib199adc53ef02b14fcea2884fe4359ad2fe81b8e Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Merge remote-tracking branch 'origin/5.11' into devQt Forward Merge Bot2018-08-111-61/+63
|\| | | | | | | Change-Id: I0f03c92b104e3d8280c0647cd13ce92c505e6472
| * QBluetoothServiceInfo - disentangle q<->d madnessTimur Pocheptsov2018-08-091-61/+63
| | | | | | | | | | | | | | | | | | | | Having d_ptr as a shared pointer in a copyable q-object, and a q_ptr in this d-object was probably the dumbest idea ever, not sure how this happened at all. Task-number: QTBUG-69857 Change-Id: I845394604d42879ca36f0b376ba94819e223df77 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Fix QBluetoothSocket::socketDescriptor() for BlueZ DBusAlex Blasche2018-08-091-0/+3
| | | | | | | | | | | | Task-number: QTBUG-68550 Change-Id: Ibc22b9a51336855c0a981f9280e6ae9ede9437b4 Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
* | Shift close/abort signaling from QBluetoothSocket to private implementationsAlex Blasche2018-08-095-19/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The private implementation can much more easily determine whether the socket closure is already done or not. On Bluez DBus, this avoids disconnected() being emitted twice. Another platform that is still delayed is Android. The patch permits the removal of Android specific ifdefs. Last but not least the patch cleans up missing signals in WinRT. Task-number: QTBUG-68550 Change-Id: I189e1dbc9f6d410522da1a82113fdf4fe79a4cbd Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>