From d40b7586c3d1f1020a9fefd52bec75161edc63e8 Mon Sep 17 00:00:00 2001
From: Ben Boeckel <MathStuf@gmail.com>
Date: Tue, 17 Mar 2009 19:07:08 -0400
Subject: Better loop for cleaning up the value list

---
 sigscript/Config.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'sigscript')

diff --git a/sigscript/Config.cpp b/sigscript/Config.cpp
index 9218ee96..370f1b4a 100644
--- a/sigscript/Config.cpp
+++ b/sigscript/Config.cpp
@@ -96,10 +96,11 @@ QStringList Config::values(const bool recursive) const
     QStringList values = m_values.keys();
     if (recursive && m_parent)
         values += m_parent->values(true);
-    for (QMutableStringListIterator i(values); i.hasNext(); i.next())
+    const QStringList keys = m_values.keys();
+    foreach (const QString& key, keys)
     {
-        if (m_values.contains(i.value()) && (m_values[i.value()].second & (Deleted | Hidden)))
-            i.remove();
+        if (m_values[key].second & (Deleted | Hidden))
+            values.removeAll(key);
     }
     return values.toSet().toList();
 }
-- 
cgit