refactor: Remove remaining std::bind, check via clang-tidy #34373

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2601-no-bind changing 5 files +22 −9
  1. maflcko commented at 7:14 pm on January 21, 2026: member

    std::bind has many issues:

    • It is verbose in a meaningless way
    • Overriden args are silently accepted and dropped at runtime without a compile error. Same for accidental duplicates.

    One could use std::bind_front similar to commit fa267551c4eaef577db92e248c4b6d31d0c8bc77. Though, I think the remaining cases are better off with lambdas.

    So do that here, and enable the modernize-avoid-bind clang-tidy rule to avoid std::bind bugs in the future.

  2. refactor: Remove remaining std::bind, check via clang-tidy faa22777fb
  3. DrahtBot added the label Refactoring on Jan 21, 2026
  4. DrahtBot commented at 7:14 pm on January 21, 2026: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/34373.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    Concept ACK hebasto, fjahr

    If your review is incorrectly listed, please copy-paste <!–meta-tag:bot-skip–> into the comment that the bot should ignore.

  5. hebasto commented at 7:16 pm on January 21, 2026: member
    Concept ACK.
  6. fjahr commented at 10:16 pm on January 21, 2026: contributor
    Concept ACK

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-01-22 03:13 UTC

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