-
-
Notifications
You must be signed in to change notification settings - Fork 307
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
add rvalue reference #5685
add rvalue reference #5685
Conversation
also this
should probably be
It works, but is somewhat unconvential in an non-templated environment: https://stackoverflow.com/questions/28828159/usage-of-stdforward-vs-stdmove. Alternatively we could capture both cases using a template
but doesnt seem very much "OpenMS-like" |
/rebase |
Co-authored-by: Hannes Roest <hannesroest@gmx.ch>
Co-authored-by: Hannes Roest <hannesroest@gmx.ch>
Co-authored-by: Hannes Roest <hannesroest@gmx.ch>
60a1c01
to
4c46577
Compare
tests pass now. |
@@ -197,6 +198,8 @@ namespace OpenMS | |||
result.addSpectrum(std::move(in[k])); | |||
} | |||
} | |||
result.ExperimentalSettings::operator=(in); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could also be std::move(in)
?
Not crucial though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
Description
Based on results from clang-tidy scan .
Checklist:
How can I get additional information on failed tests during CI:
If your PR is failing you can check out
Note: