summaryrefslogtreecommitdiff
path: root/src/buildtool/execution_api/execution_service/execution_server.cpp
diff options
context:
space:
mode:
authorMaksim Denisov <denisov.maksim@huawei.com>2024-08-29 17:31:02 +0200
committerMaksim Denisov <denisov.maksim@huawei.com>2024-08-30 17:17:09 +0200
commitc01123bd19be5398f20730d2872ad337e5b5d73a (patch)
treeba4cd2d962a208ecd056650737c86a52a7ea838e /src/buildtool/execution_api/execution_service/execution_server.cpp
parent013d83eed166049be110d47f00feb4c6af1315af (diff)
downloadjustbuild-c01123bd19be5398f20730d2872ad337e5b5d73a.tar.gz
Cast ArtifactDigest to bazel_re::Digest explicitly
...to simplify further refactoring.
Diffstat (limited to 'src/buildtool/execution_api/execution_service/execution_server.cpp')
-rw-r--r--src/buildtool/execution_api/execution_service/execution_server.cpp16
1 files changed, 10 insertions, 6 deletions
diff --git a/src/buildtool/execution_api/execution_service/execution_server.cpp b/src/buildtool/execution_api/execution_service/execution_server.cpp
index c89f06fe..65e63ce9 100644
--- a/src/buildtool/execution_api/execution_service/execution_server.cpp
+++ b/src/buildtool/execution_api/execution_service/execution_server.cpp
@@ -109,7 +109,8 @@ auto ExecutionServiceImpl::ToBazelExecuteResponse(
fmt::format("Could not store stderr of action {}",
i_execution_response->ActionDigest())};
}
- action_result.mutable_stderr_digest()->CopyFrom(*cas_digest);
+ (*action_result.mutable_stderr_digest()) =
+ static_cast<bazel_re::Digest>(*cas_digest);
}
if (i_execution_response->HasStdOut()) {
@@ -121,7 +122,8 @@ auto ExecutionServiceImpl::ToBazelExecuteResponse(
fmt::format("Could not store stdout of action {}",
i_execution_response->ActionDigest())};
}
- action_result.mutable_stdout_digest()->CopyFrom(*cas_digest);
+ (*action_result.mutable_stdout_digest()) =
+ static_cast<bazel_re::Digest>(*cas_digest);
}
::bazel_re::ExecuteResponse bazel_response{};
@@ -282,11 +284,13 @@ namespace {
digest.hash());
return unexpected{std::move(error)};
}
- *(out_dir.mutable_tree_digest()) = *std::move(cas_digest);
+ (*out_dir.mutable_tree_digest()) =
+ static_cast<bazel_re::Digest>(*cas_digest);
}
else {
// In native mode: Set the directory digest directly.
- *(out_dir.mutable_tree_digest()) = digest;
+ (*out_dir.mutable_tree_digest()) =
+ static_cast<bazel_re::Digest>(digest);
}
return std::move(out_dir);
}
@@ -320,8 +324,8 @@ namespace {
Artifact::ObjectInfo const& info) noexcept
-> ::bazel_re::OutputFile {
::bazel_re::OutputFile out_file{};
- *(out_file.mutable_path()) = std::move(path);
- *(out_file.mutable_digest()) = info.digest;
+ (*out_file.mutable_path()) = std::move(path);
+ (*out_file.mutable_digest()) = static_cast<bazel_re::Digest>(info.digest);
out_file.set_is_executable(IsExecutableObject(info.type));
return out_file;
}