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.
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-
MozirDmitriy commented at 7:22 AM on September 4, 2025: contributor
-
fix(wallet_policies): use descriptor instead of desc to prevent NameError 25ffcfcf36
- 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 - jonatack added the label Bug fix on Sep 4, 2025
- jonatack approved
-
jonatack commented at 8:33 PM on September 4, 2025: member
ACK 25ffcfcf36ae068e47a639ad9bc36aefa4cfc1a5
This appears to be a typo, as no
descis defined in that function. - jonatack merged this on Sep 4, 2025
- jonatack closed this on Sep 4, 2025
Contributors
Labels