rust-bitcoin / rust-bitcoin / 9257889878

83%

push

github

Merge rust-bitcoin/rust-bitcoin#2808: Add an `AddressData` type

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/b5ef7db3c010e9385fe7a3d276f62b6958cfcdd4">b5ef7db3c<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/1741229526432005fa180d03108f78ae72dd0d37"> api: Run just check-api (Tobin C. Harding)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/1b40550ce870850fd1aef906b155bf259ba50a17">1b40550ce</a> Add an AddressData type (Tobin C. Harding)

Pull request description:

  In the 0.32.0 release we removed the `address::Payload` struct because it was deemed an implementation detail. As a byproduct of doing so we made it impossible for users to match on an enum and get the address payload (or data).

  - Add a public `AddressData` enum that holds an address' encoded data.
  - Add a conversion function to `Address` that returns the data enum.

  This patch is additive and is expected to be backported and release as a `0.32` point release.

ACKs for top commit:
  apoelstra:
    ACK b5ef7db3c010e9385fe7a3d276f62b6958cfcdd4 I still feel a little partial to calling the struct "DecodedAddress" and the method "decode"...but this is good, and I do not want to bikeshed

Tree-SHA512: d97836bb205d4765a189294d46b424c135c

0 of 8 new or added lines in 1 file covered. (0.0%)

19320 of 23198 relevant lines covered (83.28%)

16759.71 hits per line