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

List:       kde-commits
Subject:    KDE/kdeplasma-addons/containments/groupingdesktop/lib
From:       Giulio Camuffo <giuliocamuffo () gmail ! com>
Date:       2010-12-07 17:26:02
Message-ID: 20101207172602.793B3AC8A5 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1204509 by camuffo:

QHash is faster than QMap. also, the private classes members shouldn't start with m_


 M  +6 -6      abstractgroup.cpp  
 M  +2 -2      abstractgroup_p.h  


--- trunk/KDE/kdeplasma-addons/containments/groupingdesktop/lib/abstractgroup.cpp #1204508:1204509
@@ -205,11 +205,11 @@
 void AbstractGroupPrivate::setChildBorders(Plasma::Applet *a, bool added)
 {
     if (added) {
-        m_savedAppletsHints.insert(a, a->backgroundHints());
+        savedAppletsHints.insert(a, a->backgroundHints());
         a->setBackgroundHints(Plasma::Applet::NoBackground);
     } else {
-        if (m_savedAppletsHints.contains(a)) {
-            a->setBackgroundHints(m_savedAppletsHints.value(a));
+        if (savedAppletsHints.contains(a)) {
+            a->setBackgroundHints(savedAppletsHints.value(a));
         }
     }
 }
@@ -217,11 +217,11 @@
 void AbstractGroupPrivate::setChildBorders(AbstractGroup *g, bool added)
 {
     if (added) {
-        m_savedGroupsHints.insert(g, g->backgroundHints());
+        savedGroupsHints.insert(g, g->backgroundHints());
         g->setBackgroundHints(AbstractGroup::PlainBackground);
     } else {
-        if (m_savedGroupsHints.contains(g)) {
-            g->setBackgroundHints(m_savedGroupsHints.value(g));
+        if (savedGroupsHints.contains(g)) {
+            g->setBackgroundHints(savedGroupsHints.value(g));
         }
     }
 }
--- trunk/KDE/kdeplasma-addons/containments/groupingdesktop/lib/abstractgroup_p.h #1204508:1204509
@@ -57,8 +57,8 @@
         bool isLoading;
         bool hasInterface;
         bool simplerBackgroundChildren;
-        QMap<Plasma::Applet *, Plasma::Applet::BackgroundHints> m_savedAppletsHints;
-        QMap<AbstractGroup *, AbstractGroup::BackgroundHints> m_savedGroupsHints;
+        QHash<Plasma::Applet *, Plasma::Applet::BackgroundHints> savedAppletsHints;
+        QHash<AbstractGroup *, AbstractGroup::BackgroundHints> savedGroupsHints;
 
     private:
         KConfigGroup *m_mainConfig;
[prev in list] [next in list] [prev in thread] [next in thread]