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.

Sign up for GitHub

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

Jump to bottom

multi: fix small timeouts #11937

Closed
wants to merge 1 commit into from
Closed

multi: fix small timeouts #11937

wants to merge 1 commit into from
+1 −1

Conversation

hwti
Copy link
Contributor

@hwti hwti commented Sep 25, 2023

Since Curl_timediff rounds down to the millisecond, timeouts which expire in less than 1ms are considered as outdated and removed from the list.
We can use Curl_timediff_us instead, big timeouts could saturate but this is not an issue.

@hwti
multi: fix small timeouts
9e63e5c 
Since Curl_timediff rounds down to the millisecond, timeouts which expire in less than 1ms
are considered as outdated and removed from the list.
We can use Curl_timediff_us instead, big timeouts could saturate but this is not an issue.
@hwti hwti mentioned this pull request Sep 25, 2023
Closed
@bagder
Copy link
Member

bagder commented Sep 25, 2023

Thanks!

@bagder bagder closed this in 579f093 Sep 25, 2023
@hwti hwti deleted the small-timeout branch September 27, 2023 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bagder bagder bagder approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@hwti @bagder