Skip to content
This repository was archived by the owner on Jun 21, 2020. It is now read-only.

Read trait #48

Merged
merged 8 commits into from
Nov 20, 2019
Merged

Read trait #48

merged 8 commits into from
Nov 20, 2019

Conversation

yoshuawuyts
Copy link
Member

@yoshuawuyts yoshuawuyts commented Nov 6, 2019

WIP, replaces Stream<Output = Bytes> with AsyncRead. Though I'd like to turn this into an AsyncBufRead instead to improve on allocations. Closes #31.

Either way, I'll be using this as a starting point for the work in http-rs/tide#336 (comment). Thanks!

@yoshuawuyts yoshuawuyts mentioned this pull request Nov 14, 2019
@yoshuawuyts yoshuawuyts marked this pull request as ready for review November 14, 2019 14:35
Copy link
Collaborator

@tirr-c tirr-c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see lots of addition of commas after type parameters, is this intended? I've never seen this style before.

Signed-off-by: Yoshua Wuyts <[email protected]>
Signed-off-by: Yoshua Wuyts <[email protected]>
Signed-off-by: Yoshua Wuyts <[email protected]>
Signed-off-by: Yoshua Wuyts <[email protected]>
Signed-off-by: Yoshua Wuyts <[email protected]>
Signed-off-by: Yoshua Wuyts <[email protected]>
Signed-off-by: Yoshua Wuyts <[email protected]>
Signed-off-by: Yoshua Wuyts <[email protected]>
@yoshuawuyts yoshuawuyts merged commit 5ad589f into master Nov 20, 2019
@yoshuawuyts yoshuawuyts deleted the read-trait branch November 20, 2019 17:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement AsyncBufRead for Body
3 participants