Review Request 126610: kwidgetitemdelegate: properly cleanup widgets on index removal

[Pino Toscano]

> On Gen. 2, 2016, 7:31 p.m., Sune Vuorela wrote:
> > Looks much better than my attempt over in https://git.reviewboard.kde.org/r/120139

Note my patch does not remove the warnings on close, although I found out how to avoid them: to the KWidgetItemDelegate construr, either pass a null parent, or the view->viewport().


- Pino


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126610/#review90477
-----------------------------------------------------------


On Gen. 2, 2016, 7:24 p.m., Pino Toscano wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126610/
> -----------------------------------------------------------
> 
> (Updated Gen. 2, 2016, 7:24 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kitemviews
> 
> 
> Description
> -------
> 
> When indexes are removed and their widgets deleted, the event filter on each widget is not removed, leading to the "you should not delete widgets manually"-alike warning.
> 
> Add an helper forgetAbout() function which performs all the actions done per-widget before deleting each, additionally removing also the event filter.
> 
> 
> Diffs
> -----
> 
>   src/kwidgetitemdelegate.cpp 779dc2a8a57148fb37f1f5a7194bc9656cb305a4 
>   src/kwidgetitemdelegatepool.cpp e916dddad8be56bb803e241da43d8cbe7a171ec3 
>   src/kwidgetitemdelegatepool_p.h 401fe193b0954d6c7c721503d4657b7f08e9fd2e 
> 
> Diff: https://git.reviewboard.kde.org/r/126610/diff/
> 
> 
> Testing
> -------
> 
> A sample application with widgets for items in the model, removing indexes: no more warning at removal time.
> 
> 
> Thanks,
> 
> Pino Toscano
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160102/f715fa68/attachment.html>