remove forever-deprecated and hidden f64 methods#153975
remove forever-deprecated and hidden f64 methods#153975cyrgani wants to merge 1 commit intorust-lang:mainfrom
f64 methods#153975Conversation
|
r? @joboet rustbot has assigned @joboet. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
@bors try @craterbot check |
This comment has been minimized.
This comment has been minimized.
remove forever-deprecated and hidden `f64` methods
|
@craterbot check |
|
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
Assuming crater results come back clean... @rfcbot merge libs-api |
|
Team member @Amanieu has proposed to merge this. The next step is review by the rest of the tagged team members: No concerns currently listed. Once a majority of reviewers approve (and at most 2 approvals are outstanding), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up! See this document for info about what commands tagged team members can give me. |
|
r? scottmcm |
|
🚧 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
🎉 Experiment
Footnotes
|
|
Looked at the crater report, it is indeed clean :) |
The methods
f64::is_positiveandf64::is_negativewere deprecated since 1.0 and marked as#[doc(hidden)]in favor off64::is_sign_positiveandf64::is_sign_negative. They also only exist onf64, not onf32. But for some unknown reason, they have been marked as stable.This PR proposes to remove both methods as they were never a part of the documented API, assuming that a crater run finds no significant breakage.