From bfe005a888625d69e461f0c4241c2e323c93b973 Mon Sep 17 00:00:00 2001 From: Lorn Potter Date: Mon, 21 Oct 2013 10:38:09 +1000 Subject: make sure to signal only changed configurations really use this Change-Id: I082f8d84da572a2c51f67355dcbc06394940c421 Reviewed-by: Robin Burchell --- src/plugins/bearer/connman/qconnmanengine.cpp | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) (limited to 'src/plugins/bearer') diff --git a/src/plugins/bearer/connman/qconnmanengine.cpp b/src/plugins/bearer/connman/qconnmanengine.cpp index d9e7a1621a..24751b509f 100644 --- a/src/plugins/bearer/connman/qconnmanengine.cpp +++ b/src/plugins/bearer/connman/qconnmanengine.cpp @@ -350,7 +350,7 @@ void QConnmanEngine::configurationChange(const QString &id) QMutexLocker locker(&mutex); if (accessPointConfigurations.contains(id)) { - + bool changed = false; QNetworkConfigurationPrivatePointer ptr = accessPointConfigurations.value(id); QString servicePath = serviceFromId(id); @@ -368,17 +368,21 @@ void QConnmanEngine::configurationChange(const QString &id) if (ptr->name != networkName) { ptr->name = networkName; + changed = true; } if (ptr->state != curState) { ptr->state = curState; + changed = true; } ptr->mutex.unlock(); - locker.unlock(); - emit configurationChanged(ptr); - locker.relock(); + if (changed) { + locker.unlock(); + emit configurationChanged(ptr); + locker.relock(); + } } locker.unlock(); -- cgit v1.2.3