<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/117777/">https://git.reviewboard.kde.org/r/117777/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 7cd8252d2c662c6db4a90788a64fec8297c6c378 by Alex Merry on behalf of Stephen Kelly to branch master.</pre>
<br />
<p>- Commit Hook</p>
<br />
<p>On April 25th, 2014, 7:18 p.m. UTC, Alex Merry wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Frameworks and Stephen Kelly.</div>
<div>By Alex Merry.</div>
<p style="color: grey;"><i>Updated April 25, 2014, 7:18 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kitemmodels
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Unit tests now pass.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>autotests/CMakeLists.txt <span style="color: grey">(4281fec2de1124162af623cf999db73eff705684)</span></li>
<li>autotests/proxymodeltestsuite/modeltest.h <span style="color: grey">(4b80850d51ca068fb8ffb0a704a9f938b4dacfa9)</span></li>
<li>autotests/proxymodeltestsuite/modeltest.cpp <span style="color: grey">(0928a3a4d78d10fcae4a5671e91e836e9fde429b)</span></li>
<li>src/kselectionproxymodel.cpp <span style="color: grey">(ea97f7a197f9181745f2979c2f7b2e5e1f0015f3)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/117777/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>