Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Missing include file in vtkImageLabelDilate3D.cxx #45

Closed
southie opened this issue Oct 26, 2023 · 1 comment · Fixed by #46
Closed

Missing include file in vtkImageLabelDilate3D.cxx #45

southie opened this issue Oct 26, 2023 · 1 comment · Fixed by #46

Comments

@southie
Copy link

southie commented Oct 26, 2023 •
edited
Loading

vtkImageLabelDilate3D.cxx added in #44 introduces a compilation error on Microsoft Visual Studio Professional 2022 (64-bit) - Current
Version 17.6.4 ; MSVC version 14.36.32532
Compilation fails with with error:
vtkAddon\vtkImageLabelDilate3D.cxx(67): error C2039: 'map': is not a member of 'std'

vtkComputeModeOfArray references std::map<T, int> but does not include the header which results in the compilation error.
Reverting to the previous commit compiles as expected. Adding #include <map> to the includes also allows compilation to continue
@jcfr
Copy link
Member

jcfr commented Oct 26, 2023

Thanks for the report. We will integrate a patch later in the day.

In the meantime, do not hesitate to propose a pull request.

@jcfr jcfr linked a pull request Oct 26, 2023 that will close this issue
COMP: Update vtkImageLabelDilate3D adding includes for used STL containers #46
Merged
@jcfr jcfr closed this as completed in #46 Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

COMP: Update vtkImageLabelDilate3D adding includes for used STL containers jcfr/vtkAddon
2 participants
@jcfr @southie