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

std: Add Default/IntoIterator/ToOwned to the prelude #24541

Merged
merged 1 commit into from
Apr 22, 2015

Conversation

alexcrichton
Copy link
Member

This is an implementation of RFC 1030 which adds these traits to the
prelude and additionally removes all inherent into_iter methods on collections
in favor of the trait implementation (which is now accessible by default).

This is technically a breaking change due to the prelude additions and removal
of inherent methods, but it is expected that essentially no code breaks in
practice.

[breaking-change]
Closes #24538

@alexcrichton
Copy link
Member Author

r? @aturon

@rust-highfive rust-highfive assigned aturon and unassigned nikomatsakis Apr 17, 2015
@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton alexcrichton force-pushed the issue-24538 branch 2 times, most recently from ad96559 to 38decbc Compare April 17, 2015 23:06
This is an implementation of [RFC 1030][rfc] which adds these traits to the
prelude and additionally removes all inherent `into_iter` methods on collections
in favor of the trait implementation (which is now accessible by default).

[rfc]: rust-lang/rfcs#1030

This is technically a breaking change due to the prelude additions and removal
of inherent methods, but it is expected that essentially no code breaks in
practice.

[breaking-change]
Closes rust-lang#24538
@aturon
Copy link
Member

aturon commented Apr 21, 2015

@bors: r+ p=10

@bors
Copy link
Contributor

bors commented Apr 21, 2015

📌 Commit 8f5b5f9 has been approved by aturon

@aturon aturon added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Apr 21, 2015
@aturon
Copy link
Member

aturon commented Apr 21, 2015

Nominating for beta inclusion.

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Apr 21, 2015
This is an implementation of [RFC 1030][rfc] which adds these traits to the
prelude and additionally removes all inherent `into_iter` methods on collections
in favor of the trait implementation (which is now accessible by default).

[rfc]: rust-lang/rfcs#1030

This is technically a breaking change due to the prelude additions and removal
of inherent methods, but it is expected that essentially no code breaks in
practice.

[breaking-change]
Closes rust-lang#24538
@bors
Copy link
Contributor

bors commented Apr 21, 2015

⌛ Testing commit 8f5b5f9 with merge 0ff9888...

@bors
Copy link
Contributor

bors commented Apr 21, 2015

⛄ The build was interrupted to prioritize another pull request.

@bors
Copy link
Contributor

bors commented Apr 22, 2015

⌛ Testing commit 8f5b5f9 with merge 977b6fd...

@bors
Copy link
Contributor

bors commented Apr 22, 2015

⛄ The build was interrupted to prioritize another pull request.

@bors bors merged commit 8f5b5f9 into rust-lang:master Apr 22, 2015
@pnkfelix
Copy link
Member

(special-casing for 1.0; i.e. a "minor" change like this would not normally go into beta, though of course it is a breaking change b/c of prelude and thus we are accepting it for beta.)

going from nominated to (nominated, accepted)

@pnkfelix pnkfelix added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Apr 23, 2015
@alexcrichton alexcrichton removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Apr 23, 2015
@alexcrichton alexcrichton deleted the issue-24538 branch April 30, 2015 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Default/ToOwned/IntoIterator to the prelude
6 participants