diff options
| author | Andrew Chow <github@achow101.com> | 2023-01-18 14:18:01 -0500 |
|---|---|---|
| committer | Andrew Chow <github@achow101.com> | 2023-01-18 14:26:39 -0500 |
| commit | 8ae2808a4354e8dcc697f76bacc5e2f2befe9220 (patch) | |
| tree | 56543964431a532f5d01af26ac042b756d8f1644 /src/rpc/client.cpp | |
| parent | aef8b4f43b0c4300aa6cf2c5cf5c19f55e73499c (diff) | |
| parent | a2ac6f9582c4c996fa36e4801fa0aac756235754 (diff) | |
Merge bitcoin/bitcoin#25659: wallet: simplify ListCoins implementation
a2ac6f9582c4c996fa36e4801fa0aac756235754 wallet: unify FindNonChangeParentOutput functions (furszy)
b3f4e827378e010cd2a5d1b876d01db52c054d26 wallet: simplify ListCoins implementation (furszy)
Pull request description:
Focused on the following changes:
1) Removed the entire locked coins lookup that was inside `ListCoins` by including them directly on the `AvailableCoins` result (where we were skipping them before).
2) Unified both `FindNonChangeParentOutput` functions (only called from `ListCoins`)
ACKs for top commit:
achow101:
ACK a2ac6f9582c4c996fa36e4801fa0aac756235754
aureleoules:
ACK a2ac6f9582c4c996fa36e4801fa0aac756235754, LGTM
theStack:
Code-review ACK a2ac6f9582c4c996fa36e4801fa0aac756235754
Tree-SHA512: f72b21ee1600c5992559b5dcd8ff260527afac2ec696737f998343a0850b84d439e7f86ea52a14cc0cddabf132a30bf5b52fb34950578ac323083d4375b937f1
Diffstat (limited to 'src/rpc/client.cpp')
0 files changed, 0 insertions, 0 deletions