志宇
志宇
Thanks for the PR! I would argue that the original was clearer because it more directly expressed what was happening, instead of having the reader go through one more layer...
This is a really cool project, cannot wait to see it take off! Just a note about the branch-and-bound provided by the `bdk_coin_select` crate... it is unique in the sense...
cc @nymius
Thanks for the progress on this! I still need to have a proper look. For now, I'm wondering whether we should just depend on the [`http`](https://docs.rs/http/latest/http/) crate for HTTP types....
The persistence test suite will make this work. It's unfortunate we didn't think about that solution earlier.
@icota have you considered using `ChangeSet`s instead? Serializing `CheckPoint`s is a bit odd since it is a linked list and the resultant serialization would be super nested. Also updates may...
@icota Yes. That is a good point about applying it on the other side. I forgot that `bdk_wallet::Wallet` does not have an `apply_changeset` method (whereas the core structures from `bdk_chain`...
This is in-progress and I'm working on it!
So just a quick update on this. Because this affects `bdk_sqlite` crate, I had to start changing that as well. This is why this PR is taking longer than expected....
Okay technically we don't need the `T` since we always append the same type. I propose the following: ## Encoding Encode each element as: ``` [Version: VarInt] [Length: VarInt] [Data:...