Skip to content
This repository has been archived by the owner on Dec 18, 2018. It is now read-only.

Always complete RequestBodyPipe.Reader #1893

Merged
merged 3 commits into from
Jun 12, 2017
Merged

Always complete RequestBodyPipe.Reader #1893

merged 3 commits into from
Jun 12, 2017

Conversation

halter73
Copy link
Member

@halter73 halter73 commented Jun 9, 2017

This is part of an effort to stop leaking memory pool blocks.

Semi related: dotnet/corefxlab#1601

@cesarblum
Copy link
Contributor

CI is upset. But the change itself looks good.

@halter73 halter73 force-pushed the halter73/leaky-blocks branch from f6958b0 to 98be991 Compare June 9, 2017 18:28
Copy link
Contributor

@cesarblum cesarblum left a comment

Choose a reason for hiding this comment

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

:shipit: when CI is green.

@halter73 halter73 force-pushed the halter73/leaky-blocks branch from 98be991 to a030ba9 Compare June 9, 2017 19:42
@halter73 halter73 merged commit d879518 into dev Jun 12, 2017
@halter73 halter73 deleted the halter73/leaky-blocks branch June 12, 2017 18:11
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.

3 participants