Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

blockchain: Separate tx input stake checks. #1452

Merged

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented Sep 13, 2018

This separates all of the validation checks in CheckTransactionInputs related to each allowed type of stake transaction into separate functions dedicated to each type.

It also renames the utxo view parameter to CheckTransactionInputs to be consistent with the name used in other functions throughout the package.

It does not perform any additional cleanup beyond the separation and does not have any functional changes.

This is work towards #1145.

@davecgh davecgh added this to the 1.4.0 milestone Sep 13, 2018
blockchain/validate.go Outdated Show resolved Hide resolved
blockchain/validate.go Outdated Show resolved Hide resolved
This separates all of the validation checks in CheckTransactionInputs
related to each allowed type of stake transaction into separate
functions dedicated to each type.

It also renames the utxo view parameter to CheckTransactionInputs to be
consistent with the name used in other functions throughout the package.

It does not perform any additional cleanup beyond the separation and
does not have any functional changes.
@davecgh davecgh force-pushed the blockchain_separate_tx_input_stake_checks branch from 50731b9 to 7ae066f Compare September 14, 2018 18:39
@davecgh davecgh merged commit 7ae066f into decred:master Sep 14, 2018
@davecgh davecgh deleted the blockchain_separate_tx_input_stake_checks branch September 14, 2018 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants