diff options
author | Jon Atack <jon@atack.com> | 2020-11-09 15:00:31 +0100 |
---|---|---|
committer | Jon Atack <jon@atack.com> | 2020-11-11 15:56:01 +0100 |
commit | 410e471fa42d3db04e8879c71f8c824dcc151a83 (patch) | |
tree | 640921f4d4e7802571c06f1803264197a793179c /src/wallet | |
parent | a0d495747320c79b27a83c216dcc526ac8df8f24 (diff) | |
download | bitcoin-410e471fa42d3db04e8879c71f8c824dcc151a83.tar.xz |
wallet: remove redundant bumpfee fee_rate checks
SetFeeEstimateMode() handles these checks now and provides a more actionable
error message.
Diffstat (limited to 'src/wallet')
-rw-r--r-- | src/wallet/rpcwallet.cpp | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/src/wallet/rpcwallet.cpp b/src/wallet/rpcwallet.cpp index 238d37ae98..73aed65439 100644 --- a/src/wallet/rpcwallet.cpp +++ b/src/wallet/rpcwallet.cpp @@ -3474,18 +3474,6 @@ static RPCHelpMan bumpfee_helper(std::string method_name) auto conf_target = options.exists("confTarget") ? options["confTarget"] : options["conf_target"]; - if (!conf_target.isNull()) { - if (options.exists("fee_rate")) { - throw JSONRPCError(RPC_INVALID_PARAMETER, "Cannot specify both conf_target and fee_rate. Please provide either a confirmation target in blocks for automatic fee estimation, or an explicit fee rate."); - } - } else if (options.exists("fee_rate")) { - CFeeRate fee_rate(AmountFromValue(options["fee_rate"])); - if (fee_rate <= CFeeRate(0)) { - throw JSONRPCError(RPC_INVALID_PARAMETER, strprintf("Invalid fee_rate %s (must be greater than 0)", fee_rate.ToString())); - } - coin_control.m_feerate = fee_rate; - } - if (options.exists("replaceable")) { coin_control.m_signal_bip125_rbf = options["replaceable"].get_bool(); } |