[Ctk-developers] Merging branch "dah" into master

[Julien Finet]

Hi Sascha,

It seems like a good idea to "publish" the dah branch !
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 :-) ).

My 2 cents,
Julien.

On Fri, Oct 22, 2010 at 9:27 AM, Zelzer Sascha
<s.zelzer at dkfz-heidelberg.de>wrote:

> Hi Folks,
>
>
>
> 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
>
>
>
> -          The dah code would benefit from future refactorings taking
> place in the master branch
>
> -          Make use of proper include path inheritance between plugins and
> libraries
>
> -          Take advantage of improvements inside ctkPluginFramework
>
> -          Improve the visibility of the dah plugins to the community
>
>
>
> After the merge, developers could still hack on new features in their own
> branches inside their CTK forks.
>
>
>
> I would take care of adapting the dah code and cleaning up some memory
> management issues.
>
>
>
> Do you have any concerns / remarks?
>
>
>
> Thanks,
>
>
>
> Sascha
>
> _______________________________________________
> 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/20101022/8031b1c5/attachment.html>