Skip to content

Memory span #183

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
Open
wants to merge 30 commits into
base: dev
Choose a base branch
from
Open

Memory span #183

wants to merge 30 commits into from

Conversation

@onizet
Copy link
Owner

@onizet onizet commented Dec 14, 2024 •
edited
Loading

Rewriting of parsing to use System.Span

Method Runtime Mean Error StdDev Ratio RatioSD Allocated
ParseWithSpan .NET 8.0 171.7 ms 17.36 ms 50.37 ms 1.08 0.42 906.87 KB
ParseWithRegex .NET 8.0 157.5 ms 7.79 ms 21.83 ms 1.02 0.19 1 MB
ParseWithSpan .NET 4.8 197.2 ms 12.04 ms 33.16 ms 1.12 0.28 1298.28 KB
ParseWithRegex .NET 4.8 192.1 ms 11.25 ms 32.27 ms 1.30 0.28 1489.05 KB

onizet and others added 30 commits September 28, 2024 21:42
@onizet
Improve performance by parsing HtmlColor with memory span
ec48fcb
@onizet
Improve regex to avoid a call to HtmlDecode
a5048cd
@onizet
Unit and Margin also now use memory span
eabbce6
@onizet
Bug fix on Span polyfill method
1dff57b
@onizet
Reuse existing .Net framework implementation of HtmlDecode if available
a1cd67e
@onizet
Fix sln markup
b523d8b
@onizet
Handle case sensitiveness on unit metric
e79b024
@onizet
Use regex compiled for performance boost
8ce2cec
@onizet
Set a timeout for regex
ca5dff9
@onizet
Optimise parsing of Style attributes
129bebb
@onizet
Ensure to catch regex timeout exception
66834d8
@onizet
Prefer usage of await and async method
4f516aa
@onizet
Optimise parsing of Style attributes
19c263f
@onizet
Update changelog
07e3f2e
@onizet
Add Benchmark project
2bd74d3
@onizet
Improve unit tests
83851e9
@onizet
Allow to run multiple runtimes benchmarks
7a5322b
@onizet
Include the csproj in the sln
52a3610
@onizet
Remove neat support for SearchValues as performant is worst than Read…
04e9302 
…OnlySpan
@onizet
Merge branch 'dev' into memory-span
2b14315
@onizet
Merge branch 'memory-span' of github.com:onizet/html2openxml into mem…
dbfb53e 
…ory-span
@onizet
Fix compilation error
89f4e5f
@onizet
Use FrozenDictionary for better performance (thanks Graham!)
e62d833
@onizet
Decrease warnings
5dc31f7
@onizet
Code simplification
03bd94d
@onizet
Rewrite parsing to improve code readability
9ec8636
@onizet
Do not allocate new array in a loop
935da6c
@onizet
AngleSharp update
af8711e
@onizet
Ensure to trim input before parsing the color
9b8f240
@onizet
Use benefits of FrozenDictionary only for net8, no cumbersome net462 …
436a600 
…with an additional package
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 10, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
7.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@onizet