Skip to content

Minor wording updates to the undefined behavior section #14

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 1 commit into from
Jan 15, 2025

Conversation

ia0
Copy link
Member

@ia0 ia0 commented Jan 15, 2025

  • Preserve the parallel between "unsafe traits and functions" at the beginning of the paragraph with "unsafe impls and blocks" in the middle, by adding "impls and".

  • Remove "even" for the condition of unsafe code being unsound at the end of the paragraph. With my understanding of English, "even when" means that the sentence holds even when (no pun intended) the "when" condition is removed, which is not true for that sentence. Sound unsafe code can trigger undefined behavior if its safety conditions are not upheld.

- Preserve the parallel between "unsafe traits and functions" at the beginning
  of the paragraph with "unsafe impls and blocks in the middle, by adding "impls
  and".

- Remove "even" for the condition of unsafe code being unsound at the end of the
  paragraph. With my understanding of English, "even when" means that the
  sentence holds even when (no pun intended) the "when" condition is removed,
  which is not true for that sentence. Sound unsafe code can trigger undefined
  behavior if its safety conditions are not upheld.
@djkoloski djkoloski merged commit d3bccf3 into google:main Jan 15, 2025
2 checks passed
@ia0 ia0 deleted the fix branch January 15, 2025 14:25
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.

2 participants