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.

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

consider replacing hex with const-hex for improved performance #2260

Open
borngraced opened this issue Oct 31, 2024 · 1 comment
Open

consider replacing hex with const-hex for improved performance #2260

borngraced opened this issue Oct 31, 2024 · 1 comment
Labels
micro-opt Optimization at very minimal level

Comments

@borngraced
Copy link
Member

It looks like the hex crate (https://github.com/KokaKiwi/rust-hex) hasn’t been actively maintained for over three years. Since we work a lot with encoding and decoding, it might be worth considering a switch to the const-hex crate (https://github.com/DaniPopes/const-hex) instead, mainly for the performance improvements

@onur-ozkan
Copy link
Member

Some dependencies don’t need active maintenance as their implementation relies on strict standards (e.g., md5). But still, I’m quite impressed by the performance difference between hex and const-hex; I think it’s worth trying out!

@onur-ozkan onur-ozkan added the micro-opt Optimization at very minimal level label Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
micro-opt Optimization at very minimal level
Projects
None yet
Development

No branches or pull requests

3 participants
@onur-ozkan @borngraced and others