Script disassembly for short byte arrays is confusing #7996

issue TheBlueMatt openend this issue on May 3, 2016
  1. TheBlueMatt commented at 8:54 pm on May 3, 2016: member
    ScriptToAsmString will convert any push shorter than 4 bytes to a number before display. This can lead to some confusion if you were expecting a hex-encoded blob as it does for longer-than-4-byte chunks. I’m not sure this is even documented anywhere.
  2. laanwj added the label Docs and Output on May 4, 2016
  3. apoelstra commented at 5:37 pm on May 4, 2016: contributor
    This is stronger than just confusion – for numbers between 10 000 000 and 2^32, it is ambiguous whether they are decimal numbers or hex-encoded data.
  4. laanwj commented at 1:38 pm on May 5, 2016: member
    I think this has always been the case. It’d make some sense if at least the hexadecimal were clearly distinguishable from decimal values, for example with a prefix. But as @apoelstra already says this is now ambiguous.

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-22 00:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me