<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
Dear Julien,<br>
<br>
I will have a look at ctkDICOMIndexer.cxx before the next hackfest.<br>
<br>
Marco<br>
<br>
<br>
Am 08.04.2010 20:10, schrieb Julien Finet:
<blockquote
cite="mid:x2q5b9fa8231004081110t916e7c0dia34dfd823a6bbe1b@mail.gmail.com"
type="cite">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 moz-do-not-send="true"
href="mailto:luis.ibanez@kitware.com">luis.ibanez@kitware.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote"
style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">The
CTK Dasboard is in need of some Though Love.<br>
<br>
<br>
A) Code Coverage: 42%<br>
<a moz-do-not-send="true"
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 moz-do-not-send="true"
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 moz-do-not-send="true" href="mailto:Ctk-developers@commontk.org">Ctk-developers@commontk.org</a><br>
<a moz-do-not-send="true"
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>
</blockquote>
<br>
<br>
<pre class="moz-signature" cols="72">--
----------------------------------------------------------------------
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: <a class="moz-txt-link-abbreviated" href="mailto:M.Nolden@dkfz.de">M.Nolden@dkfz.de</a>
</pre>
</body>
</html>