Skip to content

Support POSIX AIO in mio again#1408

Merged
Thomasdezeeuw merged 2 commits intotokio-rs:masterfrom
asomers:aio
Dec 1, 2020
Merged

Support POSIX AIO in mio again#1408
Thomasdezeeuw merged 2 commits intotokio-rs:masterfrom
asomers:aio

Conversation

@asomers
Copy link
Copy Markdown
Collaborator

@asomers asomers commented Dec 1, 2020

  • impl AsRawFd for Registry
  • Add smoke tests for POSIX AIO

See also https://github.com/asomers/mio-aio

This is needed for POSIX AIO.

Fixes tokio-rs#1407
@asomers
Copy link
Copy Markdown
Collaborator Author

asomers commented Dec 1, 2020

The OSX test failures look unrelated.

Copy link
Copy Markdown
Collaborator

@Thomasdezeeuw Thomasdezeeuw left a comment

Choose a reason for hiding this comment

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

Add some comments about some small things, but overall LGTM.

@Thomasdezeeuw
Copy link
Copy Markdown
Collaborator

I forgot you need to call init from the test module before using temp_file. If you can fix that and squash the last two commits (leaving 2 commits; AsRawFd and tests), I'm good with merging.

@Thomasdezeeuw Thomasdezeeuw merged commit 6fad879 into tokio-rs:master Dec 1, 2020
@Thomasdezeeuw
Copy link
Copy Markdown
Collaborator

Thanks @asomers.

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