Skip to content

Enable no_std when std feature is not enabled - #565

Merged
folkertdev merged 2 commits into
trifectatechfoundation:mainfrom
RRRadicalEdward:allow-rust-allocator-in-no-std
Sep 17, 2026
Merged

folkertdev merged 2 commits into
trifectatechfoundation:mainfrom
RRRadicalEdward:allow-rust-allocator-in-no-std

Conversation

@RRRadicalEdward

@RRRadicalEdward RRRadicalEdward commented Sep 17, 2026 •

Copy link
Copy Markdown

The primary motivation is to be able to use Rust's allocator (via the rust-allocator feature flag) when targeting no_std.

@RRRadicalEdward
RRRadicalEdward force-pushed the allow-rust-allocator-in-no-std branch from ccfd754 to 71b8885 Compare September 17, 2026 16:03
@RRRadicalEdward RRRadicalEdward changed the title Allow rust-allocator in no_std Enable no_std when std feature is not enabled Sep 17, 2026
@RRRadicalEdward RRRadicalEdward changed the title Enable no_std when std feature is not enabled Enable no_std when std feature is not enabled Sep 17, 2026
@codecov

codecov Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
fuzz-compress ?
fuzz-decompress ?
test-aarch64-apple-darwin 91.10% <100.00%> (-0.02%) ⬇️
test-aarch64-unknown-linux-gnu 85.11% <100.00%> (-0.04%) ⬇️
test-i686-unknown-linux-gnu 84.60% <100.00%> (-0.03%) ⬇️
test-x86_64-apple-darwin 89.94% <100.00%> (-0.02%) ⬇️
test-x86_64-unknown-linux-gnu 93.26% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
zlib-rs/src/allocate.rs 95.13% <100.00%> (ø)
zlib-rs/src/lib.rs 89.28% <ø> (ø)

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@folkertdev

Copy link
Copy Markdown
Member

Nice. To make sure this keeps working, can you add this configuration to the CI checks too? As a part of link-c-dynamic-library maybe, it already has some similar configurations.

@RRRadicalEdward

Copy link
Copy Markdown
Author

Nice. To make sure this keeps working, can you add this configuration to the CI checks too? As a part of link-c-dynamic-library maybe, it already has some similar configurations.

Sure!

@folkertdev
folkertdev merged commit 7909c0f into trifectatechfoundation:main Sep 17, 2026
41 checks passed
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.

3 participants