<table><tr><td style="">bruns created this revision.<br />bruns added reviewers: Baloo, ngraham, astippich, poboiko.<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/D21835">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The positions() method is only used inside the PhraseAndIterator and<br />
requires either a VectorPositionInfoIterator or a DBPositionIterator,<br />
as these are the only ones reimplementing the positions() method.</p>

<p>Actually, VectorPositionInfoIterator and DBPositionIterator have<br />
identical implementations, save the constructor argument. The conversion<br />
of the constructor argument can be trivially inlined and DBPositionIterator<br />
can be removed completely.</p>

<p>This indirectly improves the test coverage (VectorPositionInfoIterator is<br />
used in the PhraseAndIteratorTest) and removes duplicate code.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>ctest</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R293 Baloo</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21835">https://phabricator.kde.org/D21835</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/unit/engine/phraseanditeratortest.cpp<br />
autotests/unit/engine/positiondbtest.cpp<br />
src/engine/phraseanditerator.cpp<br />
src/engine/phraseanditerator.h<br />
src/engine/positiondb.cpp<br />
src/engine/positiondb.h<br />
src/engine/postingiterator.cpp<br />
src/engine/postingiterator.h<br />
src/engine/transaction.cpp<br />
src/engine/vectorpositioninfoiterator.cpp<br />
src/engine/vectorpositioninfoiterator.h</div></div></div><br /><div><strong>To: </strong>bruns, Baloo, ngraham, astippich, poboiko<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams<br /></div>