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

Rename hir::ExprAgain to hir::ExprContinue #51727

Merged
merged 2 commits into from
Jun 23, 2018

Conversation

varkor
Copy link
Member

@varkor varkor commented Jun 23, 2018

The current name is confusing and historical.

I also used this PR to clean up the annoying indentation in check/mod.rs. If that's viewed as too tangential a change, I'll split it up, but it seemed reasonable to slip it in to reduce @bors's work. It's easy to compare for the two commits individually.

r? @petrochenkov

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

I can't believe somebody has finally done this 😆
@bors r+

@bors
Copy link
Contributor

bors commented Jun 23, 2018

📌 Commit 621047b 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 Jun 23, 2018
@bors
Copy link
Contributor

bors commented Jun 23, 2018

⌛ Testing commit 621047b with merge 4fe88c0...

bors added a commit that referenced this pull request Jun 23, 2018
Rename hir::ExprAgain to hir::ExprContinue

The current name is confusing and historical.

I also used this PR to clean up the annoying indentation in `check/mod.rs`. If that's viewed as too tangential a change, I'll split it up, but it seemed reasonable to slip it in to reduce @bors's work. It's easy to compare for the two commits individually.

r? @petrochenkov
@bors
Copy link
Contributor

bors commented Jun 23, 2018

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

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