jonhoo / inferno

A Rust port of FlameGraph
Other
1.64k stars 117 forks source link

Avoid unsound usage of `from_utf8_unchecked` in test #289

Closed danielhenrymantilla closed 1 year ago

danielhenrymantilla commented 1 year ago

By operating on byte sequences / byte-strings directly

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (bf0d001) 91.14% compared to head (d7c2b59) 91.15%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #289 +/- ## ======================================= Coverage 91.14% 91.15% ======================================= Files 19 19 Lines 4237 4238 +1 ======================================= + Hits 3862 3863 +1 Misses 375 375 ``` | [Impacted Files](https://codecov.io/gh/jonhoo/inferno/pull/289?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jon+Gjengset) | Coverage Δ | | |---|---|---| | [src/collapse/dtrace.rs](https://codecov.io/gh/jonhoo/inferno/pull/289?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jon+Gjengset#diff-c3JjL2NvbGxhcHNlL2R0cmFjZS5ycw==) | `81.14% <100.00%> (+0.05%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jon+Gjengset). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jon+Gjengset)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.