Page MenuHomePhabricator
Differential D342

Add a Coin abstraction
ClosedPublic
Actions

Authored by deadalnix on Jul 19 2017, 10:32.

Details

Reviewers
freetrader
Group Reviewers
Restricted Project
Commits
rSTAGING73e13e3da72a: Add a Coin abstraction
rABC73e13e3da72a: Add a Coin abstraction
Summary

This is a step toward managing the UTXO on a per output basis instead of a per transaction basis. This has several advantages. First the memory footprint is smaller and we require to read less data from disk. In addition each UTXO will be immutable so optimizations can be made based on that fact.

This is based on work from Pieter Wuille.

Test Plan

Added a unittest

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

deadalnix created this revision.Jul 19 2017, 10:32
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 19 2017, 10:32
deadalnix added a child revision: D343: Update HaveCoins API to use the Coin abstraction..Jul 19 2017, 11:18
deadalnix mentioned this in D346: Update GetCoins API to use the Coin abstraction..Jul 19 2017, 12:46
freetrader accepted this revision.Jul 19 2017, 13:27
This revision is now accepted and ready to land.Jul 19 2017, 13:27
Closed by commit rABC73e13e3da72a: Add a Coin abstraction (authored by deadalnix, committed by deadalnix). · Explain WhyJul 19 2017, 13:51
This revision was automatically updated to reflect the committed changes.
deadalnix mentioned this in rABC7d568aeddb8b: Update GetCoins API to use the Coin abstraction..
deadalnix mentioned this in D348: Uncache UTXO on a per outpoint basis instead of a per transaction basis.Jul 19 2017, 21:38
deadalnix mentioned this in rABC055242781488: Uncache UTXO on a per outpoint basis instead of a per transaction basis.Jul 20 2017, 01:17
deadalnix mentioned this in D349: Update AccessCoins API to use the Coin abstraction.Jul 20 2017, 02:14
deadalnix mentioned this in rABCf701ed902693: Update AccessCoins API to use the Coin abstraction.Jul 20 2017, 13:25
deadalnix mentioned this in D362: Remove version from UTXO and undo datastructures..Jul 21 2017, 14:45
deadalnix mentioned this in D365: Include height and coinbase flag in all txos..Jul 21 2017, 21:45
deadalnix mentioned this in D368: Use the Coin abstraction to undo spends..Jul 22 2017, 15:48
deadalnix mentioned this in D470: Add the API to add/spend coin on a per UTXO basis..Aug 14 2017, 20:36
deadalnix mentioned this in D471: Kill ModifyCoin.Aug 14 2017, 23:55
deadalnix mentioned this in D491: Add SipHashUint256Extra as a specialization of siphash for a 256bits int and a 32 bit one..Aug 26 2017, 19:27
deadalnix mentioned this in rABC42a837318e8c: Add SipHashUint256Extra as a specialization of siphash for a 256bits int and a….Aug 28 2017, 18:09
deadalnix mentioned this in rABC80df982ab2f6: Remove version from UTXO and undo datastructures..Aug 29 2017, 16:43
deadalnix mentioned this in rABCd6cdd9454092: Include height and coinbase flag in all txos..
deadalnix mentioned this in rABC421bf43c7b8a: Use the Coin abstraction to undo spends..
deadalnix mentioned this in rABCacd289859064: Add the API to add/spend coin on a per UTXO basis..
deadalnix mentioned this in rABC2f67fe3e1c25: Kill ModifyCoin.
deadalnix mentioned this in D512: Store the UTXO set on a per output basis rather than a per transaction basis.Aug 30 2017, 21:59
deadalnix mentioned this in rABC611284f473fb: Store the UTXO set on a per output basis rather than a per transaction basis.Sep 6 2017, 19:56
Fabien mentioned this in D8254: Merge CCoinsViewCache's GetOutputFor and AccessCoin.Nov 3 2020, 16:59
Fabien mentioned this in rABC6a02152a2b27: Merge CCoinsViewCache's GetOutputFor and AccessCoin.Nov 4 2020, 12:52

Revision Contents
Changeset List

PathSize
src/
52 lines
test/
59 lines

Diff 867

View Options

src/coins.h

Loading...
View Options

src/test/coins_tests.cpp

Loading...