Skip to content

Conversation

@hjmjohnson
Copy link
Member

No description provided.

…acro

Added two new macro's, intended to replace the old 'itkTypeMacro' and
'itkTypeMacroNoParent'.

The main aim is to be clearer about what those macro's do: add a virtual
'GetNameOfClass()' member function and override it. Unlike 'itkTypeMacro',
'itkOverrideGetNameOfClassMacro' does not have a 'superclass' parameter, as it
was not used anyway.

Note that originally 'itkTypeMacro' did not use its 'superclass' parameter
either, looking at commit 699b66cb04d410e555656828e8892107add38ccb, Will
Schroeder, June 27, 2001:
https://github.com/InsightSoftwareConsortium/ITK/blob/699b66cb04d410e555656828e8892107add38ccb/Code/Common/itkMacro.h#L331-L337
…mation()

ITKv5_CONST enables backwards compatible behavior when ITKV4_COMPATIBILITY
is turned ON for methods which have acquired 'const' qualifier in ITKv5.
Breaking changes were originally introduced by ITK  commits
3e6b6f5bd6772316aa0fa6b89f31b42bef562112 (on 2018-10-18) and
16eae15c1bb6cc1bae9fba3e09a3102bdc02e955 (on 2018-10-23).
In preparation for release.

Ensure that cmake is 3.16.3 or greater to
match v5.4.2 minimum requirements.

Set minimum python to 3.9

Increment version number.

Replace keyword of "ITK" with lowercase "itk"
in some instances to be consistent.
Set the default build package tags to v5.4.3 for
capturing the ITKRemoteModuleBuildTestPackageAction
shared scripts.

This pulls the default configuration items needed to
build against ITK version v5.4.3.
itkShrinkDecimateImageFilter.hxx:184:42: warning: ignoring return value
of ‘bool
itk::ImageBase<VImageDimension>::TransformPhysicalPointToIndex(const
  itk::Point<TCoordinate, VImageDimension>&, IndexType&) const [with
TCoordinate = double; unsigned int VImageDimension = 3; IndexType =
itk::Index<3>]’, declared with attribute ‘nodiscard’ [-Wunused-result]
@hjmjohnson hjmjohnson closed this May 28, 2025
@hjmjohnson hjmjohnson deleted the update-clang-format branch May 28, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants