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

[FIX] Cysharp/ObservableCollections#11 #12

Merged

neuecc merged 1 commit into Cysharp:master from yossiepon:fix/issue_11

Sep 30, 2022

Contributor

yossiepon commented Aug 25, 2022

Modified ArrayPool.Shared.Rent(count) to allocate a minimum buffer size of 16, because at this point the value of count is 0, which unintentionally creates an empty array.
The doubling of the buffer should be done in the branch whose index has reached the end of the buffer, as well as the deallocation.


          [FIX] Cysharp#11

973bee3

Member

neuecc commented Sep 30, 2022

thanks!

neuecc merged commit 6d6aa74 into Cysharp:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet