p2p: Clarify control flow in ProcessMessage()
ClosedPublic
Actions

Authored by deadalnix on Nov 27 2019, 23:29.

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGING96a51590b6c6: p2p: Clarify control flow in ProcessMessage()
rABC96a51590b6c6: p2p: Clarify control flow in ProcessMessage()

Summary

This is a backport of Core PR13946

Test Plan

make check
./test/functional/test_runner.py --extended

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr13946

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 8318
Build 14655: Default Diff Build & Tests
Build 14654: arc lint + arc unit

Event Timeline

deadalnix created this revision.Nov 27 2019, 23:29

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 27 2019, 23:29

Build Bitcoin-ABC / Diffs / Diff Testing started.

Harbormaster completed remote builds in B8318: Diff 14476.Nov 27 2019, 23:40

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Fabien edited the summary of this revision. (Show Details)Nov 28 2019, 07:04

Fabien accepted this revision.Nov 28 2019, 07:15

This revision is now accepted and ready to land.Nov 28 2019, 07:15

Closed by commit rABC96a51590b6c6: p2p: Clarify control flow in ProcessMessage() (authored by MarcoFalke <falke.marco@gmail.com>, committed by deadalnix). · Explain WhyNov 28 2019, 13:25

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABC96a51590b6c6: p2p: Clarify control flow in ProcessMessage().

deadalnix added a commit: rSTAGING96a51590b6c6: p2p: Clarify control flow in ProcessMessage().Apr 7 2020, 21:54

deadalnix mentioned this in D6120: Have Seeder's ProcessMessage return fall through.May 18 2020, 18:46

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

87 lines

Diff 14476

View Options

p2p: Clarify control flow in ProcessMessage()ClosedPublicActions