Skip to content
This repository has been archived by the owner on Feb 3, 2022. It is now read-only.

Simple refactoring, renaming things to be clearer #39

Closed
nczhu opened this issue Mar 31, 2020 · 0 comments · Fixed by #45
Closed

Simple refactoring, renaming things to be clearer #39

nczhu opened this issue Mar 31, 2020 · 0 comments · Fixed by #45
Assignees
Projects

Comments

@nczhu
Copy link
Collaborator

nczhu commented Mar 31, 2020

Renaming the following:

UnspentOutputs -> UtxoStore: map of hash and transactionOutputs

TransactionInput (s):

  • outpoint : UTXO hash corresponding to tOutputs in utxoStore
  • sigScript : Signature of current Transaction, by the corresponding key as the outpoint; The script contains two components, a signature and a public key. The public key must match the hash given in the script of the redeemed output. The public key is used to verify the redeemers signature, which is the second component. More precisely, the second component is an ECDSA signature over a hash of a simplified version of the transaction.

TransactionOutput (s):

  • value : no change
  • pubKeyScript : the publickey condition in order to spend this utxo

Type names:

  • Un-alias confusing typenames, just use the H512, H256 original naming
  • leftover -> reward
  • initial_utxo -> genesis_utxo
@nczhu nczhu self-assigned this Mar 31, 2020
@nczhu nczhu added this to To do in Kanban Apr 2, 2020
@nczhu nczhu moved this from To do to In progress in Kanban Apr 2, 2020
@nczhu nczhu changed the title Refactor UTXO Runtime (tasks need to be subdivided) Simple refactoring, renaming things to be clearer Apr 5, 2020
@nczhu nczhu moved this from In progress to Pending review in Kanban Apr 5, 2020
@nczhu nczhu linked a pull request Apr 6, 2020 that will close this issue
@nczhu nczhu closed this as completed in #45 Apr 9, 2020
Kanban automation moved this from Pending review to Done Apr 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Kanban
  
Done
Development

Successfully merging a pull request may close this issue.

1 participant