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.

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

Implement zero-alloc support for reading field names #123

Merged
merged 3 commits into from
Apr 16, 2023
Merged

Implement zero-alloc support for reading field names #123

merged 3 commits into from
Apr 16, 2023

Commits on Apr 16, 2023

  1. Implement zero-alloc support for reading field names 

    Currently the custom type visitor just deserializes field names as
strings, which allocates a new System.String for each field. However, the
API supports passing through a ReadOnlySpan<byte> for UTF8 strings. This change
should provide support using the ROS<byte> in the normal path and avoid allocation.
    @agocke
    agocke committed Apr 16, 2023
    Configuration menu
    Copy the full SHA
    ec04640 View commit details
    Browse the repository at this point in the history

  2. Use struct visitor

    @agocke
    agocke committed Apr 16, 2023
    Configuration menu
    Copy the full SHA
    ce294e1 View commit details
    Browse the repository at this point in the history

  3. Re-baseline

    @agocke
    agocke committed Apr 16, 2023
    Configuration menu
    Copy the full SHA
    a07e5b1 View commit details
    Browse the repository at this point in the history