code-Update minisketch.py #1725

pull romashka-btc wants to merge 1 commits into bitcoin:master from romashka-btc:fix/err changing 1 files +1 −1
  1. romashka-btc commented at 6:33 pm on December 18, 2024: none

    Fix: Replaced unused loop variable with _ for clarity

    Changes

    • bip-0330/minisketch.py:106: Replaced the unused loop index i with _.

      • Before:

        0for i in range(FIELD_BITS - 1):
        1    out += [0, a]
        
      • After:

        0for _ in range(FIELD_BITS - 1):
        1    out += [0, a]
        

    Purpose

    • Improved code clarity by explicitly marking the loop variable as unused.
    • Followed Python best practices for clean and maintainable code.
  2. code-Update minisketch.py 9837f17328
  3. jonatack commented at 6:50 pm on December 18, 2024: member
    Thank you, but this change isn’t worth asking the BIP authors for their review time. Closing.
  4. jonatack closed this on Dec 18, 2024

  5. jonatack commented at 7:05 pm on December 18, 2024: member
    (If I may make a suggestion, don’t hesitate to provide thoughtful review on currently open pull requests.)

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: 2024-12-21 16:10 UTC

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