Page MenuHomePhabricator

Remove unused depends list from `getblocktemplate` transactions
ClosedPublic

Authored by schancel on Apr 7 2019, 04:31.

Details

Summary

Prior to CTOR, each transaction provided with getblocktemplate included a numerical list
of prior transcations (by index) that the given transaction depended on. (See: BIP022)

However, once CTOR was activated, this list became meaningless as there is no longer a
topological ordering contraint. Worse, since dependent transactions can appear later
in a block, this data does not indicate reliably to clients if there are dependencies.

Finally, the specification allows this value to be omitted. (See BIP022)

This commit, therefore, removes the extraneous and erroreous calculation.

Test Plan

make check && ./test/functional/test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
depends (branched from master)
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5404
Build 8870: Bitcoin ABC Buildbot (legacy)
Build 8869: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Apr 7 2019, 04:31
jasonbcox requested changes to this revision.Apr 7 2019, 04:46
jasonbcox added a subscriber: jasonbcox.

This should be mentioned in release notes.

This revision now requires changes to proceed.Apr 7 2019, 04:46
jasonbcox added inline comments.
doc/release-notes.md
9 ↗(On Diff #7988)

Since these release notes tend to be consumed by non-contributors, I think the diff link should be left out. A git blame can easily provide the diff if needed.

This revision is now accepted and ready to land.Apr 9 2019, 00:01
This revision was automatically updated to reflect the committed changes.