Skip to content
This repository has been archived by the owner on Dec 3, 2021. It is now read-only.

Mimic the API of embedded-rust/itm and ultimately replace it #20

Open
tmplt opened this issue Nov 20, 2021 · 2 comments
Open

Mimic the API of embedded-rust/itm and ultimately replace it #20

tmplt opened this issue Nov 20, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@tmplt
Copy link
Member

tmplt commented Nov 20, 2021

The current API of itm_decodes leaves a lot to desire. The API of embedded-rust/itm is more proper and should be mimiced if we mean to replace it.

@tmplt tmplt added this to the v0.7.0 milestone Nov 20, 2021
@tmplt tmplt added the enhancement New feature or request label Nov 20, 2021
@tmplt
Copy link
Member Author

tmplt commented Nov 21, 2021

As discussed in the Embedded Rust Matrix channel, itm-decode is queued to replace embedded-rust/itm following a Cortex-M team vote. The API of this crate will be refactored to better follow Rust API guidelines by copying some ideas from the existing itm crate, but also by implementing the API change discussed in #19. This work is done on the replace-itm-crate branch. When this is done (and presuming that the vote passes), a PR to embedded-rust/itm will be created. Following the merge, embedded-rust/itm v0.4.0 will be released and this crate will be deprecated.

@tmplt tmplt changed the title Mimic the API of embedded-rust/itm Mimic the API of embedded-rust/itm and ultimately replace it Nov 21, 2021
@tmplt tmplt removed this from the v0.7.0 milestone Nov 21, 2021
@tmplt
Copy link
Member Author

tmplt commented Dec 3, 2021

This has been handles as of rtic-scope/itm@3931431.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant