rust-bitcoin / rust-bitcoin / 9551233748 / 1

83%

master: 83%

push

github

Merge rust-bitcoin/rust-bitcoin#2872: Update PushBytes::read_scriptint(x) to x.read_scriptint()

<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/e7f33a2a1246a14b57b6015ee322fe92bb8d1969">e7f33a2a1<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/0ae6f49cea8276bc62d2d5c6e1fc710a66be81d3"> Update PushBytes::read_scriptint(x) to x.read_scriptint() (Shing Him Ng)

Pull request description:

  Fixes #2849

  There were a few other usages of PushBytes::read_scriptint(_) in the tests such as [this](https://github.com/rust-bitcoin/rust-bitcoin/blob/<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/406e3486ab4c0bee059ed6442dde0cd60ee4277f">406e3486a</a>/bitcoin/src/blockdata/script/tests.rs#L315), but they couldn't be updated as cleanly as the changes in this PR. I wasn't sure if the intention of this issue was to fix those as well, but I can update this PR if needed

ACKs for top commit:
  tcharding:
    ACK e7f33a2a1246a14b57b6015ee322fe92bb8d1969
  Kixunil:
    ACK e7f33a2a1246a14b57b6015ee322fe92bb8d1969

Tree-SHA512: 4620f497249c734cc26121e51d0f53ba579

19581 of 23555 relevant lines covered (83.13%)

16309.3 hits per line