[Ctk-developers] CTK Dasboard: Low Code Coverage : 42% .and. 608 Valgrind Errors.

[Julien Finet]

Agree, let's test it all ! :-)

I already have a test ready (in Slicer) for ctkModelTester. I like testing
testers.

I can also look at ctkDICOMIndexer.cxx, but I'd probably need some help from
Marco...

Julien.

On Thu, Apr 8, 2010 at 1:58 PM, Luis Ibanez <luis.ibanez at kitware.com> wrote:

> The CTK Dasboard is in need of some Though Love.
>
>
> A) Code Coverage:  42%
> http://my.cdash.org/viewCoverage.php?buildid=57880
>
>
> B) 608 Valgring Errors
> http://my.cdash.org/viewDynamicAnalysis.php?buildid=57892
>
>
>         We must fix this before the Hackaton.
>
>
> It will be a disaster to have a lot of new code to be put
> inside a repository with a weak testing infrastructure.
>
>
> Any volunteer for adopting the orphan classes ?
>
> ./Libs/Core/ctkDependencyGraph.h
> ./Libs/DICOM/Core/ctkDICOMIndexer.cxx
> ./Libs/Core/ctkDependencyGraph.cxx
> ./Libs/Core/ctkModelTester.cxx
>
>
>
> Many of the memory leaks seems to come from :
>
> ==9718==    at 0x4C25153: malloc (vg_replace_malloc.c:195)
> ==9718==    by 0x75331A1: strdup (strdup.c:43)
> ==9718==    by 0x89F0C49: IceRegisterForProtocolSetup (in
> /usr/lib/libICE.so.6.3.0)
>
>
> Any reason for using the old strdup instead
> of using "std::string"     ?
>
>
> Finally,
>
> There are no Windows nor Mac builds in the Dashboard...
>
> Any volunteers for setting up additional builds ?
>
>
>
>      Luis
> _______________________________________________
> Ctk-developers mailing list
> Ctk-developers at commontk.org
> http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/ctk-developers/attachments/20100408/1c655cba/attachment.htm>