[PATCH] crash fix for dir-selection dialog (BR67737)
Waldo Bastian
bastian at kde.org
Wed Aug 11 13:52:11 BST 2004
See http://bugs.kde.org/show_bug.cgi?id=67737
KFileTree has serious issues, very much like we have fixed in all the other
treeviews that deal with KDirLister + redirection caused by the devices:
urls. (Sidebar anyone?)
The main issue in BR67737 is that kfiletreeview.cpp keeps pointers to
QListViewItems around but fails to notice when these items get removed. It's
very hard to get proper notification of this, so the next best thing is to
check if they still exist when they are accessed.
The attached patch fixes the crash but it doesn't fix the code. The devices:
URL still doesn't work because the KFileTree doesn't keep track of the
redirections so it gets listings for URLs that it can't find in the treeview.
Might be a good idea to consolidate the three file tree view classes
(kio/kfile, konqueror/sidebar, konqueror/listview) in KDE 4 and to take
advantage of the new Qt list view stuff.
Cheers,
Waldo
--
bastian at kde.org | KDE Community World Summit 2004 | bastian at suse.com
bastian at kde.org | 21-29 August, Ludwigsburg, Germany | bastian at suse.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20040811/54f10799/attachment.sig>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kfiletree.patch
Type: text/x-diff
Size: 4708 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20040811/54f10799/attachment.patch>
More information about the kde-core-devel
mailing list