<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, vhanda.<br />broulik set the repository for this revision to rMILOU Milou.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D2395" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Simplify queryString once, rather than during every iteration</li>
<li class="remarkup-list-item">Foreach instead of potentially detaching range-for</li>
<li class="remarkup-list-item">Use reverse iterators (new in Qt 5.6) instead of Java-style iterators</li>
<li class="remarkup-list-item">Remove traces of begin/endInsert/RemoveRows feature - while it would be nice to not reset the model with all this sorting and merging going on it's pretty difficult to predict where an item will end up which is probably why it never got implemented. It still calculated tons of indices that went unused, so remove this and go a different route in the future</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><p>Seems to work</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rMILOU Milou</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2395" rel="noreferrer">https://phabricator.kde.org/D2395</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>lib/sourcesmodel.cpp<br />
lib/sourcesmodel.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, vhanda<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>