From 8687f8cfdaf689aea5fb2cdb8043c5c4d5a93658 Mon Sep 17 00:00:00 2001 From: Ali Abdel-Qader Date: Sat, 6 May 2023 15:51:53 -0400 Subject: [PATCH] Use explicit constructor for QSslCertificate with value initialized argument Previously the BluetoothDeviceLink::certificate() method was returning a value initialized object which I believe default initializes the object. However, Clang throws a build error at this because QSslCertificate has explicit constructors. This change uses one of those constructors and uses value intialization to default construct/initialize the argument for it. It fixes the build and hopefully doesn't break anything since this is a TODO anyways! BUG: 469428 Signed-off-by: Ali Abdel-Qader (cherry picked from commit bbac0aa0852b14bc5cc9a1b8c3be0f55adda2428) --- core/backends/bluetooth/bluetoothdevicelink.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/backends/bluetooth/bluetoothdevicelink.cpp b/core/backends/bluetooth/bluetoothdevicelink.cpp index 1ab401fc9..facf5deed 100644 --- a/core/backends/bluetooth/bluetoothdevicelink.cpp +++ b/core/backends/bluetooth/bluetoothdevicelink.cpp @@ -99,5 +99,5 @@ void BluetoothDeviceLink::dataReceived() QSslCertificate BluetoothDeviceLink::certificate() const { - return {}; // TODO Not sure what to do here. For LanDeviceLink we use the SSL connection's certificate, but we don't have that here + return QSslCertificate({}); // TODO Not sure what to do here. For LanDeviceLink we use the SSL connection's certificate, but we don't have that here }