Use vec! instead of mut and push

As suggested by Clippy, use the `vec!` macro directly instead of
declaring a mutable vector and pushing elements onto it.

Remove useless question mark operator

Clippy emits:

  warning: Question mark operator is useless here

No need to use the `?` operator inside an `Ok()` statement when
returning, just return directly.

Do not compare vtable

Clippy emits error:

 comparing trait object pointers compares a non-unique vtable address

The vtable is an implementation detail, it may change in future. we
should not be comparing vtable addresses for equality. Instead we can
get a pointer to the data field of a fat pointer and compare on that.

Refactor db/batch matching

Remove the TODO; refactor matching to correctly handle conditionally
built `Sled` variants. Use `unreachable` instead of `unimplemented` with
a comment hinting that this is a bug, this makes it explicit, both at
runtime and when reading the code, that this match arm should not be hit.

Conditionally compile constructor

The `ChunksIterator` constructor is only used when either `electrum` or
`esplora` features are enabled. Conditionally build it so that we do not
get a clippy warning when building without these features.

Allow mutex_atomic

 Clippy complains about use of a mutex, suggesting we use an
 `AtomicUsize`. While the same functionality _could_ be achieved using an
 `AtomicUsize` and a CAS loop it makes the code harder to reason about
 for little gain. Lets just quieten clippy with an allow attribute and
 document why we did so.

Use default pattern

Clippy emits warning:

  warning: field assignment outside of initializer for an instance
  created with Default::default()

Do as suggested by clippy and use the default init pattern.

```
    let foo = Foo {
     bar: ...,
        Default::default()
    }
```

Use next instead of nth(0)

As suggested by clippy we can use `.next()` on an iterator instead of
`nth(0)`. Although arguably no clearer, and certainly no worse, it keeps
clippy quiet and a clean lint is a good thing.

Use count instead of collect and len

Clippy emits warning:

warning: avoid using `collect()` when not needed

As suggested by clippy just use `count` directly on the iterator instead
of `collect` followed by `len`.

Remove needles explicit reference

Clippy emits warning:

warning: needlessly taken reference of both operands

Remove the explicit reference's as suggested.

Remove redundant calls to clone

No need to clone copy types, found by clippy.

Remove static lifetime

const str types do not need an explicit lifetime, remove it. Found by
clippy.

Use ! is_empty instead of len > 0

As directed by clippy use `!a.is_empty()` instead of `a.len() > 0`.

Use `unwrap_or_else`

As directed by clippy use `unwrap_or_else` in order to take advantage of
lazy evaluation.

[ci] Rename MAGICAL_ env vars to BDK_, for tests use wallet name in RPC calls

[ci] Update start-core.sh to create default wallet for bitcoind 0.21.0

Merge branch 'release/0.4.0'

Fix CHANGELOG 'Unreleased' link

Bump version to 0.4.1-dev

Bump version to 0.4.0

Update 'bdk-macros', 'bdk-testutils', 'bdk-testutils-macros' dep versions

Bump 'bdk-macros' version to 0.3.0

Bump 'bdk-testutils' version to 0.3.0

Bump 'bdk-testutils-macros' version to 0.3.0

Small CHANGELOG cleanup

[descriptor] Perform additional checks before using a descriptor

Fixes #287

Merge commit 'refs/pull/290/head' of github.com:bitcoindevkit/bdk

Don't fix tokio minor version

This is also what they give as an example in their docs: https://docs.rs/tokio/1.2.0/tokio/

Don't fix tokio minor version

This is also what they give as an example in their docs: https://docs.rs/tokio/1.2.0/tokio/

[ci] Manually set rust stable version in CI pipeline

[policy] Allow specifying a policy path for `Multisig`

While technically it's not required since there are no timelocks inside,
it's still less confusing for the end user if we allow this instead of
failing like we do currently.

[policy] Remove the `TooManyItemsSelected` error

The `TooManyItemsSelected` error has been removed, since it's not technically an
error but potentailly more of an "over-constraint" over a tx: for instance,
given a `thresh(3,pk(a),pk(b),older(10),older(20))` descriptor one could create
a spending tx with the `[0,1,2]` items that would only be spendable after `10`
blocks, or a tx with the `[0,2,3]` items that would be spendable after `20`.

In this case specifying more items than the threshold would create a tx with
the maximum constraint possible, in this case the `20` blocks. This is not
necessarily an error, so we should allow it without failing.

Merge branch 'master' into release/0.4.0

Merging in fixes for the CI after Rust 1.50.0

Rename ToDescriptorKey to IntoDescriptorKey

Rename ToWalletDescriptor to IntoWalletDescriptor

Update CHANGELOG for rust 1.50.0 clippy changes

Fix clippy warn on compact_filters peer::_recv()

Fix clippy warning 'wrong_self_convention'

Fix clippy warning 'unnecessary_wraps'

Update version in the examples

Update CHANGELOG for release v0.4.0

Bump version to 0.4.0-rc.1

compact filters balance example

Merge commit 'refs/pull/278/head' of github.com:bitcoindevkit/bdk

Properly handle the `Signet` network

Closes #62

[compact_filters] Use the new rust-bitcoin API

Update changelog

Update bitcoin, miniscript, electrum-client

Update bitcoin, miniscript, electrum-client

Merge commit 'refs/pull/274/head' of github.com:bitcoindevkit/bdk

Un-pin the version of `cc`

Fixes #183

Emit specific compile error if incompatible features are enabled

This is motivated by the feature `electrum` being part of the
`default` features of this crate. It is easy to naively enable
`esplora` and `async-interface` and forget that `electrum` is enabled
by default, running into not so obvious compile errors.

