Skip to content

Add floating-point read/write to Reader/Writer #5235

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

Closed
wants to merge 1 commit into from

Conversation

yjh0502
Copy link
Contributor

@yjh0502 yjh0502 commented Mar 5, 2013

When parsing bytes from a wire, there is a need to parse floating-point bytes to float values ([u8_4] to f32, [u8_8] to f64). This can be done via cast::transmute, but there is no way to do it safely.

It's quite common, so I think I't better to support it in core library.

@yjh0502
Copy link
Contributor Author

yjh0502 commented Mar 6, 2013

@graydon I fixed a typo on a comment.

bors added a commit that referenced this pull request Mar 6, 2013
When parsing bytes from a wire, there is a need to parse floating-point bytes to float values ([u8*4] to f32, [u8*8] to f64). This can be done via cast::transmute, but there is no way to do it safely.

It's quite common, so I think I't better to support it in core library.
@bors bors closed this Mar 6, 2013
bors added a commit to rust-lang-ci/rust that referenced this pull request May 2, 2020
Fix deploy script for tag deploys

By hand updating the documentation for the `rust-1.41.1` release, I found, that the deploy script was buggy for tag deploys. Luckily we weren't affected yet (will only run if the commit of the tag already has the workflow config, so probably ~`rust-1.43.0`)

The bug:

When a tag was deployed, until now it also updated the master documentation to the state of the tag. This was because the `git add .` not only updated the files created or modified by the tag commit, but also the master lints.json file.

changelog: none
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.

3 participants