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.

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

Example could use a for loop to show processing results, clarify Results #8

Open
tv42 opened this issue Jul 14, 2021 · 0 comments
Open

Comments

@tv42
Copy link

tv42 commented Jul 14, 2021

rust-analyzer thinks the example's store.search return type is <&str as Searcher<Book>>::Item, which is not very useful. Store::search docs are not great either, leaving it unclear what I::Item is.

An extra source of confusion is that Item sounds like an item of an iterator, where in this case it apparently refers to results, plural.

It took a couple of tries, e.g. attempting to iterate the return value directly, before I stumbled on Results and saw how to iterate the results correctly.

It's not very clear how one would navigate from Store::search to Results based purely on the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant