refactor: Fix coinselection.h include, Make COutput a struct #24666

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2203-coinselection-refactor-🚿 changing 3 files +15 −17

MarcoFalke commented at 9:02 AM on March 25, 2022: member
- Fix include (see commit message)
- {}-init, see #24091 (review)
- struct, see #24091 (review)

wallet: Fix coinselection include

coinselection.h is not used by wallet.h but by qt/coincontroldialog.cpp

fa61cdf464

Clarify that COutput is a struct, not a class
Also, use {}-initialization
fab287cedd
MarcoFalke added the label Refactoring on Mar 25, 2022
theStack approved
theStack commented at 2:17 PM on March 25, 2022: member

Code-review ACK fab287cedde85b21622d767d3ece65291e18b0bf
fanquake merged this on Mar 25, 2022
fanquake closed this on Mar 25, 2022
achow101 commented at 3:52 PM on March 25, 2022: member

Post merge ACK fab287cedde85b21622d767d3ece65291e18b0bf
MarcoFalke deleted the branch on Mar 25, 2022
DrahtBot locked this on Mar 25, 2023

Contributors

Labels