Skip to content

asyn-ares: remove typecast, fix expire - #15145

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/c-ares-getsock
Closed

asyn-ares: remove typecast, fix expire#15145
bagder wants to merge 1 commit into
masterfrom
bagder/c-ares-getsock

Conversation

@bagder

@bagder bagder commented Oct 3, 2024

Copy link
Copy Markdown
Member
  • Use the appropriate variable type for the curlx_tvtoms() return code: timediff_t and remove the typecast.

  • Simplify the function and avoid the odd expire adjustment that probably is a rest from ancient days when the expire function did not handle zero millisecond timeouts.

- Use the appropriate variable type for the curlx_tvtoms() return code:
  timediff_t and remove the typecast.

- Simplify the function and avoid the odd expire adjustment that
  probably is a rest from ancient days when the expire function did not
  handle zero millisecond timeouts.
@bagder bagder added name lookup DNS and related tech tidy-up labels Oct 3, 2024
@bagder bagder closed this in 6f454ba Oct 4, 2024
@bagder
bagder deleted the bagder/c-ares-getsock branch October 4, 2024 12:08
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
- Use the appropriate variable type for the curlx_tvtoms() return code:
  timediff_t and remove the typecast.

- Simplify the function and avoid the odd expire adjustment that
  probably is a rest from ancient days when the expire function did not
  handle zero millisecond timeouts.

Closes curl#15145
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

name lookup DNS and related tech tidy-up

Development

Successfully merging this pull request may close these issues.

1 participant