xconfig: clean up
authorLi Zefan <lizf@cn.fujitsu.com>
Fri, 7 May 2010 05:58:04 +0000 (13:58 +0800)
committerMichal Marek <mmarek@suse.cz>
Wed, 2 Jun 2010 13:10:33 +0000 (15:10 +0200)
@ok is a pointer to a bool var, so we should check the value of
*ok. But actually we don't need to check it, so just remove the
if statement.

Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Michal Marek <mmarek@suse.cz>
scripts/kconfig/qconf.cc

index 00c51507cfcc44acfe434fc35e08827c14ff7f0d..47cdeae8378c62a46488936d9197db9dbca6fcf1 100644 (file)
@@ -58,11 +58,10 @@ QValueList<int> ConfigSettings::readSizes(const QString& key, bool *ok)
 {
        QValueList<int> result;
        QStringList entryList = readListEntry(key, ok);
-       if (ok) {
-               QStringList::Iterator it;
-               for (it = entryList.begin(); it != entryList.end(); ++it)
-                       result.push_back((*it).toInt());
-       }
+       QStringList::Iterator it;
+
+       for (it = entryList.begin(); it != entryList.end(); ++it)
+               result.push_back((*it).toInt());
 
        return result;
 }