[PATCH] rtems-tools/covoar: Wrong coverage computed

Joel Sherrill joel at rtems.org
Fri Jan 25 14:38:05 UTC 2019


On Fri, Jan 25, 2019 at 8:10 AM Jiri Gaisler <jiri at gaisler.se> wrote:

> Here is a final fix that also corrects the coverage computations. The
> parsing of the symbol table made all symbols 2 bytes too large, hence the
> wrong annotations. I have manually verified the correlation between sis
> coverage output and the covoar coverage calculations and annotations for a
> couple of RTEMS tests programs, and there is now a 100% match.
>
> Please review and let me know if I can push this.
>

This looks good except it combines the change to allow selecting the format
from the command line with the "+1 / -1" fix. They should be separate
patches.

They are technically good. Just push them as separate patches.

>
> Jiri.
>
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20190125/67c67ab4/attachment-0002.html>


More information about the devel mailing list