rust-bitcoin / rust-bitcoin / 15201971164

83%

push

github

Merge rust-bitcoin/rust-bitcoin#4542: Fix FeeRate::checked_add/sub

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/395252c6d95fdd581ce5e8ad5d7978515aafea23">395252c6d Fix FeeRate::checked_add/sub (Tobin C. Harding)

Pull request description:

  Currently the `checked_add` and `checked_sub` functions use a `u64` for the right hand side. This leaks the inner unit because the RHS value is implicitly in the same unit.

  Make the functions have `FeeRate` on the RHS.

ACKs for top commit:
  apoelstra:
    ACK 395252c6d95fdd581ce5e8ad5d7978515aafea23; successfully ran local tests; will one-ACK merge as this is units-only

Tree-SHA512: 1f0893f75d2724d0feeea78c5275786fca6

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

23524 of 27746 relevant lines covered (84.78%)

16011.36 hits per line