rust-bitcoin / rust-bitcoin / 15448828568

83%

Build Type

push

github

Committed by apoelstra

Commit Message

Merge rust-bitcoin/rust-bitcoin#4511: Modify `locktime` `serde` implementations

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/4621d2bde14d71b3d6ef0b14258a7479c049ba3b">4621d2bde<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/fa07198f21821fe33e53726da5c1bd3c185f525b"> Modify locktime serde implemenations (Tobin C. Harding)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/200c27631521ad139e2070a04c1cf223791a1277">200c27631</a> bitcoin: Make test code spacing uniform (Tobin C. Harding)

Pull request description:

  Patch 1 is preparatory clean up. Patch 2 is the meat and potatoes. See commit log there for full explanation.

  Briefly:

      - Remove `serde` stuff from `units::locktime`
      - Manually implement `serde` traits on `relative::LockTime`
      - Fix the regression test to use the new format

ACKs for top commit:
  apoelstra:
    ACK 4621d2bde14d71b3d6ef0b14258a7479c049ba3b; successfully ran local tests

Tree-SHA512: bc96d79dda448b31e2c00e8393c9b24090e

Coverage Stats

8 of 15 new or added lines in 2 files covered. (53.33%)

6 existing lines in 1 file now uncovered.

23461 of 27815 relevant lines covered (84.35%)

15957.19 hits per line