Skip to content

BUG: Fix GDCM posix_memalign undefined with mingw#5777

Merged
blowekamp merged 1 commit intoInsightSoftwareConsortium:mainfrom
blowekamp:gdcm_mingw_aligned
Feb 11, 2026
Merged

BUG: Fix GDCM posix_memalign undefined with mingw#5777
blowekamp merged 1 commit intoInsightSoftwareConsortium:mainfrom
blowekamp:gdcm_mingw_aligned

Conversation

@blowekamp
Copy link
Member

Address compilation error, by switching to use _aligned_malloc.

PR Checklist

  • No API changes were made (or the changes have been approved)
  • No major design changes were made (or the changes have been approved)
  • Added test (or behavior not changed)
  • Updated API documentation (or API not changed)
  • Added license to new files (if any)
  • Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
  • Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

Address compilation error, by switching to use _aligned_malloc.
@github-actions github-actions bot added type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots area:ThirdParty Issues affecting the ThirdParty module labels Feb 6, 2026
@blowekamp
Copy link
Member Author

PR made upstream: malaterre/GDCM#206

@blowekamp
Copy link
Member Author

@malaterre I have created another PR for GDCM from ITK work.

@blowekamp blowekamp marked this pull request as ready for review February 9, 2026 21:35
@blowekamp
Copy link
Member Author

\azp ITK.Windows

@blowekamp blowekamp merged commit 8366558 into InsightSoftwareConsortium:main Feb 11, 2026
15 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ThirdParty Issues affecting the ThirdParty module type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants