Use std library find algorithm

Also avoids mixing const and non-const iterators.
lxqt · May 3, 2024 · b218e9e · b218e9e
1 parent a8d1e67
commit b218e9e
Showing 1 changed file with 18 additions and 14 deletions.
diff --git a/lib/kprocess.cpp b/lib/kprocess.cpp
@@ -85,14 +85,16 @@ void KProcess::setEnv(const QString &name, const QString &value, bool overwrite)
     }
     QString fname(name);
     fname.append(QLatin1Char('='));
-    for (QStringList::Iterator it = env.begin(); it != env.end(); ++it)
-        if ((*it).startsWith(fname)) {
-            if (overwrite) {
-                *it = fname.append(value);
-                setEnvironment(env);
-            }
-            return;
+    auto it = std::find_if(env.begin(), env.end(), [&fname](const QString &s) {
+        return s.startsWith(fname);
+    });
+    if (it != env.end()) {
+        if (overwrite) {
+            *it = fname.append(value);
+            setEnvironment(env);
         }
+        return;
+    }
     env.append(fname.append(value));
     setEnvironment(env);
 }
@@ -106,14 +108,16 @@ void KProcess::unsetEnv(const QString &name)
     }
     QString fname(name);
     fname.append(QLatin1Char('='));
-    for (QStringList::Iterator it = env.begin(); it != env.end(); ++it)
-        if ((*it).startsWith(fname)) {
-            env.erase(it);
-            if (env.isEmpty())
-                env.append(QString::fromLatin1(DUMMYENV));
-            setEnvironment(env);
-            return;
+    auto it = std::find_if(env.begin(), env.end(), [&fname](const QString &s) {
+        return s.startsWith(fname);
+    });
+    if (it != env.end()) {
+        env.erase(it);
+        if (env.isEmpty()) {
+            env.append(QString::fromLatin1(DUMMYENV));
         }
+        setEnvironment(env);
+    }
 }
 
 void KProcess::setProgram(const QString &exe, const QStringList &args)