aboutsummaryrefslogtreecommitdiff
path: root/src/core
diff options
context:
space:
mode:
Diffstat (limited to 'src/core')
-rw-r--r--src/core/kauthorized.cpp2
-rw-r--r--src/core/kconfig.cpp17
-rw-r--r--src/core/kconfiggroup.h7
-rw-r--r--src/core/kconfigini.cpp2
-rw-r--r--src/core/kcoreconfigskeleton.cpp6
-rw-r--r--src/core/kdesktopfile.cpp14
-rw-r--r--src/core/ksharedconfig.cpp2
7 files changed, 28 insertions, 22 deletions
diff --git a/src/core/kauthorized.cpp b/src/core/kauthorized.cpp
index b6054d76..12068dc5 100644
--- a/src/core/kauthorized.cpp
+++ b/src/core/kauthorized.cpp
@@ -391,7 +391,7 @@ KCONFIGCORE_EXPORT bool authorizeUrlActionInternal(const QString &action, const
QUrl destURL(_destURL);
destURL.setPath(QDir::cleanPath(destURL.path()));
- Q_FOREACH (const URLActionRule &rule, d->urlActionRestrictions) {
+ for (const URLActionRule &rule : qAsConst(d->urlActionRestrictions)) {
if ((result != rule.permission) && // No need to check if it doesn't make a difference
(action == QLatin1String(rule.action.constData())) &&
rule.baseMatch(baseURL, baseClass) &&
diff --git a/src/core/kconfig.cpp b/src/core/kconfig.cpp
index bdf89b1e..b78196d6 100644
--- a/src/core/kconfig.cpp
+++ b/src/core/kconfig.cpp
@@ -706,10 +706,10 @@ QStringList KConfigPrivate::getGlobalFiles() const
const bool useEtcKderc = !etc_kderc.isEmpty();
s_globalFiles()->reserve(paths1.size() + paths2.size() + (useEtcKderc ? 1 : 0));
- Q_FOREACH (const QString &dir1, paths1) {
+ for (const QString &dir1 : paths1) {
s_globalFiles()->push_front(dir1);
}
- Q_FOREACH (const QString &dir2, paths2) {
+ for (const QString &dir2 : paths2) {
s_globalFiles()->push_front(dir2);
}
@@ -729,7 +729,7 @@ void KConfigPrivate::parseGlobalFiles()
// TODO: can we cache the values in etc_kderc / other global files
// on a per-application basis?
const QByteArray utf8Locale = locale.toUtf8();
- Q_FOREACH (const QString &file, globalFiles) {
+ for (const QString &file : globalFiles) {
KConfigBackend::ParseOptions parseOpts = KConfigBackend::ParseGlobal | KConfigBackend::ParseExpansions;
if (file.compare(*sGlobalFileName, sPathCaseSensitivity) != 0)
@@ -760,7 +760,8 @@ void KConfigPrivate::parseConfigFiles()
files << canonicalFile;
}
} else {
- Q_FOREACH (const QString &f, QStandardPaths::locateAll(resourceType, fileName)) {
+ const QStringList localFilesPath = QStandardPaths::locateAll(resourceType, fileName);
+ for (const QString &f : localFilesPath) {
files.prepend(QFileInfo(f).canonicalFilePath());
}
@@ -781,7 +782,7 @@ void KConfigPrivate::parseConfigFiles()
// qDebug() << "parsing local files" << files;
const QByteArray utf8Locale = locale.toUtf8();
- foreach (const QString &file, files) {
+ for (const QString &file : qAsConst(files)) {
if (file.compare(mBackend->filePath(), sPathCaseSensitivity) == 0) {
switch (mBackend->parseConfig(utf8Locale, entryMap, KConfigBackend::ParseExpansions)) {
case KConfigBackend::ParseOk:
@@ -816,7 +817,7 @@ KConfig::AccessMode KConfig::accessMode() const
void KConfig::addConfigSources(const QStringList &files)
{
Q_D(KConfig);
- Q_FOREACH (const QString &file, files) {
+ for (const QString &file : files) {
d->extraFiles.push(file);
}
@@ -931,9 +932,9 @@ void KConfig::deleteGroupImpl(const QByteArray &aGroup, WriteConfigFlags flags)
KEntryMap::EntryOptions options = convertToOptions(flags) | KEntryMap::EntryDeleted;
const QSet<QByteArray> groups = d->allSubGroups(aGroup);
- Q_FOREACH (const QByteArray &group, groups) {
+ for (const QByteArray &group : groups) {
const QStringList keys = d->keyListImpl(group);
- Q_FOREACH (const QString &_key, keys) {
+ for (const QString &_key : keys) {
const QByteArray &key = _key.toUtf8();
if (d->canWriteEntry(group, key.constData())) {
d->entryMap.setEntry(group, key, QByteArray(), options);
diff --git a/src/core/kconfiggroup.h b/src/core/kconfiggroup.h
index f5e16fdb..0cedeaa7 100644
--- a/src/core/kconfiggroup.h
+++ b/src/core/kconfiggroup.h
@@ -731,12 +731,13 @@ QList<T> KConfigGroup::readEntry(const char *key, const QList<T> &defaultValue)
QVariantList data;
- Q_FOREACH (const T &value, defaultValue) {
+ for (const T &value : defaultValue) {
data.append(qVariantFromValue(value));
}
QList<T> list;
- Q_FOREACH (const QVariant &value, readEntry<QVariantList>(key, data)) {
+ const auto variantList = readEntry<QVariantList>(key, data);
+ for (const QVariant &value : variantList) {
Q_ASSERT(value.canConvert<T>());
list.append(qvariant_cast<T>(value));
}
@@ -757,7 +758,7 @@ void KConfigGroup::writeEntry(const char *key, const QList<T> &list, WriteConfig
ConversionCheck::to_QVariant<T>();
ConversionCheck::to_QString<T>();
QVariantList data;
- Q_FOREACH (const T &value, list) {
+ for (const T &value : list) {
data.append(qVariantFromValue(value));
}
diff --git a/src/core/kconfigini.cpp b/src/core/kconfigini.cpp
index 87c4a8af..26d8390c 100644
--- a/src/core/kconfigini.cpp
+++ b/src/core/kconfigini.cpp
@@ -299,7 +299,7 @@ KConfigIniBackend::parseConfig(const QByteArray &currentLocale, KEntryMap &entry
}
// now make sure immutable groups are marked immutable
- Q_FOREACH (const QByteArray &group, immutableGroups) {
+ for (const QByteArray &group : qAsConst(immutableGroups)) {
entryMap.setEntry(group, QByteArray(), QByteArray(), KEntryMap::EntryImmutable);
}
diff --git a/src/core/kcoreconfigskeleton.cpp b/src/core/kcoreconfigskeleton.cpp
index 01f69cbe..50f91e93 100644
--- a/src/core/kcoreconfigskeleton.cpp
+++ b/src/core/kcoreconfigskeleton.cpp
@@ -902,12 +902,12 @@ void KCoreConfigSkeleton::ItemUrlList::readConfig(KConfig *config)
mReference = mDefault;
} else {
QStringList strList;
- Q_FOREACH (const QUrl &url, mDefault) {
+ for (const QUrl &url : qAsConst(mDefault)) {
strList.append(url.toString());
}
mReference.clear();
const QStringList readList = cg.readEntry<QStringList>(mKey, strList);
- Q_FOREACH (const QString &str, readList) {
+ for (const QString &str : readList) {
mReference.append(QUrl(str));
}
}
@@ -924,7 +924,7 @@ void KCoreConfigSkeleton::ItemUrlList::writeConfig(KConfig *config)
cg.revertToDefault(mKey);
} else {
QStringList strList;
- Q_FOREACH (const QUrl &url, mReference) {
+ for (const QUrl &url : qAsConst(mReference)) {
strList.append(url.toString());
}
cg.writeEntry<QStringList>(mKey, strList);
diff --git a/src/core/kdesktopfile.cpp b/src/core/kdesktopfile.cpp
index 96d7b118..d9283cef 100644
--- a/src/core/kdesktopfile.cpp
+++ b/src/core/kdesktopfile.cpp
@@ -81,14 +81,16 @@ QString KDesktopFile::locateLocal(const QString &path)
QString relativePath;
QChar plus(QLatin1Char('/'));
// Relative to config? (e.g. for autostart)
- Q_FOREACH (const QString &dir, QStandardPaths::standardLocations(QStandardPaths::GenericConfigLocation)) {
+ const QStringList lstGenericConfigLocation = QStandardPaths::standardLocations(QStandardPaths::GenericConfigLocation);
+ for (const QString &dir : lstGenericConfigLocation) {
if (path.startsWith(dir + plus)) {
relativePath = path.mid(dir.length() + 1);
return QStandardPaths::writableLocation(QStandardPaths::GenericConfigLocation) + QLatin1Char('/') + relativePath;
}
}
// Relative to xdg data dir? (much more common)
- Q_FOREACH (const QString &dir, QStandardPaths::standardLocations(QStandardPaths::GenericDataLocation)) {
+ const QStringList lstGenericDataLocation = QStandardPaths::standardLocations(QStandardPaths::GenericDataLocation);
+ for (const QString &dir : lstGenericDataLocation) {
if (path.startsWith(dir + plus)) {
relativePath = path.mid(dir.length() + 1);
}
@@ -128,13 +130,14 @@ bool KDesktopFile::isAuthorizedDesktopFile(const QString &path)
// Check if the .desktop file is installed as part of KDE or XDG.
const QStringList appsDirs = QStandardPaths::standardLocations(QStandardPaths::ApplicationsLocation);
- Q_FOREACH (const QString &prefix, appsDirs) {
+ for (const QString &prefix : appsDirs) {
if (QDir(prefix).exists() && realPath.startsWith(QFileInfo(prefix).canonicalFilePath(), sensitivity)) {
return true;
}
}
const QString servicesDir = QStringLiteral("kservices5/"); // KGlobal::dirs()->xdgDataRelativePath("services")
- Q_FOREACH (const QString &xdgDataPrefix, QStandardPaths::standardLocations(QStandardPaths::GenericDataLocation)) {
+ const QStringList lstGenericDataLocation = QStandardPaths::standardLocations(QStandardPaths::GenericDataLocation);
+ for (const QString &xdgDataPrefix : lstGenericDataLocation) {
if (QDir(xdgDataPrefix).exists()) {
const QString prefix = QFileInfo(xdgDataPrefix).canonicalFilePath();
if (realPath.startsWith(prefix + QLatin1Char('/') + servicesDir, sensitivity)) {
@@ -143,7 +146,8 @@ bool KDesktopFile::isAuthorizedDesktopFile(const QString &path)
}
}
const QString autostartDir = QStringLiteral("autostart/");
- Q_FOREACH (const QString &xdgDataPrefix, QStandardPaths::standardLocations(QStandardPaths::GenericConfigLocation)) {
+ const QStringList lstConfigPath = QStandardPaths::standardLocations(QStandardPaths::GenericConfigLocation);
+ for (const QString &xdgDataPrefix : lstConfigPath) {
if (QDir(xdgDataPrefix).exists()) {
const QString prefix = QFileInfo(xdgDataPrefix).canonicalFilePath();
if (realPath.startsWith(prefix + QLatin1Char('/') + autostartDir, sensitivity)) {
diff --git a/src/core/ksharedconfig.cpp b/src/core/ksharedconfig.cpp
index 52a2d23d..0b2b9c35 100644
--- a/src/core/ksharedconfig.cpp
+++ b/src/core/ksharedconfig.cpp
@@ -90,7 +90,7 @@ KSharedConfigPtr KSharedConfig::openConfig(const QString &_fileName,
list->mainConfig = nullptr;
}
- foreach (auto cfg, *static_cast<const GlobalSharedConfigList*>(list)) {
+ for (auto cfg : qAsConst(*list)) {
if (cfg->name() == fileName &&
cfg->d_ptr->openFlags == flags &&
cfg->locationType() == resType
generated by cgit v1.2.1 (git 2.18.0) at 2025-05-04 21:34:00 +0000