[Ctk-developers] ctkUtils::rmdir - Delete recursively a directory

[Sascha Zelzer]

Even better!

+1 [if you add an 'r' ;-) ]

Thanks,
Sascha

On 09/01/2011 06:55 PM, Jean-Christophe Fillion-Robin wrote:
> Hi Sasha,
>
> Since the Qt equivalent was named QDir::rmdir, I adopted the same 
> name. Changing the name wouldn't be a problem.
>
> What about  "removeDirRecusively" instead ?
>
> Jc
>
> On Thu, Sep 1, 2011 at 12:43 PM, Sascha Zelzer 
> <s.zelzer at dkfz-heidelberg.de <mailto:s.zelzer at dkfz-heidelberg.de>> wrote:
>
>     Hi Jc,
>
>     Looks good. I guess the CTKCore dependency of the
>     ctkPluginFramework was just a matter of time, so I am good with it.
>
>     Just a little "style" comment: The "rmdir" function name looks a
>     little out of place when compared with the others. What about
>     "removeDir"? Just my two cents.
>
>     Thanks for pushing code reuse,
>
>     Sascha
>
>
>     On 09/01/2011 06:26 PM, Jean-Christophe Fillion-Robin wrote:
>>     Hi Folks,
>>
>>     I was looking for a function to delete recursively a function and
>>     I found out that such code has present in multiple location:
>>      - ctkPluginFrameworkUtil_p.h
>>     <https://github.com/jcfr/CTK/commit/8714424163b376c4efb693448278a10e13a8690a#diff-4>
>>      - ctkPluginGenerator.cpp
>>     <https://github.com/jcfr/CTK/commit/8714424163b376c4efb693448278a10e13a8690a#diff-0>
>>
>>     Topic "ctkutils-rmdir" has been integrated to master. It includes:
>>       - addition of "rmdir" function to ctkUtils
>>       - implementation of the corresponding test in ctkUtilsTest4.cpp
>>       - See
>>     https://github.com/commontk/CTK/commit/fd8cfead27c56ae607e379d764d2eb790d16af3e
>>
>>     Topic "reuse-ctkutil-rmdir-in-toolkit" has NOT been integrated to
>>     master, indeed it implies that ctkPluginFramework will depend on
>>     CTKCore. Does that sounds reasonable ?
>>       See
>>     https://github.com/jcfr/CTK/tree/reuse-ctkutil-rmdir-in-toolkit
>>     and
>>     https://github.com/jcfr/CTK/commit/8714424163b376c4efb693448278a10e13a8690a
>>
>>     Thanks
>>     Jc
>>
>>
>>
>>     -- 
>>     +1 919 869 8849 <tel:%2B1%20919%20869%208849>
>>
>
>
>     _______________________________________________
>     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
>
>
>
>
> -- 
> +1 919 869 8849
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/ctk-developers/attachments/20110901/48fe3258/attachment.html>