Skip to content

Use Option<Duration> for timeouts.#55

Open
reitermarkus wants to merge 1 commit intodcuddeback:masterfrom
reitermarkus:optional-timeout
Open

Use Option<Duration> for timeouts.#55
reitermarkus wants to merge 1 commit intodcuddeback:masterfrom
reitermarkus:optional-timeout

Conversation

@reitermarkus
Copy link
Copy Markdown

A timeout of None means that operations will block indefinitely. This is similar to what std::net::TcpStream does.

@reitermarkus reitermarkus force-pushed the optional-timeout branch 8 times, most recently from 87bc421 to 1bc266a Compare December 25, 2018 00:31
@reitermarkus
Copy link
Copy Markdown
Author

@dcuddeback, ping.

1 similar comment
@reitermarkus
Copy link
Copy Markdown
Author

@dcuddeback, ping.

@dcuddeback
Copy link
Copy Markdown
Owner

@dcuddeback, ping.

Please don't do that. I'm busy, and there's no reason your PR should receive priority over anything else on my agenda.

@reitermarkus
Copy link
Copy Markdown
Author

@dcuddeback, please forgive me for being a nag. I don't want preferential treatment, I just hate PRs going stale. I can understand you're busy, I'd volunteer as a collaborator on this project if you'd like, to take some workload off of you.

@detly
Copy link
Copy Markdown

detly commented Jul 8, 2021

Into<Option<Duration>> might be better, because then callers can still just pass a Duration if they want a timeout. See eg. Calloop's run() method.

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