Minor code clean up

Signed-off-by: Michele Calgaro <michele.calgaro@yahoo.it>

1 files changed, 6 insertions, 14 deletions

diff --git a/src/tdepowersave.cpp b/src/tdepowersave.cpp
index bac1b8e..a08e8de 100644
--- a/src/tdepowersave.cpp
+++ b/src/tdepowersave.cpp
@@ -2651,22 +2651,18 @@ TQStringList tdepowersave::listCPUFreqPolicies() {
 bool tdepowersave::do_setCPUFreqPolicy( TQString policy ) {
 	if (trace) kdDebug() << funcinfo << "IN: " << "policy: " << policy << endl;
 
-
-	bool ret = true;
-
+	bool ret = false;
 	if (hwinfo->isCpuFreqAllowed() && hwinfo->isOnline()) {
 		if (policy == "PERFORMANCE") {
 			hwinfo->setCPUFreq(PERFORMANCE);
+			ret = true;
 		} else if (policy == "DYNAMIC") {
 			hwinfo->setCPUFreq(DYNAMIC, settings->cpuFreqDynamicPerformance);
+			ret = true;
 		} else if (policy == "POWERSAVE") {
 			hwinfo->setCPUFreq(POWERSAVE);
-		} else {
-			kdDebugFuncOut(trace);
-			ret = false;
+			ret = true;
 		}
-	} else {
-		ret = false;
 	}
 
 	kdDebugFuncOut(trace);
@@ -2746,18 +2742,14 @@ TQStringList tdepowersave::listSchemes(){
 bool tdepowersave::do_setScheme( TQString _scheme ) {
 	if (trace) kdDebug() << funcinfo << "IN: scheme: " << _scheme << endl;
 
-	int index;
-	index = settings->schemes.findIndex(_scheme);
+	int index = settings->schemes.findIndex(_scheme);
 	if (trace) kdDebug() << funcinfo << " index: " << index << endl;
 	if (index != -1) {
 		do_setActiveScheme(index);
 		kdDebugFuncOut(trace);
 		return true;
 	}
-	else {
-		kdDebugFuncOut(trace);
-		return false;
-	}
+
 	kdDebugFuncOut(trace);
 	return false;
 }