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

Stabilize float_bits_conv for Rust 1.20 #43055

Merged
merged 1 commit into from
Jul 17, 2017

Conversation

est31
Copy link
Member

@est31 est31 commented Jul 4, 2017

Stabilizes the float_bits_conv lib feature for the 1.20 release of Rust. I've initially implemented the feature in #39271 and later made PR #43025 to output quiet NaNs even on platforms with different encodings, which seems to have been the only unresolved issue of the API.

Due to PR #43025 being only applied to master this stabilisation can't happen for Rust 1.19 through the usual "stabilisation on beta" system that is being done for library APIs.

r? @BurntSushi

closes #40470.

@Mark-Simulacrum Mark-Simulacrum added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Jul 4, 2017
@aidanhs aidanhs added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 5, 2017
@est31 est31 changed the title Stabilize float_bits_conv for Rust 1.20 Stabilize float_bits_conv for Rust 1.21 Jul 6, 2017
@est31 est31 force-pushed the stabilize_float_bits_conv branch from b98547b to 9fb16a6 Compare July 6, 2017 13:14
@est31
Copy link
Member Author

est31 commented Jul 6, 2017

I guess the 1.20 estimate is wrong because of the FCP which delays by >= 10 days and the new beta branches in a week already. 1.21 it is then!

@est31 est31 force-pushed the stabilize_float_bits_conv branch from 9fb16a6 to 010dea1 Compare July 6, 2017 13:17
@carols10cents
Copy link
Member

friendly ping @BurntSushi!

@BurntSushi
Copy link
Member

Doesn't FCP in #40470 need to finish first?

@carols10cents
Copy link
Member

Oh sorry, that wasn't clear in this issue.

@carols10cents carols10cents added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 10, 2017
@seppalai
Copy link

Is there any way to work around this in Rust 1.18?

@est31
Copy link
Member Author

est31 commented Jul 13, 2017

@i-dash what do you mean by that? Right now its possible to do such conversion with the help of unsafe Rust. Or have you stumbled into huonw/ieee754#2 ? For that, you can use a cargo replace, like doing the following in your Cargo.toml (maybe you need to adjust the version, also untested):

[replace]
"ieee754:0.2.1" = { git = "https://github.com/clarcharr/ieee754" }

@est31
Copy link
Member Author

est31 commented Jul 16, 2017

r? @BurntSushi

@sfackler
Copy link
Member

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Jul 17, 2017

📌 Commit 010dea1 has been approved by sfackler

@bors
Copy link
Contributor

bors commented Jul 17, 2017

⌛ Testing commit 010dea1 with merge 56071f6...

bors added a commit that referenced this pull request Jul 17, 2017
Stabilize float_bits_conv for Rust 1.21

Stabilizes the `float_bits_conv` lib feature for the 1.20 release of Rust. I've initially implemented the feature in #39271 and later made PR #43025 to output quiet NaNs even on platforms with different encodings, which seems to have been the only unresolved issue of the API.

Due to PR #43025 being only applied to master this stabilisation can't happen for Rust 1.19 through the usual "stabilisation on beta" system that is being done for library APIs.

r? @BurntSushi

closes #40470.
@bors
Copy link
Contributor

bors commented Jul 17, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: sfackler
Pushing 56071f6 to master...

@bors bors merged commit 010dea1 into rust-lang:master Jul 17, 2017
@alexcrichton alexcrichton added the relnotes Marks issues that should be documented in the release notes of the next release. label Jul 20, 2017
@alexcrichton alexcrichton changed the title Stabilize float_bits_conv for Rust 1.21 Stabilize float_bits_conv for Rust 1.20 Jul 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relnotes Marks issues that should be documented in the release notes of the next release. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. 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.

Tracking issue for float_bits_conv feature
9 participants