Review Request 129657: Fix assert (in beginRemoveRows) when deselecting an empty child of a selected child in korganizer

[David Faure]

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

Review request for KDE Frameworks and Stephen Kelly.


Repository: kitemmodels


Description
-------

After
   int proxyEndRemove = proxyStartRemove;
   proxyEndRemove += rc; was adding 0 (empty root)
and then --proxyEndRemove; was making us end up with proxyEndRemove < proxyStartRemove.


Diffs
-----

  autotests/kselectionproxymodeltest.cpp 483e7c42dabab6aa560622ff0418ee7f90363e15 
  src/kselectionproxymodel.cpp 0f57c70f05d4cbde9b14f4257bff1365ef5443f6 

Diff: https://git.reviewboard.kde.org/r/129657/diff/


Testing
-------

New unittest + unchecking calendar in korganizer no longer asserts.


Thanks,

David Faure

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