Use two boolean literals instead of re-using variable
ClosedPublic
Actions

Authored by jasonbcox on Feb 27 2019, 21:51.

Details

Reviewers

deadalnix
Fabien

Group Reviewers

Restricted Project

Commits

rSTAGING9bae8c8bf6f1: Use two boolean literals instead of re-using variable
rABC9bae8c8bf6f1: Use two boolean literals instead of re-using variable

Summary

Partial backport of Core PR10898 (commit 4971a9a)
https://github.com/bitcoin/bitcoin/pull/10898/commits/4971a9a3c928235ef4af29a7f72379cbb567e446

Test Plan

ninja check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr10898-4

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 5100
Build 8263: Bitcoin ABC Buildbot (legacy)
Build 8262: arc lint + arc unit

Event Timeline

jasonbcox created this revision.Feb 27 2019, 21:51

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 27 2019, 21:51

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 B5100: Diff 7514.Feb 27 2019, 22:03

Fabien accepted this revision.Feb 28 2019, 07:27

This revision is now accepted and ready to land.Feb 28 2019, 07:27

Closed by commit rABC9bae8c8bf6f1: Use two boolean literals instead of re-using variable (authored by practicalswift <practicalswift@users.noreply.github.com>, committed by jasonbcox). · Explain WhyMar 1 2019, 00:40

This revision was automatically updated to reflect the committed changes.

jasonbcox added a commit: rSTAGING9bae8c8bf6f1: Use two boolean literals instead of re-using variable.Sep 18 2019, 22:36

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

27 lines

Diff 7514

View Options

Use two boolean literals instead of re-using variableClosedPublicActions