itm icon indicating copy to clipboard operation
itm copied to clipboard

Crate reimplemented with more features. Replace?

Open tmplt opened this issue 4 years ago • 2 comments

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 avatar Mar 11 '21 21:03 tmplt

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 avatar Mar 11 '21 21:03 tmplt

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 avatar Nov 21 '21 02:11 tmplt