diff options
| author | 2 * r + 2 * t <61896496+soramanew@users.noreply.github.com> | 2025-09-23 17:34:20 +1000 |
|---|---|---|
| committer | 2 * r + 2 * t <61896496+soramanew@users.noreply.github.com> | 2025-09-23 17:34:20 +1000 |
| commit | 7aaa14438f7d674707d4b8591364ee457248d0e6 (patch) | |
| tree | 83c01a7433dd178d3033086c68a91ad10079993b /plugin/src/Caelestia/appdb.cpp | |
| parent | dev: build with clazy (diff) | |
| download | caelestia-shell-7aaa14438f7d674707d4b8591364ee457248d0e6.tar.gz caelestia-shell-7aaa14438f7d674707d4b8591364ee457248d0e6.tar.bz2 caelestia-shell-7aaa14438f7d674707d4b8591364ee457248d0e6.zip | |
plugin: fix clazy warnings
Diffstat (limited to 'plugin/src/Caelestia/appdb.cpp')
| -rw-r--r-- | plugin/src/Caelestia/appdb.cpp | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/plugin/src/Caelestia/appdb.cpp b/plugin/src/Caelestia/appdb.cpp index 19db096..1fa2d77 100644 --- a/plugin/src/Caelestia/appdb.cpp +++ b/plugin/src/Caelestia/appdb.cpp @@ -154,7 +154,7 @@ void AppDb::incrementFrequency(const QString& id) { query.bindValue(":id", id); query.exec(); - for (auto app : m_apps) { + for (auto* app : std::as_const(m_apps)) { if (app->id() == id) { const auto before = apps(); @@ -186,7 +186,7 @@ quint32 AppDb::getFrequency(const QString& id) const { } void AppDb::updateAppFrequencies() { - for (auto app : m_apps) { + for (auto* app : std::as_const(m_apps)) { app->setFrequency(getFrequency(app->id())); } } @@ -194,7 +194,7 @@ void AppDb::updateAppFrequencies() { void AppDb::updateApps() { bool dirty = false; - for (auto entry : m_entries) { + for (const auto& entry : std::as_const(m_entries)) { const auto id = entry->property("id").toString(); if (!m_apps.contains(id)) { dirty = true; @@ -203,12 +203,13 @@ void AppDb::updateApps() { } QSet<QString> newIds; - for (auto entry : m_entries) { + for (const auto& entry : std::as_const(m_entries)) { newIds.insert(entry->property("id").toString()); } QList<AppEntry*> toDelete; - for (auto id : m_apps.keys()) { + for (auto it = m_apps.keyBegin(); it != m_apps.keyEnd(); ++it) { + const auto& id = *it; if (!newIds.contains(id)) { dirty = true; toDelete << m_apps.take(id); |