Implement move constructor and move assignment operator.
The default (implicit) methods created by the compiler will be incorrect, since this class allocates dynamically. Thus, custom implementations were added.
See also !596 (merged). That the code does not crash is only due to the fact that move operation were not applied on this class.
Merge request reports
Activity
added Framework label
added 19 commits
-
2d3f2a09...bd4b2aef - 18 commits from branch
computing:master
- 449cdc9b - Prevent automatic move constructor and move assignment operator.
-
2d3f2a09...bd4b2aef - 18 commits from branch
fine from my side. Please fell free to merge if you are also fine with the change.
- Resolved by Florian Uhlig
Seems that move IS needed at least compiler-wise...some weird c++17 stuff...
Dear @v.friese,
you have been identified as code owner of at least one file which was changed with this merge request.
Please check the changes and approve them or request changes.
added CodeOwners label
added 6 commits
-
a178a77e...aa16daad - 4 commits from branch
computing:master
- f8f2c129 - Prevent automatic move constructor and move assignment operator.
- 7fb45388 - Replace automatic move constructor and move assignment operator by custom ones.
-
a178a77e...aa16daad - 4 commits from branch
enabled an automatic merge when the pipeline for 75b8ec4b succeeds