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

Emmanuel Pescosta emmanuelpescosta099 at gmail.com
Sun Nov 9 16:57:58 GMT 2014


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

Ship it!


Ship It!

- Emmanuel Pescosta


On Nov. 9, 2014, 5:55 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:55 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/93c8e52b/attachment.htm>


More information about the kfm-devel mailing list