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

Remove items that are unstable and deprecated #41437

Merged
merged 10 commits into from
Apr 23, 2017

Conversation

cuviper
Copy link
Member

@cuviper cuviper commented Apr 21, 2017

This removes unstable items that have been deprecated for more than one cycle.

  • Since 1.16.0, #![feature(enumset)]
    • All of mod collections::enum_set
  • Since 1.15.0, #![feature(borrow_state)]
    • cell::BorrowState
    • RefCell::borrow_state()
  • Since 1.15.0, #![feature(is_unique)]
    • Rc::is_unique() (made private like Arc::is_unique())
  • Since 1.15.0, #![feature(rc_would_unwrap)]
    • Rc::would_wrap()
  • Since 1.13.0, #![feature(binary_heap_extras)]
    • BinaryHeap::push_pop()
    • BinaryHeap::replace()
  • Since 1.12.0, #![feature(as_unsafe_cell)]
    • Cell::as_unsafe_cell()
    • RefCell::as_unsafe_cell()
  • Since 1.12.0, #![feature(map_entry_recover_keys)]
    • btree_map::OccupiedEntry::remove_pair()
    • hash_map::OccupiedEntry::remove_pair()
  • Since 1.11.0, #![feature(float_extras)]
    • Float::nan()
    • Float::infinity()
    • Float::neg_infinity()
    • Float::neg_zero()
    • Float::zero()
    • Float::one()
    • Float::integer_decode()
    • f32::integer_decode()
    • f32::ldexp()
    • f32::frexp()
    • f32::next_after()
    • f64::integer_decode()
    • f64::ldexp()
    • f64::frexp()
    • f64::next_after()
  • Since 1.11.0, #![feature(zero_one)]
    • num::Zero
    • num::One

@rust-highfive
Copy link
Collaborator

r? @alexcrichton

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

@cuviper
Copy link
Member Author

cuviper commented Apr 21, 2017

Things I didn't include (yet):

  • core::hash::SipHasher13
    • used in std::collections::hash_map::DefaultHasher
    • should move it across crates, properly hidden in DefaultHasher?
  • core::hash::SipHasher24
    • used in the stable+deprecated core::hash::SipHasher
    • could just make it private, only used in SipHasher
  • rustc_plugin::Registry::register_custom_derive()
    • the function alone is easy enough to remove, but there's probably a lot of supporting code that could go away with it, which I haven't explored yet.

@bors
Copy link
Contributor

bors commented Apr 21, 2017

