Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix documentation of getResults #597

Merged
merged 2 commits into from
Aug 4, 2021
Merged

Fix documentation of getResults #597

merged 2 commits into from
Aug 4, 2021

Commits on Aug 4, 2021

  1. Add getResult unit test for in between range searching 

    Search::getResult(start, end) seems to be broken for getting results in
range that not starting with 0.
    @maneeshpm @kelson42
    maneeshpm authored and kelson42 committed Aug 4, 2021
    Configuration menu
    Copy the full SHA
    953cba7 View commit details
    Browse the repository at this point in the history

  2. Fix documentation of getResults 

    getReturn should follow the same convention used by Xapian for
consistency. The second argument maxResults is the maximum number of
results to return starting from the first argument start.
    @maneeshpm @kelson42
    maneeshpm authored and kelson42 committed Aug 4, 2021
    Configuration menu
    Copy the full SHA
    6a0bcbe View commit details
    Browse the repository at this point in the history