From dec2572ec616fdb2818632a49dcad00447d9c680 Mon Sep 17 00:00:00 2001 From: Tomaz Canabrava Date: Fri, 8 Jan 2016 15:32:06 -0200 Subject: Don't if true then true Set the values directly, seems much more sane. Signed-off-by: Tomaz Canabrava Signed-off-by: Dirk Hohndel --- desktop-widgets/printoptions.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/desktop-widgets/printoptions.cpp b/desktop-widgets/printoptions.cpp index 29aba313d..75d6ad8c4 100644 --- a/desktop-widgets/printoptions.cpp +++ b/desktop-widgets/printoptions.cpp @@ -34,10 +34,8 @@ void PrintOptions::setup() setupTemplates(); // general print option checkboxes - if (printOptions->color_selected) - ui.printInColor->setChecked(true); - if (printOptions->print_selected) - ui.printSelected->setChecked(true); + ui.printInColor->setChecked(printOptions->color_selected); + ui.printSelected->setChecked(printOptions->print_selected); // connect slots only once if (hasSetupSlots) -- cgit v1.2.3-70-g09d2