rust-bitcoin / rust-bitcoin / 10647897784

83%

push

github

Merge rust-bitcoin/rust-bitcoin#3215: Improve const_assert

<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/3c7c8c44b6e5524a6649655d14d207fd441ebf55">3c7c8c44b Improve const_assert (Tobin C. Harding)

Pull request description:

  Now that we can panic in const context we can improve the `const_assert` macro by adding a message string.

  Original idea by Kix:

    https://github.com/rust-bitcoin/rust-bitcoin/pull/2972#discussion_r1726328228

ACKs for top commit:
  Kixunil:
    ACK 3c7c8c44b6e5524a6649655d14d207fd441ebf55 in the sense that it does what it's supposed to with the only tiny issue being that the `bool` looks weird but not broken in any other way I can think of.
  apoelstra:
    ACK 3c7c8c44b6e5524a6649655d14d207fd441ebf55 successfully ran local tests

Tree-SHA512: 5ff721c00291eb4b04e794ec3fc640d2572

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

19707 of 23818 relevant lines covered (82.74%)

15626.2 hits per line