Agree, let's test it all ! :-)<div><br></div><div>I already have a test ready (in Slicer) for ctkModelTester. I like testing testers.<br><br></div><div>I can also look at ctkDICOMIndexer.cxx, but I'd probably need some help from Marco...</div>
<div><br></div><div>Julien.</div><div><br><div class="gmail_quote">On Thu, Apr 8, 2010 at 1:58 PM, Luis Ibanez <span dir="ltr"><<a href="mailto:luis.ibanez@kitware.com">luis.ibanez@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">The CTK Dasboard is in need of some Though Love.<br>
<br>
<br>
A) Code Coverage: 42%<br>
<a href="http://my.cdash.org/viewCoverage.php?buildid=57880" target="_blank">http://my.cdash.org/viewCoverage.php?buildid=57880</a><br>
<br>
<br>
B) 608 Valgring Errors<br>
<a href="http://my.cdash.org/viewDynamicAnalysis.php?buildid=57892" target="_blank">http://my.cdash.org/viewDynamicAnalysis.php?buildid=57892</a><br>
<br>
<br>
We must fix this before the Hackaton.<br>
<br>
<br>
It will be a disaster to have a lot of new code to be put<br>
inside a repository with a weak testing infrastructure.<br>
<br>
<br>
Any volunteer for adopting the orphan classes ?<br>
<br>
./Libs/Core/ctkDependencyGraph.h<br>
./Libs/DICOM/Core/ctkDICOMIndexer.cxx<br>
./Libs/Core/ctkDependencyGraph.cxx<br>
./Libs/Core/ctkModelTester.cxx<br>
<br>
<br>
<br>
Many of the memory leaks seems to come from :<br>
<br>
==9718== at 0x4C25153: malloc (vg_replace_malloc.c:195)<br>
==9718== by 0x75331A1: strdup (strdup.c:43)<br>
==9718== by 0x89F0C49: IceRegisterForProtocolSetup (in<br>
/usr/lib/libICE.so.6.3.0)<br>
<br>
<br>
Any reason for using the old strdup instead<br>
of using "std::string" ?<br>
<br>
<br>
Finally,<br>
<br>
There are no Windows nor Mac builds in the Dashboard...<br>
<br>
Any volunteers for setting up additional builds ?<br>
<br>
<br>
<br>
Luis<br>
_______________________________________________<br>
Ctk-developers mailing list<br>
<a href="mailto:Ctk-developers@commontk.org">Ctk-developers@commontk.org</a><br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers</a><br>
</blockquote></div><br></div>