Skip to content

Remove dependency on winapi 0.2#25

Merged
carllerche merged 2 commits intocarllerche:v0.1.xfrom
EmbarkStudios:v0.1.x-remove-winapi-0.2
Oct 9, 2019
Merged

Remove dependency on winapi 0.2#25
carllerche merged 2 commits intocarllerche:v0.1.xfrom
EmbarkStudios:v0.1.x-remove-winapi-0.2

Conversation

@repi
Copy link

@repi repi commented Sep 20, 2019

Instead declare the few types we use directly in the crate to avoid bringing in the old winapi 0.2 crate.

Fixes #16.
Replaces previous attempts #19 #22 #24.

Instead declare the few types we use directly in the crate to avoid bringing in the old winapi 0.2 dependency.

Fixes carllerche#16.
Replaces previous attempts carllerche#19 carllerche#22 carllerche#24.
@repi
Copy link
Author

repi commented Sep 23, 2019

@carllerche does this look good?

@carllerche carllerche merged commit d564f78 into carllerche:v0.1.x Oct 9, 2019
@carllerche
Copy link
Owner

Thanks. I'll push this out. Unfortunately, it won't help any on the Tokio / mio stack until the next major release, but it is a step.

type ULONG = u32;
type CHAR = i8;

struct WSABUF {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be #[repr(C)] here.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you're right, missed that, damn. will do PR

@repi repi mentioned this pull request Oct 9, 2019
This was referenced Dec 16, 2019
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.

3 participants