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

std: Relax UnwindSafe impl for Unique #40020

Merged
merged 1 commit into from
Feb 25, 2017

Conversation

alexcrichton
Copy link
Member

Add the ?Sized bound as we don't require the type to be sized.

Closes #40011

Add the `?Sized` bound as we don't require the type to be sized.

Closes rust-lang#40011
@rust-highfive
Copy link
Collaborator

r? @sfackler

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

@alexcrichton
Copy link
Member Author

Nominating for a beta backport as well as the linked issue is a beta regression

@alexcrichton alexcrichton added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Feb 21, 2017
@sfackler
Copy link
Member

@bors r+

(boo UnwindSafe!)

@bors
Copy link
Contributor

bors commented Feb 21, 2017

📌 Commit 347e1af has been approved by sfackler

eddyb added a commit to eddyb/rust that referenced this pull request Feb 25, 2017
…ckler

std: Relax UnwindSafe impl for Unique

Add the `?Sized` bound as we don't require the type to be sized.

Closes rust-lang#40011
eddyb added a commit to eddyb/rust that referenced this pull request Feb 25, 2017
…ckler

std: Relax UnwindSafe impl for Unique

Add the `?Sized` bound as we don't require the type to be sized.

Closes rust-lang#40011
bors added a commit that referenced this pull request Feb 25, 2017
@bors bors merged commit 347e1af into rust-lang:master Feb 25, 2017
@alexcrichton alexcrichton deleted the fix-unwind-safe branch February 25, 2017 21:52
@alexcrichton
Copy link
Member Author

Accepting for a backport as it's a regression

@alexcrichton alexcrichton added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Feb 25, 2017
@brson brson mentioned this pull request Feb 25, 2017
@alexcrichton alexcrichton removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Feb 25, 2017
bors added a commit that referenced this pull request Mar 2, 2017
[beta] next

- #39913
- #39730
- #39674
- #39602
- #39586
- #39471
- #39980
- #40020
- #40135

@nikomatsakis [this commit](3787d33) did not pick cleanly. You might peek at it.

I took the liberty of accepting all the nominations myself, but the [packed struct alignment](#39586) PR is quite large. It did pick fine though and there's a comment there suggesting it works on beta cc @rust-lang/compiler.

cc @alexcrichton
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants