Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add IO::Sized #2972

Merged
merged 2 commits into from
Jul 11, 2016
Merged

Add IO::Sized #2972

merged 2 commits into from
Jul 11, 2016
+144 −14

Conversation

RX14
Copy link
Member

@RX14 RX14 commented Jul 9, 2016

This class is a wrapper around an IO which has a limited read size. A similar class is already used in the crystal stdlib for handling Content-Length headers in HTTP, so I replaced that with a implementation that extends IO::Sized.

Apart from that, this is a useful complement to IO::Delimited.

@RX14 RX14 force-pushed the feature/sized-io branch from ee51696 to 2944292 Compare July 9, 2016 19:17
@RX14 RX14 force-pushed the feature/sized-io branch from 2944292 to 3228c0b Compare July 9, 2016 19:18
@RX14 RX14 mentioned this pull request Jul 9, 2016
Closed
@jhass jhass added this to the 0.19.0 milestone Jul 11, 2016
@jhass jhass merged commit d4f50a9 into crystal-lang:master Jul 11, 2016
@jhass
Copy link
Member

jhass commented Jul 11, 2016

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
topic:stdlib
Projects
None yet
Milestone
0.19.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@RX14 @jhass @asterite