diff options
| author | fanquake <fanquake@gmail.com> | 2022-01-02 08:16:31 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2022-01-02 08:37:12 +0800 |
| commit | 9d099b02d8f02042f46401bcdbe5a729ef607308 (patch) | |
| tree | 013c230cee2a17f6cb42854e9a3c942ccdf07944 /src/wallet/test/wallet_test_fixture.cpp | |
| parent | 653577251011d077adea198c1c1437a56e2ae2ee (diff) | |
| parent | 4523d28b6bd550bf9f9c724f9aa90c3d2d4ebe3b (diff) | |
Merge bitcoin/bitcoin#23836: rest: Expose block filters follow-ups
4523d28b6bd550bf9f9c724f9aa90c3d2d4ebe3b [test] compare filter and header with the result of the getblockfilter RPC (Niklas Gögge)
3a2464f21619c3065ec2f5d8f7431703c30c964a [rest] drop superfluous rpc serializations flags for block filters (Niklas Gögge)
064abd14a55e0fa1bff530237816a748d01e0ddb [rest] add a more verbose error message for invalid header counts (Niklas Gögge)
83b8f3a8961baa34a136ecfaf62c3ea0d133b6d6 [refactor] various style fix-ups (Niklas Gögge)
Pull request description:
This PR addresses unresolved review comments from [#17631](https://github.com/bitcoin/bitcoin/pull/17631).
This includes:
* various style fix-ups
* returning a more verbose error message for invalid header counts
* removing superfluous rpc serializations flags for block filters
* improving the test to include comparing the block filters returned from the rest with the ones returned from the `getblockfilter` RPC.
ACKs for top commit:
jnewbery:
ACK 4523d28b6bd550bf9f9c724f9aa90c3d2d4ebe3b
brunoerg:
tACK 4523d28b6bd550bf9f9c724f9aa90c3d2d4ebe3b
Tree-SHA512: 634e6b2ae3e1d5f31675a50cfe11a5e74bf5a51b9e7b512d9e18879bf2ed424fc0ac6ec633023f192e3ad12cf0c73b0b51de79dd7ec00844dba3e1493d823b8c
Diffstat (limited to 'src/wallet/test/wallet_test_fixture.cpp')
0 files changed, 0 insertions, 0 deletions