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

perf: using plus 1 count for other collection types #1073

Merged
merged 2 commits into from
Apr 14, 2022

Conversation

James-Frowen
Copy link
Member

For byte[], 0 is written for null and count+1 is written for count, we can re-use this for other collections

Splitting these methods into seperate fucntions so they can be used for all collections

Also changing are ArraySegment is read, it now has its own extension methods that checks if the array is null or not before creating the segment

For byte[], 0 is written for null and count+1 is written for count, we can re-use this for other collections

Splitting these methods into seperate fucntions so they can be used for all collections

Also changing are ArraySegment<T> is read, it now has its own extension methods that checks if the array is null or not before creating the segment
@sonarcloud
Copy link

sonarcloud bot commented Apr 13, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@James-Frowen James-Frowen merged commit 25ab6f3 into master Apr 14, 2022
@James-Frowen James-Frowen deleted the write-plus-one branch April 14, 2022 13:57
github-actions bot pushed a commit that referenced this pull request Apr 14, 2022
## [122.0.2](v122.0.1...v122.0.2) (2022-04-14)

### Bug Fixes

* increasing log to warning when receiving known type without handler ([#1072](#1072)) ([05db6cf](05db6cf))

### Performance Improvements

* using plus 1 count for other collection types ([#1073](#1073)) ([25ab6f3](25ab6f3))
@github-actions
Copy link
Contributor

🎉 This PR is included in version 122.0.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

1 participant