diff options
| author | fanquake <fanquake@gmail.com> | 2020-05-12 16:45:06 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2020-05-12 17:05:40 +0800 |
| commit | 0f2fa599aee0fa45cd65ef5a5973212d20024c07 (patch) | |
| tree | 9f5ed10fe78e39c1e91fbf33e05bb70e98f27935 /test | |
| parent | ad1b7b18177afb2a2d919cef5442e7c35fd7fe1e (diff) | |
| parent | 83da576f4416c64b5d520819208a722b2273739a (diff) | |
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 <naumenko.gs@gmail.com>
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 'test')
0 files changed, 0 insertions, 0 deletions