Paths

Table of Contentst

Differential D4089

Move SocketHandler logic to private method.
ClosedPublic
Actions

Authored by deadalnix on Sep 15 2019, 19:12.

Tags

None

Subscribers

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGING5d541c439495: Move SocketHandler logic to private method.
rABC5d541c439495: Move SocketHandler logic to private method.

Summary

This is a partical backport of Core PR14335 : https://github.com/bitcoin/bitcoin/pull/14335/commits/032488e6e7b4c2d37f3f434d575d25806ddcb0b0

Depends on D4088

Test Plan

make check
./test/functional/test_runner.py

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr14335-4

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 7473
Build 12989: Bitcoin ABC Buildbot (legacy)
Build 12988: arc lint + arc unit

Event Timeline

deadalnix created this revision.Sep 15 2019, 19:12

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 15 2019, 19:12

Build Bitcoin-ABC / Diffs / Diff Testing started.

Harbormaster completed remote builds in B7472: Diff 11330.Sep 15 2019, 19:24

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Fabien requested changes to this revision.Sep 16 2019, 08:13

Fabien added a subscriber: Fabien.

Fabien added inline comments.

src/net.cpp
1347 ↗	(On Diff #11330)	Revert the comment
1380 ↗	(On Diff #11330)	Fix the comments layout.
1418 ↗	(On Diff #11330)	Braces
1424 ↗	(On Diff #11330)	Braces
1445 ↗	(On Diff #11330)	Braces
1449 ↗	(On Diff #11330)	Braces
1469 ↗	(On Diff #11330)	Revert to `int32_t`
1480 ↗	(On Diff #11330)	Braces
1487 ↗	(On Diff #11330)	Braces
1513 ↗	(On Diff #11330)	Braces
1536 ↗	(On Diff #11330)	Braces

This revision now requires changes to proceed.Sep 16 2019, 08:13

Fix braces and various othe small changes

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B7473: Diff 11334.Sep 16 2019, 08:49

Fabien accepted this revision.Sep 16 2019, 08:59

Fabien added inline comments.

src/net.cpp
1347	NIt: restore capitalization.

This revision is now accepted and ready to land.Sep 16 2019, 08:59

Closed by commit rABC5d541c439495: Move SocketHandler logic to private method. (authored by Patrick Strateman <patrick.strateman@gmail.com>, committed by deadalnix). · Explain WhySep 16 2019, 09:18

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABC5d541c439495: Move SocketHandler logic to private method..

deadalnix added a commit: rSTAGING5d541c439495: Move SocketHandler logic to private method..Sep 19 2019, 02:23

Revision Contents
Changeset List

Path

Size

src/

1 line

350 lines

Diff 11334

src/net.h

Loading...

src/net.cpp

Loading...