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

Deprecate advance method on Iterators. #15514

Closed
wants to merge 6 commits into from

Conversation

Projects
None yet
4 participants
@luqmana
Copy link
Member

luqmana commented Jul 7, 2014

Closes #15492.

@huonw

This comment has been minimized.

Copy link
Member

huonw commented Jul 7, 2014

This should theoretically #[deprecated = "use a forloop or theall method"] the advance method.

@luqmana luqmana changed the title Remove advance from Iterator. Deprecate advance method on Iterators, Jul 7, 2014

@luqmana luqmana changed the title Deprecate advance method on Iterators, Deprecate advance method on Iterators. Jul 7, 2014

bors added a commit that referenced this pull request Jul 8, 2014

bors added a commit that referenced this pull request Jul 9, 2014

@cmr

This comment has been minimized.

Copy link

cmr commented on 5d39d0b Jul 9, 2014

r+

@bors

This comment has been minimized.

Copy link
Contributor

bors commented on 5d39d0b Jul 9, 2014

saw approval from cmr
at luqmana@5d39d0b

This comment has been minimized.

Copy link
Contributor

bors replied Jul 9, 2014

merging luqmana/rust/die-advance-die = 5d39d0b into auto

This comment has been minimized.

Copy link
Contributor

bors replied Jul 9, 2014

luqmana/rust/die-advance-die = 5d39d0b merged ok, testing candidate = 1b8e671

This comment has been minimized.

Copy link
Contributor

bors replied Jul 10, 2014

fast-forwarding master to auto = 1b8e671

bors added a commit that referenced this pull request Jul 9, 2014

@bors bors closed this Jul 10, 2014

@luqmana luqmana deleted the luqmana:die-advance-die branch Jul 10, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.