Skip to content

Conversation

@yecril71pl
Copy link
Contributor

@yecril71pl yecril71pl commented Jan 5, 2022

Summary

BinaryReader.Dispose releases the BaseStream unless configured otherwise.

`BinaryReader.Dispose` releases the `BaseStream` unless configured otherwise.
@yecril71pl yecril71pl requested a review from a team as a code owner January 5, 2022 15:31
@ghost ghost added the area-System.IO label Jan 5, 2022
@ghost
Copy link

ghost commented Jan 5, 2022

Tagging subscribers to this area: @carlossanlop
See info in area-owners.md if you want to be subscribed.

Issue Details

Summary

BinaryReader.Dispose releases the BaseStream unless configured otherwise.

Fixes #Issue_Number (if available)

Author: yecril71pl
Assignees: -
Labels:

area-System.IO

Milestone: -

@opbld33
Copy link

opbld33 commented Jan 5, 2022

Docs Build status updates of commit e7723ad:

✅ Validation status: passed

File Status Preview URL Details
xml/System.IO/BinaryReader.xml ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

Copy link
Member

@adamsitnik adamsitnik left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @yecril71pl !

@adamsitnik adamsitnik merged commit f5192c4 into dotnet:main Mar 31, 2022
@yecril71pl yecril71pl deleted the patch-6 branch April 2, 2022 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants