Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Less truncating when casting timeout values for Poll::poll #420

Closed
wants to merge 1 commit into from

Conversation

tbu-
Copy link
Contributor

@tbu- tbu- commented Jul 2, 2016

No description provided.

@tbu- tbu- force-pushed the pr_timeout_less_truncating branch 5 times, most recently from fcaf5a5 to d06747f Compare July 3, 2016 12:09
@tbu- tbu- force-pushed the pr_timeout_less_truncating branch from d06747f to 68fd1a8 Compare July 3, 2016 12:22
@tbu-
Copy link
Contributor Author

tbu- commented Jul 3, 2016

The remaining failures on Travis look like they're not caused by this PR.

@carllerche
Copy link
Member

Thanks. Merged into dev 7108b3d

@carllerche carllerche closed this Jul 18, 2016
tbu- added a commit to tbu-/mio that referenced this pull request Dec 25, 2022
Round up submillisecond wait times to the next millisecond on platforms
where submillisecond sleep times aren't supported. This makes the
behavior actually match the documentation.

See also tokio-rs#420, where this was implemented for the first time.
tbu- added a commit to tbu-/mio that referenced this pull request Dec 25, 2022
Round up submillisecond wait times to the next millisecond on platforms
where submillisecond sleep times aren't supported. This makes the
behavior actually match the documentation.

See also tokio-rs#420, where this was implemented for the first time.
Thomasdezeeuw pushed a commit that referenced this pull request Dec 27, 2022
Round up submillisecond wait times to the next millisecond on platforms
where submillisecond sleep times aren't supported. This makes the
behavior actually match the documentation.

See also #420, where this was implemented for the first time.
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