Skip to content

Commit

Permalink
STYLE: CoordRepType -> CoordinateType code readability
Browse files Browse the repository at this point in the history
For the sake of code readability, a new 'CoordinateType' alias is added for
each nested 'CoordRepType' alias. The old 'CoordRepType' aliases will still be
available with ITK 6.0, but it is recommended to use 'CoordinateType' instead.
The 'CoordRepType' aliases will be removed when 'ITK_FUTURE_LEGACY_REMOVE' is
enabled. Similarly, 'InputCoordinateType', 'OutputCoordinateType', and
'ImagePointCoordinateType' replace 'InputCoordRepType', 'OutputCoordRepType',
and 'ImagePointCoordRepType', respectively.
  • Loading branch information
hjmjohnson committed Jan 26, 2025
1 parent 4d35307 commit b81b523
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions include/itkTileMergeImageFilter.h
Original file line number Diff line number Diff line change
Expand Up @@ -49,15 +49,15 @@ template <typename TImageType,
class ITK_TEMPLATE_EXPORT TileMergeImageFilter
: public TileMontage<
Image<typename NumericTraits<typename TImageType::PixelType>::ValueType, TImageType::ImageDimension>,
typename TInterpolator::CoordRepType>
typename TInterpolator::CoordinateType>
{
public:
ITK_DISALLOW_COPY_AND_MOVE(TileMergeImageFilter);

/** We define superclass with scalar pixel type, to enable compiling even when RGB pixel is supplied. */
using Superclass =
TileMontage<Image<typename NumericTraits<typename TImageType::PixelType>::ValueType, TImageType::ImageDimension>,
typename TInterpolator::CoordRepType>;
typename TInterpolator::CoordinateType>;

/** Standard class type aliases. */
using Self = TileMergeImageFilter;
Expand Down
10 changes: 5 additions & 5 deletions include/itkTileMergeImageFilter.hxx
Original file line number Diff line number Diff line change
Expand Up @@ -265,10 +265,10 @@ TileMergeImageFilter<TImageType, TPixelAccumulateType, TInterpolator>::GenerateO
if (m_Montage.IsNull())
{
// initialize mosaic bounds
this->m_MinInner.Fill(NumericTraits<typename TInterpolator::CoordRepType>::NonpositiveMin());
this->m_MinOuter.Fill(NumericTraits<typename TInterpolator::CoordRepType>::max());
this->m_MaxOuter.Fill(NumericTraits<typename TInterpolator::CoordRepType>::NonpositiveMin());
this->m_MaxInner.Fill(NumericTraits<typename TInterpolator::CoordRepType>::max());
this->m_MinInner.Fill(NumericTraits<typename TInterpolator::CoordinateType>::NonpositiveMin());
this->m_MinOuter.Fill(NumericTraits<typename TInterpolator::CoordinateType>::max());
this->m_MaxOuter.Fill(NumericTraits<typename TInterpolator::CoordinateType>::NonpositiveMin());
this->m_MaxInner.Fill(NumericTraits<typename TInterpolator::CoordinateType>::max());

for (SizeValueType i = 0; i < this->m_LinearMontageSize; i++)
{
Expand Down Expand Up @@ -315,7 +315,7 @@ TileMergeImageFilter<TImageType, TPixelAccumulateType, TInterpolator>::GenerateO
iOrigin = inverseT->TransformPoint(iOrigin);

const ContinuousIndexType ci =
outputImage->template TransformPhysicalPointToContinuousIndex<typename TInterpolator::CoordRepType,
outputImage->template TransformPhysicalPointToContinuousIndex<typename TInterpolator::CoordinateType,
typename PointType::ValueType>(iOrigin);
for (unsigned d = 0; d < ImageDimension; d++)
{
Expand Down

0 comments on commit b81b523

Please sign in to comment.