Skip to content

Rename String::into_boxed_slice -> into_boxed_str #27696

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

Merged
merged 1 commit into from
Aug 14, 2015

Conversation

bluss
Copy link
Member

@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
Member

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
Member 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
Member 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
Member 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

💔 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