Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Reconsider naming of Timeout::KeepAlive #15

Closed
46bit opened this issue Mar 23, 2017 · 1 comment
Closed

Reconsider naming of Timeout::KeepAlive #15

46bit opened this issue Mar 23, 2017 · 1 comment
Labels
enhancement
Milestone
First relatively ...

Comments

@46bit
Copy link
Owner

46bit commented Mar 23, 2017

[Ralith] and "keepalive" is a confusing name for "wake up at a regular interval"

I rather agree
@46bit 46bit mentioned this issue Mar 23, 2017
Closed
46bit added a commit that referenced this issue Mar 23, 2017
@46bit
Heavily reworked timeout mechanisms to be compositional. #16
cd5e45b 
Closes #17
Closes #15
Fixes Stream half of #13

```rust
client.receive().and_then(|(msg, client)| { … })
client.receive().with_hard_timeout(duration).and_then(|(msg, client)| { … })
client.receive().with_soft_timeout(duration).and_then(|(maybe_msg, client)| { … })

room.receive().and_then(|(msgs, room)| { … })
room.receive().with_hard_timeout(duration).and_then(|(msgs, room)| { … })
room.receive().with_soft_timeout(duration).and_then(|(msgs, room)| { … })
```
@46bit
Copy link
Owner Author

46bit commented Mar 23, 2017

Timeout type no longer exists.

@46bit 46bit closed this as completed Mar 23, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
First relatively stable API
Development

No branches or pull requests
1 participant
@46bit