Filter out lines which begin with '# ' in inferno-flamegraph

[mrob95]

implements #237

also fixes #238 by moving the check for empty/ignored lines out of merge and into from_lines. I think this makes more sense as the block in from_lines which handles opt.reverse_stack_order pre-processes the samples before calling merge, and seems to assume that the lines it deals with are not empty. I'm not very experienced with rust though, so please let me know if there is a better way!

I've added an empty line to the bad-lines test file as a regression test for #238, and a couple of test files for input from the austin profiler.

[codecov]

Codecov Report

Merging #239 (8d4cf04) into master (29fb054) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #239      +/-   ##
==========================================
+ Coverage   87.17%   87.21%   +0.04%     
==========================================
  Files          18       18              
  Lines        2526     2527       +1     
==========================================
+ Hits         2202     2204       +2     
+ Misses        324      323       -1     

Impacted Files	Coverage Δ
src/flamegraph/merge.rs	87.23% <ø> (+0.77%)	⬆️
src/flamegraph/mod.rs	87.03% <100.00%> (+0.13%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29fb054...8d4cf04. Read the comment docs.

[jonhoo]

Just one small nit and then I think we're good to land this. Thanks for taking the time to fix the issue too!

[jonhoo]

Thanks! I'll fix the MSRV issue separately — that's not caused by your change.

[jonhoo]

Published as 0.11.3 🎉

[mrob95]

Awesome! Thanks 😊