Don't drop everything after "(" inside C++ template parameters

[adamnovak]

This should eliminate weird partial things like std::function<void from flame graphs.

Fixes #120 Fixes #192 Fixes #214

[christoph-cullmann]

Tested here at AbsInt, very useful to avoid strange cuttings for template names.

[Hi-Angel]

Note: since the project doesn't have an active maintainer to address issues or even PRs, there's an active analogous project that you can use. It's written in Rust. Screenshot for a C++ testcase from this project:

[brendangregg]

I worked on this project last month. These PRs that change the regexps require a lot of testing, and I've been meaning to rewrite the testing framework before merging them. That's a lot of work but will pay off for future regexp changes.

[pizzard]

This works great for me.

[SoftwareApe]

@Hi-Angel I like the much cleaner interface, however with the rust tool (v0.3.0 installed using cargo install flamegraph) I get the exact same "function<bool" partial function names.

EDIT: Now fixed in inferno v0.9.7, which should at some point be referenced in flamegraph-rs.