diff --git a/src/kcms/kio/kcookiesmanagement.cpp b/src/kcms/kio/kcookiesmanagement.cpp --- a/src/kcms/kio/kcookiesmanagement.cpp +++ b/src/kcms/kio/kcookiesmanagement.cpp @@ -163,9 +163,10 @@ { cookiesDom.next(); CookiePropList list = cookiesDom.value(); - foreach(CookieProp *cookie, list) - { - QDBusInterface kded(QStringLiteral("org.kde.kcookiejar5"), QStringLiteral("/modules/kcookiejar"), QStringLiteral("org.kde.KCookieServer"), QDBusConnection::sessionBus()); + for (auto it = list.begin(); it < list.end(); ++it) { + CookieProp *cookie = *it; + QDBusInterface kded(QStringLiteral("org.kde.kcookiejar5"), QStringLiteral("/modules/kcookiejar"), + QStringLiteral("org.kde.KCookieServer"), QDBusConnection::sessionBus()); QDBusReply reply = kded.call( QStringLiteral("deleteCookie"), cookie->domain, cookie->host, cookie->path, cookie->name ); diff --git a/src/kcms/kio/kcookiespolicies.cpp b/src/kcms/kio/kcookiespolicies.cpp --- a/src/kcms/kio/kcookiespolicies.cpp +++ b/src/kcms/kio/kcookiespolicies.cpp @@ -247,7 +247,8 @@ { QTreeWidgetItem* nextItem = nullptr; - Q_FOREACH (QTreeWidgetItem * item, mUi.policyTreeWidget->selectedItems()) { + const QList selectedItems = mUi.policyTreeWidget->selectedItems(); + for (const QTreeWidgetItem *item : selectedItems) { nextItem = mUi.policyTreeWidget->itemBelow (item); if (!nextItem) nextItem = mUi.policyTreeWidget->itemAbove (item); diff --git a/src/kcms/webshortcuts/main.h b/src/kcms/webshortcuts/main.h --- a/src/kcms/webshortcuts/main.h +++ b/src/kcms/webshortcuts/main.h @@ -31,7 +31,7 @@ KURIFilterModule(QWidget *parent, const QVariantList &args); ~KURIFilterModule(); - void load() override; + void load() override; // TODO KF6: remove it, not needed void save() override; void defaults() override; diff --git a/src/kcms/webshortcuts/main.cpp b/src/kcms/webshortcuts/main.cpp --- a/src/kcms/webshortcuts/main.cpp +++ b/src/kcms/webshortcuts/main.cpp @@ -108,14 +108,14 @@ void KURIFilterModule::save() { - foreach(KCModule * module, modules) { + for (KCModule *module : qAsConst(modules)) { module->save(); } } void KURIFilterModule::defaults() { - foreach(KCModule * module, modules) { + for (KCModule *module : qAsConst(modules)) { module->defaults(); } }