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

replace shallowCopy with move #217

Merged
merged 1 commit into from
Jul 24, 2022
Merged

replace shallowCopy with move #217

merged 1 commit into from
Jul 24, 2022

Conversation

ringabout
Copy link
Contributor

ref nim-lang/Nim#20070

Imo, shallowCopy only works for string/seq and is about to be removed for ARC/ORC. Inn this case, move is better.

@c-blake c-blake merged commit 839286c into c-blake:master Jul 24, 2022
@c-blake
Copy link
Owner

c-blake commented Jul 24, 2022

Ok. Thanks!

@ringabout ringabout deleted the patch-1 branch July 24, 2022 12:49
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.

2 participants