Closed ordishs closed 1 year ago
Base: 83.70% // Head: 83.70% // No change to project coverage :thumbsup:
Coverage data is based on head (
3743c11
) compared to base (4766f6a
). Patch coverage: 0.00% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
This pull request looks stale. Feel free to reopen it if you think it's a mistake.
When using go-bt with a 32 bit architecture we are checking if len([]byte) > 0xffffffff which can never be the case. This causes a compile warning:
bscript/script.go:432:32: 0xffffffff (untyped int constant 4294967295) overflows int
This check only needs to be done on a 64 bit architecture where the length of a byte slice can be 0xffffffffffffffff, but OP_PUSHDATA4 is the largest of the PUSH opcodes in bitcoin.