Re-layout comments in ProcessGetBlockData and ProcessGetData
ClosedPublic
Actions

Authored by Fabien on Feb 28 2019, 11:21.

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGING6a8b6fb9fa22: Re-layout comments in ProcessGetBlockData and ProcessGetData
rABC6a8b6fb9fa22: Re-layout comments in ProcessGetBlockData and ProcessGetData

Summary

This is a follow-up for a suggestion in D2606. Due the the functions
refactoring, blocks of comment ended up left-shifted and the layout can
be updated to look better.

Depends on D2607

Test Plan

Read the comments

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

comment_layout_netprocessing

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 5104
Build 8271: Bitcoin ABC Buildbot (legacy)
Build 8270: arc lint + arc unit

Event Timeline

Fabien created this revision.Feb 28 2019, 11:21

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 28 2019, 11:21

Herald added a reviewer: deadalnix. · View Herald Transcript

Herald added a subscriber: schancel. · View Herald Transcript

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

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

Harbormaster completed remote builds in B5104: Diff 7521.Feb 28 2019, 11:34

deadalnix accepted this revision.Feb 28 2019, 14:23

This revision is now accepted and ready to land.Feb 28 2019, 14:23

Closed by commit rABC6a8b6fb9fa22: Re-layout comments in ProcessGetBlockData and ProcessGetData (authored by Fabien). · Explain WhyFeb 28 2019, 18:57

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rSTAGING6a8b6fb9fa22: Re-layout comments in ProcessGetBlockData and ProcessGetData.Sep 18 2019, 22:35

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

66 lines

Diff 7521

View Options

Re-layout comments in ProcessGetBlockData and ProcessGetDataClosedPublicActions