Skip to content

Commit 58c7288

Browse files
author
MarcoFalke
committed
Merge #18268: rpc: Remove redundant types from descriptions
8a2a652 Remove redundant type information from rpc docs (David O'Callaghan) Pull request description: Simple edit of the RPC calls to remove redundant text ("A json object/array ...") from the beginning of help. Fixes: #18258 Top commit has no ACKs. Tree-SHA512: cbbf760e0b7b4eda61c40b420ed77f5d878318e37b0eb13e63567212240b2c4ecc15d84030e98075e21c9ae9016539adfd201e5661ea824166a76d335180c32f
2 parents 249114b + 8a2a652 commit 58c7288

File tree

4 files changed

+24
-24
lines changed

4 files changed

+24
-24
lines changed

‎src/rpc/mining.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -342,7 +342,7 @@ static UniValue getblocktemplate(const JSONRPCRequest& request)
342342
" https://github.com/bitcoin/bips/blob/master/bip-0009.mediawiki#getblocktemplate_changes\n"
343343
" https://github.com/bitcoin/bips/blob/master/bip-0145.mediawiki\n",
344344
{
345-
{"template_request", RPCArg::Type::OBJ, "{}", "A json object in the following spec",
345+
{"template_request", RPCArg::Type::OBJ, "{}", "Format of the template",
346346
{
347347
{"mode", RPCArg::Type::STR, /* treat as named arg */ RPCArg::Optional::OMITTED_NAMED_ARG, "This must be set to \"template\", \"proposal\" (see BIP 23), or omitted"},
348348
{"capabilities", RPCArg::Type::ARR, /* treat as named arg */ RPCArg::Optional::OMITTED_NAMED_ARG, "A list of strings",

‎src/rpc/misc.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ static UniValue createmultisig(const JSONRPCRequest& request)
7676
"It returns a json object with the address and redeemScript.\n",
7777
{
7878
{"nrequired", RPCArg::Type::NUM, RPCArg::Optional::NO, "The number of required signatures out of the n keys."},
79-
{"keys", RPCArg::Type::ARR, RPCArg::Optional::NO, "A json array of hex-encoded public keys.",
79+
{"keys", RPCArg::Type::ARR, RPCArg::Optional::NO, "The hex-encoded public keys.",
8080
{
8181
{"key", RPCArg::Type::STR_HEX, RPCArg::Optional::OMITTED, "The hex-encoded public key"},
8282
}},
@@ -515,11 +515,11 @@ UniValue logging(const JSONRPCRequest& request)
515515
" - \"none\", \"0\" : even if other logging categories are specified, ignore all of them.\n"
516516
,
517517
{
518-
{"include", RPCArg::Type::ARR, RPCArg::Optional::OMITTED_NAMED_ARG, "A json array of categories to add debug logging",
518+
{"include", RPCArg::Type::ARR, RPCArg::Optional::OMITTED_NAMED_ARG, "The categories to add to debug logging",
519519
{
520520
{"include_category", RPCArg::Type::STR, RPCArg::Optional::OMITTED, "the valid logging category"},
521521
}},
522-
{"exclude", RPCArg::Type::ARR, RPCArg::Optional::OMITTED_NAMED_ARG, "A json array of categories to remove debug logging",
522+
{"exclude", RPCArg::Type::ARR, RPCArg::Optional::OMITTED_NAMED_ARG, "The categories to remove from debug logging",
523523
{
524524
{"exclude_category", RPCArg::Type::STR, RPCArg::Optional::OMITTED, "the valid logging category"},
525525
}},

‎src/rpc/rawtransaction.cpp

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,7 @@ static UniValue gettxoutproof(const JSONRPCRequest& request)
230230
"you need to maintain a transaction index, using the -txindex command line option or\n"
231231
"specify the block in which the transaction is included manually (by blockhash).\n",
232232
{
233-
{"txids", RPCArg::Type::ARR, RPCArg::Optional::NO, "A json array of txids to filter",
233+
{"txids", RPCArg::Type::ARR, RPCArg::Optional::NO, "The txids to filter",
234234
{
235235
{"txid", RPCArg::Type::STR_HEX, RPCArg::Optional::OMITTED, "A transaction hash"},
236236
},
@@ -368,7 +368,7 @@ static UniValue createrawtransaction(const JSONRPCRequest& request)
368368
"Note that the transaction's inputs are not signed, and\n"
369369
"it is not stored in the wallet or transmitted to the network.\n",
370370
{
371-
{"inputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "A json array of json objects",
371+
{"inputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "The inputs",
372372
{
373373
{"", RPCArg::Type::OBJ, RPCArg::Optional::OMITTED, "",
374374
{
@@ -379,7 +379,7 @@ static UniValue createrawtransaction(const JSONRPCRequest& request)
379379
},
380380
},
381381
},
382-
{"outputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "a json array with outputs (key-value pairs), where none of the keys are duplicated.\n"
382+
{"outputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "The outputs (key-value pairs), where none of the keys are duplicated.\n"
383383
"That is, each address can only appear once and there can only be one 'data' object.\n"
384384
"For compatibility reasons, a dictionary, which holds the key-value pairs directly, is also\n"
385385
" accepted as second parameter.",
@@ -622,7 +622,7 @@ static UniValue combinerawtransaction(const JSONRPCRequest& request)
622622
"The combined transaction may be another partially signed transaction or a \n"
623623
"fully signed transaction.",
624624
{
625-
{"txs", RPCArg::Type::ARR, RPCArg::Optional::NO, "A json array of hex strings of partially signed transactions",
625+
{"txs", RPCArg::Type::ARR, RPCArg::Optional::NO, "The hex strings of partially signed transactions",
626626
{
627627
{"hexstring", RPCArg::Type::STR_HEX, RPCArg::Optional::OMITTED, "A transaction hash"},
628628
},
@@ -708,12 +708,12 @@ static UniValue signrawtransactionwithkey(const JSONRPCRequest& request)
708708
"this transaction depends on but may not yet be in the block chain.\n",
709709
{
710710
{"hexstring", RPCArg::Type::STR, RPCArg::Optional::NO, "The transaction hex string"},
711-
{"privkeys", RPCArg::Type::ARR, RPCArg::Optional::NO, "A json array of base58-encoded private keys for signing",
711+
{"privkeys", RPCArg::Type::ARR, RPCArg::Optional::NO, "The base58-encoded private keys for signing",
712712
{
713713
{"privatekey", RPCArg::Type::STR_HEX, RPCArg::Optional::OMITTED, "private key in base58-encoding"},
714714
},
715715
},
716-
{"prevtxs", RPCArg::Type::ARR, RPCArg::Optional::OMITTED_NAMED_ARG, "A json array of previous dependent transaction outputs",
716+
{"prevtxs", RPCArg::Type::ARR, RPCArg::Optional::OMITTED_NAMED_ARG, "The previous dependent transaction outputs",
717717
{
718718
{"", RPCArg::Type::OBJ, RPCArg::Optional::OMITTED, "",
719719
{
@@ -1279,7 +1279,7 @@ UniValue combinepsbt(const JSONRPCRequest& request)
12791279
"\nCombine multiple partially signed Bitcoin transactions into one transaction.\n"
12801280
"Implements the Combiner role.\n",
12811281
{
1282-
{"txs", RPCArg::Type::ARR, RPCArg::Optional::NO, "A json array of base64 strings of partially signed transactions",
1282+
{"txs", RPCArg::Type::ARR, RPCArg::Optional::NO, "The base64 strings of partially signed transactions",
12831283
{
12841284
{"psbt", RPCArg::Type::STR, RPCArg::Optional::OMITTED, "A base64 string of a PSBT"},
12851285
},
@@ -1384,7 +1384,7 @@ UniValue createpsbt(const JSONRPCRequest& request)
13841384
"\nCreates a transaction in the Partially Signed Transaction format.\n"
13851385
"Implements the Creator role.\n",
13861386
{
1387-
{"inputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "A json array of json objects",
1387+
{"inputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "The json objects",
13881388
{
13891389
{"", RPCArg::Type::OBJ, RPCArg::Optional::OMITTED, "",
13901390
{
@@ -1395,7 +1395,7 @@ UniValue createpsbt(const JSONRPCRequest& request)
13951395
},
13961396
},
13971397
},
1398-
{"outputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "a json array with outputs (key-value pairs), where none of the keys are duplicated.\n"
1398+
{"outputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "The outputs (key-value pairs), where none of the keys are duplicated.\n"
13991399
"That is, each address can only appear once and there can only be one 'data' object.\n"
14001400
"For compatibility reasons, a dictionary, which holds the key-value pairs directly, is also\n"
14011401
" accepted as second parameter.",
@@ -1617,7 +1617,7 @@ UniValue joinpsbts(const JSONRPCRequest& request)
16171617
"\nJoins multiple distinct PSBTs with different inputs and outputs into one PSBT with inputs and outputs from all of the PSBTs\n"
16181618
"No input in any of the PSBTs can be in more than one of the PSBTs.\n",
16191619
{
1620-
{"txs", RPCArg::Type::ARR, RPCArg::Optional::NO, "A json array of base64 strings of partially signed transactions",
1620+
{"txs", RPCArg::Type::ARR, RPCArg::Optional::NO, "The base64 strings of partially signed transactions",
16211621
{
16221622
{"psbt", RPCArg::Type::STR, RPCArg::Optional::NO, "A base64 string of a PSBT"}
16231623
}}

‎src/wallet/rpcwallet.cpp

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -813,14 +813,14 @@ static UniValue sendmany(const JSONRPCRequest& request)
813813
HelpRequiringPassphrase(pwallet) + "\n",
814814
{
815815
{"dummy", RPCArg::Type::STR, RPCArg::Optional::NO, "Must be set to \"\" for backwards compatibility.", "\"\""},
816-
{"amounts", RPCArg::Type::OBJ, RPCArg::Optional::NO, "A json object with addresses and amounts",
816+
{"amounts", RPCArg::Type::OBJ, RPCArg::Optional::NO, "The addresses and amounts",
817817
{
818818
{"address", RPCArg::Type::AMOUNT, RPCArg::Optional::NO, "The bitcoin address is the key, the numeric amount (can be string) in " + CURRENCY_UNIT + " is the value"},
819819
},
820820
},
821821
{"minconf", RPCArg::Type::NUM, RPCArg::Optional::OMITTED_NAMED_ARG, "Ignored dummy value"},
822822
{"comment", RPCArg::Type::STR, RPCArg::Optional::OMITTED_NAMED_ARG, "A comment"},
823-
{"subtractfeefrom", RPCArg::Type::ARR, RPCArg::Optional::OMITTED_NAMED_ARG, "A json array with addresses.\n"
823+
{"subtractfeefrom", RPCArg::Type::ARR, RPCArg::Optional::OMITTED_NAMED_ARG, "The addresses.\n"
824824
" The fee will be equally deducted from the amount of each selected address.\n"
825825
" Those recipients will receive less bitcoins than you enter in their corresponding amount field.\n"
826826
" If no addresses are specified here, the sender pays the fee.",
@@ -951,7 +951,7 @@ static UniValue addmultisigaddress(const JSONRPCRequest& request)
951951
"If 'label' is specified, assign address to that label.\n",
952952
{
953953
{"nrequired", RPCArg::Type::NUM, RPCArg::Optional::NO, "The number of required signatures out of the n keys or addresses."},
954-
{"keys", RPCArg::Type::ARR, RPCArg::Optional::NO, "A json array of bitcoin addresses or hex-encoded public keys",
954+
{"keys", RPCArg::Type::ARR, RPCArg::Optional::NO, "The bitcoin addresses or hex-encoded public keys",
955955
{
956956
{"key", RPCArg::Type::STR, RPCArg::Optional::OMITTED, "bitcoin address or hex-encoded public key"},
957957
},
@@ -2144,7 +2144,7 @@ static UniValue lockunspent(const JSONRPCRequest& request)
21442144
"Also see the listunspent call\n",
21452145
{
21462146
{"unlock", RPCArg::Type::BOOL, RPCArg::Optional::NO, "Whether to unlock (true) or lock (false) the specified transactions"},
2147-
{"transactions", RPCArg::Type::ARR, /* default */ "empty array", "A json array of objects. Each object the txid (string) vout (numeric).",
2147+
{"transactions", RPCArg::Type::ARR, /* default */ "empty array", "The transaction outputs and within each, the txid (string) vout (numeric).",
21482148
{
21492149
{"", RPCArg::Type::OBJ, RPCArg::Optional::OMITTED, "",
21502150
{
@@ -2817,7 +2817,7 @@ static UniValue listunspent(const JSONRPCRequest& request)
28172817
{
28182818
{"minconf", RPCArg::Type::NUM, /* default */ "1", "The minimum confirmations to filter"},
28192819
{"maxconf", RPCArg::Type::NUM, /* default */ "9999999", "The maximum confirmations to filter"},
2820-
{"addresses", RPCArg::Type::ARR, /* default */ "empty array", "A json array of bitcoin addresses to filter",
2820+
{"addresses", RPCArg::Type::ARR, /* default */ "empty array", "The bitcoin addresses to filter",
28212821
{
28222822
{"address", RPCArg::Type::STR, RPCArg::Optional::OMITTED, "bitcoin address"},
28232823
},
@@ -3167,7 +3167,7 @@ static UniValue fundrawtransaction(const JSONRPCRequest& request)
31673167
"e.g. with 'importpubkey' or 'importmulti' with the 'pubkeys' or 'desc' field."},
31683168
{"lockUnspents", RPCArg::Type::BOOL, /* default */ "false", "Lock selected unspent outputs"},
31693169
{"feeRate", RPCArg::Type::AMOUNT, /* default */ "not set: makes wallet determine the fee", "Set a specific fee rate in " + CURRENCY_UNIT + "/kB"},
3170-
{"subtractFeeFromOutputs", RPCArg::Type::ARR, /* default */ "empty array", "A json array of integers.\n"
3170+
{"subtractFeeFromOutputs", RPCArg::Type::ARR, /* default */ "empty array", "The integers.\n"
31713171
" The fee will be equally deducted from the amount of each specified output.\n"
31723172
" Those recipients will receive less bitcoins than you enter in their corresponding amount field.\n"
31733173
" If no outputs are specified here, the sender pays the fee.",
@@ -3250,7 +3250,7 @@ UniValue signrawtransactionwithwallet(const JSONRPCRequest& request)
32503250
HelpRequiringPassphrase(pwallet) + "\n",
32513251
{
32523252
{"hexstring", RPCArg::Type::STR, RPCArg::Optional::NO, "The transaction hex string"},
3253-
{"prevtxs", RPCArg::Type::ARR, RPCArg::Optional::OMITTED_NAMED_ARG, "A json array of previous dependent transaction outputs",
3253+
{"prevtxs", RPCArg::Type::ARR, RPCArg::Optional::OMITTED_NAMED_ARG, "The previous dependent transaction outputs",
32543254
{
32553255
{"", RPCArg::Type::OBJ, RPCArg::Optional::OMITTED, "",
32563256
{
@@ -4149,7 +4149,7 @@ UniValue walletcreatefundedpsbt(const JSONRPCRequest& request)
41494149
"\nCreates and funds a transaction in the Partially Signed Transaction format. Inputs will be added if supplied inputs are not enough\n"
41504150
"Implements the Creator and Updater roles.\n",
41514151
{
4152-
{"inputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "A json array of json objects",
4152+
{"inputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "The inputs",
41534153
{
41544154
{"", RPCArg::Type::OBJ, RPCArg::Optional::OMITTED, "",
41554155
{
@@ -4160,7 +4160,7 @@ UniValue walletcreatefundedpsbt(const JSONRPCRequest& request)
41604160
},
41614161
},
41624162
},
4163-
{"outputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "a json array with outputs (key-value pairs), where none of the keys are duplicated.\n"
4163+
{"outputs", RPCArg::Type::ARR, RPCArg::Optional::NO, "The outputs (key-value pairs), where none of the keys are duplicated.\n"
41644164
"That is, each address can only appear once and there can only be one 'data' object.\n"
41654165
"For compatibility reasons, a dictionary, which holds the key-value pairs directly, is also\n"
41664166
" accepted as second parameter.",
@@ -4186,7 +4186,7 @@ UniValue walletcreatefundedpsbt(const JSONRPCRequest& request)
41864186
{"includeWatching", RPCArg::Type::BOOL, /* default */ "true for watch-only wallets, otherwise false", "Also select inputs which are watch only"},
41874187
{"lockUnspents", RPCArg::Type::BOOL, /* default */ "false", "Lock selected unspent outputs"},
41884188
{"feeRate", RPCArg::Type::AMOUNT, /* default */ "not set: makes wallet determine the fee", "Set a specific fee rate in " + CURRENCY_UNIT + "/kB"},
4189-
{"subtractFeeFromOutputs", RPCArg::Type::ARR, /* default */ "empty array", "A json array of integers.\n"
4189+
{"subtractFeeFromOutputs", RPCArg::Type::ARR, /* default */ "empty array", "The outputs to subtract the fee from.\n"
41904190
" The fee will be equally deducted from the amount of each specified output.\n"
41914191
" Those recipients will receive less bitcoins than you enter in their corresponding amount field.\n"
41924192
" If no outputs are specified here, the sender pays the fee.",

0 commit comments

Comments
 (0)