Page MenuHomePhabricator

Move SocketHandler logic to private method.
ClosedPublic

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

Details

Summary
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

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

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