Hi Sascha, <div><br></div><div>It seems like a good idea to "publish" the dah branch ! </div><div>I've a little request though (as you asked for it :-) ). Last time I checked, there was not many unit tests (2?) for all the DICOM classes. I think that DAH would greatly benefit from some more testing. I believe it's a good timing to add tests (if you want to merge it means the design is quite stable meaning tests can be written :-) ).</div>

<div><br></div><div>My 2 cents,</div><div>Julien.<br><br><div class="gmail_quote">On Fri, Oct 22, 2010 at 9:27 AM, Zelzer Sascha <span dir="ltr"><<a href="mailto:s.zelzer@dkfz-heidelberg.de">s.zelzer@dkfz-heidelberg.de</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div lang="DE" link="blue" vlink="purple"><div><p class="MsoNormal"><span lang="EN-US">Hi Folks,</span></p><p class="MsoNormal">

<span lang="EN-US"> </span></p><p class="MsoNormal"><span lang="EN-US">CTK has seen many improvements since the Barcelona hackfest and I would like to merge the dah branch “dicom application hosting” into master to take advantage of the new features. The advantages would be</span></p>

<p class="MsoNormal"><span lang="EN-US"> </span></p><p><span lang="EN-US"><span>-<span style="font:7.0pt "Times New Roman"">          </span></span></span><span lang="EN-US">The dah code would benefit from future refactorings taking place in the master branch</span></p>

<p><span lang="EN-US"><span>-<span style="font:7.0pt "Times New Roman"">          </span></span></span><span lang="EN-US">Make use of proper include path inheritance between plugins and libraries</span></p><p><span lang="EN-US"><span>-<span style="font:7.0pt "Times New Roman"">          </span></span></span><span lang="EN-US">Take advantage of improvements inside ctkPluginFramework</span></p>

<p><span lang="EN-US"><span>-<span style="font:7.0pt "Times New Roman"">          </span></span></span><span lang="EN-US">Improve the visibility of the dah plugins to the community</span></p><p class="MsoNormal">

<span lang="EN-US"> </span></p><p class="MsoNormal"><span lang="EN-US">After the merge, developers could still hack on new features in their own branches inside their CTK forks.</span></p><p class="MsoNormal"><span lang="EN-US"> </span></p>

<p class="MsoNormal"><span lang="EN-US">I would take care of adapting the dah code and cleaning up some memory management issues.</span></p><p class="MsoNormal"><span lang="EN-US"> </span></p><p class="MsoNormal"><span lang="EN-US">Do you have any concerns / remarks?</span></p>

<p class="MsoNormal"><span lang="EN-US"> </span></p><p class="MsoNormal"><span lang="EN-US">Thanks,</span></p><p class="MsoNormal"><span lang="EN-US"> </span></p><font color="#888888"><p class="MsoNormal"><span lang="EN-US">Sascha</span></p>

</font></div></div><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>
<br></blockquote></div><br></div>