[Insight-developers] LandmarkBasedTransformInitializaer

[Johnson, Hans J]

ITK Developers,

I know there is concern about scope creep, so I wanted to get some feedback before we spend time working on cleaning this up.

I was reviewing:  itkLandmarkBasedTransformInitializer.h, and found it very odd:
1)  It is templated over images, but never uses images
        2)  There are member variables for m_FixedImage, and m_MovingImage, but they are never used anywhere in the class

I'd propose removing the private  images that are never used, and then templated  only over a single PointType.

Shall we move forward with this effort?

Hans







________________________________
Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged.  If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited.  Please reply to the sender that you have received the message in error, then delete it.  Thank you.
________________________________
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110608/28d3930b/attachment.htm>