Skip to content

salvage some of core::old_iter for core::iter, remove the rest #6104

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

Closed
thestinger opened this issue Apr 29, 2013 · 3 comments
Closed

salvage some of core::old_iter for core::iter, remove the rest #6104

thestinger opened this issue Apr 29, 2013 · 3 comments
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup.

Comments

@thestinger
Copy link
Contributor

All of the adaptors implemented on top of BaseIter should be removed and reimplemented as generic adaptors for any internal iterator. The traits are generally useful, but BaseIter is the wrong place for a size hint to be.

The only adaptors that can really take advantage of the hint are ones that convert to a container with the concept of reserving space. It should be possible to take advantage of the reserve on any iterator with a known size, not just each.

@thestinger
Copy link
Contributor Author

Nominating for the backwards compatible milestone. I think most of this will end up scrapped and replaced with external iterators and an Iterable trait.

We do need generic internal iterator adaptors if we want good support for them, but there aren't actually any in this module working for anything beyond BaseIter.

@thestinger
Copy link
Contributor Author

Nothing to salvage, just going to remove it.

@thestinger
Copy link
Contributor Author

#7334

flip1995 pushed a commit to flip1995/rust that referenced this issue Oct 9, 2020
Allow exponent separator

Fixes rust-lang#6096

changelog: Fixed bug in `clippy::inconsistent_digit_grouping` when floating point exponent is used
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup.
Projects
None yet
Development

No branches or pull requests

1 participant