rust-bitcoin / rust-bitcoin / 10509915686

83%

push

github

Merge rust-bitcoin/rust-bitcoin#3202: Document the new build script magic

<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/39f10339ad205f364a7e622f4be259cd4961a0e8">39f10339a Document the new build script magic (Tobin C. Harding)

Pull request description:

  This is a follow up to #3182 which introduced a new way of conditionally including code based on the compiler version.

  When originally reviewing I missed the fact that the two loops were controlled by the current compiler version (`minor`) so the created macro is different dependent on the compiler used to build the code.

  To help the next guy notice, add a comment.

ACKs for top commit:
  Kixunil:
    ACK 39f10339ad205f364a7e622f4be259cd4961a0e8
  apoelstra:
    ACK 39f10339ad205f364a7e622f4be259cd4961a0e8 successfully ran local tests

Tree-SHA512: 9a223718eed967e157e69d0d50f1a032b6a

19655 of 23759 relevant lines covered (82.73%)

15543.63 hits per line