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

perf(encode): vec alloc & utf8 unchecked #180

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

perf(encode): vec alloc & utf8 unchecked #180

wants to merge 2 commits into from

Commits on Mar 6, 2022

  1. perf(encode): vec alloc & utf8 unchecked 

    Improves throughput by 10-20%

We avoid zeroing the allocated vec since we'll write into it and we don't need to check utf8 when building the String since all alphabets emit ascii/utf8
    @AaronO
    AaronO committed Mar 6, 2022
    Configuration menu
    Copy the full SHA
    1e00cdc View commit details
    Browse the repository at this point in the history

  2. fmt

    @AaronO
    AaronO committed Mar 6, 2022
    Configuration menu
    Copy the full SHA
    392ec8f View commit details
    Browse the repository at this point in the history