From 4a3996e98b3fe4bb5ca06380264ae86715706cbc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?M=C3=A5rten=20Nordheim?= Date: Wed, 3 Nov 2021 15:14:51 +0100 Subject: QNI NetworkManager: Small refactorings and correctness fix Extract and reuse a lambda for initialization and slot object. Pass a QDBusObjectPath parameter by const-ref. Avoid implicit ascii-to-utf16 string conversion. Change-Id: I591ae6e58e010ccabcf9012db866949492e0191d Reviewed-by: Edward Welbourne Reviewed-by: Marc Mutz --- .../qnetworkmanagernetworkinformationbackend.cpp | 38 +++++++++------------- 1 file changed, 16 insertions(+), 22 deletions(-) (limited to 'src/plugins/networkinformation/networkmanager/qnetworkmanagernetworkinformationbackend.cpp') diff --git a/src/plugins/networkinformation/networkmanager/qnetworkmanagernetworkinformationbackend.cpp b/src/plugins/networkinformation/networkmanager/qnetworkmanagernetworkinformationbackend.cpp index ac16ad4b8d..7018504abe 100644 --- a/src/plugins/networkinformation/networkmanager/qnetworkmanagernetworkinformationbackend.cpp +++ b/src/plugins/networkinformation/networkmanager/qnetworkmanagernetworkinformationbackend.cpp @@ -204,32 +204,26 @@ private: QNetworkManagerNetworkInformationBackend::QNetworkManagerNetworkInformationBackend() { - using NMState = QNetworkManagerInterface::NMState; - setReachability(reachabilityFromNMState(iface.state())); - connect(&iface, &QNetworkManagerInterface::stateChanged, this, - [this](NMState newState) { - setReachability(reachabilityFromNMState(newState)); - }); - - using ConnectivityState = QNetworkManagerInterface::NMConnectivityState; - - const auto connectivityState = iface.connectivityState(); - const bool behindPortal = (connectivityState == ConnectivityState::NM_CONNECTIVITY_PORTAL); - setBehindCaptivePortal(behindPortal); + auto updateReachability = [this](QNetworkManagerInterface::NMState newState) { + setReachability(reachabilityFromNMState(newState)); + }; + updateReachability(iface.state()); + connect(&iface, &QNetworkManagerInterface::stateChanged, this, std::move(updateReachability)); + auto updateBehindCaptivePortal = [this](QNetworkManagerInterface::NMConnectivityState state) { + const bool behindPortal = (state == QNetworkManagerInterface::NM_CONNECTIVITY_PORTAL); + setBehindCaptivePortal(behindPortal); + }; + updateBehindCaptivePortal(iface.connectivityState()); connect(&iface, &QNetworkManagerInterface::connectivityChanged, this, - [this](ConnectivityState state) { - const bool behindPortal = (state == ConnectivityState::NM_CONNECTIVITY_PORTAL); - setBehindCaptivePortal(behindPortal); - }); - - using NMDeviceType = QNetworkManagerInterface::NMDeviceType; - setTransportMedium(transportMediumFromDeviceType(iface.deviceType())); + std::move(updateBehindCaptivePortal)); + auto updateTransportMedium = [this](QNetworkManagerInterface::NMDeviceType newDevice) { + setTransportMedium(transportMediumFromDeviceType(newDevice)); + }; + updateTransportMedium(iface.deviceType()); connect(&iface, &QNetworkManagerInterface::deviceTypeChanged, this, - [this](NMDeviceType newDevice) { - setTransportMedium(transportMediumFromDeviceType(newDevice)); - }); + std::move(updateTransportMedium)); auto updateMetered = [this](QNetworkManagerInterface::NMMetered metered) { setMetered(isMeteredFromNMMetered(metered)); -- cgit v1.2.3