Review Request: KWidgetItemDelegate crashed for KTorrent

Frederik Gladhorn gladhorn at kde.org
Mon May 3 07:38:42 BST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3874/
-----------------------------------------------------------

Review request for kdelibs, Kevin Ottens and Rafael Fernández López.


Summary
-------

ktorrent uses kwidgetitemdelegate in a way that broke with making it possible to delete the delegate without simultanously removing the view.
It passes the view as parent and view into KWidgetItemDelegate(QAbstractItemView *itemView, QObject *parent = 0).
When the view is deleted, the delegate will be deleted in the qobject destructor that deletes the view children. This leads to double deletion of the widgets created by kwidgetitemdelegate, since they are reparented by the view.
This patch makes ktorrent work again. (bug 233783)


This addresses bug 233783.
    https://bugs.kde.org/show_bug.cgi?id=233783


Diffs
-----

  trunk/KDE/kdelibs/kdeui/itemviews/kwidgetitemdelegatepool.cpp 1121890 
  trunk/KDE/kdelibs/kdeui/itemviews/kwidgetitemdelegatepool_p.h 1121890 

Diff: http://reviewboard.kde.org/r/3874/diff


Testing
-------


Thanks,

Frederik





More information about the kde-core-devel mailing list