Enhance LCOV formatter. #215
Conversation
|
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. |
Add number of instrumented lines 'LF' and number of lines with a non-zero execution count 'LH'
|
CLAs look good, thanks! |
|
@cbracken can you merge this PR? |
| final lines = v.keys.toList()..sort(); | ||
| lines.forEach((int k) { | ||
| buf.write('DA:$k,${v[k]}\n'); | ||
| }); |
There was a problem hiding this comment.
While you're here, would you mind switching this to use a for-loop here rather than .forEach()? (I'm the guilty party on the original)
There was a problem hiding this comment.
Actually never mind, I'll follow up and eliminate my previous shameful commit separately ;)
|
LGTM -- thanks for the patch and apologies for missing this earlier!
|
Add number of instrumented lines 'LF' and number of lines with a non-zero execution count 'LH'
Add number of instrumented lines 'LF' and number of lines with a non-zero execution count 'LH'