diff --git a/cli/kdeconnect-cli.cpp b/cli/kdeconnect-cli.cpp --- a/cli/kdeconnect-cli.cpp +++ b/cli/kdeconnect-cli.cpp @@ -152,7 +152,8 @@ urls.append(url); // Check for more arguments - for (const QString& input : parser.positionalArguments()) { + const auto args = parser.positionalArguments(); + for (const QString& input : args) { QUrl url = QUrl::fromUserInput(input, QDir::currentPath()); urls.append(url); } diff --git a/core/daemon.cpp b/core/daemon.cpp --- a/core/daemon.cpp +++ b/core/daemon.cpp @@ -289,7 +289,7 @@ QStringList Daemon::pairingRequests() const { QStringList ret; - for(Device* dev: d->m_devices) { + for(Device* dev: qAsConst(d->m_devices)) { if (dev->hasPairingRequests()) ret += dev->id(); } diff --git a/core/device.cpp b/core/device.cpp --- a/core/device.cpp +++ b/core/device.cpp @@ -441,7 +441,7 @@ QHostAddress Device::getLocalIpAddress() const { - for (DeviceLink* dl : d->m_deviceLinks) { + for (DeviceLink* dl : qAsConst(d->m_deviceLinks)) { LanDeviceLink* ldl = dynamic_cast(dl); if (ldl) { return ldl->hostAddress(); diff --git a/interfaces/remotesinksmodel.cpp b/interfaces/remotesinksmodel.cpp --- a/interfaces/remotesinksmodel.cpp +++ b/interfaces/remotesinksmodel.cpp @@ -76,16 +76,16 @@ this, &RemoteSinksModel::refreshSinkList); connect(m_dbusInterface, &OrgKdeKdeconnectDeviceRemotesystemvolumeInterface::volumeChanged, this, [this](const QString& name, int volume) { - for (Sink* s: m_sinkList) { + for (Sink* s: qAsConst(m_sinkList)) { if (s->name == name) { s->volume = volume; Q_EMIT dataChanged(index(0,0), index(m_sinkList.size() - 1, 0)); } } }); connect(m_dbusInterface, &OrgKdeKdeconnectDeviceRemotesystemvolumeInterface::mutedChanged, this, [this](const QString& name, bool muted) { - for (Sink* s: m_sinkList) { + for (Sink* s: qAsConst(m_sinkList)) { if (s->name == name) { s->muted = muted; Q_EMIT dataChanged(index(0,0), index(m_sinkList.size() - 1, 0)); diff --git a/plugins/sendnotifications/sendnotifications_config.cpp b/plugins/sendnotifications/sendnotifications_config.cpp --- a/plugins/sendnotifications/sendnotifications_config.cpp +++ b/plugins/sendnotifications/sendnotifications_config.cpp @@ -109,8 +109,9 @@ config()->set(QStringLiteral("generalUrgency"), m_ui->spin_urgency->value()); QVariantList list; - list.reserve(appModel->apps().size()); - for (const auto& a: appModel->apps()) { + const auto apps = appModel->apps(); + list.reserve(apps.size()); + for (const auto& a: apps) { list.append(QVariant::fromValue(a)); } config()->setList(QStringLiteral("applications"), list); diff --git a/plugins/systemvolume/systemvolumeplugin-pulse.cpp b/plugins/systemvolume/systemvolumeplugin-pulse.cpp --- a/plugins/systemvolume/systemvolumeplugin-pulse.cpp +++ b/plugins/systemvolume/systemvolumeplugin-pulse.cpp @@ -74,7 +74,8 @@ sinksMap.clear(); - for (PulseAudioQt::Sink* sink : PulseAudioQt::Context::instance()->sinks()) { + const auto sinks = PulseAudioQt::Context::instance()->sinks(); + for (PulseAudioQt::Sink* sink : sinks) { sinksMap.insert(sink->name(), sink); connect(sink, &PulseAudioQt::Sink::volumeChanged, this, [this, sink] { @@ -118,7 +119,8 @@ sendSinkList(); }); - for (PulseAudioQt::Sink* sink : PulseAudioQt::Context::instance()->sinks()) { + const auto sinks = PulseAudioQt::Context::instance()->sinks(); + for (PulseAudioQt::Sink* sink : sinks) { sinksMap.insert(sink->name(), sink); } } diff --git a/tests/testsocketlinereader.cpp b/tests/testsocketlinereader.cpp --- a/tests/testsocketlinereader.cpp +++ b/tests/testsocketlinereader.cpp @@ -67,7 +67,7 @@ { QList dataToSend; dataToSend << "foobar\n" << "barfoo\n" << "foobar?\n" << "\n" << "barfoo!\n" << "panda\n"; - for (const QByteArray& line : dataToSend) { + for (const QByteArray& line : qAsConst(dataToSend)) { m_conn->write(line); } m_conn->flush(); diff --git a/tests/testsslsocketlinereader.cpp b/tests/testsslsocketlinereader.cpp --- a/tests/testsslsocketlinereader.cpp +++ b/tests/testsslsocketlinereader.cpp @@ -138,7 +138,7 @@ QList dataToSend; dataToSend << "foobar\n" << "barfoo\n" << "foobar?\n" << "\n" << "barfoo!\n" << "panda\n"; - for (const QByteArray& line : dataToSend) { + for (const QByteArray& line : qAsConst(dataToSend)) { m_clientSocket->write(line); } m_clientSocket->flush(); @@ -197,7 +197,7 @@ QList dataToSend; dataToSend << "foobar\n" << "barfoo\n" << "foobar?\n" << "\n" << "barfoo!\n" << "panda\n"; - for (const QByteArray& line : dataToSend) { + for (const QByteArray& line : qAsConst(dataToSend)) { m_clientSocket->write(line); } m_clientSocket->flush();