rust-bitcoin / rust-bitcoin / 14602336468

83%

Build Type

push

github

Committed by apoelstra

Commit Message

Merge rust-bitcoin/rust-bitcoin#4376: primitives: replace error propagation with `expect`

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/<a class="double-link" href="https://git"><a class=hub.com/rust-bitcoin/rust-bitcoin/commit/cf42c511d83b6563065d3cfee12d20f0a9db88b1">cf42c511d primitives: replace error propagation with `expect` (Andrew Poelstra)

Pull request description:

  This error path cannot happen, and if it could, failing formatting is not the correct source of action, because the std docs say that formatting may only fail if the formatter says so.

  Fixes comment in #4269

ACKs for top commit:
  tcharding:
    ACK cf42c511d83b6563065d3cfee12d20f0a9db88b1
  Kixunil:
    ACK cf42c511d83b6563065d3cfee12d20f0a9db88b1

Tree-SHA512: 0ad47bb21bf27133780a7e65236440cfcf9

Coverage Stats

1 of 1 new or added line in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

22609 of 26898 relevant lines covered (84.05%)

16559.9 hits per line