Differential D28844

[PhraseAndIterator] Replace recursive next() implementation
ClosedPublic
Actions

Authored by bruns on Apr 14 2020, 9:03 PM.

Details

Reviewers
ngraham
Group Reviewers
Baloo
Commits
R293:c00cf2022763: [PhraseAndIterator] Replace recursive next() implementation
Summary

This changes the code for the PhraseAndIterator in the
same way as done for the AndPostingIterator in D28843.

The only difference is the additional
checkIfPositionsMatch in the break criterion

Depends on D28839

Diff Detail

Repository
R293 Baloo
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
bruns created this revision.Apr 14 2020, 9:03 PM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptApr 14 2020, 9:03 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
bruns requested review of this revision.Apr 14 2020, 9:03 PM
ngraham accepted this revision.Apr 15 2020, 6:04 PM
This revision is now accepted and ready to land.Apr 15 2020, 6:04 PM
Closed by commit R293:c00cf2022763: [PhraseAndIterator] Replace recursive next() implementation (authored by bruns). · Explain WhyApr 15 2020, 10:04 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 80234

View Options

src/engine/phraseanditerator.h

Loading...
View Options

src/engine/phraseanditerator.cpp

Loading...