From 185d67664ff4e6cd87a4190ce233bd0cf1a40f98 Mon Sep 17 00:00:00 2001 From: Maksim Denisov Date: Thu, 20 Feb 2025 12:44:37 +0100 Subject: ByteStreamUtils: Simplify ReadRequest and WriteRequest --- .../execution_api/remote/bazel/bytestream_client.hpp | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) (limited to 'src/buildtool/execution_api/remote/bazel') diff --git a/src/buildtool/execution_api/remote/bazel/bytestream_client.hpp b/src/buildtool/execution_api/remote/bazel/bytestream_client.hpp index e4f8f465..213e4793 100644 --- a/src/buildtool/execution_api/remote/bazel/bytestream_client.hpp +++ b/src/buildtool/execution_api/remote/bazel/bytestream_client.hpp @@ -83,10 +83,8 @@ class ByteStreamClient { Logger const* logger) : logger_{logger} { google::bytestream::ReadRequest request{}; - - auto resource_name = - ByteStreamUtils::ReadRequest{instance_name, digest}.ToString(); - request.set_resource_name(std::move(resource_name)); + request.set_resource_name( + ByteStreamUtils::ReadRequest::ToString(instance_name, digest)); reader_ = stub->Read(&ctx_, request); } }; @@ -138,9 +136,8 @@ class ByteStreamClient { google::bytestream::WriteResponse response{}; auto writer = stub_->Write(&ctx, &response); - auto const resource_name = - ByteStreamUtils::WriteRequest{instance_name, uuid, blob.digest} - .ToString(); + auto const resource_name = ByteStreamUtils::WriteRequest::ToString( + instance_name, uuid, blob.digest); google::bytestream::WriteRequest request{}; request.set_resource_name(resource_name); -- cgit v1.2.3