Skip to content

track recursion limit when expanding existential impl trait #47529

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 26, 2018
Merged

track recursion limit when expanding existential impl trait #47529

merged 1 commit into from
Jan 26, 2018

Conversation

nikomatsakis
Copy link
Contributor

@aidanhs aidanhs added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 17, 2018
@cramertj cramertj mentioned this pull request Jan 22, 2018
Closed
@cramertj
Copy link
Member

@bors r+

I've created #47659 to track improving the error message, and I've linked it from the impl Trait tracking issue so that we don't forget about it.

@bors
Copy link
Collaborator

bors commented Jan 22, 2018

📌 Commit 072c3da has been approved by cramertj

@nikomatsakis
Copy link
Contributor Author

@cramertj sounds good

@kennytm kennytm 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 24, 2018
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Jan 25, 2018
@alexcrichton
Rollup merge of rust-lang#47529 - nikomatsakis:impl-trait-issue-38064…
31f1aa5 
…, r=cramertj

track recursion limit when expanding existential impl trait

r? @cramertj
@alexcrichton alexcrichton mentioned this pull request Jan 25, 2018
Merged
bors added a commit that referenced this pull request Jan 26, 2018
@bors
Auto merge of #47748 - alexcrichton:rollup, r=alexcrichton
bacb5c5 
Rollup of 19 pull requests

- Successful merges: #47415, #47437, #47439, #47453, #47460, #47502, #47529, #47600, #47607, #47618, #47626, #47656, #47668, #47696, #47701, #47705, #47710, #47711, #47719
- Failed merges: #47455, #47521
@bors bors merged commit 072c3da into rust-lang:master Jan 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@cramertj cramertj

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@nikomatsakis @cramertj @bors @kennytm @aidanhs