Skip to content

Fixed wording in Rust 1.45 Blog Post #654

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

Merged
merged 3 commits into from
Jul 21, 2020
Merged

Fixed wording in Rust 1.45 Blog Post #654

merged 3 commits into from
Jul 21, 2020

Conversation

JosephTLyons
Copy link
Contributor

This sentence read a little awkwardly to me, so I tried to make it more clear.

@JosephTLyons JosephTLyons requested a review from a team July 16, 2020 15:49
@tesuji
Copy link
Contributor

tesuji commented Jul 16, 2020

How about "there is not unsafe code" here?

Comment on lines 90 to 91
keyword has not be specified in this code block. This is what we call a
"soundness" bug, that is, it is a bug where the compiler does the wrong thing.
Copy link

Choose a reason for hiding this comment

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

Suggested change
keyword has not be specified in this code block. This is what we call a
"soundness" bug, that is, it is a bug where the compiler does the wrong thing.
keyword has not been specified in this code block. This is what we call a
"soundness" bug, where the compiler does the wrong thing.

minor suggestion re be vs been. using that is in the middle of a sentence is pretty weird as well, so maybe we can get two fixes in one.

Copy link
Member

Choose a reason for hiding this comment

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

This suggestion is good re: grammar, but using "specified" here sounds odd, because specification has a different meaning, technically.

I can appreciate this sentence maybe sounding odd, but I think moving it in this direction makes it sound more odd, not less.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops, yeah the "be" was a typo. Ironic.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What about

But the unsafe keyword is nowhere to be found within this block of code.

?

Copy link
Contributor

Choose a reason for hiding this comment

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

I prefer this latter formulation, fwiw.

@XAMPPRocky
Copy link
Member

r? @steveklabnik || @pietroalbini

@JosephTLyons
Copy link
Contributor Author

I pushed up the 2nd proposal for the change. Feel free to close this PR if there are remaining objections to it. Thanks guys.

Copy link
Member

@steveklabnik steveklabnik left a comment

Choose a reason for hiding this comment

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

Thank you!

@steveklabnik steveklabnik merged commit 1a65620 into rust-lang:master Jul 21, 2020
@JosephTLyons JosephTLyons deleted the Fixed-wording-in-Rust-1.45-Blog-Post branch July 21, 2020 21:44
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.

6 participants