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

Add no_std support #17

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Add no_std support #17

wants to merge 7 commits into from

Conversation

Cxarli
Copy link

@Cxarli Cxarli commented Mar 3, 2021

This PR adds the possibility to disable the std feature which makes sure the crate can run on no_std environments such as embedded devices. It also adds the alloc feature to allow using Vec even in no_std environments. Since it would be a breaking API change, it has been made opt-in.

To run the tests for this environment, you will need to run it like cargo test --no-default-features --features alloc.

Running the benchmarks locally, the performance goes slightly up.

@stusmall
Copy link
Owner

stusmall commented Mar 4, 2021

Oh this is so great. Thank you. Give me a little bit to go through and review this but this has been on my personal TODO list for a while.

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.

2 participants