Adopt changes from upstream pull requests about inferno HOT 11 OPEN

Hi @versable I am trying to help out the inferno project which is a rewrite of Flamegraph in Rust. Amoung other improvements to the port, we are looking at existing open changes in this project that we might be able to bring over. Would you be ok with us adapting your PR for SVG improvements for inferno? I am not certain all of the changes apply, but at least a few of them seem very promising!

from inferno.

@jbcden, Sure, go ahead. Very interesting project btw, I love it!

from inferno.

Ah, derp, sorry, that's my lack of reading. I apologize!

from inferno.

@versable, looks like this PR is completely in JS, so you can easily backport your changes into: https://github.com/jonhoo/inferno/blob/master/src/flamegraph/flamegraph.js

from inferno.

@jonhoo, I am waiting for some examples to try generate new regex.

from inferno.

@jonhoo: It looks like there are currently 2 PRs in the queue, one of which you seem to be actively handling (#32) and the other is waiting on the OP to fix some upstream issues they discovered with the PR (#38). Is there anything left for this issue specifically?

from inferno.

@ErichDonGubler ah, so this issue is about PRs to the original Perl implementation that we may be able to port over to the Rust version. I'm sure there are plenty of PRs from there that would be good to port!

from inferno.

Is there any interest in brendangregg/FlameGraph#198? If so, I'd love to get my feet wet with Rust, and this would be really up my alley ;-)

from inferno.

@versable oooh, that seems really neat, so absolutely! I'd be happy to help with a PR :)

from inferno.

@jonhoo, What do you think about "Support AsyncProfiler generated stack trace": brendangregg/FlameGraph#234

from inferno.

@AnderEnder That looks like a good thing to support, though the fix looks dubious to me. The proposed regex, ^[^/].*\., matches anything that starts with a non-slash and contains a full stop, which seems overly broad to say "it's Java"?

from inferno.