Skip to content

[Docs] Fix minor typos in the ownership manifesto. #19349

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
Sep 17, 2018

Conversation

ravikandhadai
Copy link
Contributor

Fixed minor typos I came across while reading this manifesto.

@@ -1381,7 +1381,7 @@ comes from two main sources:
Otherwise, the feature itself is pretty small. The compiler
implicitly emits moves instead of copies, just like we discussed
above for the `move` intrinsic, and then diagnoses anything
that that didn't work for.
that didn't work with the `move` semantics.

This comment was marked as outdated.

Copy link
Contributor

@rjmccall rjmccall left a comment

Choose a reason for hiding this comment

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

The three that thats are not errors. I appreciate the other two fixes, though.

@ravikandhadai
Copy link
Contributor Author

@rjmccall I rolled back the changes to that thats.

Copy link
Contributor

@rjmccall rjmccall left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM.

@ravikandhadai
Copy link
Contributor Author

@swift-ci please smoke test and merge

@DougGregor DougGregor merged commit b6efa97 into swiftlang:master Sep 17, 2018
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.

4 participants