From 8390a321319f1fdf8c92ae40408ee6db02b75908 Mon Sep 17 00:00:00 2001 From: hjk Date: Tue, 28 May 2013 15:17:20 +0200 Subject: Settings: Simplify some user code This is still way too much boiler plate code for the task at hand. Change-Id: Ia03bff3d6947ea6372e3f5d0217c116728742b74 Reviewed-by: Eike Ziller Reviewed-by: Friedemann Kleint --- src/plugins/classview/classviewnavigationwidgetfactory.cpp | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) (limited to 'src/plugins/classview/classviewnavigationwidgetfactory.cpp') diff --git a/src/plugins/classview/classviewnavigationwidgetfactory.cpp b/src/plugins/classview/classviewnavigationwidgetfactory.cpp index 7b19e0144f..b3a4fb7e63 100644 --- a/src/plugins/classview/classviewnavigationwidgetfactory.cpp +++ b/src/plugins/classview/classviewnavigationwidgetfactory.cpp @@ -108,14 +108,11 @@ void NavigationWidgetFactory::saveSettings(int position, QWidget *widget) NavigationWidget *pw = qobject_cast(widget); QTC_ASSERT(pw, return); - QSettings *settings = Core::ICore::settings(); - QTC_ASSERT(settings, return); - // .beginGroup is not used - to prevent simultaneous access QString group = settingsPrefix(position); - // save settings - settings->setValue(group, pw->flatMode()); + // Save settings + Core::ICore::settings()->setValue(group, pw->flatMode()); } void NavigationWidgetFactory::restoreSettings(int position, QWidget *widget) @@ -123,14 +120,11 @@ void NavigationWidgetFactory::restoreSettings(int position, QWidget *widget) NavigationWidget *pw = qobject_cast(widget); QTC_ASSERT(pw, return); - QSettings *settings = Core::ICore::settings(); - QTC_ASSERT(settings, return); - // .beginGroup is not used - to prevent simultaneous access QString group = settingsPrefix(position); - // load settings - pw->setFlatMode(settings->value(group, false).toBool()); + // Load settings + pw->setFlatMode(Core::ICore::settings()->value(group, false).toBool()); } } // namespace Internal -- cgit v1.2.1