diff options
| author | Jon Atack <[email protected]> | 2020-11-09 15:00:31 +0100 |
|---|---|---|
| committer | Jon Atack <[email protected]> | 2020-11-11 15:56:01 +0100 |
| commit | 410e471fa42d3db04e8879c71f8c824dcc151a83 (patch) | |
| tree | 640921f4d4e7802571c06f1803264197a793179c /src/wallet/rpcwallet.cpp | |
| parent | wallet: introduce fee_rate (sat/vB) param/option (diff) | |
| download | discoin-410e471fa42d3db04e8879c71f8c824dcc151a83.tar.xz discoin-410e471fa42d3db04e8879c71f8c824dcc151a83.zip | |
wallet: remove redundant bumpfee fee_rate checks
SetFeeEstimateMode() handles these checks now and provides a more actionable
error message.
Diffstat (limited to 'src/wallet/rpcwallet.cpp')
| -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 238d37ae9..73aed6543 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(); } |