diff options
| author | furszy <matiasfurszyfer@protonmail.com> | 2022-08-05 11:58:08 -0300 |
|---|---|---|
| committer | furszy <matiasfurszyfer@protonmail.com> | 2022-10-26 15:47:51 -0300 |
| commit | 94c0766b0cd1990c1399a745c88c2ba4c685d8d1 (patch) | |
| tree | cc82777f6bbac49bb8d0ef3252d0995563296e96 /src/wallet/coincontrol.h | |
| parent | fabc0310480b49e159a15d494525c5aa15072cba (diff) | |
| download | bitcoin-94c0766b0cd1990c1399a745c88c2ba4c685d8d1.tar.xz | |
wallet: skip available coins fetch if "other inputs" are disallowed
no need to waste resources calculating the wallet available coins if
they are not going to be used.
The 'm_allow_other_inputs=true` default value change is to correct
an ugly misleading behavior:
The tx creation process was having a workaround patch to automatically
fall back to select coins from the wallet if `m_allow_other_inputs=false`
(previous default value) and no manual inputs were selected.
This could be seen in master in flows like `sendtoaddress`, `sendmany`
and even the GUI, where the `m_allow_other_inputs` value isn't customized
and the wallet still selects and adds coins to the tx internally.
Diffstat (limited to 'src/wallet/coincontrol.h')
| -rw-r--r-- | src/wallet/coincontrol.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/wallet/coincontrol.h b/src/wallet/coincontrol.h index d08d3664c4..b56a6d3aee 100644 --- a/src/wallet/coincontrol.h +++ b/src/wallet/coincontrol.h @@ -37,7 +37,7 @@ public: bool m_include_unsafe_inputs = false; //! If true, the selection process can add extra unselected inputs from the wallet //! while requires all selected inputs be used - bool m_allow_other_inputs = false; + bool m_allow_other_inputs = true; //! Includes watch only addresses which are solvable bool fAllowWatchOnly = false; //! Override automatic min/max checks on fee, m_feerate must be set if true |