Skip to content

Conversation

@AdamKMeller
Copy link
Contributor

Description

Adds a gated import of toOwned which allows tracepoints to compile properly in a no_std environment.

Closes #173

API Stability

  • This PR does not require a breaking API change

Checklist

  • Documentation
    • Ensured any public-facing rustdoc formatting looks good (via cargo doc)
    • (if appropriate) Added feature to "Debugging Features" in README.md
  • Validation
    • Included output of running examples/armv4t with RUST_LOG=trace + any relevant GDB output under the "Validation" section below
    • Included output of running ./example_no_std/check_size.sh before/after changes under the "Validation" section below
  • If implementing a new protocol extension IDET
    • Included a basic sample implementation in examples/armv4t
    • IDET can be optimized out (confirmed via ./example_no_std/check_size.sh)
    • OR implementation requires introducing non-optional binary bloat (please elaborate under "Description")
  • If upstreaming an Arch implementation
    • I have tested this code in my project, and to the best of my knowledge, it is working as intended.

Validation

Changes are super minimal and definitely allows it to build in a no_std env. I will try and actually test changes and add comments with validation output soon.

Copy link
Owner

@daniel5151 daniel5151 left a comment

Choose a reason for hiding this comment

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

Thanks for sending this in! Will push out 0.7.7 with this fix shortly :)

@daniel5151 daniel5151 merged commit 432aa03 into daniel5151:master Aug 15, 2025
2 checks passed
@daniel5151
Copy link
Owner

0.7.7 has been released with this fix 🚀

@AdamKMeller
Copy link
Contributor Author

Perfect, I really appreciate how fast you fixed the issue.

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.

NoStd No Alloc build failures

2 participants