fix: bad opcode err msg includes reserved opcode #28234

pull ChrisCho-H wants to merge 1 commits into bitcoin:master from ChrisCho-H:script/bad-opcode-reserved changing 2 files +2 −2
  1. ChrisCho-H commented at 3:22 pm on August 7, 2023: none
    bad opcode err msg includes reserved, because reserved opcode is evaluated as BAD_OPCODE(and it’s wrong to state missing or not undestood for it).
  2. DrahtBot commented at 3:22 pm on August 7, 2023: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  3. jonatack commented at 3:37 pm on August 7, 2023: member
    If this change is correct, the failing tests would need to be updated in the same commit (see https://jonatack.github.io/articles/how-to-contribute-pull-requests-to-bitcoin-core for more details).
  4. fix: bad opcode err msg includes reserved opcode 39888f29e9
  5. ChrisCho-H force-pushed on Aug 7, 2023
  6. DrahtBot added the label CI failed on Aug 7, 2023
  7. DrahtBot removed the label CI failed on Aug 8, 2023
  8. ChrisCho-H commented at 12:21 pm on August 8, 2023: none
    Thx, I updated in same commit.
  9. ChrisCho-H commented at 1:38 pm on August 10, 2023: none
    closed as it would be better to integrate this update here https://github.com/bitcoin/bitcoin/pull/28169
  10. ChrisCho-H closed this on Aug 10, 2023

  11. bitcoin locked this on Aug 9, 2024

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-23 09:12 UTC

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