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

Remove IReadOnlyList implementation from BlockChain<T> #606

Closed
earlbread opened this issue Oct 23, 2019 · 0 comments · Fixed by #630
Closed

Remove IReadOnlyList implementation from BlockChain<T> #606

earlbread opened this issue Oct 23, 2019 · 0 comments · Fixed by #630

Comments

@earlbread
Copy link
Member

@earlbread earlbread commented Oct 23, 2019

Currently, BlockChain<T> implements IReadOnlyList<T>. This allows BlockChain<T> to use Linq operations. However, when the number of blocks increases, operations that traverse the entire blocks should not be used. So we better leave out the IReadOnlyList <T> implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
1 participant
You can’t perform that action at this time.