BIP388: Fix incorrect use of return for raising exception #1752

pull brawncode wants to merge 1 commits into bitcoin:master from brawncode:patch-1 changing 1 files +1 −1
  1. brawncode commented at 7:50 pm on January 25, 2025: contributor
    return was being used incorrectly to handle an exception. Instead of returning the exception, it should be raised using raise.
  2. fix: Fix incorrect use of return for raising exceptions Update wallet_policies.py 607cac148e
  3. jonatack approved
  4. jonatack commented at 6:19 pm on January 29, 2025: member

    ACK 607cac148e7ca9a22a7784b481d464d894b94623

    This is the only exception of 7 present in this file that is not raised, and without a reason for it that I can see.

  5. jonatack commented at 6:20 pm on January 29, 2025: member
    Pinging @bigspider for approval just in case.
  6. bigspider approved
  7. bigspider commented at 6:25 pm on January 29, 2025: contributor
    Thanks for the fix!
  8. jonatack renamed this:
    fix: Fix incorrect use of return for raising exceptions Update wallet…
    BIP388: Fix incorrect use of return for raising exception
    on Jan 29, 2025
  9. jonatack merged this on Jan 29, 2025
  10. jonatack closed this on Jan 29, 2025


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-02-22 08:10 UTC

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