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

Library stabilizations/deprecations for 1.15 release #38369

Merged
merged 12 commits into from
Dec 18, 2016

Conversation

aturon
Copy link
Member

@aturon aturon commented Dec 14, 2016

Stabilized:

  • std::iter::Iterator::{min_by, max_by}
  • std::os::*::fs::FileExt
  • std::sync::atomic::Atomic*::{get_mut, into_inner}
  • std::vec::IntoIter::{as_slice, as_mut_slice}
  • std::sync::mpsc::Receiver::try_iter
  • std::os::unix::process::CommandExt::before_exec
  • std::rc::Rc::{strong_count, weak_count}
  • std::sync::Arc::{strong_count, weak_count}
  • std::char::{encode_utf8, encode_utf16}
  • std::cell::Ref::clone
  • std::io::Take::into_inner

Deprecated:

  • std::rc::Rc::{would_unwrap, is_unique}
  • std::cell::RefCell::borrow_state

Closes #23755
Closes #27733
Closes #27746
Closes #27784
Closes #28356
Closes #31398
Closes #34931
Closes #35601
Closes #35603
Closes #35918
Closes #36105

@rust-highfive
Copy link
Collaborator

r? @sfackler

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

@aturon
Copy link
Member Author

aturon commented Dec 14, 2016

r? @alexcrichton

I'll push up a final commit to take care of any remaining build failures before sending to bors, but you can go ahead and review.

@aturon
Copy link
Member Author

aturon commented Dec 14, 2016

cc @rust-lang/libs

@alexcrichton
Copy link
Member

r+ from me, feel free to r=me and follow bors failures

Looks like make tidy may be failing though?

@achanda
Copy link
Contributor

achanda commented Dec 14, 2016

Can we add #36949 here?

@aturon
Copy link
Member Author

aturon commented Dec 14, 2016

@achanda No, that issue has not entered its "final comment period" for stabilization yet -- we like to make sure that people are aware that the feature is headed for stabilization and have a chance to make final comments before we land it. I expect we'll pick it up for the 1.16 release.

@achanda
Copy link
Contributor

achanda commented Dec 14, 2016

Got it, thanks @aturon!

@aturon
Copy link
Member Author

aturon commented Dec 14, 2016

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Dec 14, 2016

📌 Commit f4c3004 has been approved by alexcrichton

@frewsxcv
Copy link
Member

It looks like there are Travis errors?

@aturon aturon force-pushed the stab-1.15 branch 3 times, most recently from c5a33f4 to 14396c2 Compare December 15, 2016 03:18
@aturon
Copy link
Member Author

aturon commented Dec 15, 2016

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Dec 15, 2016

📌 Commit 14396c2 has been approved by alexcrichton

@frewsxcv
Copy link
Member

Looks like there are a couple more errors.

@aturon
Copy link
Member Author

aturon commented Dec 15, 2016

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Dec 15, 2016

📌 Commit fd8191d has been approved by alexcrichton

@frewsxcv
Copy link
Member

More errors again. I'll stop responding if you've got it under control :)

@bluss
Copy link
Member

bluss commented Dec 15, 2016

I'm unsure about ExactSizeIterator::is_empty #35428

@aturon
Copy link
Member Author

aturon commented Dec 15, 2016

I'm removing the is_empty stabilization, to allow more time for discussion.

@bors
Copy link
Contributor

bors commented Dec 16, 2016

💔 Test failed - auto-win-msvc-64-opt

@alexcrichton
Copy link
Member

alexcrichton commented Dec 16, 2016 via email

@bors
Copy link
Contributor

bors commented Dec 16, 2016

⌛ Testing commit 4ab77b7 with merge cc44dbe...

@bors
Copy link
Contributor

bors commented Dec 16, 2016

💔 Test failed - auto-win-msvc-64-opt-rustbuild

@alexcrichton
Copy link
Member

alexcrichton commented Dec 16, 2016 via email

@bors
Copy link
Contributor

bors commented Dec 16, 2016

⌛ Testing commit 4ab77b7 with merge 3361a9d...

@bors
Copy link
Contributor

bors commented Dec 16, 2016

💔 Test failed - auto-win-msvc-64-opt

@alexcrichton
Copy link
Member

alexcrichton commented Dec 16, 2016 via email

@bors
Copy link
Contributor

bors commented Dec 16, 2016

⌛ Testing commit 4ab77b7 with merge 5e72a3e...

@bors
Copy link
Contributor

bors commented Dec 16, 2016

💔 Test failed - auto-win-msvc-64-opt

@aturon
Copy link
Member Author

aturon commented Dec 17, 2016

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Dec 17, 2016

📌 Commit f720b8b has been approved by alexcrichton

@aturon
Copy link
Member Author

aturon commented Dec 17, 2016

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Dec 17, 2016

📌 Commit 9a5cef4 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Dec 18, 2016

💔 Test failed - auto-win-msvc-64-cargotest

@alexcrichton
Copy link
Member

alexcrichton commented Dec 18, 2016 via email

@bors
Copy link
Contributor

bors commented Dec 18, 2016

⌛ Testing commit 9a5cef4 with merge 1f965cc...

bors added a commit that referenced this pull request Dec 18, 2016
Library stabilizations/deprecations for 1.15 release

Stabilized:

- `std::iter::Iterator::{min_by, max_by}`
- `std::os::*::fs::FileExt`
- `std::sync::atomic::Atomic*::{get_mut, into_inner}`
- `std::vec::IntoIter::{as_slice, as_mut_slice}`
- `std::sync::mpsc::Receiver::try_iter`
- `std::os::unix::process::CommandExt::before_exec`
- `std::rc::Rc::{strong_count, weak_count}`
- `std::sync::Arc::{strong_count, weak_count}`
- `std::char::{encode_utf8, encode_utf16}`
- `std::cell::Ref::clone`
- `std::io::Take::into_inner`

Deprecated:

- `std::rc::Rc::{would_unwrap, is_unique}`
- `std::cell::RefCell::borrow_state`

Closes #23755
Closes #27733
Closes #27746
Closes #27784
Closes #28356
Closes #31398
Closes #34931
Closes #35601
Closes #35603
Closes #35918
Closes #36105
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants