BIP345: fix OP_SUCCESS188 hex value #1745

1. 
   shesek commented at 4:12 am on January 11, 2025: contributor

   OP_SUCCESS188 is 0xbc, not 0xbb.
2. Fix typo in BIP 345

   `OP_SUCCESS188` is `0xbc`, not `0xbb`.

   c532b53ca2
3. jonatack added the label Bug fix on Jan 11, 2025
4. jonatack approved
5. 
   jonatack commented at 5:59 pm on January 11, 2025: member

   ACK c532b53ca2bfc4fbcf9a8d8e45c061db7ee8ce54, OP_SUCCESS188 is constrained with new rules in this BIP to implement OP_VAULT_RECOVER and 188 is 0xbc.
6. 
   jonatack commented at 6:01 pm on January 11, 2025: member

   Pinging @jamesob and @instagibbs for approval.
7. jonatack added the label Pending acceptance on Jan 11, 2025
8. Scutua approved
9. jonatack renamed this:
   Fix typo in BIP 345
   Fix OP_SUCCESS188 hex vslue of typo in BIP 345
   on Jan 12, 2025
10. jonatack renamed this:
    Fix OP_SUCCESS188 hex vslue of typo in BIP 345
    BIP345: fix OP_SUCCESS188 hex value
    on Jan 12, 2025
11. murchandamus approved
12. 
    murchandamus commented at 7:48 pm on January 13, 2025: contributor

    I was wondering whether it was obvious whether OP_SUCCESS187 was meant and 0xbb was correct, or whether OP_SUCCESS188 was correct and 0xbc was the typo.

    As the line corrected in this PR follows another prior mention that introduces the two new opcodes, and the introductions of the paragraphs repeat except in the opcode name, it seems clear that the lower mention is a copy error from the previous header, and I agree that that this is correct.

    The tapscript opcodes OP_SUCCESS187 (0xbb) and OP_SUCCESS188 (0xbc) are constrained with new rules to implement OP_VAULT and OP_VAULT_RECOVER, respectively.

13. 
    murchandamus commented at 7:52 pm on January 13, 2025: contributor

    The reference implementation also confirms this interpretation:

    Merging.

14. murchandamus merged this on Jan 13, 2025
15. murchandamus closed this on Jan 13, 2025

    ---

16. jonatack removed the label Pending acceptance on Jan 13, 2025
17. bitcoin deleted a comment on Jan 15, 2025
18. bitcoin deleted a comment on Jan 15, 2025

Contributors
shesek jonatack Scutua murchandamus

Labels
Bug fix