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

List:       kde-commits
Subject:    KDE/kdebase/workspace/libs/plasmagenericshell
From:       Aaron J. Seigo <aseigo () kde ! org>
Date:       2010-12-08 19:35:16
Message-ID: 20101208193516.97C72AC8A5 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1204709 by aseigo:

constify and clean a bit


 M  +5 -5      backgrounddialog.cpp  


--- trunk/KDE/kdebase/workspace/libs/plasmagenericshell/backgrounddialog.cpp \
#1204708:1204709 @@ -474,9 +474,9 @@
 
 void BackgroundDialog::saveConfig()
 {
-    QString wallpaperPlugin = \
d->backgroundDialogUi.wallpaperMode->itemData(d->backgroundDialogUi.wallpaperMode->currentIndex()).value<WallpaperInfo>().first;
                
-    QString wallpaperMode = \
d->backgroundDialogUi.wallpaperMode->itemData(d->backgroundDialogUi.wallpaperMode->currentIndex()).value<WallpaperInfo>().second;
                
-    QString containment = \
d->backgroundDialogUi.containmentComboBox->itemData(d->backgroundDialogUi.containmentComboBox->currentIndex(),
 +    const QString wallpaperPlugin = \
d->backgroundDialogUi.wallpaperMode->itemData(d->backgroundDialogUi.wallpaperMode->currentIndex()).value<WallpaperInfo>().first;
 +    const QString wallpaperMode = \
d->backgroundDialogUi.wallpaperMode->itemData(d->backgroundDialogUi.wallpaperMode->currentIndex()).value<WallpaperInfo>().second;
 +    const QString containment = \
d->backgroundDialogUi.containmentComboBox->itemData(d->backgroundDialogUi.containmentComboBox->currentIndex(),
                
                                                           \
AppletDelegate::PluginNameRole).toString();  
     // Containment
@@ -502,8 +502,8 @@
 
                     KPageWidgetItem *item = m->item(idx);
 
-                    if (item && item != d->appearanceItem && item != d->mouseItem
-                      && item != d->activityItem) {
+                    if (item && item != d->appearanceItem &&
+                        item != d->mouseItem && item != d->activityItem) {
                         itemsToRemove.append(item);
                     }
                 }


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

Configure | About | News | Add a list | Sponsored by KoreLogic