Skip to content

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
Contributor

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
Collaborator

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
Collaborator

bors commented Apr 21, 2015

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

@bors
Copy link
Collaborator

bors commented Apr 21, 2015

⛄ The build was interrupted to prioritize another pull request.

@bors
Copy link
Collaborator

bors commented Apr 22, 2015

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

@bors
Copy link
Collaborator

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