diff options
author | Maksim Denisov <denisov.maksim@huawei.com> | 2024-08-29 17:31:02 +0200 |
---|---|---|
committer | Maksim Denisov <denisov.maksim@huawei.com> | 2024-08-30 17:17:09 +0200 |
commit | c01123bd19be5398f20730d2872ad337e5b5d73a (patch) | |
tree | ba4cd2d962a208ecd056650737c86a52a7ea838e /src/buildtool/execution_api/execution_service/execution_server.cpp | |
parent | 013d83eed166049be110d47f00feb4c6af1315af (diff) | |
download | justbuild-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.cpp | 16 |
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; } |