Merge branch 'master' into make_txbuilder_take_ref_to_wallet

Fix whitespace and curse emacs

Fix CHANGELOG to mention s/utxos/add_utxos/

Merge commit 'refs/pull/270/head' of github.com:bitcoindevkit/bdk

[keys] Improve the API of `DerivableKey`

A new `ExtendedKey` type has been added, which is basically an enum of
`bip32::ExtendedPubKey` and `bip32::ExtendedPrivKey`, with some extra metadata
regarding the `ScriptContext`.

This type has some methods that make it very easy to extract its content as
either an `xprv` or `xpub`.

The `DerivableKey` trait has been updated so that the user now only has to
implement a method (`DerivableKey::into_extended_key()`) to perform the
conversion into an `ExtendedKey`.

The method that was previously called `add_metadata()` has now been renamed
to `into_descriptor_key()`, and it has
a blanket implementation.

Make maintain_single_recipient return a Result

preferable to panicking.

[ci] Update rust toolchains

[signer] Add `Signer::id()`

Closes #261

Fix the crate version in `src/lib.rs`

Add a function to return the version of BDK at runtime

Merge branch 'master' into make_txbuilder_take_ref_to_wallet

Merge commit 'refs/pull/174/head' of github.com:bitcoindevkit/bdk

Add "add_utxos" method on TxBuilder

To replace the previously existing ".utxos"

Remove not very useful comment

Thanks @tcharding.

Derive Clone + Debug for TxBuilder

And make Wallet Debug while I'm at it.

Revert back to Vec to hold utxos in builder

Due to brain malfunction I made utxos into a BTree. This made a test
pass but is incorrect. The test itself was incorrect as per comment in

https://github.com/bitcoindevkit/bdk/pull/258#issuecomment-758370380

So I (1) reverted utxos back to a Vec, (2) fixed the test and expanded
the comment in the test.

Remove Option trickery from TxBuilder API

see: https://github.com/bitcoindevkit/bdk/pull/258#issuecomment-754685962

Assert that `.finish()` hasn't been called already in coin_selection

Add comment explaining why params and coin_selection are Options

Fix wrong doc links

Make testutils dependency path relative

Update Changelog for Tx creation overhaul

[wallet] Use doctest_wallet!() to remove some no_runs from doctests

...and improve the fee bumping example while trying to make it
no_run (but failed).

[wallet] Overhaul TxBuilder internals and externals

Fixes #251

TxBuilders are now not created directly but are created through the
wallet with `build_tx` and `build_fee_bump`.
The advantages of this realised in this commit are:

1. Normal tx creation and fee bumping use the code internally. The only
difference between normal tx and fee bump is how the builder is created.
2. The TxBuilder now has a refernce to the wallet and can therefore
lookup things as methods are called on it. `add_utxo` now uses this to
look up UTXO deta when it is called (rather than having to do it and
possibly error later on).

To support these changes `get_utxo` and `get_descriptor_for_keychain`
public methods have been added to Wallet. I could have kept them
pub(crate) but they seem like fine APIs to have publicly.

Change "received_tx" into "populate_test_db" macro

A `[cfg(test)]` function is not as helpful as a macro since it can't be
called in the context of a doctest.

Also adds doctest_wallet macro which can be used to create a wallet in a
doctest.

Merge branch 'release/0.3.0'

Bump version to 0.3.1-dev

Bump version to 0.3.0

Update CHANGELOG.md in preparation of tag `v0.3.0`

Use contains combinator

As suggested by clippy, use the `contains` combinator instead of doing
manual range check on floats.

Use contains combinator

As suggested by clippy, use the `contains` combinator instead of doing
manual range check on floats.

Add full stops to list items

Super anal patch to make list items uniform, add full stop to the items
where it is missing.

Remove unexplainable newlines

It seems the documentation of this project uses arbitrarily long
lines (i.e. no set column width) along with the occasional newline
before some sentences (within a paragraph). When to split a sentence
onto a newline does not seem to follow any discernible pattern.

There are a few instances of newline characters appearing randomly in
the middle of a sentence and since, as observed above, there is no
fixed column width is use these new lines are out of place.

Remove them so the documentation is slightly more uniform and nice to
read in an editor.

This patch is whitespace only, no other textual changes.

Do minor grammar fix

[wallet] Add "needed" and "available" metadata to Error::InsufficientFunds

Update CHANGELOG

[wallet] Add "needed" and "available" metadata to Error::InsufficientFunds

[blockchain] Upgrade tokio

- Also upgrade reqwest
- Switch to `tokio::runtime::Builder::new_single_thread()` because
`tokio::runtime::Runtime::new()` changed it's behavior to create a
multithreaded runtime.
- `enable_all` enables time and io resource drivers as explained
[here](https://docs.rs/tokio/0.2.24/tokio/runtime/index.html#resource-drivers)

Bump version to 0.3.0-rc.1

Merge commit 'refs/pull/264/head' of github.com:bitcoindevkit/bdk

descriptor: Use `DescriptorError` instead of `Error` when reasonable

Change the return type of the `descriptor!()` macro and `ToWalletDescriptor` to
avoid having to map errors.

Also introduce more checks to validate descriptors built using the macro.

Permit to not set timeout in ElectrumBlockchainConfig

Allowing to use socks5 which requires None timeout

Fix crates.io license info

Merge commit 'refs/pull/260/head' of github.com:bitcoindevkit/bdk

Merge commit 'refs/pull/248/head' of github.com:bitcoindevkit/bdk

Merge commit 'refs/pull/257/head' of github.com:bitcoindevkit/bdk

[ci] Fail 'Build docs' job if warnings

Remove 'cli.rs' module, 'cli-utils' feature and 'repl.rs' example