rust-bitcoin / rust-bitcoin / 12069274951

83%

push

github

Merge rust-bitcoin/rust-bitcoin#3663: Fix internal weight calculation

<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/<a class="double-link" href="https://git"><a class=hub.com/rust-bitcoin/rust-bitcoin/commit/f961f3c0eca4e9b175761756f48699012b63d70a">f961f3c0e<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/4123f2c00fff30b0580d3493fbad4a6d91733023"> Add test (yancy)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/335203289201b2e34cb57a6a107547f946475109">335203289</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/4123f2c00fff30b0580d3493fbad4a6d91733023"> Fix internal weight calculation (yancy)

Pull request description:

  A fix for https://github.com/rust-bitcoin/rust-bitcoin/pull/3653/commits/</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/261c8d8ae659d98fd3f63228f16d8f2eb4010b7e">261c8d8ae</a>.  Maybe it's better to roll back that commit instead.  Clearly there needs to be a test or two added here as well since this change causes not test failures.

ACKs for top commit:
  apoelstra:
    ACK f961f3c0eca4e9b175761756f48699012b63d70a; successfully ran local tests
  sanket1729:
    utACK f961f3c0eca4e9b175761756f48699012b63d70a
  tcharding:
    ACK f961f3c0eca4e9b175761756f48699012b63d70a

Tree-SHA512: e6bb6d492bd0c31081bd4eea039e8cfc5fd

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

20139 of 24325 relevant lines covered (82.79%)

17886.03 hits per line