☔ The latest upstream changes (presumably #41342) made this pull request unmergeable. Please resolve the merge conflicts.

[unstable, deprecated since 1.16.0]
[unstable, deprecated since 1.13.0]
[unstable, deprecated since 1.12.0]
[unstable, deprecated since 1.15.0]
[unstable, deprecated since 1.15.0]
[unstable, deprecated since 1.12.0]
[unstable, deprecated since 1.15.0]
[unstable, deprecated since 1.11.0]
[unstable, deprecated since 1.11.0]
@cuviper
Copy link
Member Author

cuviper commented Apr 21, 2017

Rebased.

i8::min_value() / -i8::one(),
i16::min_value() / -i16::one(),
i32::min_value() / -i32::one(),
i64::min_value() / -i64::one(),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why start using one here?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah. Because I needed a local zero implemention anyway, so adding one is easy and gets rid of the compile-time warning that this is going to overflow. We could also go the other direction and use plain literals for all, which will then warn on every line. Or we could leave it as-is with zero() and -1, but the inconsistency bothered me. 😀

I don't think it matters to the actual test.

@alexcrichton
Copy link
Member

@bors: r+

Looks great to me, thanks @cuviper!

@bors
Copy link
Contributor

bors commented Apr 21, 2017

📌 Commit c1aaa60 has been approved by alexcrichton

@alexcrichton alexcrichton added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 21, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 21, 2017
… r=alexcrichton

Remove items that are unstable and deprecated

This removes unstable items that have been deprecated for more than one cycle.

- Since 1.16.0, `#![feature(enumset)]`
    - All of `mod collections::enum_set`
- Since 1.15.0, `#![feature(borrow_state)]`
    - `cell::BorrowState`
    - `RefCell::borrow_state()`
- Since 1.15.0, `#![feature(is_unique)]`
    - `Rc::is_unique()` (made private like `Arc::is_unique()`)
- Since 1.15.0, `#![feature(rc_would_unwrap)]`
    - `Rc::would_wrap()`
- Since 1.13.0, `#![feature(binary_heap_extras)]`
    - `BinaryHeap::push_pop()`
    - `BinaryHeap::replace()`
- Since 1.12.0, `#![feature(as_unsafe_cell)]`
    - `Cell::as_unsafe_cell()`
    - `RefCell::as_unsafe_cell()`
- Since 1.12.0, `#![feature(map_entry_recover_keys)]`
    - `btree_map::OccupiedEntry::remove_pair()`
    - `hash_map::OccupiedEntry::remove_pair()`
- Since 1.11.0, `#![feature(float_extras)]`
    - `Float::nan()`
    - `Float::infinity()`
    - `Float::neg_infinity()`
    - `Float::neg_zero()`
    - `Float::zero()`
    - `Float::one()`
    - `Float::integer_decode()`
    - `f32::integer_decode()`
    - `f32::ldexp()`
    - `f32::frexp()`
    - `f32::next_after()`
    - `f64::integer_decode()`
    - `f64::ldexp()`
    - `f64::frexp()`
    - `f64::next_after()`
- Since 1.11.0, `#![feature(zero_one)]`
    - `num::Zero`
    - `num::One`
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 22, 2017
… r=alexcrichton

Remove items that are unstable and deprecated

This removes unstable items that have been deprecated for more than one cycle.

- Since 1.16.0, `#![feature(enumset)]`
    - All of `mod collections::enum_set`
- Since 1.15.0, `#![feature(borrow_state)]`
    - `cell::BorrowState`
    - `RefCell::borrow_state()`
- Since 1.15.0, `#![feature(is_unique)]`
    - `Rc::is_unique()` (made private like `Arc::is_unique()`)
- Since 1.15.0, `#![feature(rc_would_unwrap)]`
    - `Rc::would_wrap()`
- Since 1.13.0, `#![feature(binary_heap_extras)]`
    - `BinaryHeap::push_pop()`
    - `BinaryHeap::replace()`
- Since 1.12.0, `#![feature(as_unsafe_cell)]`
    - `Cell::as_unsafe_cell()`
    - `RefCell::as_unsafe_cell()`
- Since 1.12.0, `#![feature(map_entry_recover_keys)]`
    - `btree_map::OccupiedEntry::remove_pair()`
    - `hash_map::OccupiedEntry::remove_pair()`
- Since 1.11.0, `#![feature(float_extras)]`
    - `Float::nan()`
    - `Float::infinity()`
    - `Float::neg_infinity()`
    - `Float::neg_zero()`
    - `Float::zero()`
    - `Float::one()`
    - `Float::integer_decode()`
    - `f32::integer_decode()`
    - `f32::ldexp()`
    - `f32::frexp()`
    - `f32::next_after()`
    - `f64::integer_decode()`
    - `f64::ldexp()`
    - `f64::frexp()`
    - `f64::next_after()`
- Since 1.11.0, `#![feature(zero_one)]`
    - `num::Zero`
    - `num::One`
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 22, 2017
… r=alexcrichton

Remove items that are unstable and deprecated

This removes unstable items that have been deprecated for more than one cycle.

- Since 1.16.0, `#![feature(enumset)]`
    - All of `mod collections::enum_set`
- Since 1.15.0, `#![feature(borrow_state)]`
    - `cell::BorrowState`
    - `RefCell::borrow_state()`
- Since 1.15.0, `#![feature(is_unique)]`
    - `Rc::is_unique()` (made private like `Arc::is_unique()`)
- Since 1.15.0, `#![feature(rc_would_unwrap)]`
    - `Rc::would_wrap()`
- Since 1.13.0, `#![feature(binary_heap_extras)]`
    - `BinaryHeap::push_pop()`
    - `BinaryHeap::replace()`
- Since 1.12.0, `#![feature(as_unsafe_cell)]`
    - `Cell::as_unsafe_cell()`
    - `RefCell::as_unsafe_cell()`
- Since 1.12.0, `#![feature(map_entry_recover_keys)]`
    - `btree_map::OccupiedEntry::remove_pair()`
    - `hash_map::OccupiedEntry::remove_pair()`
- Since 1.11.0, `#![feature(float_extras)]`
    - `Float::nan()`
    - `Float::infinity()`
    - `Float::neg_infinity()`
    - `Float::neg_zero()`
    - `Float::zero()`
    - `Float::one()`
    - `Float::integer_decode()`
    - `f32::integer_decode()`
    - `f32::ldexp()`
    - `f32::frexp()`
    - `f32::next_after()`
    - `f64::integer_decode()`
    - `f64::ldexp()`
    - `f64::frexp()`
    - `f64::next_after()`
- Since 1.11.0, `#![feature(zero_one)]`
    - `num::Zero`
    - `num::One`
@frewsxcv
Copy link
Member

@cuviper
Copy link
Member Author

cuviper commented Apr 22, 2017

Argh, OK. It's going to be a couple days before I can get back to this, unless a maintainer wants to push to my branch. There's no urgency here though.

@Mark-Simulacrum Mark-Simulacrum added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Apr 23, 2017
@Mark-Simulacrum Mark-Simulacrum removed the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 23, 2017
@frewsxcv
Copy link
Member

@bors r=alexcrichton

@bors
Copy link
Contributor

bors commented Apr 23, 2017

📌 Commit 13d2534 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Apr 23, 2017

⌛ Testing commit 13d2534 with merge a941244...

bors added a commit that referenced this pull request Apr 23, 2017
…hton

Remove items that are unstable and deprecated

This removes unstable items that have been deprecated for more than one cycle.

- Since 1.16.0, `#![feature(enumset)]`
    - All of `mod collections::enum_set`
- Since 1.15.0, `#![feature(borrow_state)]`
    - `cell::BorrowState`
    - `RefCell::borrow_state()`
- Since 1.15.0, `#![feature(is_unique)]`
    - `Rc::is_unique()` (made private like `Arc::is_unique()`)
- Since 1.15.0, `#![feature(rc_would_unwrap)]`
    - `Rc::would_wrap()`
- Since 1.13.0, `#![feature(binary_heap_extras)]`
    - `BinaryHeap::push_pop()`
    - `BinaryHeap::replace()`
- Since 1.12.0, `#![feature(as_unsafe_cell)]`
    - `Cell::as_unsafe_cell()`
    - `RefCell::as_unsafe_cell()`
- Since 1.12.0, `#![feature(map_entry_recover_keys)]`
    - `btree_map::OccupiedEntry::remove_pair()`
    - `hash_map::OccupiedEntry::remove_pair()`
- Since 1.11.0, `#![feature(float_extras)]`
    - `Float::nan()`
    - `Float::infinity()`
    - `Float::neg_infinity()`
    - `Float::neg_zero()`
    - `Float::zero()`
    - `Float::one()`
    - `Float::integer_decode()`
    - `f32::integer_decode()`
    - `f32::ldexp()`
    - `f32::frexp()`
    - `f32::next_after()`
    - `f64::integer_decode()`
    - `f64::ldexp()`
    - `f64::frexp()`
    - `f64::next_after()`
- Since 1.11.0, `#![feature(zero_one)]`
    - `num::Zero`
    - `num::One`
@bors
Copy link
Contributor

bors commented Apr 23, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing a941244 to master...

@cuviper
Copy link
Member Author

cuviper commented Apr 23, 2017

Thanks for the fixup @frewsxcv!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants