rust-bitcoin / rust-bitcoin / 12085606805

83%

push

github

Merge rust-bitcoin/rust-bitcoin#3677: units: Implement `iter::Sum` for all types that implement `ops::Add`

<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/433f70939c3ecc10702ab6502e3f9bcd94dab739">433f70939<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/85d1eb8289dfe694c9121b58fbb4c968b0e64308"> Implement iter::Sum for BlockInterval (Tobin C. Harding)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/0369e64b56346f04aaaea60c159761651424f336">0369e64b5</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/85d1eb8289dfe694c9121b58fbb4c968b0e64308"> Implement Sum for an iterator of references to amounts (Tobin C. Harding)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/31f883ac00e9d6887791fb41ac35d3e10b387d58">31f883ac0</a> Implement iter::Sum for FeeRate (Tobin C. Harding)

Pull request description:

  Enables summing an iterator of values. Note that this does not include either `LockTime`s. `absolute::LockTime` should not be added and for `relative::LockTime` we have https://github.com/rust-bitcoin/rust-bitcoin/issues/3676

  Close: #1638

ACKs for top commit:
  apoelstra:
    ACK 433f70939c3ecc10702ab6502e3f9bcd94dab739; successfully ran local tests; nice!
  sanket1729:
    utACK 433f70939c3ecc10702ab6502e3f9bcd94dab739

Tree-SHA512: 1eda00f3babc268da6af30148c70e84d562

16 of 39 new or added lines in 5 files covered. (41.03%)

20183 of 24365 relevant lines covered (82.84%)

17838.17 hits per line