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: Refactor main block index logic. #990

Merged
merged 1 commit into from
Jan 30, 2018

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented Jan 29, 2018

This requires #989.

This refactors the block index logic into a separate struct and introduces an individual lock for it so it can be queried independent of the chain lock.

It also modifies the newBlockNode function to accept nil for the ticket spend information parameter and updates all of the test code that doesn't require it to use nil.

@davecgh
Copy link
Member Author

davecgh commented Jan 29, 2018

Rebased to deal with the merge of #915 and also modified newBlockNode to accept nil for the new ticket parameter and updates the test code to use nil accordingly.

@davecgh davecgh force-pushed the blockchain_separate_blockindex branch 4 times, most recently from 11798eb to d44733d Compare January 29, 2018 17:56
This refactors the block index logic into a separate struct and
introduces an individual lock for it so it can be queried independent of
the chain lock.

It also modifies the `newBlockNode` function to accept nil for the
ticket spend information parameter and updates all of the test code that
doesn't require it to use nil.
@davecgh davecgh force-pushed the blockchain_separate_blockindex branch from d44733d to 2296b31 Compare January 30, 2018 19:22
@davecgh davecgh merged commit 2296b31 into decred:master Jan 30, 2018
@davecgh davecgh deleted the blockchain_separate_blockindex branch January 30, 2018 19:36
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