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

Implement generics for CheckPoint, LocalChain, and spk_client types #1757

Open
LagginTimes opened this issue Dec 4, 2024 · 1 comment
Open
Assignees
Labels
module-blockchain new feature New feature or request

Comments

@LagginTimes
Copy link
Contributor

Currently, the CheckPoint, LocalChain, and spk_client components are not designed to handle generic types, which limits their flexibility. To support caching of Merkle proofs (#1699) and median time passed calculations, it would be beneficial for these components to accept a generic type, specifically to accommodate Headers.

@evanlinjin
Copy link
Member

I'm rooting for this change. Being able to have CheckPoint<Header> means we can guarantee that we have a Header per checkpoint.

Currently, bdk_electrum_c (the streaming Electrum BDK chain source I'm working on) requires some complexity in the Headers struct to ensure we get the right headers per block.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module-blockchain new feature New feature or request
Projects
Status: Discussion
Development

No branches or pull requests

3 participants