Page MenuHomePhabricator

Remove redundant assignments (dead stores)
ClosedPublic

Authored by fpelliccioni on Oct 1 2019, 15:41.

Details

Summary

Remove redundant assignments (dead stores).

Backport of Bitcoin Core PR13767
https://github.com/bitcoin/bitcoin/pull/13767

Test Plan
ninja check
ninja bench-bitcoin
./src/bench/bitcoin-bench

Diff Detail

Repository
rABC Bitcoin ABC
Branch
feature-backport-fea4e9eca
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7677
Build 13393: Bitcoin ABC Buildbot (legacy)
Build 13392: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Oct 1 2019, 18:35

test plan.

This revision now requires changes to proceed.Oct 1 2019, 18:35
deadalnix requested changes to this revision.Oct 3 2019, 15:44

The test plan doesn't run.

The test plan is meant to describe the step you took to ensure that this patch is good. Clearly, the step described do not match what you have done, as they simply do not work.

This revision now requires changes to proceed.Oct 3 2019, 15:44

The test plan doesn't run.

The test plan is meant to describe the step you took to ensure that this patch is good. Clearly, the step described do not match what you have done, as they simply do not work.

Yes, you are right, I ran the tests on my machine and they worked correctly, but I made a typo when writing it in the test plan.

fpelliccioni edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.Oct 6 2019, 17:56