Skip to content

Conversation

estebank
Copy link
Contributor

  • Point at def span on incorrect panic or oom function
  • Use structured suggestion instead of note for += that can be performed on a dereference of the left binding
  • Small code formatting cleanup

@rust-highfive
Copy link
Contributor

r? @petrochenkov

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 30, 2018
@petrochenkov
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 31, 2018

📌 Commit 013710e has been approved by petrochenkov

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 31, 2018
@bors
Copy link
Collaborator

bors commented Sep 1, 2018

⌛ Testing commit 013710e with merge a1a8c44...

bors added a commit that referenced this pull request Sep 1, 2018
Various small diagnostic and code clean up

 - Point at def span on incorrect `panic` or `oom` function
 - Use structured suggestion instead of note for `+=` that can be performed on a dereference of the left binding
 - Small code formatting cleanup
@bors
Copy link
Collaborator

bors commented Sep 2, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: petrochenkov
Pushing a1a8c44 to master...

@bors bors merged commit 013710e into rust-lang:master Sep 2, 2018
@estebank estebank deleted the various branch November 9, 2023 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants