Skip to content

[pull] master from InsightSoftwareConsortium:master#3

Open
pull[bot] wants to merge 1607 commits into151706061:masterfrom
InsightSoftwareConsortium:master
Open

[pull] master from InsightSoftwareConsortium:master#3
pull[bot] wants to merge 1607 commits into151706061:masterfrom
InsightSoftwareConsortium:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Mar 13, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

thewtex and others added 27 commits June 19, 2025 17:28
DOC: Fix links to 5.4.4 release artifacts
Primary branch name transition.
Also use __has_warning to guard againist trying to disable warnings that are unknown (ex: in older Clangs).
COMP: suppress Clang -Wduplicate-enum
- removed workaround for Xcode 10
- emit #error for AppleClang less than Xcode 12.4
- removed workaround for broken fail() added in 1fcd007 in 2003. Hopefully the workaround is no longer needed. I'm not sure, but all tests pass on my arm64 macOS 15.5.

Related to issue #5369.
Directly initialized `MapPixelType::CType` to its intended value in the primary
template definition of `MapPixelType`, and removed its specializations.

Removed `IMAGEIOBASE_TYPEMAP`, following C++ Core Guidelines, May 8, 2025,
"Don’t use macros for program text manipulation",
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es30-dont-use-macros-for-program-text-manipulation
Support transition from master to main as the primary branch.

Changes the local and remove branch from master to main. Also works on a
topic branch.
Replaced function calls of the form `instance->MemberFunction(x)` with
`itk::T::MemberFunction(x)`, for the static member functions
`GetPixelTypeAsString` and `GetComponentTypeAsString`.

Removed the non-static version of the string <-> type conversion tests from
itkImageIOBaseTest.

Follow-up to pull request #5391
commit 45e93bb
COMP: removed workarounds for old Apple stuff
Added three static member functions to ImageIOBase:

    IsComponentTypeFloatingPoint(IOComponentEnum)
    IsComponentTypeUnsigned(IOComponentEnum)
    GetNumberOfBitsOfComponentType(IOComponentEnum)

Eases estimating whether a component type is floating point,  whether the type
is unsigned, and estimating its number of bits, respectively.
The github 'ghostworkflow' fails when the subject
line is greater than 78 characters in length.

Add a check to enforce the subject line length
during initial commit when it is easy to fix.

The function 'hooks_config' does not exist, so it
always failed and prevented the script from performing
the intended task.

The testing for correct ITK commit prefix
was migrated form commit-msg to this script.
The commit-msg script is never used.  It was replaced with
kw-commit-msg script that is called for pre-commit.
Replaced function calls of the form `instance->MemberFunction(x)` with
`itk::T::MemberFunction(x)`, for the static member functions
`GetPixelTypeAsString` and `GetComponentTypeAsString`, in Examples.

Follow-up to pull request #5391
commit 45e93bb
Optimize calls to std::string::find() and friends when the
search item passed is a single character string literal.
The character literal overload is more efficient.
Changes for improved readability in DartMeasurement output
Refactored multiple `#define` macros into strongly-typed `enum`
declarations for improved readability, type safety, and maintainability.

modernize-macro-to-enum
Refactored constructors to utilize `std::move` for transferring
ownership of string and vector arguments, improving efficiency and
modernizing the implementation. Included necessary `<utility>` headers.
…-msg-checking

BUG: Fix kw commit msg checking
ImageIO: Add member functions to get pixel component type traits, remove MapPixelType specializations
@thewtex thewtex deleted the branch 151706061:master June 23, 2025 18:55
@thewtex thewtex deleted the master branch June 23, 2025 18:55
@N-Dekker N-Dekker deleted the branch 151706061:master January 6, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.