diff options
| author | fanquake <[email protected]> | 2020-05-12 16:45:06 +0800 |
|---|---|---|
| committer | fanquake <[email protected]> | 2020-05-12 17:05:40 +0800 |
| commit | 0f2fa599aee0fa45cd65ef5a5973212d20024c07 (patch) | |
| tree | 9f5ed10fe78e39c1e91fbf33e05bb70e98f27935 /src/interfaces/node.cpp | |
| parent | Merge #18957: Add a link from ZMQ doc to ZMQ example in contrib/ (diff) | |
| parent | net: use CMessageHeader::HEADER_SIZE, add missing include (diff) | |
| download | discoin-0f2fa599aee0fa45cd65ef5a5973212d20024c07.tar.xz discoin-0f2fa599aee0fa45cd65ef5a5973212d20024c07.zip | |
Merge #18931: net: use CMessageHeader::HEADER_SIZE, add missing include
83da576f4416c64b5d520819208a722b2273739a net: use CMessageHeader::HEADER_SIZE, add missing include (Jon Atack)
Pull request description:
as suggested 16 months ago by Gleb Naumenko in https://github.com/bitcoin/bitcoin/pull/15197#issuecomment-456181865.
`static constexpr CMessageHeader::HEADER_SIZE` is already used in this file, `src/net.cpp`, in 2 instances. This commit replaces the remaining 2 integer values in the file with it and adds the explicit include header.
Co-authored by: Gleb Naumenko <[email protected]>
ACKs for top commit:
naumenkogs:
utACK 83da576
practicalswift:
ACK 83da576f4416c64b5d520819208a722b2273739a -- patch looks correct
theStack:
ACK 83da576f4416c64b5d520819208a722b2273739a -- verified that its just magic number elimination refactoring and additionally checked that all tests pass :+1:
Tree-SHA512: 5b915483bca4ea162c259865a1b615d73b88a1b1db3f82db05f770d10b8a42494d948f5b21badbcce2d9efa5915b8cbb6af83073867c23d2f152c0d35ac37b96
Diffstat (limited to 'src/interfaces/node.cpp')
0 files changed, 0 insertions, 0 deletions