net: use CMessageHeader::HEADER_SIZE, add missing include
ClosedPublic
Actions

Authored by PiRK on Jan 28 2021, 07:37.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC52644a5e025e: net: use CMessageHeader::HEADER_SIZE, add missing include

Summary

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 with it and adds the explicit include header.

Co-authored by: Gleb Naumenko <naumenko.gs@gmail.com>

This is a backport of Core PR18931

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr18931

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15028
Build 30007: Build Diff
Build 30006: arc lint + arc unit

Event Timeline

PiRK created this revision.Jan 28 2021, 07:37

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 28 2021, 07:37

PiRK requested review of this revision.Jan 28 2021, 07:37

Harbormaster completed remote builds in B15028: Diff 27382.Jan 28 2021, 07:37

teamcity edited the summary of this revision. (Show Details)Jan 28 2021, 07:37

Fabien accepted this revision.Jan 28 2021, 09:40

This revision is now accepted and ready to land.Jan 28 2021, 09:40

rebase

Harbormaster completed remote builds in B15033: Diff 27391.Jan 28 2021, 10:14

Closed by commit rABC52644a5e025e: net: use CMessageHeader::HEADER_SIZE, add missing include (authored by Jon Atack <jon@atack.com>, committed by PiRK). · Explain WhyJan 28 2021, 10:58

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC52644a5e025e: net: use CMessageHeader::HEADER_SIZE, add missing include.

Revision Contents
Changeset List

Path

Size

src/

net.cpp

5 lines

Diff 27382

View Options

net: use CMessageHeader::HEADER_SIZE, add missing includeClosedPublicActions