You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
hash160/ripemd160: concrete hash missing in output script #126
Did a double-take there - would be good to make this consistent and document which format is used for clarity (or maybe use the 0x prefix for hex values).
There's also a subtle ~typo. hash160(H) & ripemd160(H) (using upper case H as a variable) ends up producing ASM with <h> (lower case h).
Keeping case sensitive would also be nice.
sha256 and hash256 work as expected.
BTW: I'd rather keep the notation <20>, where 20 is in fact 0x20 when enclosed within angle quotation marks.
Activity
benma commentedon Jan 5, 2023
I also just noticed this inconsistency:
In the translation table for the hash functions, the Bitcoin Script uses decimal
<32>:SIZE <32> EQUALVERIFY SHA256 <h> EQUALWhile the script structure compilation output prints it in hex
<20>:OP_SIZE <20> OP_EQUALVERIFY OP_HASH160 <h> OP_EQUALDid a double-take there - would be good to make this consistent and document which format is used for clarity (or maybe use the
0xprefix for hex values).landabaso commentedon Jan 23, 2023
There's also a subtle ~typo.
hash160(H)&ripemd160(H)(using upper case H as a variable) ends up producing ASM with<h>(lower case h).Keeping case sensitive would also be nice.
sha256 and hash256 work as expected.
BTW: I'd rather keep the notation
<20>, where 20 is in fact 0x20 when enclosed within angle quotation marks.Fixes sipa#126