[Insight-developers] TransformFileReaderWithFactory inconsistent interface

[Vincent Magnotta]

Hans,

I would suggest the later since it makes the signature a little more clear.
Most of the C++ references that I reviewed, most use this type of signature.

Vince



On 5/10/09 10:26 AM, "Hans Johnson" <hans-johnson at uiowa.edu> wrote:

> Vince,
> 
> There has been some discusssion of moving the TransformIO from the Review
> directory.  I am fully in favor of this.  I did, however, notice an
> inconsistency in the member function signatures that would be nice to clear up
> before the transition occurs (and we are locked in to backwards
> compatibility).
> 
> itkTransformFileReaderWithFactory.h:  TransformListType * GetTransformList()
> {return & m_TransformList;}
> itkTransformIOBase.h:  TransformListType &GetTransformList() { return
> m_ReadTransformList; }
> 
> Could you please give some advice on which signature should be used?
> 
> Thanks,
> Hans
> 

----------------------
Associate Professor
Department of Radiology
0453-D JCP
200 Hawkins Drive
Iowa City, IA 52242
E-mail: vincent-magnotta at uiowa.edu
Phone: 319-356-8255 Fax: 319-353-6275
Website: http://www.radiology.uiowa.edu



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20090510/65fac5d7/attachment.htm>