Skip to content

Useless peekable iterator #854

New issue

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

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

Already on GitHub? # to your account

Closed
mcarton opened this issue Apr 13, 2016 · 3 comments · Fixed by #9258
Closed

Useless peekable iterator #854

mcarton opened this issue Apr 13, 2016 · 3 comments · Fixed by #9258
Assignees
Labels
A-lint Area: New lints E-medium Call for participation: Medium difficulty level problem and requires some initial experience.

Comments

@mcarton
Copy link
Member

mcarton commented Apr 13, 2016

Just noticed that when refactoring. We could warn when making an iterator peekable, but never actually peeking it(/returning it/passing it to a function).

@mcarton mcarton added E-medium Call for participation: Medium difficulty level problem and requires some initial experience. A-lint Area: New lints labels Apr 13, 2016
@birkenfeld
Copy link
Contributor

Sounds pretty specific. Are you sure it's worth it?

@Serial-ATA
Copy link
Contributor

@rustbot claim

@lukaslueg
Copy link
Contributor

@birkenfeld FWIW I've seen this in the wild, too

@bors bors closed this as completed in 2091142 Aug 19, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-lint Area: New lints E-medium Call for participation: Medium difficulty level problem and requires some initial experience.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants