summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorbabenko <[email protected]>2025-05-28 10:12:40 +0300
committerbabenko <[email protected]>2025-05-28 10:48:30 +0300
commit4b796727b1c97992a1fffcd1550bf226103fca59 (patch)
tree64ae9110e48e814b921977d88f34a20e20a8c025
parent896fcc7f6b80e06dfca1c2564723b020dfea24d3 (diff)
Fix redundant std::moves in return statements
commit_hash:d2018bced62a6351718a6b30119493e55aa49f4e
-rw-r--r--yt/yt/client/api/rpc_proxy/client_base.cpp2
-rw-r--r--yt/yt/core/misc/memory_usage_tracker.cpp4
-rw-r--r--yt/yt/core/misc/unittests/consistent_hashing_ut.cpp2
-rw-r--r--yt/yt/core/rpc/client.cpp6
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)