Fixing #47517

[Michael Brade]

Answering Robert's private mail to me as well with this (hope this is ok for 
you): yes, I knew about this crash but I didn't know about this bugreport... 
otherwise I'd have already fixed it ;) I ran out of time when trying to fix 
it. So, now it should be fixed, patch attached, explanation as follows:

Robert is right, the problem was that m_dictSubDirs contained references to 
already deleted items which caused the crash. (For me it crashed in 
saveState, which also accesses the items in m_dictSubDirs). KDirLister 
emitted deleteItem not for all subdirs (which is ok, the view should 
recursively remove the items itself) and thus the treeview didn't remove the 
subdir from m_dictSubDirs. The solution is to make 
KonqTreeViewWidget::removeSubDir() recursive.

BTW, we need to "delete" every single item in deleted directories with 
m_pBrowserView->deleteItem() to remove it from the statusbar counts.
However, my patch seems to miss a case as the statusbar is still wrong for the 
treeview... too late to fix it now, will do in a few days if nobody else did 
it by then (Robert..? ;-)

Please review, test and give me an ok to apply :-)

-- 
Michael Brade;                 KDE Developer, Student of Computer Science
  |-mail: echo brade !#|tr -d "c oh"|s\e\d 's/e/\@/2;s/$/.org/;s/bra/k/2'
  °--web: http://www.kde.org/people/michaelb.html

KDE 3: The Next Generation in Desktop Experience

-------------- next part --------------
A non-text attachment was scrubbed...
Name: treeview.patch
Type: text/x-diff
Size: 4420 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20021120/ce1ead19/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: signature
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20021120/ce1ead19/attachment.sig>