diff options
author | Oliver Reiche <oliver.reiche@huawei.com> | 2024-12-16 13:34:26 +0100 |
---|---|---|
committer | Oliver Reiche <oliver.reiche@huawei.com> | 2024-12-19 16:17:46 +0100 |
commit | 0042c50301d2e959e9e7487f0f4ab37cef245979 (patch) | |
tree | 5b3b274e001b7add63663dcbafd1e29fd53448a0 /src/buildtool/execution_api/execution_service/execution_server.cpp | |
parent | ae42d0cf824f6dd08eee400bf6c4ad01ecd4e567 (diff) | |
download | justbuild-0042c50301d2e959e9e7487f0f4ab37cef245979.tar.gz |
Remove unnecessary moves
Diffstat (limited to 'src/buildtool/execution_api/execution_service/execution_server.cpp')
-rw-r--r-- | src/buildtool/execution_api/execution_service/execution_server.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/buildtool/execution_api/execution_service/execution_server.cpp b/src/buildtool/execution_api/execution_service/execution_server.cpp index d199989a..6e57fdb1 100644 --- a/src/buildtool/execution_api/execution_service/execution_server.cpp +++ b/src/buildtool/execution_api/execution_service/execution_server.cpp @@ -104,7 +104,7 @@ auto ExecutionServiceImpl::ToIExecutionAction( execution_action->SetCacheFlag( action.do_not_cache() ? IExecutionAction::CacheFlag::DoNotCacheOutput : IExecutionAction::CacheFlag::CacheOutput); - return std::move(execution_action); + return execution_action; } auto ExecutionServiceImpl::ToBazelExecuteResponse( @@ -160,7 +160,7 @@ auto ExecutionServiceImpl::ToBazelExecuteResponse( // we run the action locally, so no communication issues should happen bazel_response.mutable_status()->set_code(grpc::StatusCode::OK); - return std::move(bazel_response); + return bazel_response; } void ExecutionServiceImpl::WriteResponse( @@ -332,7 +332,7 @@ namespace { (*out_dir.mutable_tree_digest()) = ArtifactDigestFactory::ToBazel(*cas_digest); } - return std::move(out_dir); + return out_dir; } [[nodiscard]] auto ToBazelOutputSymlink(std::string path, @@ -363,7 +363,7 @@ namespace { } *(out_link.mutable_target()) = *std::move(content); - return std::move(out_link); + return out_link; } [[nodiscard]] auto ToBazelOutputFile(std::string path, @@ -419,7 +419,7 @@ namespace { result_files.Add(ToBazelOutputFile(path, info)); } } - return std::move(result); + return result; } [[nodiscard]] auto ToBazelAction(ArtifactDigest const& action_digest, @@ -454,7 +454,7 @@ namespace { fmt::format("could not retrieve input root {} from cas", input_root_digest->hash())}; } - return std::move(action); + return action; } [[nodiscard]] auto ToBazelCommand(bazel_re::Action const& action, @@ -477,6 +477,6 @@ namespace { return unexpected{fmt::format("Failed to parse command from blob {}", command_digest->hash())}; } - return std::move(c); + return c; } } // namespace |