Skip to content

blockchain: Remove redundant stake ver calc func.#1087

Merged
davecgh merged 1 commit intodecred:masterfrom
davecgh:blockchain_remove_calcstakeversionbynode
Feb 25, 2018
Merged

blockchain: Remove redundant stake ver calc func.#1087
davecgh merged 1 commit intodecred:masterfrom
davecgh:blockchain_remove_calcstakeversionbynode

Conversation

@davecgh
Copy link
Copy Markdown
Member

@davecgh davecgh commented Feb 25, 2018

This removes the calcStakeVersionByNode in favor of just using calcStakeVersion directly since the latter already takes the node and the former is a simple pass through with no additional handling.

Copy link
Copy Markdown
Member

@dajohi dajohi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

This removes the calcStakeVersionByNode in favor of just using
calcStakeVersion directly since the latter already takes the node and
the former is a simple pass through with no additional handling.
@davecgh davecgh force-pushed the blockchain_remove_calcstakeversionbynode branch from ac2b289 to b661ebd Compare February 25, 2018 21:42
@davecgh davecgh merged commit b661ebd into decred:master Feb 25, 2018
@davecgh davecgh deleted the blockchain_remove_calcstakeversionbynode branch February 25, 2018 21:59
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.

2 participants