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

transmute: avoid suggesting from/to bits in const #5919

Merged
merged 2 commits into from
Aug 20, 2020

Conversation

ebroto
Copy link
Member

@ebroto ebroto commented Aug 17, 2020

changelog: Avoid suggesting from/to bits in const contexts in [transmute_int_to_float]

Fixes #5747

@rust-highfive
Copy link

r? @matthiaskrgr

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 17, 2020
@matthiaskrgr
Copy link
Member

Does f64 need this kind of handling as well?

@ebroto
Copy link
Member Author

ebroto commented Aug 18, 2020

Oh good point, I think the code handles this but adding some tests won't hurt, and the description of the change should be updated.

I'll fix it later, thanks!

@ebroto ebroto force-pushed the 5747_from_bits_in_const branch from 34035e2 to 6a12bae Compare August 18, 2020 20:21
@matthiaskrgr
Copy link
Member

Thanks!
@bors r+

@bors
Copy link
Contributor

bors commented Aug 20, 2020

📌 Commit 6a12bae has been approved by matthiaskrgr

@bors
Copy link
Contributor

bors commented Aug 20, 2020

⌛ Testing commit 6a12bae with merge d891954...

@bors
Copy link
Contributor

bors commented Aug 20, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: matthiaskrgr
Pushing d891954 to master...

@bors bors merged commit d891954 into rust-lang:master Aug 20, 2020
@ebroto ebroto deleted the 5747_from_bits_in_const branch August 23, 2020 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

transmute_int_to_float false positive
4 participants