Skip to content

impl Sync for IdleHandle#2189

Merged
raphlinus merged 2 commits intomasterfrom
sync_idle
May 18, 2022
Merged

impl Sync for IdleHandle#2189
raphlinus merged 2 commits intomasterfrom
sync_idle

Conversation

@raphlinus
Copy link
Contributor

Make IdleHandle implement the Sync bound on all platforms. This is required to wrap IdleHandle in any Arc, motivated in particular by ArcWaker which is in turn motivated by async Xilem experiments (see #2184)

Make IdleHandle implement the Sync bound on all platforms. This is required to wrap IdleHandle in any Arc, motivated in particular by ArcWaker which is in turn motivated by async Xilem experiments (see #2184)
@jneem
Copy link
Member

jneem commented May 18, 2022

Could you also add Sync to the assertion at the end of druid-shell/src/window.rs?

@raphlinus raphlinus merged commit 6c72829 into master May 18, 2022
@cmyr cmyr deleted the sync_idle branch June 20, 2022 13:45
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.

2 participants