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: Calculate the lottery IV on demand. #1065

Merged
merged 1 commit into from
Feb 21, 2018

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented Feb 21, 2018

This provides a function to calculate the lottery initialization vector for a node on demand versus calculating it at node creation time and storing the result.

This is being done because it saves an additional 32 bytes in the block node structure and the ultimate goal is to move towards having all block nodes in memory to greatly simplify and optimize the code. Since there will be a lot of nodes in memory, these extra bytes can add up to significant savings.

@davecgh davecgh force-pushed the blockchain_calc_lottery_iv branch 3 times, most recently from dde301f to b65b46d Compare February 21, 2018 22:19
Copy link
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

@davecgh davecgh force-pushed the blockchain_calc_lottery_iv branch from b65b46d to 0523885 Compare February 21, 2018 22:41
This provides a function to calculate the lottery initialization vector
for a node on demand versus calculating it at node creation time and
storing the result.

This is being done because it saves an additional 32 bytes in the block
node structure and the ultimate goal is to move towards having all block
nodes in memory to greatly simplify and optimize the code.  Since there
will be a lot of nodes in memory, these extra bytes can add up to
significant savings.
@davecgh davecgh force-pushed the blockchain_calc_lottery_iv branch from 0523885 to 17f957f Compare February 21, 2018 22:45
@davecgh davecgh merged commit 17f957f into decred:master Feb 21, 2018
@davecgh davecgh deleted the blockchain_calc_lottery_iv branch February 21, 2018 22:58
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