Skip to content

Conversation

@jesses2025smith
Copy link
Contributor

  1. update windows to 0.61 and fix;
  2. add anyhow and update rand for example;
  3. add [PoisonError] to [Error] for replace unwrap to map_err;

@jesses2025smith
Copy link
Contributor Author

fixed #403

@qdot
Copy link
Contributor

qdot commented Apr 20, 2025

In windows-rs 0.61, how does .get() work? Is that the blocking form, since it isn't awaited now?

@qdot
Copy link
Contributor

qdot commented Apr 20, 2025

Ok, yup, answered my own question by looking up the relevant windows-rs PR: microsoft/windows-rs#3213

.get() is blocking. This can't come in as is, we need to use the IntoFuture() casts and await as we currently do.

qdot added a commit that referenced this pull request Apr 20, 2025
Can't block, but the conversion is easy enough that I just want to get
it done, so I'm handling it.
@qdot qdot merged commit 7a4a405 into deviceplug:dev Apr 20, 2025
@qdot
Copy link
Contributor

qdot commented Apr 20, 2025

Ok, I did the fix myself 'cause it was simple and I'd like to get windows 0.61 in.

qdot added a commit that referenced this pull request Apr 21, 2025
Can't block, but the conversion is easy enough that I just want to get
it done, so I'm handling it.
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