<table><tr><td style="">bruns created this revision.<br />bruns added a reviewer: Baloo.<br />Herald added projects: Frameworks, Baloo.<br />Herald added a subscriber: kde-frameworks-devel.<br />bruns requested review of this revision.
</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/D24502">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>In case many consecutive files are skipped in a directory, the<br />
recursive next() implementation recursed for every file, i.e.<br />
calling itself repeatedly.</p>

<p>Instead, loop over all directory items until either a file or directory<br />
is found which should be indexed, or there are no more items left.</p>

<p>Move the "shouldIndexHidden" variable out of the loop, as it is<br />
invariant.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>ctest -R filtereddiriterator</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R293 Baloo</div></div></div><br /><div><strong>BRANCH</strong><div><div>iterative_filtereddiriterator</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24502">https://phabricator.kde.org/D24502</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/file/filtereddiriterator.cpp</div></div></div><br /><div><strong>To: </strong>bruns, Baloo<br /><strong>Cc: </strong>kde-frameworks-devel, lots0logs, LeGast00n, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams<br /></div>