Skip to content

Replace mpmc_large feature using LenType - #682

Open
bmgxyz wants to merge 9 commits into
rust-embedded:mainfrom
bmgxyz:0561-remove-mpmc-large
Open

Replace mpmc_large feature using LenType#682
bmgxyz wants to merge 9 commits into
rust-embedded:mainfrom
bmgxyz:0561-remove-mpmc-large

Conversation

@bmgxyz

@bmgxyz bmgxyz commented Aug 5, 2026

Copy link
Copy Markdown
  • Remove mpmc_large feature
  • Add new generic LenT to Sealed (supertrait of LenType) and propagate to mpmc::Queue, mpmc::QueueInner, and storage::SealedStorage; set LenT = u8 in for Queue and QueueInner to maintain backwards compatibility
  • Add LenType::Atomic and LenType::Signed to support transition to LenT
  • Add Atomic trait that shadows Atomic in the standard library, which is currently unstable
  • Update and expand tests in mpmc and add a new cfail test

This PR introduces breaking changes.

Closes #561

LLM Usage Disclosure: I used an LLM to plan my changes, find solutions for certain compiler errors, and write test boilerplate. I wrote about half of the code by hand and reviewed all code myself.

@bmgxyz

bmgxyz commented Aug 5, 2026

Copy link
Copy Markdown
Author

Is it necessary to update the benchmark? I would do it myself, but I don't know how.

@sgued sgued added the API Break Breaking changes label Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API Break Breaking changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace mpmc_large feature using LenType.

2 participants