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

[Marco Nolden]

Dear Julien,

I will have a look at ctkDICOMIndexer.cxx before the next hackfest.

Marco


Am 08.04.2010 20:10, schrieb 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 
> <mailto: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 <mailto:Ctk-developers at commontk.org>
>     http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers
>
>


-- 
----------------------------------------------------------------------
Dipl.-Inform. Med. Marco Nolden
Deutsches Krebsforschungszentrum       (German Cancer Research Center)
Div. Medical&  Biological Informatics          Tel: (+49) 6221-42 2325
Im Neuenheimer Feld 280                        Fax: (+49) 6221-42 2345
D-69120 Heidelberg                             eMail: M.Nolden at dkfz.de

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/ctk-developers/attachments/20100409/62146306/attachment.htm>