diff options
author | Andrew Chow <github@achow101.com> | 2022-09-14 16:04:57 -0400 |
---|---|---|
committer | Andrew Chow <github@achow101.com> | 2022-09-14 16:15:03 -0400 |
commit | 2e3cd26a1a5e071407b79cf02c35f6cf42aa6191 (patch) | |
tree | daa321574e54b7c7ac8e72823712174436120d3d /build_msvc | |
parent | d19000370066dfa0aabea23f244628d0f54891e9 (diff) | |
parent | b00fc44ca5cb938f18d31cde7feb4e1c968dcc2f (diff) | |
download | bitcoin-2e3cd26a1a5e071407b79cf02c35f6cf42aa6191.tar.xz |
Merge bitcoin/bitcoin#26053: rpc: bugfix, 'add_inputs' default value is true unless 'inputs' are provided
b00fc44ca5cb938f18d31cde7feb4e1c968dcc2f test: add coverage for 'add_inputs' dynamic default value (furszy)
ddbcfdf3d050061f4e8979a0e2bb63bba5a43c47 RPC: bugfix, 'add_inputs' default value is true unless 'inputs' are provided (furszy)
Pull request description:
This bugfix was meant to be in #25685, but decoupled it to try to make it part of 24.0 release.
It's a truly misleading functionality.
This PR doesn't change behavior in any way. Just fixes two invalid RPC help messages and adds test
coverage for the current behavior.
#### Description
In both RPC commands `send()` and `walletcreatefundedpsbt` the help message says
that `add_inputs` default value is false when it's actually dynamically set by the following statement:
```c++
coin_control.m_allow_other_inputs = rawTx.vin.size() == 0;
```
Which means that, by default, `add_inputs` is true unless there is any pre-set input, in which
case, the default is false.
ACKs for top commit:
achow101:
ACK b00fc44ca5cb938f18d31cde7feb4e1c968dcc2f
S3RK:
ACK b00fc44ca5cb938f18d31cde7feb4e1c968dcc2f
Tree-SHA512: 5c68a40d81c994e0ab6de0817db69c4d3dea3a9a64a60362531bf583b7a4c37d524b740905a3f3a89cdbf221913ff5b504746625adb8622788aea93a35bbcd40
Diffstat (limited to 'build_msvc')
0 files changed, 0 insertions, 0 deletions