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

Performance improvements to CoberturaParser #691

Merged
merged 1 commit into from
Aug 28, 2024
Merged

Performance improvements to CoberturaParser #691

merged 1 commit into from
Aug 28, 2024

Commits on Aug 28, 2024

  1. Performance improvements to CoberturaParser 

    - Try and re-use already filtered XElements in `CoberturaParser` rather than re-filtering the xml document all over the place.
- Optimise some `string.Concat` in `ProcessClass` method which were showing up in hot paths in profiling

In local testing, this reduces parsing time from roughly 11.5secs to 9 seconds, as well as reducing memory allocations from  17GB to 2.2GB.
    @afscrome
    afscrome committed Aug 28, 2024
    Configuration menu
    Copy the full SHA
    e5e2775 View commit details
    Browse the repository at this point in the history