[Insight-developers] ConvertPointToContinousIndex to ConvertPointToContinuousIndex

[Hans Johnson]

All,

There must be some sort of deprecation process for these sorts of issues.
Backwards compatibility is great, but there needs to be some sort of
mechanism to eventually phase these errors.

I thought that items listed as deprecated only needed to be maintained for 2
two revisions after the change was made.

======
Along similar lines, at some point we need to move beyond the requirement of
compilation under compilers that were readily available in 2000, but are now
unsupported.

Hans
-- 
Hans J. Johnson, Ph.D.
Hans-johnson at uiowa.edu

278 GH
The University of Iowa
Iowa City, IA 52241
(319) 353 8587


> From: Stephen Aylward <Stephen.Aylward at Kitware.com>
> Date: Tue, 15 Apr 2008 14:35:44 -0400
> To: Bill Lorensen <bill.lorensen at gmail.com>
> Cc: "Insight-developers (E-mail)" <insight-developers at itk.org>
> Subject: Re: [Insight-developers] ConvertPointToContinousIndex to
> ConvertPointToContinuousIndex
> 
> I think both spellings should be maintained.
> 
> s
> 
> On Tue, Apr 15, 2008 at 10:17 AM, Bill Lorensen <bill.lorensen at gmail.com>
> wrote:
>> Marius,
>> 
>>  I made that change a couple of months ago. Since I am the author of
>>  the backward compatibility policy, I struggled with keeping the old
>>  spelling around. In the past, I have fixed spelling errors but left
>>  the old spelling as well. For example, in itkArrowSpatialObject, the
>>  method GetLenght() was retained when its name was corrected to
>>  GetLength().
>> 
>>  Perhaps the old name should be retained as well. After all, we did
>>  release the software with the bad method name. It was the development
>>  team's fault, not your fault.
>> 
>>  Bill
>> 
>> 
>> 
>>  On Tue, Apr 15, 2008 at 9:58 AM, Marius Staring <marius at isi.uu.nl> wrote:
>>> Hi,
>>> 
>>> The function
>>> 
>>>     void ConvertPointToContinousIndex( const PointType & point,
>>> ContinuousIndexType & cindex ) const
>>> 
>>> in itkImageFunction.h
>>> 
>>> was renamed in itk 3.6 to ConvertPointToContinuousIndex, which is indeed
>>> proper English. However, this change broke my application. I could quickly
>>> fix it, but this seems to violate the backwards compatibility policy. Or
>>> not?
>>> 
>>> Anyway, I just wanted to let you know.
>>> 
>>> Regards,
>>> 
>>> Marius Staring
>>> 
>>> 
>>> _______________________________________________
>>> Insight-developers mailing list
>>> Insight-developers at itk.org
>>> http://www.itk.org/mailman/listinfo/insight-developers
>>> 
>>> 
>>  _______________________________________________
>>  Insight-developers mailing list
>>  Insight-developers at itk.org
>>  http://www.itk.org/mailman/listinfo/insight-developers
>> 
> 
> 
> 
> -- 
> Stephen R. Aylward, Ph.D.
> Chief Medical Scientist
> Kitware, Inc. - Chapel Hill Office
> http://www.kitware.com
> (518) 371-3971 x300
> _______________________________________________
> Insight-developers mailing list
> Insight-developers at itk.org
> http://www.itk.org/mailman/listinfo/insight-developers