refactor: fix wallet and related named args #24677

pull fanquake wants to merge 3 commits into bitcoin:master from fanquake:fix_wallet_and_co_named_args changing 3 files +32 −32

fanquake commented at 9:30 PM on March 25, 2022: member

Should be one of the last changes split from #24661.

Motivation:

Incorrect named args are source of bugs, like #22979.

To allow them being checked by clang-tidy, use a format it can understand.
bench: fix incorrect named args in coin_selection bench 6fc00f7331
test: fix incorrect named args in coin_selection tests 8b0e776718
test: fix incorrect named args in wallet tests 21db4eb3ff
fanquake added the label Refactoring on Mar 25, 2022
fanquake requested review from MarcoFalke on Mar 25, 2022
MarcoFalke commented at 12:56 PM on March 28, 2022: member

Concept ACK 21db4eb3ff481334e0fdf4724f853556f6a0028f
MarcoFalke merged this on Mar 28, 2022
MarcoFalke closed this on Mar 28, 2022
fanquake deleted the branch on Mar 28, 2022
bitcoin locked this on Mar 28, 2023

Contributors

Review Requested
MarcoFalke

Labels