[Insight-developers] Conflicting itkPointSetToPointSetMetric headers

[Nicholas Tustison]

Sorry about that. 

So the idea would be to ultimately remove the old PointSetToPointSetMetric
and derived classes.  There's only one derived class, the ICP metric, and 
we reimplemented that in a more efficient manner.  However, we don't have
a PointSetToPointSetRegistrationMethod class, yet, to replace the current
one.  

I don't know that I have a preference but do we want to delete the old classes or
should we simply rename the new classes?

Nick

On Oct 7, 2011, at 11:03 AM, Brad King wrote:

> Hi Folks,
> 
> This commit:
> 
>  http://itk.org/gitweb?p=ITK.git;a=commitdiff;h=3076fe37
> 
> added
> 
>  Modules/Registration/HighDimensionalMetrics/include/itkPointSetToPointSetMetric.h
>  Modules/Registration/HighDimensionalMetrics/include/itkPointSetToPointSetMetric.hxx
> 
> but that header name already existed at
> 
>  Modules/Registration/Common/include/itkPointSetToPointSetMetric.h
>  Modules/Registration/Common/include/itkPointSetToPointSetMetric.hxx
> 
> The headers clobber each other in an install tree where they are not
> divided by module.  This was exposed by my "TestExternal" build that
> builds the module tests against an installed ITK:
> 
>  http://www.cdash.org/CDash/viewBuildError.php?buildid=1602106
> 
> -Brad