Skip to content

Revert web-async to std::sync::Mutex - #35

Merged
kixelated merged 5 commits into
mainfrom
parking_lot
Mar 3, 2026
Merged

Revert web-async to std::sync::Mutex#35
kixelated merged 5 commits into
mainfrom
parking_lot

Conversation

@kixelated

@kixelated kixelated commented Feb 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • Replace parking_lot::Mutex with std::sync::Mutex in web-async
  • Remove deadlock feature and deadlock.rs (depended on parking_lot)
  • Remove parking_lot and ctor dependencies

parking_lot::Mutex doesn't implement UnwindSafe/RefUnwindSafe, which strips those auto-trait impls from any type containing Lock<T>. This caused 8 auto_trait_impl_removed semver failures in downstream crates (moq-lite, hang).

Test plan

  • cargo check --target aarch64-apple-darwin passes
  • CI passes

🤖 Generated with Claude Code

@kixelated
kixelated enabled auto-merge (squash) February 18, 2026 20:54
@kixelated
kixelated disabled auto-merge February 18, 2026 20:54
kixelated and others added 2 commits February 18, 2026 12:55
parking_lot::Mutex doesn't implement UnwindSafe/RefUnwindSafe, which
removes auto-trait impls from any type containing Lock<T>. This caused
8 semver failures in downstream crates (moq-lite, hang).

Switch back to std::sync::Mutex which preserves these auto-traits.
Remove the deadlock detection feature since it depended on parking_lot.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kixelated kixelated changed the title Switch to parking lot for optional deadlock detection Revert web-async to std::sync::Mutex Mar 3, 2026
@kixelated
kixelated enabled auto-merge (squash) March 3, 2026 01:09
@kixelated
kixelated disabled auto-merge March 3, 2026 03:03
@kixelated
kixelated merged commit 8f49ba8 into main Mar 3, 2026
0 of 2 checks passed
@kixelated
kixelated deleted the parking_lot branch March 3, 2026 03:03
@github-actions github-actions Bot mentioned this pull request Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant