Review Request: Clear the previous selection when pasting or dropping items (fixing a recent 4.9 branch regression)

[Commit Hook]

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


This review has been submitted with commit 5f6c60f0e1e82c235eca855e1f78edc8f8c7a67a by Frank Reininghaus to branch KDE/4.9.

- Commit Hook


On Nov. 19, 2012, 9:02 p.m., Frank Reininghaus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107389/
> -----------------------------------------------------------
> 
> (Updated Nov. 19, 2012, 9:02 p.m.)
> 
> 
> Review request for Dolphin and Emmanuel Pescosta.
> 
> 
> Description
> -------
> 
> To reduce the risk of regressions, I thought I'd better let other people have a look at non-trivial patches of mine before pushing. In particular, selecting items is an area where a lot can go wrong. Emmanuel, can you have a look at the patch? You're doing some work in the area at the moment, so maybe you see if that breaks something or if there's an easier way to do it (I doubt that we can avoid the new member though if we don't want to bring bug 217575 back though). This fix is mostly independent of your patch https://git.reviewboard.kde.org/r/107351/. That one should also go in, of course - I'm uploading the fix for the regression now because I want to fix it before KDE 4.9.4 is tagged.
> 
> Others are of course also welcome to comment on the patch!
> 
> 
> This addresses bug 310365.
>     http://bugs.kde.org/show_bug.cgi?id=310365
> 
> 
> Diffs
> -----
> 
>   dolphin/src/views/dolphinview.h 7d8e8b7 
>   dolphin/src/views/dolphinview.cpp 13db989 
> 
> Diff: http://git.reviewboard.kde.org/r/107389/diff/
> 
> 
> Testing
> -------
> 
> Fixes the bug for me, and I haven't found any new regressions so far.
> 
> 
> Thanks,
> 
> Frank Reininghaus
> 
>

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