diff options
author | Lorn Potter <lorn.potter@jollamobile.com> | 2013-10-21 10:38:09 +1000 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-12-16 17:32:33 +0100 |
commit | bfe005a888625d69e461f0c4241c2e323c93b973 (patch) | |
tree | 1c75b10182e72889c0535e940d85cd9afa7e189f /src/plugins | |
parent | 9518c17837ba37f743ca87890105654cfb3d1564 (diff) |
make sure to signal only changed configurations
really use this
Change-Id: I082f8d84da572a2c51f67355dcbc06394940c421
Reviewed-by: Robin Burchell <robin+qt@viroteck.net>
Diffstat (limited to 'src/plugins')
-rw-r--r-- | src/plugins/bearer/connman/qconnmanengine.cpp | 12 |
1 files changed, 8 insertions, 4 deletions
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(); |