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

Added necessary "at" in line 170 #3688

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DreadedHippy
Copy link

I had to read this line more than 4 times to understand it. Having the "at" makes it more readable

I had to read this line more than 4 times to understand it. Having the "at" makes it more readable
Copy link
Contributor

@chriskrycho chriskrycho left a comment

Choose a reason for hiding this comment

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

I can definitely sympathize with that experience, so I am sorry you had it! Unfortunately, the change actually makes the sentence incorrect grammatically, and the original is correct grammatically. We might be able to tweak it so the sentence reads like this, though:

Note that a reference’s scope starts from where it is introduced and continues through the last time that reference is used. For instance, this code will compile because the last usage of the immutable references, in the println!, occurs before the mutable reference is introduced:

That is still grammatically correct, preserving the structure of the sentence. Would it have helped with your confusion here?

@chriskrycho chriskrycho added this to the ch4 milestone Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants