diff options
| author | fanquake <fanquake@gmail.com> | 2020-09-19 16:51:35 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2020-09-19 17:13:28 +0800 |
| commit | 831b0ecea9156447a2b6a67d28858bc26d302c1c (patch) | |
| tree | 30591da5c34512510715b813d4f51727a067c247 /test/functional/p2p_compactblocks.py | |
| parent | 83b23848f7300addf5bbfcfe47a6f8b75a3599f8 (diff) | |
| parent | 6fe2ef2acb00b1df7f6a0c0dea1a81a1924be0e1 (diff) | |
Merge #13686: ZMQ: Small cleanups in the ZMQ code
6fe2ef2acb00b1df7f6a0c0dea1a81a1924be0e1 scripted-diff: Rename SendMessage to SendZmqMessage. (Daniel Kraft)
a3ffb6ebebd753cec294c91cef7c603a30cf217e Replace zmqconfig.h by a simple zmqutil. (Daniel Kraft)
7f2ad1b9acef4ccc1b3e1a9f551416235d95cbfd Use std::unique_ptr for CZMQNotifierFactory. (Daniel Kraft)
b93b9d54569145bfcec6cee10968284fe05fe254 Simplify and fix notifier removal on error. (Daniel Kraft)
e15b1cfc310df739b92bd281112dbeb31d3bb30a Various cleanups in zmqnotificationinterface. (Daniel Kraft)
Pull request description:
This contains various small code cleanups that make the ZMQ code easier to read and maintain (at least in my opinion). The only functional change is that a potential memory leak is fixed that would have occured when a notifier is removed from the `notifiers` list after its callback function returned `false` (which is likely not relevant in practice but still a bug).
ACKs for top commit:
instagibbs:
utACK 6fe2ef2acb00b1df7f6a0c0dea1a81a1924be0e1
hebasto:
re-ACK 6fe2ef2acb00b1df7f6a0c0dea1a81a1924be0e1, only the latest commit got a scripted-diff since my [previous](https://github.com/bitcoin/bitcoin/pull/13686#pullrequestreview-487649808) review.
Tree-SHA512: 8206f8713bf3698d7cd4cb235f6657dc1c4dd920f50a8c5f371a559dd17ce5ab6d94d6281165eef860a22fc844a6bb25489ada12c83ebc780efd7ccdc0860f70
Diffstat (limited to 'test/functional/p2p_compactblocks.py')
0 files changed, 0 insertions, 0 deletions