codecov[bot]
commented
1 year ago Codecov Report
Patch coverage: 88.97% and project coverage change: -0.07% :warning:
Comparison is base (
a497e3e) 91.08% compared to head (ba6d131) 91.01%.
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #291 +/- ## ========================================== - Coverage 91.08% 91.01% -0.07% ========================================== Files 18 19 +1 Lines 4249 4385 +136 ========================================== + Hits 3870 3991 +121 - Misses 379 394 +15 ``` | [Files Changed](https://app.codecov.io/gh/jonhoo/inferno/pull/291?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jon+Gjengset) | Coverage Δ | | |---|---|---| | [src/collapse/mod.rs](https://app.codecov.io/gh/jonhoo/inferno/pull/291?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jon+Gjengset#diff-c3JjL2NvbGxhcHNlL21vZC5ycw==) | `96.96% <ø> (ø)` | | | [src/collapse/recursive.rs](https://app.codecov.io/gh/jonhoo/inferno/pull/291?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jon+Gjengset#diff-c3JjL2NvbGxhcHNlL3JlY3Vyc2l2ZS5ycw==) | `88.97% <88.97%> (ø)` | |:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
jonhoo
Original script - https://github.com/brendangregg/FlameGraph/blob/master/stackcollapse-recursive.pl
I was a bit unsure whether this fit better as a standalone script or as an option to the existing collapse scripts, but for a first pass I ported this as a standalone script, as
--squash-recursiveoption to one of the other collapse scripts after you decide you want to collapse recursive calls.I can definitely also see benefits to adding this as an option to the existing collapse scripts, so I'm happy to switch to that if you'd prefer!
Fixes #275.