LBRYCommunity/lbrycrd
1
0
Fork 0
Code Issues 29 Pull requests 1 Projects Releases 32 Packages Wiki Activity

RPCHelpMan: Support required arguments after optional ones

Browse source
This commit is contained in:
MarcoFalke 2018-12-05 11:43:55 -05:00
parent 5f23460c7e
commit fa9a5bc1a0
No known key found for this signature in database
GPG key ID: CE2B75697E69A548
2 changed files with 8 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/rpc/mining.cpp
View file

@ -241,7 +241,7 @@ static UniValue prioritisetransaction(const JSONRPCRequest& request)
"Accepts the transaction into mined blocks at a higher (or lower) priority\n", "Accepts the transaction into mined blocks at a higher (or lower) priority\n",
{ {
{"txid", RPCArg::Type::STR_HEX, /* opt */ false, /* default_val */ "", "The transaction id."}, {"txid", RPCArg::Type::STR_HEX, /* opt */ false, /* default_val */ "", "The transaction id."},
{"dummy", RPCArg::Type::NUM, /* opt */ false, /* default_val */ "", "API-Compatibility for previous API. Must be zero or null.\n" {"dummy", RPCArg::Type::NUM, /* opt */ true, /* default_val */ "null", "API-Compatibility for previous API. Must be zero or null.\n"
" DEPRECATED. For forward compatibility use named arguments and omit this parameter."}, " DEPRECATED. For forward compatibility use named arguments and omit this parameter."},
{"fee_delta", RPCArg::Type::NUM, /* opt */ false, /* default_val */ "", "The fee value (in satoshis) to add (or subtract, if negative).\n" {"fee_delta", RPCArg::Type::NUM, /* opt */ false, /* default_val */ "", "The fee value (in satoshis) to add (or subtract, if negative).\n"
" Note, that this value is not a fee rate. It is a value to modify absolute fee of the TX.\n" " Note, that this value is not a fee rate. It is a value to modify absolute fee of the TX.\n"

16
src/rpc/util.cpp
View file

@ -258,20 +258,19 @@ std::string RPCHelpMan::ToString() const
// Oneline summary // Oneline summary
ret += m_name; ret += m_name;
bool is_optional{false}; bool was_optional{false};
for (const auto& arg : m_args) { for (const auto& arg : m_args) {
ret += " "; ret += " ";
if (arg.m_optional) { if (arg.m_optional) {
if (!is_optional) ret += "( "; if (!was_optional) ret += "( ";
is_optional = true; was_optional = true;
} else { } else {
// Currently we still support unnamed arguments, so any argument following an optional argument must also be optional if (was_optional) ret += ") ";
// If support for positional arguments is deprecated in the future, remove this line was_optional = false;
assert(!is_optional);
} }
ret += arg.ToString(/* oneline */ true); ret += arg.ToString(/* oneline */ true);
} }
if (is_optional) ret += " )"; if (was_optional) ret += " )";
ret += "\n"; ret += "\n";
// Description // Description
@ -285,8 +284,7 @@ std::string RPCHelpMan::ToString() const
if (i == 0) ret += "\nArguments:\n"; if (i == 0) ret += "\nArguments:\n";
// Push named argument name and description // Push named argument name and description
const auto str_wrapper = (arg.m_type == RPCArg::Type::STR || arg.m_type == RPCArg::Type::STR_HEX) ? "\"" : ""; sections.m_sections.emplace_back(std::to_string(i + 1) + ". " + arg.m_name, arg.ToDescriptionString());
sections.m_sections.emplace_back(std::to_string(i + 1) + ". " + str_wrapper + arg.m_name + str_wrapper, arg.ToDescriptionString());
sections.m_max_pad = std::max(sections.m_max_pad, sections.m_sections.back().m_left.size()); sections.m_max_pad = std::max(sections.m_max_pad, sections.m_sections.back().m_left.size());
// Recursively push nested args // Recursively push nested args