diff options
author | MacroFake <falke.marco@gmail.com> | 2022-07-20 07:49:16 +0200 |
---|---|---|
committer | MacroFake <falke.marco@gmail.com> | 2022-07-20 07:49:25 +0200 |
commit | 0897b189e4242321cd03339e7dde5c4d963d0e48 (patch) | |
tree | f3dded08b411a00c505e5834e6b91034c6873d94 /src/test/coinstatsindex_tests.cpp | |
parent | 5560682a4464852eb3c244c1ddf9eea02dc962b2 (diff) | |
parent | 1e761a0169ebdbd3b5784af39fc2248b4546eeea (diff) |
Merge bitcoin/bitcoin#25308: refactor: Reduce number of LoadChainstate parameters and return values
1e761a0169ebdbd3b5784af39fc2248b4546eeea ci: Enable IWYU in src/kernel directory (Ryan Ofsky)
6db6552377ad6316626b3ab8605a98f96f22c3d2 refactor: Reduce number of SanityChecks return values (Ryan Ofsky)
b3e7de7ee6efb186efc272855ff1af5d9254b971 refactor: Reduce number of LoadChainstate return values (Russell Yanofsky)
3b91d4b9947adbec74721f538e46c712db22587c refactor: Reduce number of LoadChainstate parameters (Russell Yanofsky)
Pull request description:
Replace long LoadChainstate parameters list with options struct. Replace long list of return values with simpler error strings.
No changes in behavior. Motivation is just to make libbitcoin_kernel API easier to use and more future-proof, and make internal code clearer and more maintainable.
ACKs for top commit:
MarcoFalke:
ACK 1e761a0169ebdbd3b5784af39fc2248b4546eeea 🕚
Tree-SHA512: 86f251ab820ca6664ade87ccac8330f79b0e48e26b98082f022f592ed1380f8eefc3cce260b85d5eea5d2f5f2531602e03d641e579c15684ecd9093b2aebcc58
Diffstat (limited to 'src/test/coinstatsindex_tests.cpp')
0 files changed, 0 insertions, 0 deletions