Differential D1879

Various nits in net_processing.cpp
ClosedPublic
Actions

Authored by deadalnix on Oct 5 2018, 09:45.

Details

Reviewers

jasonbcox

Group Reviewers

Restricted Project

Commits

rSTAGINGee28ccaa316a: Various nits in net_processing.cpp
rABCee28ccaa316a: Various nits in net_processing.cpp

Summary

As per title.

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

netprocessingnits

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3514
Build 5104: Bitcoin ABC Buildbot (legacy)
Build 5103: arc lint + arc unit

Event Timeline

deadalnix created this revision.Oct 5 2018, 09:45

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 5 2018, 09:45

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B3514: Diff 5281.Oct 5 2018, 10:03

jasonbcox accepted this revision.Oct 5 2018, 20:10

This revision is now accepted and ready to land.Oct 5 2018, 20:10

Closed by commit rABCee28ccaa316a: Various nits in net_processing.cpp (authored by deadalnix). · Explain WhyOct 6 2018, 09:35

This revision was automatically updated to reflect the committed changes.

nakihito mentioned this in D3293: Merge #13163: Make it clear which functions that are intended to be translation unit local.Jun 12 2019, 23:30

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

51 lines

Diff 5281

View Options

Various nits in net_processing.cppClosedPublicActions