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

Remove explicit .iter() / deref from fmt chapter #1695

Merged
merged 1 commit into from
Mar 31, 2023

Conversation

dmartin
Copy link
Contributor

@dmartin dmartin commented Mar 31, 2023

Removes unnecessary usage of the dereferencing operator from the early formatting chapter.
With this change, for-loop desugaring and the introduction of the dereferencing operator is deferred until (I think) chapter 8.4 ("for and range").

@rustbot
Copy link
Collaborator

rustbot commented Mar 31, 2023

Failed to set assignee to marioidival: cannot assign: HTTP status client error (403 Forbidden) for url (https://api.github.com/repos/rust-lang/rust-by-example/issues/1695/assignees)

Note: Only org members, users with write permissions, or people who have commented on the PR may be assigned.

@rustbot
Copy link
Collaborator

rustbot commented Mar 31, 2023

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @marioidival (or someone else) soon.

@marioidival marioidival merged commit 5020979 into rust-lang:master Mar 31, 2023
@dmartin dmartin deleted the remove-deref branch March 31, 2023 17:32
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Apr 3, 2023
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