p2p, rpc, fuzz: various tiny follow-ups #22169

pull jonatack wants to merge 1 commits into bitcoin:master from jonatack:p2p-rpc-fuzz-various-followups changing 3 files +5 −5

jonatack commented at 1:57 PM on June 6, 2021: member
- p2p: pass Span by value per #22143 (comment) as a follow-up to 8be56f0f8ecc54744
- rpc: remove duplicate CAddress constructor per #22043 (review)
- fuzz: rename 3 fuzz targets changed in eba9a94b9f56be2fda623e77f19b960425ea1eb5 back to their original names per https://github.com/bitcoin-core/qa-assets/pull/63#issuecomment-855281865
p2p, rpc, fuzz: various tiny follow-ups c274574458
MarcoFalke commented at 2:51 PM on June 6, 2021: member

cr ACK c274574458e5921be4d1f3e86e6bba72a7cd3e65
DrahtBot added the label P2P on Jun 6, 2021
DrahtBot added the label RPC/REST/ZMQ on Jun 6, 2021
jarolrod commented at 5:01 PM on June 6, 2021: member

ACK c274574458e5921be4d1f3e86e6bba72a7cd3e65

Patch looks correct following the linked discussions for the p2p and rpc changes. Did not fuzz.
practicalswift commented at 8:42 PM on June 6, 2021: contributor

cr ACK c274574458e5921be4d1f3e86e6bba72a7cd3e65: patch looks correct
fanquake merged this on Jun 7, 2021
fanquake closed this on Jun 7, 2021
jonatack deleted the branch on Jun 7, 2021
sidhujag referenced this in commit 8739d50db8 on Jun 9, 2021
gwillen referenced this in commit af93aa29dc on Jun 1, 2022
DrahtBot locked this on Aug 18, 2022

Contributors

Labels