[Insight-developers] Deceptive Coverage Numbers

Gaëtan Lehmann gaetan.lehmann at jouy.inra.fr
Thu Apr 16 15:48:21 EDT 2009


Hi Luis,

Le 16 avr. 09 à 18:22, Luis Ibanez a écrit :

>
> Hi Bill,
>
> Good catch...
>
> Visual inspection is probably the best idea at this point,
> since it also provides a code review and will prompt many
> improvements.
>
> Another option that comes to mind is to wrap this classes,
> (for some common instantiations) in which case, all the
> templated methods will be instantiated.

Wrapping only ensures that the code builds - already a nice beginning  
- but I'm afraid it won't help to enhance the code coverage.
Also, with wrapitk, itk::Statistics::Histogram is not build currently,  
because of the nested namespace (It's ok in the unstable version).

Regards,

Gaëtan


>
>
>
>   Luis
>
>
> ----------------------
> Bill Lorensen wrote:
>> Luis,
>> Take a look at the coverage for this file:
>> http://www.cdash.org/CDash/viewCoverageFile.php?buildid=314097&fileid=10830502
>> Note that several methods do not show up as program lines and thus do
>> not show up as uncovered lines. This is not limited to the new
>> statistics coverage, we have the same issues in ITK.
>> I believe it is because the template code is never referenced and
>> therefore is not considered code. I think the only remedy for now  
>> is a
>> visual inspection.
>> Bill
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
>
> Please keep messages on-topic and check the ITK FAQ at: http://www.itk.org/Wiki/ITK_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://www.itk.org/mailman/listinfo/insight-developers

-- 
Gaëtan Lehmann
Biologie du Développement et de la Reproduction
INRA de Jouy-en-Josas (France)
tel: +33 1 34 65 29 66    fax: 01 34 65 29 09
http://voxel.jouy.inra.fr  http://www.mandriva.org
http://www.itk.org  http://www.clavier-dvorak.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 186 bytes
Desc: Ceci est une signature ?lectronique PGP
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20090416/0f0698e4/attachment.pgp>


More information about the Insight-developers mailing list