diff options
author | Ben Boeckel <MathStuf@gmail.com> | 2009-03-17 19:07:08 -0400 |
---|---|---|
committer | Ben Boeckel <MathStuf@gmail.com> | 2009-03-17 19:07:08 -0400 |
commit | d40b7586c3d1f1020a9fefd52bec75161edc63e8 (patch) | |
tree | d70d692438bd0720c1f34a00130fab7762bc36f8 /sigscript | |
parent | f59bbe43dacc5a17a8917d3b67e7d79f46e2bd72 (diff) | |
download | sigen-d40b7586c3d1f1020a9fefd52bec75161edc63e8.tar.gz sigen-d40b7586c3d1f1020a9fefd52bec75161edc63e8.tar.xz sigen-d40b7586c3d1f1020a9fefd52bec75161edc63e8.zip |
Better loop for cleaning up the value list
Diffstat (limited to 'sigscript')
-rw-r--r-- | sigscript/Config.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
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(); } |