readd capture disjoint fields gate#85564
Merged
bors merged 4 commits intorust-lang:masterfrom May 27, 2021
Merged
Conversation
(and thus exposed underlying issue rust-lang#85561).
Contributor
|
r? @lcnr (rust-highfive has picked a reviewer for you, use r? to override) |
rust-highfive
added
the
S-waiting-on-review
pnkfelix
commented
May 21, 2021
src/test/ui/unsafe/issue-85435-unsafe-op-in-let-under-unsafe-under-closure.rs
Outdated
Show resolved
Hide resolved
pnkfelix
commented
May 21, 2021
src/test/ui/unsafe/issue-85435-unsafe-op-in-let-under-unsafe-under-closure.rs
Outdated
Show resolved
Hide resolved
pnkfelix
commented
May 21, 2021
src/test/ui/unsafe/issue-85435-unsafe-op-in-let-under-unsafe-under-closure.rs
Outdated
Show resolved
Hide resolved
(removing confusing comment from my test, since the comment reflects the bad undesirable behavior that is being fixed here.)
pnkfelix
added
T-compiler
This was referenced May 21, 2021
src/test/ui/unsafe/issue-85435-unsafe-op-in-let-under-unsafe-under-closure.rs
Show resolved
Hide resolved
Contributor
|
I am not able to review any PRs in the near future |
test THIR unsafeck too Co-authored-by: Léo Lanteri Thauvin <leseulartichaut@gmail.com>
nikomatsakis
approved these changes
May 24, 2021
Contributor
|
@bors r+ |
Collaborator
|
📌 Commit 1c1d4f9 has been approved by |
bors
added
S-waiting-on-bors
Collaborator
|
⌛ Testing commit 1c1d4f9 with merge 74f4e1780f795236c859d645045b4accc8394963... |
Collaborator
|
A job failed! Check out the build log: (web) (plain) Click to see the possible cause of the failure (guessed by this bot) |
Collaborator
|
💔 Test failed - checks-actions |
bors
added
S-waiting-on-review
bors
added
S-waiting-on-bors
Contributor
Author
bors
added
S-waiting-on-author
Contributor
|
@bors r+ |
Collaborator
|
📌 Commit 1c1d4f9 has been approved by |
bors
added
S-waiting-on-bors
Contributor
|
We decided to land and backport this PR |
Dylan-DPC-zz
pushed a commit
to Dylan-DPC-zz/rust
that referenced
this pull request
May 26, 2021
…disjoint-fields-gate, r=nikomatsakis readd capture disjoint fields gate This readds a feature gate guard that was added in PR rust-lang#83521. (Basically, there were unintended consequences to the code exposed by removing the feature gate guard.) The root bug still remains to be resolved, as discussed in issue rust-lang#85561. This is just a band-aid suitable for a beta backport. Cc issue rust-lang#85435 Note that the latter issue is unfixed until we backport this (or another fix) to 1.53 beta
Dylan-DPC-zz
pushed a commit
to Dylan-DPC-zz/rust
that referenced
this pull request
May 27, 2021
…disjoint-fields-gate, r=nikomatsakis readd capture disjoint fields gate This readds a feature gate guard that was added in PR rust-lang#83521. (Basically, there were unintended consequences to the code exposed by removing the feature gate guard.) The root bug still remains to be resolved, as discussed in issue rust-lang#85561. This is just a band-aid suitable for a beta backport. Cc issue rust-lang#85435 Note that the latter issue is unfixed until we backport this (or another fix) to 1.53 beta
This was referenced May 27, 2021
bors
added a commit
to rust-lang-ci/rust
that referenced
this pull request
May 27, 2021
Rollup of 8 pull requests Successful merges: - rust-lang#84221 (E0599 suggestions and elision of generic argument if no canditate is found) - rust-lang#84701 (stabilize member constraints) - rust-lang#85564 ( readd capture disjoint fields gate) - rust-lang#85583 (Get rid of PreviousDepGraph.) - rust-lang#85649 (Update cc) - rust-lang#85689 (Remove Iterator #[rustc_on_unimplemented]s that no longer apply.) - rust-lang#85719 (Add inline attr to CString::into_inner so it can optimize out NonNull checks) - rust-lang#85725 (Remove unneeded workaround) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Contributor
rustbot
added
the
beta-accepted
Dylan-DPC-zz
pushed a commit
to Dylan-DPC-zz/rust
that referenced
this pull request
Jun 3, 2021
…nikomatsakis Fix issue 85435 by restricting Fake Read precision This PR fixes the root bug of issue rust-lang#85435 by restricting Fake Read precision in closures and removing the feature gate introduced in PR rust-lang#85564. More info [here](rust-lang#85561 (comment)) and [here](rust-lang#85561 (comment)). Closes rust-lang#85561 r? `@nikomatsakis`
JohnTitor
added a commit
to JohnTitor/rust
that referenced
this pull request
Jun 3, 2021
…nikomatsakis Fix issue 85435 by restricting Fake Read precision This PR fixes the root bug of issue rust-lang#85435 by restricting Fake Read precision in closures and removing the feature gate introduced in PR rust-lang#85564. More info [here](rust-lang#85561 (comment)) and [here](rust-lang#85561 (comment)). Closes rust-lang#85561 r? ``@nikomatsakis``
bors
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Jun 8, 2021
…acrum [beta] backport of readd capture disjoint fields gate Beta backport of "readd capture disjoint fields gate", PR rust-lang#85564 Fix issue rust-lang#85435
Mark-Simulacrum
removed
the
beta-nominated
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This readds a feature gate guard that was added in PR #83521. (Basically, there were unintended consequences to the code exposed by removing the feature gate guard.)
The root bug still remains to be resolved, as discussed in issue #85561. This is just a band-aid suitable for a beta backport.
Cc issue #85435
Note that the latter issue is unfixed until we backport this (or another fix) to 1.53 beta