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

List:       kde-commits
Subject:    kdebase/konqueror/sidebar/trees
From:       Carsten Pfeiffer <carpdjih () mailbox ! tu-berlin ! de>
Date:       2003-05-29 9:59:39
[Download RAW message or body]

CVS commit by pfeiffer: 

don't delete the module's items before deleting the module
Waldo: did you have a reason for that? This caused the history
module to crash.

CCMAIL: bastian@kde.org


  M +1 -3      konq_sidebartree.cpp   1.28


--- kdebase/konqueror/sidebar/trees/konq_sidebartree.cpp  #1.27:1.28
@@ -169,9 +169,9 @@ void KonqSidebarTree::itemDestructed( Ko
 void KonqSidebarTree::clearTree()
 {
+    m_lstModules.clear();
     m_topLevelItems.clear();
     m_mapCurrentOpeningFolders.clear();
     m_currentBeforeDropItem = 0;
     clear();
-    m_lstModules.clear();
 
     if (m_dirtreeDir.type==VIRT_Folder)
@@ -644,6 +644,4 @@ void KonqSidebarTree::loadTopLevelItem( 
     name = cfg.readEntry( "Name", name );
     KonqSidebarTreeModule * module = 0L;
-
-  module=0;
 
     // Here's where we need to create the right module...


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