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.

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

Updated Microsoft.IO.RecyclableMemoryStream to version 3.0.0. #266

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dxdjgl
Copy link

@dxdjgl dxdjgl commented Dec 30, 2023

No description provided.

@trivalik
Copy link
Contributor

trivalik commented Feb 5, 2024

This is indeed a good idea, since version 3 has a breaking change in RecyclableMemoryStreamManager.GetStream(string) (return value has changed). alternative would be to set Version="[2.2.1,3.0.0)".

@dxdjgl dxdjgl force-pushed the master branch 3 times, most recently from 7a815c9 to 7cbdc2e Compare February 10, 2024 12:21
Copy link
Contributor

@trivalik trivalik left a comment

Choose a reason for hiding this comment

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

This is good, this allows to use both versions.

@dxdjgl
Copy link
Author

dxdjgl commented Feb 13, 2024

Hope it can merge and a build can be release soon.

@awcullen
Copy link
Contributor

I have updated Microsoft.IO.RecyclableMemoryStream to version 3.0.0 and found no issues. I like the new functions in RecyclableMemoryStream. In the future, we could improve the encoder performance.

@trivalik
Copy link
Contributor

@awcullen You would get a issue in case you have another library which require use of RecyclableMemoryStream with version 2, because of the breaking change mentioned above.

see microsoft/Microsoft.IO.RecyclableMemoryStream#324

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants