Fixed FPS limit being enabled by default

There were some inconsistencies between the values set up in
`Client::initConfiguration` and those in `getConfigDefaults`. These
duplicates have now been removed.

For some of these settings the code getting the values had to be
adjusted to use getBoolValue, to actually rely on the provided default
instead of one provided as a parameter.

1 files changed, 5 insertions, 5 deletions

diff --git a/src/gui/setup_video.cpp b/src/gui/setup_video.cpp
index 0d6c9cce..5f7dc7b4 100644
--- a/src/gui/setup_video.cpp
+++ b/src/gui/setup_video.cpp
@@ -397,22 +397,22 @@ void Setup_Video::apply()
     // If LowCPU is enabled from a disabled state we warn the user
     else if (mDisableSDLTransparencyCheckBox->isSelected())
     {
-        if (config.getValue("disableTransparency", true) == false)
+        if (!config.getBoolValue("disableTransparency"))
         {
             new OkDialog(_("Transparency disabled"),
-                 _("You must restart to apply changes."));
+                         _("You must restart to apply changes."));
         }
     }
     else
     {
-        if (config.getValue("disableTransparency", true) == true)
+        if (config.getBoolValue("disableTransparency"))
         {
             new OkDialog(_("Transparency enabled"),
-                 _("You must restart to apply changes."));
+                         _("You must restart to apply changes."));
         }
     }
     config.setValue("disableTransparency",
-                                 mDisableSDLTransparencyCheckBox->isSelected());
+                    mDisableSDLTransparencyCheckBox->isSelected());
 
     mFps = mFpsCheckBox->isSelected() ? (int) mFpsSlider->getValue() : 0;
     mFpsSlider->setEnabled(mFps > 0);