[Insight-developers] [ITK + Wrapping] module name

[Brad King]

Benoit Regrain wrote:
> ----- Original Message ----- From: "Brad King" <brad.king at kitware.com>
> To: "Benoit Regrain" <benoit.regrain at creatis.insa-lyon.fr>
> Cc: <insight-developers at itk.org>
> Sent: Friday, June 17, 2005 5:51 PM
> Subject: Re: [Insight-developers] [ITK + Wrapping] module name
> 
> 
>> Benoit Regrain wrote:
>>
>>> I'm observing the ITK wrapping. All wrapped classes are organized by 
>>> groups.
>>> But I don't understand the rules to choose the group name.
>>>
>>> In ITK/Wrapping/CSwig/Common we can find files :
>>> 1- wrap_itkTransform.cxx : group name is itkTransform
>>> 1- wrap_itkVersorTransform.cxx : group name is itkVersorTransformGroup
>>
>> The second one should not have "Group" on the end.  That is a mistake.
> 
> Ok
> 
> 
>>> 2- wrap_itkScaleTransform.cxx : group name begins with itk
>>
>> There is an "itk::ScaleTransform" class to which this corresponds.
>>
>>> 2- wrap_ITKRigidTransforms.cxx : group name begins with ITK
>>
>> There is no "itk::RigidTransforms" class.  The file is meant for 
>> several classes, so the group gets an ITK prefix.
> 
> 
> But in the wrap_itkScaleTransform.cxx file, we wrap the itkScaleTransform
> class and to the itkScaleLogarithmicTransform. couldn't it be consider 
> as a group (2 classes in a same file...) ?

This is also a mistake.  The file should be split into two or renamed to 
wrap_ITKScaleTransforms.cxx.

-Brad