Hi Stolee,
On Fri, 12 Oct 2018, Derrick Stolee wrote:
>
> In an effort to ensure new code is reasonably covered by the test suite, we
> now have contrib/coverage-diff.sh to combine the gcov output from 'make
> coverage-test ; make coverage-report' with the output from 'git diff A B' to
> discover _new_ lines of code that are not covered.
Thanks for doing this. I do notice, though, that there are a few mentions
of BUG() lines, e.g.
> 0af129b2ed builtin/rebase--interactive2.c 267) BUG("invalid command '%d'",
> command);
I do not think that we will ever want to ensure that all of these lines
get code coverage ;-) Maybe it would be possible to exclude those lines
from the analysis?
Thank you,
Dscho
