Skip to content

Fix PipeOptions ctor pauseWriterThreshold #4491

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
Merged
merged 1 commit into from
Jul 21, 2020
Merged

Fix PipeOptions ctor pauseWriterThreshold #4491

merged 1 commit into from
Jul 21, 2020

Conversation

carlossanlop
Copy link
Contributor

Should be zero instead of negative.

Fixes dotnet/runtime#476

@carlossanlop
Fix PipeOptions ctor pauseWriterThreshold
b9dbff8 
Should be zero instead of negative.
@carlossanlop carlossanlop added the doc-bug Problem with the content; needs to be fixed label Jul 9, 2020
@carlossanlop carlossanlop requested a review from halter73 July 9, 2020 00:58
@carlossanlop carlossanlop self-assigned this Jul 9, 2020
@dotnet-bot dotnet-bot added this to the July 2020 milestone Jul 9, 2020
@carlossanlop carlossanlop mentioned this pull request Jul 9, 2020
Closed
@carlossanlop
Copy link
Contributor Author

ping @halter73

@halter73
Copy link
Member

Sorry for the delay.

@carlossanlop carlossanlop merged commit 948347a into master Jul 21, 2020
@carlossanlop carlossanlop deleted the PipeOptionsPauseWriterThreshold branch July 21, 2020 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@halter73 halter73 halter73 approved these changes

Assignees

@carlossanlop carlossanlop

Labels
area-System.IO.Pipelines doc-bug Problem with the content; needs to be fixed
Projects
None yet
Milestone
July 2020
Development

Successfully merging this pull request may close these issues.

PipeOptions overrides default
4 participants
@carlossanlop @halter73 @maryamariyan @dotnet-bot