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

COMMON/DIRECTOR: Move semantics for Arrays and some DIRECTOR structs #3480

Closed
wants to merge 4 commits into from

Conversation

@henke37
Copy link
Contributor

@henke37 henke37 commented Nov 1, 2021

To celebrate the support for c++11 everywhere, I added move semantics to the director::InfoEntry struct.

Turns out that didn't work with Common::Array as is, so I had to add support for move semantics. While at it I figured that I might as well make use of move semantics where possible without changing semantics to speed things up.

henke37 added 4 commits Oct 31, 2021
It's what was desired all along, so away with the copy semantics.
Another pointless copy operation sidestepped by the power of move semantics.
@sev-
Copy link
Member

@sev- sev- commented Nov 1, 2021

We moved to C++11 but not to STL.

Loading

@sev- sev- closed this Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants