Page MenuHomePhabricator

Remove unused depends list from `getblocktemplate` transactions

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



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/

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

schancel created this revision.Apr 7 2019, 04:31
Owners added a reviewer: Restricted Owners Package.Apr 7 2019, 04:31
Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 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
schancel updated this revision to Diff 7988.Apr 8 2019, 01:49

Update release notes

jasonbcox accepted this revision.Apr 9 2019, 00:01
jasonbcox added inline comments.
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.