Skip to content

Fix regression in defaults #49344 #49704

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

Merged
merged 1 commit into from
Apr 8, 2018
Merged

Conversation

leoyvens
Copy link
Contributor

@leoyvens leoyvens commented Apr 5, 2018

Fixes #49344 by not checking the well-formedness wrt defaults of predicates that contain lifetimes, which is consistent with not checking generic predicates.

r? @nikomatsakis

@nikomatsakis
Copy link
Contributor

@leodasvacas thanks for jumping on this!

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 6, 2018

📌 Commit 933f9eb has been approved by nikomatsakis

@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 Apr 6, 2018
@nikomatsakis nikomatsakis added beta-nominated Nominated for backporting to the compiler in the beta channel. and removed beta-nominated Nominated for backporting to the compiler in the beta channel. labels Apr 6, 2018
@leoyvens
Copy link
Contributor Author

leoyvens commented Apr 6, 2018

This has indeed hit beta, we may consider backporting.

@bors
Copy link
Collaborator

bors commented Apr 8, 2018

⌛ Testing commit 933f9eb with merge 08ad376...

bors added a commit that referenced this pull request Apr 8, 2018
Fix regression in defaults #49344

Fixes #49344 by not checking the well-formedness wrt defaults of predicates that contain lifetimes, which is consistent with not checking generic predicates.

r? @nikomatsakis
@bors
Copy link
Collaborator

bors commented Apr 8, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: nikomatsakis
Pushing 08ad376 to master...

@bors bors merged commit 933f9eb into rust-lang:master Apr 8, 2018
@kennytm kennytm added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Apr 14, 2018
bors added a commit that referenced this pull request Apr 14, 2018
[beta] Backport fix for regression in defaults

This proposes a backport of #49704, which fixes a regression that affects beta.

r? @nikomatsakis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. 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