diff options
author | babenko <[email protected]> | 2025-05-28 10:12:40 +0300 |
---|---|---|
committer | babenko <[email protected]> | 2025-05-28 10:48:30 +0300 |
commit | 4b796727b1c97992a1fffcd1550bf226103fca59 (patch) | |
tree | 64ae9110e48e814b921977d88f34a20e20a8c025 | |
parent | 896fcc7f6b80e06dfca1c2564723b020dfea24d3 (diff) |
Fix redundant std::moves in return statements
commit_hash:d2018bced62a6351718a6b30119493e55aa49f4e
-rw-r--r-- | yt/yt/client/api/rpc_proxy/client_base.cpp | 2 | ||||
-rw-r--r-- | yt/yt/core/misc/memory_usage_tracker.cpp | 4 | ||||
-rw-r--r-- | yt/yt/core/misc/unittests/consistent_hashing_ut.cpp | 2 | ||||
-rw-r--r-- | yt/yt/core/rpc/client.cpp | 6 |
4 files changed, 6 insertions, 8 deletions
diff --git a/yt/yt/client/api/rpc_proxy/client_base.cpp b/yt/yt/client/api/rpc_proxy/client_base.cpp index cdcc9b408c9..594987f73f2 100644 --- a/yt/yt/client/api/rpc_proxy/client_base.cpp +++ b/yt/yt/client/api/rpc_proxy/client_base.cpp @@ -821,7 +821,7 @@ TFuture<TDistributedWriteSessionWithCookies> TClientBase::StartDistributedWriteS TDistributedWriteSessionWithCookies sessionWithCookies; sessionWithCookies.Session = ConvertTo<TSignedDistributedWriteSessionPtr>(TYsonString(result->signed_session())), sessionWithCookies.Cookies = std::move(cookies); - return std::move(sessionWithCookies); + return sessionWithCookies; })); } diff --git a/yt/yt/core/misc/memory_usage_tracker.cpp b/yt/yt/core/misc/memory_usage_tracker.cpp index e9d0e855aad..58b18307b90 100644 --- a/yt/yt/core/misc/memory_usage_tracker.cpp +++ b/yt/yt/core/misc/memory_usage_tracker.cpp @@ -163,7 +163,7 @@ TErrorOr<TMemoryUsageTrackerGuard> TMemoryUsageTrackerGuard::TryAcquire( guard.Size_ = size; guard.AcquiredSize_ = size; guard.Granularity_ = granularity; - return std::move(guard); + return guard; } void TMemoryUsageTrackerGuard::Release() @@ -258,7 +258,7 @@ TMemoryUsageTrackerGuard TMemoryUsageTrackerGuard::TransferMemory(i64 size) guard.Size_ = size; guard.AcquiredSize_ = acquiredDelta; guard.Granularity_ = Granularity_; - return std::move(guard); + return guard; } //////////////////////////////////////////////////////////////////////////////// diff --git a/yt/yt/core/misc/unittests/consistent_hashing_ut.cpp b/yt/yt/core/misc/unittests/consistent_hashing_ut.cpp index dfe692f419a..8633819cac6 100644 --- a/yt/yt/core/misc/unittests/consistent_hashing_ut.cpp +++ b/yt/yt/core/misc/unittests/consistent_hashing_ut.cpp @@ -254,7 +254,7 @@ public: std::swap(Items_.back(), Items_[deleteIndex]); Items_.pop_back(); - return std::move(deleteItem); + return deleteItem; } std::vector<TCrpItemWithToken>::iterator begin() diff --git a/yt/yt/core/rpc/client.cpp b/yt/yt/core/rpc/client.cpp index c96eba676c2..1953601581a 100644 --- a/yt/yt/core/rpc/client.cpp +++ b/yt/yt/core/rpc/client.cpp @@ -105,8 +105,7 @@ TSharedRefArray TClientRequest::Serialize() auto headerlessMessage = GetHeaderlessMessage(); if (!retry) { - auto output = CreateRequestMessage(Header_, headerlessMessage); - return std::move(output); + return CreateRequestMessage(Header_, headerlessMessage); } if (StreamingEnabled_) { @@ -116,8 +115,7 @@ TSharedRefArray TClientRequest::Serialize() auto patchedHeader = Header_; patchedHeader.set_retry(true); - auto output = CreateRequestMessage(patchedHeader, headerlessMessage); - return std::move(output); + return CreateRequestMessage(patchedHeader, headerlessMessage); } IClientRequestControlPtr TClientRequest::Send(IClientResponseHandlerPtr responseHandler) |