common/str_list.h: fix clang warning about std::move #12570

Merged
merged 1 commit into from Dec 20, 2016

Projects

None yet

3 participants

@wjwithagen
Contributor

/home/jenkins/workspace/ceph-master/src/include/str_list.h:99:10: warning: moving a local object in a return statement prevents copy elision [-Wpessimizing-move]
return std::move(str_vec);
^
/home/jenkins/workspace/ceph-master/src/include/str_list.h:99:10: note: remove std::move call here
return std::move(str_vec);
^~ ~
1 warning generated.

Signed-off-by: Willem Jan Withagen wjw@digiware.nl

@wjwithagen wjwithagen Clang/str_list.h: fix clang warning about std::move
/home/jenkins/workspace/ceph-master/src/include/str_list.h:99:10: warning: moving a local object in a return statement prevents copy elision [-Wpessimizing-move]
  return std::move(str_vec);
         ^
/home/jenkins/workspace/ceph-master/src/include/str_list.h:99:10: note: remove std::move call here
  return std::move(str_vec);
         ^~~~~~~~~~       ~
1 warning generated.

Signed-off-by: Willem Jan Withagen <wjw@digiware.nl>
d9acbf5
@liewegas liewegas changed the title from Clang/str_list.h: fix clang warning about std::move to common/str_list.h: fix clang warning about std::move Dec 19, 2016
@liewegas liewegas added this to the kraken milestone Dec 19, 2016
@liewegas liewegas merged commit 73fa838 into ceph:master Dec 20, 2016

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
@wjwithagen wjwithagen deleted the wjwithagen:patch-9 branch Jan 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment