Skip to content

STYLE: Missed conversions for new macro name#2059

Merged
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:masterfrom
hjmjohnson:style-fix-macro-name-conversion
Oct 14, 2020
Merged

STYLE: Missed conversions for new macro name#2059
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:masterfrom
hjmjohnson:style-fix-macro-name-conversion

Conversation

@hjmjohnson
Copy link
Copy Markdown
Member

Call ITK_DISALLOW_COPY_AND_MOVE, not ITK_DISALLOW_COPY_AND_ASSIGN
Some conversions were missed during initial update of macro name.

Comment thread Utilities/ITKv5Preparation/Move_DISALLOW_COPY_to_public_section.cpp Outdated
Comment thread Utilities/Doxygen/doxygen.config.in
@dzenanz dzenanz requested a review from N-Dekker October 14, 2020 14:32
Call ITK_DISALLOW_COPY_AND_MOVE, not ITK_DISALLOW_COPY_AND_ASSIGN
Some conversions were missed during initial update of macro name.
@hjmjohnson hjmjohnson force-pushed the style-fix-macro-name-conversion branch from b872699 to 9c3cc79 Compare October 14, 2020 14:36
@jhlegarreta jhlegarreta self-requested a review October 14, 2020 15:11
Copy link
Copy Markdown
Contributor

@N-Dekker N-Dekker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Hans @hjmjohnson ! Sorry for the extra work!

@hjmjohnson
Copy link
Copy Markdown
Member Author

Python failure is unrelated.

@hjmjohnson hjmjohnson merged commit bf30fc0 into InsightSoftwareConsortium:master Oct 14, 2020
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.

4 participants