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

Rollup of 5 pull requests #69235

Closed
wants to merge 10 commits into from

Conversation

Dylan-DPC-zz
Copy link

Successful merges:

Failed merges:

r? @ghost

ollie27 and others added 10 commits January 28, 2020 12:39
The main improvement is to make `last` no longer recursive.
…ow554

recursion_limit parsing handles overflows

This PR adds overflow handling to `#![recursion_limit]` attribute parsing. If parsing the given value results in an `IntErrorKind::Overflow`, then the recursion_limit is set to `usize::max_value()`.

closes rust-lang#67265
Simplify `Skip::nth` and `Skip::last` implementations

The main improvement is to make `last` no longer recursive.
macOS: avoid calling pthread_self() twice
Do not ICE when encountering `yield` inside `async` block

Fix rust-lang#67158.
@Dylan-DPC-zz Dylan-DPC-zz added the rollup A PR which is a rollup label Feb 17, 2020
@Dylan-DPC-zz
Copy link
Author

@bors r+ p=5 rollup=never

@bors
Copy link
Contributor

bors commented Feb 17, 2020

📌 Commit 386b494 has been approved by Dylan-DPC

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Feb 17, 2020
@bors
Copy link
Contributor

bors commented Feb 17, 2020

⌛ Testing commit 386b494 with merge 096100359173be3c634ad7a01f1b64b448eac8fc...

@bors
Copy link
Contributor

bors commented Feb 17, 2020

💔 Test failed - checks-azure

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Feb 17, 2020
@Dylan-DPC-zz
Copy link
Author

@bors retry

@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 Feb 17, 2020
@bors
Copy link
Contributor

bors commented Feb 17, 2020

⌛ Testing commit 386b494 with merge be0c6e6d79e248ea43c16266e5b5fe78042b72d0...

@bors
Copy link
Contributor

bors commented Feb 18, 2020

💔 Test failed - checks-azure

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants