Skip to content

Conversation

@bluss
Copy link
Contributor

@bluss bluss commented Aug 12, 2015

Rename String::into_boxed_slice -> into_boxed_str

This is the name that was decided in rust-lang/rfcs#1152, and it's
better if we say “boxed str” for Box<str>.

The old name String::into_boxed_slice is deprecated.

@rust-highfive
Copy link
Contributor

r? @huonw

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

@killercup
Copy link
Contributor

This is a breaking-change for anyone using feature(box_str), right?

@alexcrichton alexcrichton added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Aug 12, 2015
@alexcrichton
Copy link
Member

Gah oops! Can you also leave behind the old method as deprecated?

@bluss
Copy link
Contributor Author

bluss commented Aug 12, 2015

Ok. Didn't think it was an issue, since it's unstable & recent. Will do.

This is the name that was decided in rust-lang/rfcs#1152, and it's
better if we say “boxed str” for `Box<str>`.

The old name `String::into_boxed_slice` is deprecated.
@bluss
Copy link
Contributor Author

bluss commented Aug 13, 2015

Updated to leave the old name in, deprecated. Thus it's not a breaking change, but it introduces a deprecation.

@alexcrichton
Copy link
Member

@bors: r+ bec6409

@bluss
Copy link
Contributor Author

bluss commented Aug 13, 2015

Thank you!

@bors
Copy link
Collaborator

bors commented Aug 13, 2015

⌛ Testing commit bec6409 with merge d83ba75...

bors added a commit that referenced this pull request Aug 13, 2015
Rename String::into_boxed_slice -> into_boxed_str

This is the name that was decided in rust-lang/rfcs#1152, and it's
better if we say “boxed str” for `Box<str>`.

The old name `String::into_boxed_slice` is deprecated.
@bors
Copy link
Collaborator

bors commented Aug 13, 2015

💔 Test failed - auto-mac-64-opt

@alexcrichton
Copy link
Member

@bors: retry

Nothing like causing a bus error in the linker

@bors
Copy link
Collaborator

bors commented Aug 13, 2015

@bors
Copy link
Collaborator

bors commented Aug 13, 2015

💔 Test failed - auto-mac-32-opt

@alexcrichton
Copy link
Member

@bors: retry

On Thu, Aug 13, 2015 at 1:34 PM, bors [email protected] wrote:

[image: 💔] Test failed - auto-mac-32-opt
http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/6118


Reply to this email directly or view it on GitHub
#27696 (comment).

bors added a commit that referenced this pull request Aug 14, 2015
Rename String::into_boxed_slice -> into_boxed_str

This is the name that was decided in rust-lang/rfcs#1152, and it's
better if we say “boxed str” for `Box<str>`.

The old name `String::into_boxed_slice` is deprecated.
@bors
Copy link
Collaborator

bors commented Aug 14, 2015

⌛ Testing commit bec6409 with merge e2bebf3...

@bors bors merged commit bec6409 into rust-lang:master Aug 14, 2015
@bluss bluss deleted the into-boxed-str branch August 14, 2015 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants