-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Make closures inherit their parent's "safety context" #85607
Make closures inherit their parent's "safety context" #85607
Conversation
This comment has been minimized.
This comment has been minimized.
f0744a1
to
7e90a53
Compare
This comment has been minimized.
This comment has been minimized.
7e90a53
to
3f1fb86
Compare
@nikomatsakis This is ready for review now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are there no tests for this/
Whoops you're right, I forgot |
3f1fb86
to
f996001
Compare
@nikomatsakis Pushed the tests. To track the tests that were fixed by this, I ran all the test suite with
Making progress :) |
This comment has been minimized.
This comment has been minimized.
f996001
to
d1f0e9f
Compare
@bors r+ |
📌 Commit d1f0e9f has been approved by |
⌛ Testing commit d1f0e9f with merge 6676f8aa363db6b8bb1b6494cd0a33ca24a04d32... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
Probably spurious failure? |
@bors retry |
☀️ Test successful - checks-actions |
Fixes rust-lang/project-thir-unsafeck#9,
blocked on #85273.r? @nikomatsakis