highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2026-09-12 05:32:22 +02:00
Code Issues Packages Projects Releases Wiki Activity

rpc: avoid moving RPC enum types

Browse Source 
`RPCArg::Type` and `RPCResult::Type` are enum values, so moving them in constructors is no different from copying them.

Initialize the stored type fields directly.
This commit is contained in:
Lőrinc
2026-02-06 10:58:10 +01:00
parent b67baed4e7
commit d9f94aa882
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/rpc/util.h
View File

@@ -236,7 +236,7 @@ struct RPCArg {
std::string description,
RPCArgOptions opts = {})
: m_names{std::move(name)},
m_type{std::move(type)},
m_type{type},
m_fallback{std::move(fallback)},
m_description{std::move(description)},
m_opts{std::move(opts)}
@@ -252,7 +252,7 @@ struct RPCArg {
std::vector<RPCArg> inner,
RPCArgOptions opts = {})
: m_names{std::move(name)},
m_type{std::move(type)},
m_type{type},
m_inner{std::move(inner)},
m_fallback{std::move(fallback)},
m_description{std::move(description)},
@@ -338,7 +338,7 @@ struct RPCResult {
std::string description,
std::vector<RPCResult> inner = {},
RPCResultOptions opts = {})
: m_type{std::move(type)},
: m_type{type},
m_key_name{std::move(m_key_name)},
m_inner{std::move(inner)},
m_optional{optional},
@@ -366,7 +366,7 @@ struct RPCResult {
std::string description,
std::vector<RPCResult> inner = {},
RPCResultOptions opts = {})
: m_type{std::move(type)},
: m_type{type},
m_key_name{std::move(m_key_name)},
m_inner{std::move(inner)},
m_optional{optional},