fktn-k / fkYAML

A C++ header-only YAML library
MIT License
67 stars 7 forks source link

Reduce string copies in parse #394

Closed fktn-k closed 3 weeks ago

fktn-k commented 3 weeks ago

This PR has significantly reduced the number & size of string copies and subsequent heap allocations during parsing of YAML sources by the following major changes:

As a result of the above changes, the benchmark scores of this library has increased by 0.5MiB/s (when parsing tool/benchmark/macos.yml file with the same environment described in README.md):

last benchmark score benchmark score after the changes
40.491 MiB/s 41.051 MiB/s

Pull Request Checklist

Read the CONTRIBUTING.md file for detailed information.

Please don't

github-actions[bot] commented 3 weeks ago

:octocat: Upload Coverage Event Notification

Coverage data has been uploaded for the commit feb0187cfbbff24a68e4be4b25ddce3894c508f4. You can download the artifact which contains the same file uploaded to the Coveralls and its HTML version.

Name fkYAML_coverage.pr394.zip
ID 1957793065
URL https://github.com/fktn-k/fkYAML/actions/runs/10958197573/artifacts/1957793065