-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Add stream ext #573
Add stream ext #573
Conversation
Cc: @carllerche Followup to #558. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, one minor change requested inline.
tests/timer.rs
Outdated
@@ -71,6 +71,7 @@ fn starving() { | |||
rx.recv().unwrap(); | |||
} | |||
|
|||
#[allow(deprecated)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you remove this? I assume you get a warning. We should fix this in a follow up PR by updating the test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed the commit which had this.
14da312
to
1310ba5
Compare
The commit that added the #[allow(deprecated)]
was removed from this branch.
Ping? |
@mathstuf Can you rebase? That should pull in some changes which fix the broken unit tests. |
Primarily, it offers a `timeout` method for streams.
1310ba5
to
b89ec6b
Compare
Rebased. |
Thanks for pinging me... Sorry for letting this slip. |
No description provided.