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.

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

COMP: Use and move ITK_DISALLOW_COPY_AND_ASSIGN call to public section #48

Merged

Conversation

jhlegarreta
Copy link
Collaborator

Use the ITK_DISALLOW_COPY_AND_ASSIGN macro to enhance consistency across
the the toolkit when disallowing the copy constructor and the assign
operator.

Move the ITK_DISALLOW_COPY_AND_ASSIGN calls to public section following
the discussion in
https://discourse.itk.org/t/itk-disallow-copy-and-assign/648

Use the `ITK_DISALLOW_COPY_AND_ASSIGN` macro to enhance consistency across
the the toolkit when disallowing the copy constructor and the assign
operator.

Move the `ITK_DISALLOW_COPY_AND_ASSIGN` calls to public section following
the discussion in
https://discourse.itk.org/t/itk-disallow-copy-and-assign/648
@dzenanz dzenanz merged commit 32e4ee6 into KitwareMedical:master Feb 20, 2020
@jhlegarreta jhlegarreta deleted the UseDisallowCopyAndAssignMacro branch February 20, 2020 22:00
@hjmjohnson
Copy link
Contributor

@jhlegarreta @dzenanz FYI: I have flagged almost 300 cases where this may still be necessary. I have my undergraduate student starting to work on this next week.

@jhlegarreta
Copy link
Collaborator Author

Sounds good. Thanks Hans.

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.

3 participants