Skip to content

Commit 037d4eb

Browse files
committed
zephyr: work: Remove debug 'Sync' implementation
The WorkWaker now really does implement Sync, as we use an Arc and Mutex to protect it. Signed-off-by: David Brown <[email protected]>
1 parent 68d1aa0 commit 037d4eb

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

zephyr/src/work.rs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -439,10 +439,6 @@ pub mod futures {
439439
// SAFETY: The worker struct is explicitly safe to send by the Zephyr docs.
440440
unsafe impl Send for WorkWaker {}
441441

442-
// SAFETY: The work is _not_ actually Sync, and this needs to be fixed. However, as long as the
443-
// 'wake' method for WorkWaker panics, we can implement Sync.
444-
unsafe impl Sync for WorkWaker {}
445-
446442
impl Wake for Mutex<WorkWaker> {
447443
fn wake(this: Arc<Self>) {
448444
// At this point, it isn't safe to implement a delayed waker because delayed wakers will

0 commit comments

Comments
 (0)