Fixed issue with interior commas in function names.

eriwen / lcov-to-cobertura-xml

Converts lcov output to Cobertura-compatible XML for CI

https://eriwen.github.io/lcov-to-cobertura-xml/

Apache License 2.0

187 stars 73 forks source link

Fixed issue with interior commas in function names. #37

Closed cgossel closed 2 years ago

cgossel commented 2 years ago

Set maximum split parameter when breaking FN and FNDA values. With certain languages (Rust), demangled LCOV FN and FNDA entries can contain interior commas. This will result in more than two sections when splitting, and cause an exception when there are two many elements to unpack.

eminence commented 2 years ago

I'm also running into this problem, and I confirm the fix here is working for my rust code base

eriwen commented 2 years ago

Thank you @cgossel for the PR.

There was a conflict and another fix PR. I've applied with 2026dfa and b7bda153ff41133264ba9aeea3452e18fa4d08db

This fix will go out with the next release shortly.