Remove Unused CTransaction tx in wallet.cpp
ClosedPublic
Actions

Authored by nakihito on Dec 11 2019, 21:28.

Details

Reviewers

deadalnix
Fabien
jasonbcox

Group Reviewers

Restricted Owners Package	(Owns No Changed Paths)
Restricted Project

Commits

rSTAGING93156fddade6: Remove Unused CTransaction tx in wallet.cpp
rABC93156fddade6: Remove Unused CTransaction tx in wallet.cpp

Summary

Variable did not get removed in D4675.

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

FixUnusedtx

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 8511
Build 15021: Default Diff Build & Tests
Build 15020: arc lint + arc unit

Event Timeline

nakihito created this revision.Dec 11 2019, 21:28

Owners added a reviewer: Restricted Owners Package.Dec 11 2019, 21:28

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 11 2019, 21:28

Build Bitcoin-ABC / Diffs / Diff Testing started.

nakihito edited the summary of this revision. (Show Details)Dec 11 2019, 21:28

nakihito edited the test plan for this revision. (Show Details)

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B8511: Diff 14783.Dec 11 2019, 21:39

Please look into why this happened. It was there when I reviewed.

This revision is now accepted and ready to land.Dec 11 2019, 21:48

Closed by commit rABC93156fddade6: Remove Unused CTransaction tx in wallet.cpp (authored by nakihito). · Explain WhyDec 11 2019, 21:50

This revision was automatically updated to reflect the committed changes.

nakihito added a commit: rABC93156fddade6: Remove Unused CTransaction tx in wallet.cpp.

nakihito added a commit: rSTAGING93156fddade6: Remove Unused CTransaction tx in wallet.cpp.Apr 7 2020, 22:03

Revision Contents
Changeset List

Path

Size

src/

wallet/

wallet.cpp

2 lines

Diff 14783

View Options

Remove Unused CTransaction tx in wallet.cppClosedPublicActions