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.
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
Remove std::
prefix from uint8_t in MathematicalMorphologyEnums, replace <cstdint>
with "itkIntTypes.h"
#4387
Remove std::
prefix from uint8_t in MathematicalMorphologyEnums, replace <cstdint>
with "itkIntTypes.h"
#4387
Conversation
Follow-up to pull request InsightSoftwareConsortium#3250 commit c173dfd "STYLE: Remove `std::` prefix from types that work without it" Lee Newberg, March 6, 2022.
Should we make similar include file changes in?:
|
"itkIntTypes.h" adds aliases of the types from `<cstdint>` to the `itk` namespace, allowing those types to be used without `std::` prefix. Suggested by Lee Newberg.
Thanks @Leengit, I just did these three, with the second commit!
Didn't do itkIntTypes.h, of course, to avoid an Didn't do itkMemoryUsageObserver.cxx, because I don't really know why it does itkMathematicalMorphologyEnums.h done by the first commit, of course 😄 |
std::
prefix from uint8_t in MathematicalMorphologyEnumsstd::
prefix from uint8_t in MathematicalMorphologyEnums, replace <cstdint>
with "itkIntTypes.h"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thank you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔢
Follow-up to pull request #3250 commit c173dfd "STYLE: Remove
std::
prefix from types that work without it" by Lee Newberg (@Leengit), March 6, 2022.Mentioned at https://discourse.itk.org/t/itkmathematicalmorphologyenums-h-compile-error-with-gcc-13-on-linux/6377/4