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

Move Bytes to read::util #336

Merged
merged 1 commit into from
Jul 25, 2021
Merged

Move Bytes to read::util #336

merged 1 commit into from
Jul 25, 2021

Conversation

philipc
Copy link
Contributor

@philipc philipc commented Jul 25, 2021

This also means we don't need alloc when no features are enabled.

Fixes #328

This also means we don't need `alloc` when no features are enabled.
@philipc philipc merged commit daf8cdc into gimli-rs:master Jul 25, 2021
@philipc philipc deleted the issue-328 branch July 25, 2021 04:45
@@ -64,6 +64,7 @@
#[cfg(feature = "cargo-all")]
compile_error!("'--all-features' is not supported; use '--features all' instead");

#[cfg(feature = "read_core")]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this also needs to be enabled for write_core.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, I didn't see any uses of alloc based on git grep alloc src/write/

mcbegamerxx954 pushed a commit to mcbegamerxx954/object that referenced this pull request Jun 15, 2024
This also means we don't need `alloc` when no features are enabled.
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.

Don't depend on alloc when disabling the read_core and write_core features
2 participants