search

LLFourn / bdk_core_staging

Staging area for bdk_core initial development
15 stars 12 forks source link

Use bitcoin::Amount everywhere? #96

Open LLFourn opened 1 year ago

LLFourn commented 1 year ago

see comment https://github.com/bitcoindevkit/bdk/issues/823#issuecomment-1354392205

One difficultly with this is I was planning to have the coin selection stuff not have any depends at all.

rajarshimaitra commented 1 year ago

From the description of that issue, it seems we are good with just Amount in the balance, and leave other internal places as it is, like coin_select.

We can do this with a balance() in keychaintracker, which already has rust-bitcoin in its dep.