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

Crate reimplemented with more features. Replace? #36

Open
tmplt opened this issue Mar 11, 2021 · 2 comments
Open

Crate reimplemented with more features. Replace? #36

tmplt opened this issue Mar 11, 2021 · 2 comments

Comments

@tmplt
Copy link

tmplt commented Mar 11, 2021

I recently published itm-decode before stumbling upon this crate just now. Comparing the two after a cursory dive, itm seems a bit more general with its application of traits; itm-decode has a more granular error enum, to help with trace streams that diverge from the specification; #34 is being implemented, and an ITM encoder may eventually be implemented for testing purposes with some other tools I'll be implementing as part of my master's. I have yet to compare the test benches. bin/itm-decode.rs shares some features with itmdump.

Perhaps the crates should be merged?

@tmplt
Copy link
Author

tmplt commented Mar 11, 2021

Additionally, I believe sync packets are incorrectly handled. c.f. https://github.com/rtic-scope/itm-decode/blob/400447e8e01260c2fafc463c742646f2107a2ad6/src/lib.rs#L667-L683

@tmplt
Copy link
Author

tmplt commented Nov 21, 2021

Another planned feature is the bookkeeping of instrumentation packets. The API is also being refactored to mimic that of this crate (i.e., read data from a Stream<R: std::io::Read>).

@tmplt tmplt changed the title Crate reimplemented. Merge? Crate reimplemented with more features. Replace? Nov 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant