Skip to content

Use drop instead of the toilet closure |_| () - #67807

Merged
bors merged 1 commit into
rust-lang:masterfrom
tesuji:toilet-closure
Jan 3, 2020
Merged

Use drop instead of the toilet closure |_| ()#67807
bors merged 1 commit into
rust-lang:masterfrom
tesuji:toilet-closure

Conversation

@tesuji

Copy link
Copy Markdown
Contributor

No description provided.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @sfackler

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

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 2, 2020
@Centril

Copy link
Copy Markdown
Contributor

Looks like we're flushing the closures down the 🚽.

r? @Centril@bors r+ rollup

@bors

bors commented Jan 2, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit dd8f072 has been approved by Centril

@rust-highfiverust-highfive assigned Centril and unassigned sfacklerJan 2, 2020
@borsbors 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 Jan 2, 2020
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Jan 2, 2020
Use drop instead of the toilet closure `|_| ()`
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Jan 2, 2020
Use drop instead of the toilet closure `|_| ()`
bors added a commit that referenced this pull request Jan 2, 2020
Rollup of 5 pull requests
Successful merges:
- #67636 (allow rustfmt key in [build] section)
- #67736 (Less-than is asymmetric, not antisymmetric)
- #67762 (Add missing links for insecure_time)
- #67777 (Use self profile infrastructure for -Z time and -Z time-passes)
- #67807 (Use drop instead of the toilet closure `|_| ()`)
Failed merges:
r? @ghost
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Jan 3, 2020
Use drop instead of the toilet closure `|_| ()`
@JohnTitorJohnTitor mentioned this pull request Jan 3, 2020
bors added a commit that referenced this pull request Jan 3, 2020
Rollup of 10 pull requests
Successful merges:
- #67450 (Allow for setting a ThinLTO import limit during bootstrap)
- #67595 (Suggest adding a lifetime constraint for opaque type)
- #67636 (allow rustfmt key in [build] section)
- #67736 (Less-than is asymmetric, not antisymmetric)
- #67762 (Add missing links for insecure_time)
- #67783 (Warn for bindings named same as variants when matching against a borrow)
- #67796 (Ensure that we process projections during MIR inlining)
- #67807 (Use drop instead of the toilet closure `|_| ()`)
- #67816 (Clean up err codes)
- #67825 (Minor: change take() docs grammar to match other docs)
Failed merges:
r? @ghost
@bors
bors merged commit dd8f072 into rust-lang:masterJan 3, 2020
@tesuji
tesuji deleted the toilet-closure branch January 3, 2020 12:27
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: 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.

5 participants

@tesuji@rust-highfive@Centril@bors@sfackler