rust-bitcoin / rust-bitcoin / 13182980585

83%

Build Type

push

github

Committed by apoelstra

Commit Message

Merge rust-bitcoin/rust-bitcoin#3989: Do `hashes` file re-org

<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/85e0330d7f057c9fe447bff5fdb6023150ead319">85e0330d7<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/16cf1f7f2a6867b76f5eddb73ab3e9ed4e57853a"> Run the formatter (Tobin C. Harding)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/7be0db730a2295c78f1a28bdd0061b62f1aa9881">7be0db730</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/16cf1f7f2a6867b76f5eddb73ab3e9ed4e57853a"> hashes: Move bench and test code into files (Tobin C. Harding)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/665fa9de991023eedd426445e436ed975efb8aeb">665fa9de9</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/16cf1f7f2a6867b76f5eddb73ab3e9ed4e57853a"> hashes: Pull crypto out into submodule (Tobin C. Harding)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/1bfd1e071a690f7073a3f4340be305c3d49d5f57">1bfd1e071</a> hashes: Make module subdirectories (Tobin C. Harding)

Pull request description:

  This is an attempt at point 3 in https://github.com/rust-bitcoin/rust-bitcoin/pull/3961#issuecomment-2619946074

  However instead of using `api.rs` and `implementation.rs` it uses `crypto.rs` for the cryptography stuff and leaves the rest in `mod.rs`.

  This whole PR is internal changes only. Almost entirely code moves, review is easy if you have your `diff` configured nicely.

ACKs for top commit:
  Kixunil:
    ACK 85e0330d7f057c9fe447bff5fdb6023150ead319
  apoelstra:
    ACK 85e0330d7f057c9fe447bff5fdb6023150ead319; successfully ran local tests; look great! thanks!

Tree-SHA512: e52e6410ef8f5c1850a551c356a0725f13b

Coverage Stats

210 of 247 new or added lines in 8 files covered. (85.02%)

16 existing lines in 2 files now uncovered.

20658 of 25042 relevant lines covered (82.49%)

17381.66 hits per line