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: use std::move() for better performance #16620

Merged
merged 1 commit into from Aug 1, 2017

Conversation

Projects
None yet
3 participants
@cfanz
Contributor

cfanz commented Jul 27, 2017

use std::move() to avoid extra copy constructions in src/common/OutputDataSocket.cc.

See the discussions in
#16525
#16555

Signed-off-by: Xinying Song songxinying@cloudin.cn

Xinying Song
common: use std::move() for better performance
use std::move() to avoid extra copy constructions in src/common/OutputDataSocket.cc.

Signed-off-by: Xinying Song <songxinying@cloudin.cn>
@joscollin

LGTM

@tchaikov tchaikov merged commit a8c2b10 into ceph:master Aug 1, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment