Review Request 121087: Use deleteLater() to delete the selection toggle in KItemListWidget::slotHoverAnimationFinished()

[Christoph Feck]

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


> doesn't make anything worse.

There is one subtle difference between delete and deleteLater, though, which indeed might make things worse: only delete is save to be called on a null ptr.

- Christoph Feck


On Nov. 9, 2014, 5:11 p.m., Frank Reininghaus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121087/
> -----------------------------------------------------------
> 
> (Updated Nov. 9, 2014, 5:11 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Bugs: 334785
>     http://bugs.kde.org/show_bug.cgi?id=334785
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> This patch should make a crash with a backtrace as provided in the bug report impossible. I'm not sure if this would really prevent the crash - it looks like the actual cause of the problem could be in QGraphicsView or anywhere else. But I'm pretty sure that this change doesn't make anything worse.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/kitemviews/kitemlistwidget.cpp e7d2951 
> 
> Diff: https://git.reviewboard.kde.org/r/121087/diff/
> 
> 
> Testing
> -------
> 
> Hovering and un-hovering icons and selection toggles works as before.
> 
> 
> Thanks,
> 
> Frank Reininghaus
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20141109/91154e9e/attachment.htm>