Page MenuHomePhabricator
Differential D1740

Simplify IsBanned code from net.cpp
ClosedPublic
Actions

Authored by deadalnix on Sep 7 2018, 12:52.

Details

Reviewers
jasonbcox
Group Reviewers
Restricted Project
Commits
rSTAGING637dce206ad7: Simplify IsBanned code from net.cpp
rABC637dce206ad7: Simplify IsBanned code from net.cpp
Summary

Reltively ovious refactoring.

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
netsimplify
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3229
Build 4545: Bitcoin ABC Buildbot (legacy)
Build 4544: arc lint + arc unit

Event Timeline

deadalnix created this revision.Sep 7 2018, 12:52
Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 7 2018, 12:52
teamcity added a subscriber: teamcity.Sep 7 2018, 12:53

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

Harbormaster completed remote builds in B3229: Diff 4816.Sep 7 2018, 13:30
teamcity added a comment.Sep 7 2018, 13:30

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

deadalnix updated this revision to Diff 4817.Sep 7 2018, 14:30

One more C++11 loop

teamcity added a comment.Sep 7 2018, 14:30

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

teamcity added a comment.Sep 7 2018, 15:11

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

Harbormaster completed remote builds in B3230: Diff 4817.Sep 7 2018, 15:11
jasonbcox accepted this revision.Sep 8 2018, 21:06
This revision is now accepted and ready to land.Sep 8 2018, 21:06
Closed by commit rABC637dce206ad7: Simplify IsBanned code from net.cpp (authored by deadalnix). · Explain WhySep 9 2018, 10:33
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
18 lines

Diff 4816

View Options

src/net.cpp

Loading...