diff options
author | fanquake <fanquake@gmail.com> | 2023-01-17 09:33:14 +0000 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2023-01-17 09:39:26 +0000 |
commit | 7799f53542a8fab8ae6338862528c9dd9bc851f1 (patch) | |
tree | 4eb3e4d57e66711f3336902e8effb791c0265598 /src/qt/walletmodel.h | |
parent | 8339f3cea841005066239fa43e16586fdfd9d19a (diff) | |
parent | fa9f6d7bcdba5f18c46fff1dcc0ac6d3dd8db75d (diff) | |
download | bitcoin-7799f53542a8fab8ae6338862528c9dd9bc851f1.tar.xz |
Merge bitcoin/bitcoin#26039: refactor: Run type check against RPCArgs (1/2)
fa9f6d7bcdba5f18c46fff1dcc0ac6d3dd8db75d rpc: Run type check against RPCArgs (MarcoFalke)
faf96721a66dcc215ea9d6affb30f9a00cc37000 test: Fix wrong types passed to RPCs (MarcoFalke)
Pull request description:
It seems brittle to require `RPCTypeCheck` being called inside the code logic. Without compile-time enforcement this will lead to places where it is forgotten and thus to inconsistencies and bugs. Fix this by removing the calls to `RPCTypeCheck` and doing the check internally.
The changes should be reviewed as refactoring changes. However, if they change behavior, it will be a bugfix. For example the changes here happen to also detect/fix bugs like the one fixed in commit 3b5fb6e77a93f58b3d03b1eec3595f5c45e633a9.
ACKs for top commit:
ajtowns:
ACK fa9f6d7bcdba5f18c46fff1dcc0ac6d3dd8db75d
Tree-SHA512: fb2c0981fe6e24da3ca7dbc06898730779ea4e02ea485458505a281cf421015e44dad0221a04023fc547ea2c660d94657909843fc85d92b847611ec097532439
Diffstat (limited to 'src/qt/walletmodel.h')
0 files changed, 0 insertions, 0 deletions