Skip to content

Conversation

weiznich
Copy link
Contributor

@weiznich weiznich commented Sep 8, 2025

This commit manually implements RefUnwindSafe for std::sync::Barrier to fix #146087. This is a fix for a regression indroduced by e95db59

@rustbot
Copy link
Collaborator

rustbot commented Sep 8, 2025

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Sep 8, 2025
@rustbot

This comment has been minimized.

This commit manually implements `RefUnwindSafe` for
`std::sync::Barrier` to fix 146087. This is a fix for a regression
indroduced by rust-lang@e95db59
@weiznich
Copy link
Contributor Author

I would appreciate if this could land before the next beta branches of so that this regression is not present there.

@joboet
Copy link
Member

joboet commented Sep 10, 2025

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Sep 10, 2025

📌 Commit ef7b036 has been approved by joboet

It is now in the queue for this repository.

@bors bors 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 Sep 10, 2025
@joboet joboet assigned joboet and unassigned Mark-Simulacrum Sep 10, 2025
bors added a commit that referenced this pull request Sep 10, 2025
Rollup of 8 pull requests

Successful merges:

 - #145327 (std: make address resolution weirdness local to SGX)
 - #145879 (default auto traits: use default supertraits instead of `Self: Trait` bounds on associated items)
 - #146123 (Suggest examples of format specifiers in error messages)
 - #146311 (Minor symbol comment fixes.)
 - #146322 (Make Barrier RefUnwindSafe again)
 - #146327 (Add tests for deref on pin)
 - #146340 (Strip frontmatter in fewer places)
 - #146342 (Improve C-variadic error messages: part 2)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit d061896 into rust-lang:master Sep 11, 2025
10 checks passed
@rustbot rustbot added this to the 1.91.0 milestone Sep 11, 2025
rust-timer added a commit that referenced this pull request Sep 11, 2025
Rollup merge of #146322 - weiznich:fix/146087, r=joboet

Make Barrier RefUnwindSafe again

This commit manually implements `RefUnwindSafe` for `std::sync::Barrier` to fix #146087. This is a fix for a regression indroduced by e95db59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

std::sync::barrier::BarrierState is not RefUnwindSafe anymore
7 participants