[Insight-developers] ITKv4 Module Names

Gaëtan Lehmann gaetan.lehmann at jouy.inra.fr
Thu Feb 10 14:53:17 EST 2011


Le 10 févr. 11 à 19:32, Brad King a écrit :

> On 02/10/2011 12:21 PM, Gaëtan Lehmann wrote:
>> IO-Base and IO-Raw look a bit strange, aren't they? Shouldn't they be
>> placed in the IO group?
>
> Perhaps.  This grouping is based on what's in the Modularization/ 
> Manifest.txt
> file in the main ITK tree.
>
>> I don't like much the 2 levels of directories, as it is sometime
>> redundant with the module name. I also guess that's because of them
>> that we have the strange IO-Raw and IO-Base?
> [snip]

The problem there is that the first layer of directory doesn't contain  
exclusive things.

In that case, it is possible to be in the core of ITK and be related  
to IO.

So maybe it would be better to keep the subdirectories for the really  
easy to group modules, like the IO modules.

>> Several IO modules don't have at least their first letter in
>> uppercase, like ITK-IO-jpeg. I suppose their name should match the
>> name of the class in ITK.
>
> See my response to Bill L. a few minutes ago.

ok

>
>> The glob would then be
>>  <top>/*/*/*itk-module.cmake
>>  <top>/*/*/*/*-itk-module.cmake
>
> I'll see if the logic can work that way, but I'm a little concerned
> about what happens if we find
>
>  foo-itk-module.cmake
>  bar-itk-module.cmake
>
> in one directory.
>

Yes you're right. It would make the logic more complex for a very  
small benefit.
Having it not called with the usual CMakeLists.txt is nice already.

Gaëtan

-- 
Gaëtan Lehmann
Biologie du Développement et de la Reproduction
INRA de Jouy-en-Josas (France)
tel: +33 1 34 65 29 66    fax: 01 34 65 29 09
http://voxel.jouy.inra.fr  http://www.itk.org
http://www.mandriva.org  http://www.bepo.fr

-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 203 bytes
Desc: Ceci est une signature ?lectronique PGP
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110210/5bb4b0e0/attachment.pgp>


More information about the Insight-developers mailing list