Include height and coinbase flag in all txos.
ClosedPublic
Actions

Authored by deadalnix on Jul 21 2017, 21:45.

Details

Reviewers

freetrader
sickpig
awemany
kyuupichan

Group Reviewers

Restricted Project

Commits

rSTAGINGd6cdd9454092: Include height and coinbase flag in all txos.
rABCd6cdd9454092: Include height and coinbase flag in all txos.

Summary

Right now, only the last UTXO from a transaction contains these information. If we are going to move from a per tx to a per txo model, as started in D342 , then this needs to change.

Test Plan

make check
../qa/pull-tester/rpc-tests.py -extended

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

deadalnix created this revision.Jul 21 2017, 21:45

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 21 2017, 21:45

freetrader accepted this revision.Jul 23 2017, 09:37

This revision is now accepted and ready to land.Jul 23 2017, 09:37

Closed by commit rABCd6cdd9454092: Include height and coinbase flag in all txos. (authored by deadalnix, committed by deadalnix). · Explain WhyAug 29 2017, 16:43

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

undo.h

14 lines

validation.cpp

29 lines

Diff 1267

View Options

src/undo.h