Review Request 117777: Make the KSelectionProxyModel unit test pass

Commit Hook null at kde.org
Sat Apr 26 15:40:46 UTC 2014


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


This review has been submitted with commit 7cd8252d2c662c6db4a90788a64fec8297c6c378 by Alex Merry on behalf of Stephen Kelly to branch master.

- Commit Hook


On April 25, 2014, 7:18 p.m., Alex Merry wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117777/
> -----------------------------------------------------------
> 
> (Updated April 25, 2014, 7:18 p.m.)
> 
> 
> Review request for KDE Frameworks and Stephen Kelly.
> 
> 
> Repository: kitemmodels
> 
> 
> Description
> -------
> 
> Make the KSelectionProxyModel unit test pass
> 
> It looks like Qt4 emitted layoutChanged in circumstances where Qt5 emits
> rowsMoved. This could be used to optimize things, but for now we just
> act the same way in either case (rebuilding the mapping).
> 
> Add a pedantic mode to modeltest.
> 
> Cherry-pick of ffd732966d1f2c632d811e8fc378a95aa1456703
> 
> This was lost when I updated ModelTest to Qt5.
> 
> Re-apply Stephen Kelly's modeltest fixes
> 
> These were lost when I replaced ModelTest with the version from Qt 5.
> 
> 
> Diffs
> -----
> 
>   autotests/CMakeLists.txt 4281fec2de1124162af623cf999db73eff705684 
>   autotests/proxymodeltestsuite/modeltest.h 4b80850d51ca068fb8ffb0a704a9f938b4dacfa9 
>   autotests/proxymodeltestsuite/modeltest.cpp 0928a3a4d78d10fcae4a5671e91e836e9fde429b 
>   src/kselectionproxymodel.cpp ea97f7a197f9181745f2979c2f7b2e5e1f0015f3 
> 
> Diff: https://git.reviewboard.kde.org/r/117777/diff/
> 
> 
> Testing
> -------
> 
> Unit tests now pass.
> 
> 
> Thanks,
> 
> Alex Merry
> 
>

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


More information about the Kde-frameworks-devel mailing list