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

Rule of five: implement move operators #2434

Closed
wants to merge 1 commit into from

Conversation

Timple
Copy link
Contributor

@Timple Timple commented Feb 23, 2024

I think @fujitatomoya had a point: #2425 (comment)

So this is the alternative to: #2425

Signed-off-by: Tim Clephas <tim.clephas@nobleo.nl>
@clalancette
Copy link
Contributor

@Timple It is preferred to update the old PR; that way the context of why we made the decision is preferred. In #2425 , you can update both the patch (by force-pushing), and the issue title (the edit button).

So with that in mind, I'm going to close this one.

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.

None yet

2 participants