rust-bitcoin / rust-bitcoin / 13706706721

83%

push

github

Merge rust-bitcoin/rust-bitcoin#4092: Add `Timestamp` newtype

<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/b3f122b3996c1a73479be2f95b7f2ae642c9c56f">b3f122b39 Add Timestamp newtype (Tobin C. Harding)

Pull request description:

  Bitcoin block headers have a timestamp. Currently we are using a `u32`. While this functions correctly it gives the compiler no chance to enforce type safety.

  Add a `Timestamp` newtype that is a thin wrapper around a `u32`. Document it and test the API surface in `api.rs`.

ACKs for top commit:
  apoelstra:
    ACK b3f122b3996c1a73479be2f95b7f2ae642c9c56f; successfully ran local tests
  Kixunil:
    ACK b3f122b3996c1a73479be2f95b7f2ae642c9c56f

Tree-SHA512: 6f4a4a5887a00612775d8ec148fff30b78a

28 of 33 new or added lines in 6 files covered. (84.85%)

21557 of 25970 relevant lines covered (83.01%)

16774.55 hits per line