BIP388: fix variable name in from_descriptor() to prevent NameError #1952

pull MozirDmitriy wants to merge 1 commits into bitcoin:master from MozirDmitriy:mzd changing 1 files +1 −1
  1. MozirDmitriy commented at 7:22 am on September 4, 2025: contributor
    Replace unintended reference to local variable name desc with the correct method parameter descriptor in from_descriptor. The previous code raised a NameError when scanning operators because desc is not defined in that scope. This change aligns with the rest of the method, which consistently uses descriptor, and allows the example in main to run without error.
  2. fix(wallet_policies): use descriptor instead of desc to prevent NameError 25ffcfcf36
  3. jonatack renamed this:
    fix(wallet_policies): use descriptor instead of desc to prevent NameError
    BIP388: fix variable name in from_descriptor() to prevent NameError
    on Sep 4, 2025
  4. jonatack added the label Bug fix on Sep 4, 2025
  5. jonatack approved
  6. jonatack commented at 8:33 pm on September 4, 2025: member

    ACK 25ffcfcf36ae068e47a639ad9bc36aefa4cfc1a5

    This appears to be a typo, as no desc is defined in that function.

  7. jonatack merged this on Sep 4, 2025
  8. jonatack closed this on Sep 4, 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-09-13 09:10 UTC

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