Skip to content

perf: use closures instead of bound functions#704

Open
jonahsnider wants to merge 1 commit intomainfrom
closures
Open

perf: use closures instead of bound functions#704
jonahsnider wants to merge 1 commit intomainfrom
closures

Conversation

@jonahsnider
Copy link
Copy Markdown
Owner

No description provided.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Nov 1, 2024

CodSpeed Performance Report

Merging #704 will improve performances by 18.3%

Comparing closures (b27aa8d) with main (e2facb1)

Summary

⚡ 3 improvements
✅ 3 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
convert 8192 bytes to best 61.2 µs 51.7 µs +18.3%
convert 2.5 liters to in3 22.5 µs 19.5 µs +15%
parse 10h to ms 82.1 µs 70.5 µs +16.39%

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant