rust-bitcoin / rust-bitcoin / 15198932718

83%

Build Type

push

github

Committed by apoelstra

Commit Message

Merge rust-bitcoin/rust-bitcoin#4539: units: re-order ceil/floor functions

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/1fef5a3dc55bfc1858c32f81f18840ec1d01c807">1fef5a3dc units: re-order ceil/floor functions (Tobin C. Harding)

Pull request description:

  We have 4 functions, 2 ceil, 2 floor but they are ordered ceil, floor, floor, ceil - this causes my head to twitch when I read it.

  The logic in the floor versions is easier so put them first, this is uniform with `fee_rate/mod.rs`.

  Code move only.

ACKs for top commit:
  apoelstra:
    ACK 1fef5a3dc55bfc1858c32f81f18840ec1d01c807; successfully ran local tests; will one-ack merge on the basis that this is trivial

Tree-SHA512: 8f15a34ee08bec01d42f1166511527fe1a5

Coverage Stats

7 of 8 new or added lines in 1 file covered. (87.5%)

1 existing line in 1 file now uncovered.

23523 of 27746 relevant lines covered (84.78%)

15993.36 hits per line