[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [brprint3d] /: Use boolean directly instead of if(true) then true else false
From:       Tomaz Canabrava <tomaz.canabrava () intel ! com>
Date:       2015-11-30 23:47:44
Message-ID: E1a3YAa-0000MT-GX () scm ! kde ! org
[Download RAW message or body]

Git commit a124c476d0de1fd186769b4a04717e376a0964b7 by Tomaz Canabrava.
Committed on 30/11/2015 at 23:38.
Pushed by tcanabrava into branch 'master'.

Use boolean directly instead of if(true) then true else false

Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>

M  +17   -54   ManualControlWidget.cpp

http://commits.kde.org/brprint3d/a124c476d0de1fd186769b4a04717e376a0964b7

diff --git a/ManualControlWidget.cpp b/ManualControlWidget.cpp
index 44ab72b..94e308b 100755
--- a/ManualControlWidget.cpp
+++ b/ManualControlWidget.cpp
@@ -195,65 +195,28 @@ void ManualControlWidget::disableExtrudersButtons(bool checked)
 {
     if(sender()==ui->bt_extruder1)
     {
-        if(checked)
-        {
-            ui->bt_extruder2->setEnabled(false);
-            ui->bt_extruder3->setEnabled(false);
-            ui->bt_extruder4->setEnabled(false);
-        }
-        else
-        {
-            ui->bt_extruder2->setEnabled(true);
-            ui->bt_extruder3->setEnabled(true);
-            ui->bt_extruder4->setEnabled(true);
-        }
-
+        ui->bt_extruder2->setEnabled(!checked);
+        ui->bt_extruder3->setEnabled(!checked);
+        ui->bt_extruder4->setEnabled(!checked);
     }
-    if(sender()==ui->bt_extruder2)
-    {   if(checked)
-        {
-            ui->bt_extruder1->setEnabled(false);
-            ui->bt_extruder3->setEnabled(false);
-            ui->bt_extruder4->setEnabled(false);
-        }
-        else
-        {
-            ui->bt_extruder1->setEnabled(true);
-            ui->bt_extruder3->setEnabled(true);
-            ui->bt_extruder4->setEnabled(true);
-        }
-
+    else if(sender()==ui->bt_extruder2)
+    {
+        ui->bt_extruder1->setEnabled(!checked);
+        ui->bt_extruder3->setEnabled(!checked);
+        ui->bt_extruder4->setEnabled(!checked);
     }
-    if(sender()==ui->bt_extruder3)
-    {    if(checked)
-        {
-            ui->bt_extruder2->setEnabled(false);
-            ui->bt_extruder1->setEnabled(false);
-            ui->bt_extruder4->setEnabled(false);
-        }
-        else
-        {
-            ui->bt_extruder2->setEnabled(true);
-            ui->bt_extruder1->setEnabled(true);
-            ui->bt_extruder4->setEnabled(true);
-        }
+    else if(sender()==ui->bt_extruder3)
+    {
+        ui->bt_extruder2->setEnabled(!checked);
+        ui->bt_extruder1->setEnabled(!checked);
+        ui->bt_extruder4->setEnabled(!checked);
 
     }
-    if(sender()==ui->bt_extruder4)
+    else if(sender()==ui->bt_extruder4)
     {
-        if(checked)
-        {
-            ui->bt_extruder2->setEnabled(false);
-            ui->bt_extruder3->setEnabled(false);
-            ui->bt_extruder1->setEnabled(false);
-        }
-        else
-        {
-            ui->bt_extruder2->setEnabled(true);
-            ui->bt_extruder3->setEnabled(true);
-            ui->bt_extruder1->setEnabled(true);
-        }
-
+        ui->bt_extruder2->setEnabled(!checked);
+        ui->bt_extruder3->setEnabled(!checked);
+        ui->bt_extruder1->setEnabled(!checked);
     }
 
 }
[prev in list] [next in list] [prev in thread] [next in thread]