summary refs log tree commit diff
path: root/scripts/kconfig
diff options
context:
space:
mode:
authorLi Zefan <lizf@cn.fujitsu.com>2010-05-07 13:58:04 +0800
committerMichal Marek <mmarek@suse.cz>2010-06-02 15:10:33 +0200
commitc1f96f091ee146836dd73ecce531f8e0a170cfca (patch)
tree99419b51a9638aebab16c03db27d9989813371ce /scripts/kconfig
parentc10d03caf303d91da07c7f093a4822453c13f9b1 (diff)
downloadlinux-c1f96f091ee146836dd73ecce531f8e0a170cfca.tar.gz
xconfig: clean up
@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>
Diffstat (limited to 'scripts/kconfig')
-rw-r--r--scripts/kconfig/qconf.cc9
1 files changed, 4 insertions, 5 deletions
diff --git a/scripts/kconfig/qconf.cc b/scripts/kconfig/qconf.cc
index 00c51507cfcc..47cdeae8378c 100644
--- a/scripts/kconfig/qconf.cc
+++ b/scripts/kconfig/qconf.cc
@@ -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;
 }