summaryrefslogtreecommitdiff
path: root/src/buildtool/execution_api/execution_service/execution_server.cpp
diff options
context:
space:
mode:
authorOliver Reiche <oliver.reiche@huawei.com>2024-12-16 13:34:26 +0100
committerOliver Reiche <oliver.reiche@huawei.com>2024-12-19 16:17:46 +0100
commit0042c50301d2e959e9e7487f0f4ab37cef245979 (patch)
tree5b3b274e001b7add63663dcbafd1e29fd53448a0 /src/buildtool/execution_api/execution_service/execution_server.cpp
parentae42d0cf824f6dd08eee400bf6c4ad01ecd4e567 (diff)
downloadjustbuild-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.cpp14
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