Page MenuHomePhabricator
Differential D7601

refactor: Remove redundant conditional
ClosedPublic
Actions

Authored by PiRK on Sep 28 2020, 10:26.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABCfd36863a2e4d: refactor: Remove redundant conditional
Summary

This condition fMaster is already guaranteed to be true by the outer if.

Backport of Core PR17971

Test Plan

Trivial change. I just made sure I did not introduce any typo by checking that it compiles and tests still run succesfully.
ninja && ninja check

Diff Detail

Event Timeline

PiRK created this revision.Sep 28 2020, 10:26
Owners added a reviewer: Restricted Owners Package.Sep 28 2020, 10:26
Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 28 2020, 10:26
PiRK requested review of this revision.Sep 28 2020, 10:26
teamcity edited the summary of this revision. (Show Details)Sep 28 2020, 10:26

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

Harbormaster completed remote builds in B12968: Diff 23878.Sep 28 2020, 10:35
deadalnix accepted this revision.Sep 28 2020, 14:34
This revision is now accepted and ready to land.Sep 28 2020, 14:34
Closed by commit rABCfd36863a2e4d: refactor: Remove redundant conditional (authored by Bushstar <bushsolo@gmail.com>, committed by PiRK). · Explain WhySep 28 2020, 14:46
This revision was automatically updated to reflect the committed changes.
PiRK added a commit: rABCfd36863a2e4d: refactor: Remove redundant conditional.

Revision Contents
Changeset List

PathSize
src/
4 lines

Diff 23878

View Options

src/checkqueue.h

Loading...