From 845744929e40dbdc81ed9c7df0152d58bbb28be6 Mon Sep 17 00:00:00 2001 From: Maksim Denisov Date: Thu, 26 Sep 2024 14:03:28 +0200 Subject: Fix automatic moves proposed by clang-tidy. Enable performance-no-automatic-move check. --- src/buildtool/execution_api/bazel_msg/bazel_msg_factory.cpp | 2 +- src/buildtool/execution_api/remote/bazel/bazel_network_reader.cpp | 2 +- src/buildtool/execution_api/remote/bazel/bazel_response.cpp | 2 +- src/buildtool/storage/large_object_cas.tpp | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/buildtool/execution_api/bazel_msg/bazel_msg_factory.cpp b/src/buildtool/execution_api/bazel_msg/bazel_msg_factory.cpp index 1992961d..58059156 100644 --- a/src/buildtool/execution_api/bazel_msg/bazel_msg_factory.cpp +++ b/src/buildtool/execution_api/bazel_msg/bazel_msg_factory.cpp @@ -305,7 +305,7 @@ auto BazelMsgFactory::CreateDirectoryDigestFromTree( return std::nullopt; } - auto const digest = bundle->blob.digest; + auto digest = bundle->blob.digest; try { if (not process_blob(std::move(bundle->blob))) { return std::nullopt; diff --git a/src/buildtool/execution_api/remote/bazel/bazel_network_reader.cpp b/src/buildtool/execution_api/remote/bazel/bazel_network_reader.cpp index d5f2f3f8..a31939b4 100644 --- a/src/buildtool/execution_api/remote/bazel/bazel_network_reader.cpp +++ b/src/buildtool/execution_api/remote/bazel/bazel_network_reader.cpp @@ -238,7 +238,7 @@ auto BazelNetworkReader::Validate(BazelBlob const& blob) const noexcept } // rehash data - auto const rehashed_info = HashInfo::HashData( + auto rehashed_info = HashInfo::HashData( hash_function_, *blob.data, requested_hash_info->IsTree()); // ensure rehashed data produce the same hash diff --git a/src/buildtool/execution_api/remote/bazel/bazel_response.cpp b/src/buildtool/execution_api/remote/bazel/bazel_response.cpp index 7771fa8e..638747cb 100644 --- a/src/buildtool/execution_api/remote/bazel/bazel_response.cpp +++ b/src/buildtool/execution_api/remote/bazel/bazel_response.cpp @@ -237,7 +237,7 @@ auto BazelResponse::Populate() noexcept -> std::optional { // have to upload them manually. auto root_digest = UploadTreeMessageDirectories(*tree); if (not root_digest) { - auto const error = + auto error = fmt::format("BazelResponse: {}", root_digest.error()); Logger::Log(LogLevel::Trace, error); return error; diff --git a/src/buildtool/storage/large_object_cas.tpp b/src/buildtool/storage/large_object_cas.tpp index e77f6355..4d2a1a6c 100644 --- a/src/buildtool/storage/large_object_cas.tpp +++ b/src/buildtool/storage/large_object_cas.tpp @@ -39,7 +39,7 @@ template auto LargeObjectCAS::GetEntryPath( ArtifactDigest const& digest) const noexcept -> std::optional { - const std::filesystem::path file_path = file_store_.GetPath(digest.hash()); + std::filesystem::path file_path = file_store_.GetPath(digest.hash()); if (FileSystemManager::IsFile(file_path)) { return file_path; } -- cgit v1.2.3