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

[Frank Reininghaus]

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

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/20121119/47e4a9c9/attachment.htm>