Repository: mesos Updated Branches: refs/heads/master 2903c0f0e -> 61fe4732d
Fixed 'CombinedAuthenticator' to avoid an extra copy. This patch adds a missing call to `std::move` in the constructor for `CombinedAuthenticatorProcess` to avoid an unnecessary copy. Review: https://reviews.apache.org/r/58173/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/6fc22d1c Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/6fc22d1c Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/6fc22d1c Branch: refs/heads/master Commit: 6fc22d1cc8d0e4bca263fe4df7ea09274de02f01 Parents: 2903c0f Author: Greg Mann <[email protected]> Authored: Tue Apr 4 10:40:21 2017 -0700 Committer: Anand Mazumdar <[email protected]> Committed: Tue Apr 4 10:41:53 2017 -0700 ---------------------------------------------------------------------- src/authentication/http/combined_authenticator.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/6fc22d1c/src/authentication/http/combined_authenticator.cpp ---------------------------------------------------------------------- diff --git a/src/authentication/http/combined_authenticator.cpp b/src/authentication/http/combined_authenticator.cpp index c734e76..2ab6081 100644 --- a/src/authentication/http/combined_authenticator.cpp +++ b/src/authentication/http/combined_authenticator.cpp @@ -100,7 +100,7 @@ CombinedAuthenticatorProcess::CombinedAuthenticatorProcess( const string& _realm, vector<Owned<Authenticator>>&& _authenticators) : ProcessBase(process::ID::generate("__combined_authenticator__")), - authenticators(_authenticators), + authenticators(std::move(_authenticators)), realm(_realm) {}
