From 0b0b9261e5e78fe9aa95f3b48ba2ea5f4a7dcd3d Mon Sep 17 00:00:00 2001
From: Ivailo Monev <xakepa10@gmail.com>
Date: Sat, 10 Jun 2023 21:16:33 +0300
Subject: [PATCH] kwin: minor KWin::Client::setShortcut() optimization

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
---
 kwin/effects/thumbnailaside/thumbnailaside.cpp | 3 ++-
 kwin/useractions.cpp                           | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/kwin/effects/thumbnailaside/thumbnailaside.cpp b/kwin/effects/thumbnailaside/thumbnailaside.cpp
index ac16484c..52e0ddd0 100644
--- a/kwin/effects/thumbnailaside/thumbnailaside.cpp
+++ b/kwin/effects/thumbnailaside/thumbnailaside.cpp
@@ -175,8 +175,9 @@ void ThumbnailAsideEffect::arrange()
 
 void ThumbnailAsideEffect::repaintAll()
 {
-    foreach (const Data & d, windows)
+    foreach (const Data & d, windows) {
         effects->addRepaint(d.rect);
+    }
 }
 
 bool ThumbnailAsideEffect::isActive() const
diff --git a/kwin/useractions.cpp b/kwin/useractions.cpp
index a94c572b..cf2b0b0b 100755
--- a/kwin/useractions.cpp
+++ b/kwin/useractions.cpp
@@ -1784,7 +1784,7 @@ void Client::setShortcut(const QString& _cut)
     }
     QList< KShortcut > keys;
     QStringList groups = cut.split(" - ");
-    foreach (const QString it, groups) {
+    foreach (const QString &it, groups) {
         QRegExp reg("(.*\\+)\\((.*)\\)");
         if (reg.indexIn(it) > -1) {
             QString base = reg.cap(1);
@@ -1804,7 +1804,7 @@ void Client::setShortcut(const QString& _cut)
             }
         }
     }
-    foreach (const KShortcut it, keys) {
+    foreach (const KShortcut &it, keys) {
         if (_shortcut == it)   // current one is in the list
             return;
         if (workspace()->shortcutAvailable(it, this)) {