[Insight-users] Possible bug in Modules/IO/TransformBase/src/itkTransformFactoryBase.cxx

[David Doria]

On Thu, Oct 11, 2012 at 1:03 PM, Tim Allman <timothy at brasscats.ca> wrote:
> I have been looking at the code (v 4.2.1) and have discovered what appears
> to be a bug in
>
> Modules/IO/TransformBase/src/itkTransformFactoryBase.cxx
>
> line 107
> Modules/IO/TransformBase/src/itkTransformFactoryBase.cxx
>
>     // We cannot register both Rigid3DTransform and
>     // itkv3::Rigid3DTransform because they both have the same name
> #ifdef ITKV3_COMPATIBILITY
>     TransformFactory< Rigid3DTransform< double > >::RegisterTransform ();
> #else
>     TransformFactory< itkv3::Rigid3DTransform< double > >::RegisterTransform
> ();
> #endif
>
> There is a similar construct at line 166.
>
> It appears that the v4 version is selected when the v3 version should be and
> vice versa.

It looks backwards to me as well. Could you push a patch to Gerrit?
(Instructions here:
http://www.itk.org/Wiki/ITK/Git/Develop#Create_a_Topic and
Share_a_Topic)

David