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

replace AddAssign example with something more evocative of addition #35806

Merged
merged 1 commit into from
Aug 20, 2016

Conversation

matthew-piziak
Copy link
Contributor

This is analogous to PR #35709 for the Add trait.

@rust-highfive
Copy link
Collaborator

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@matthew-piziak
Copy link
Contributor Author

r? @steveklabnik

@rust-highfive rust-highfive assigned steveklabnik and unassigned aturon Aug 18, 2016
/// *self = Point {
/// x: self.x + other.x,
/// y: self.y + other.y,
/// }
Copy link
Member

Choose a reason for hiding this comment

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

while this doesn't require a ; here, I would put one.

@steveklabnik
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Aug 18, 2016

📌 Commit 6c66eaa has been approved by steveklabnik

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Aug 18, 2016
…=steveklabnik

replace `AddAssign` example with something more evocative of addition

This is analogous to PR rust-lang#35709 for the `Add` trait.
sophiajt pushed a commit to sophiajt/rust that referenced this pull request Aug 19, 2016
…=steveklabnik

replace `AddAssign` example with something more evocative of addition

This is analogous to PR rust-lang#35709 for the `Add` trait.
sophiajt pushed a commit to sophiajt/rust that referenced this pull request Aug 19, 2016
…=steveklabnik

replace `AddAssign` example with something more evocative of addition

This is analogous to PR rust-lang#35709 for the `Add` trait.
sophiajt pushed a commit to sophiajt/rust that referenced this pull request Aug 20, 2016
…=steveklabnik

replace `AddAssign` example with something more evocative of addition

This is analogous to PR rust-lang#35709 for the `Add` trait.
sophiajt pushed a commit to sophiajt/rust that referenced this pull request Aug 20, 2016
…=steveklabnik

replace `AddAssign` example with something more evocative of addition

This is analogous to PR rust-lang#35709 for the `Add` trait.
bors added a commit that referenced this pull request Aug 20, 2016
@bors bors merged commit 6c66eaa into rust-lang:master Aug 20, 2016
matthew-piziak added a commit to matthew-piziak/rust that referenced this pull request Aug 22, 2016
These examples are exactly analogous to those in PRs rust-lang#35709 and rust-lang#35806. I'll probably remove the `fn main` wrappers for `Add` and `Sub` once this is merged in.

Part of rust-lang#29365.

r? @steveklabnik
@matthew-piziak matthew-piziak deleted the addassign-example branch August 23, 2016 15:12
sophiajt pushed a commit to sophiajt/rust that referenced this pull request Aug 24, 2016
…laumeGomez

more evocative examples for `Sub` and `SubAssign`

These examples are exactly analogous to those in PRs rust-lang#35709 and rust-lang#35806. I'll probably remove the `fn main` wrappers for `Add` and `Sub` once this is merged in.

Part of rust-lang#29365.

r? @steveklabnik
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.

5 participants