diff options
author | Jon Atack <jon@atack.com> | 2021-04-03 20:22:55 +0200 |
---|---|---|
committer | Jon Atack <jon@atack.com> | 2021-04-07 12:57:09 +0200 |
commit | 1b9189866af26ed0003c1afe8dd5652ebe9b2e4a (patch) | |
tree | 2d6ca110d8ff3123ac9f5e1cae909a20b7744dea /src | |
parent | 3bb6e7b6555f3c8743a697cb9d509620714dc483 (diff) |
rpc: simplify/constify getnodeaddresses code
Diffstat (limited to 'src')
-rw-r--r-- | src/rpc/net.cpp | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/src/rpc/net.cpp b/src/rpc/net.cpp index aff9bf7ae1..96c195a20e 100644 --- a/src/rpc/net.cpp +++ b/src/rpc/net.cpp @@ -863,15 +863,11 @@ static RPCHelpMan getnodeaddresses() throw JSONRPCError(RPC_CLIENT_P2P_DISABLED, "Error: Peer-to-peer functionality missing or disabled"); } - int count = 1; - if (!request.params[0].isNull()) { - count = request.params[0].get_int(); - if (count < 0) { - throw JSONRPCError(RPC_INVALID_PARAMETER, "Address count out of range"); - } - } + const int count{request.params[0].isNull() ? 1 : request.params[0].get_int()}; + if (count < 0) throw JSONRPCError(RPC_INVALID_PARAMETER, "Address count out of range"); + // returns a shuffled list of CAddress - std::vector<CAddress> vAddr = node.connman->GetAddresses(count, /* max_pct */ 0); + const std::vector<CAddress> vAddr{node.connman->GetAddresses(count, /* max_pct */ 0)}; UniValue ret(UniValue::VARR); for (const CAddress& addr : vAddr) { |