summaryrefslogtreecommitdiffstats
path: root/desktop-widgets/printoptions.cpp
diff options
context:
space:
mode:
authorGravatar Tomaz Canabrava <tomaz.canabrava@intel.com>2016-01-08 15:32:06 -0200
committerGravatar Dirk Hohndel <dirk@hohndel.org>2016-01-08 10:02:20 -0800
commitdec2572ec616fdb2818632a49dcad00447d9c680 (patch)
tree67cce0685a18eb296060b73e96deed5ccfb64af0 /desktop-widgets/printoptions.cpp
parent1210d4b575728f2acacb2813a69cf0ee4920a9be (diff)
downloadsubsurface-dec2572ec616fdb2818632a49dcad00447d9c680.tar.gz
Don't if true then true
Set the values directly, seems much more sane. Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'desktop-widgets/printoptions.cpp')
-rw-r--r--desktop-widgets/printoptions.cpp6
1 files 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)