validation: Remove stale BlockManager param from ContextualCheckBlockHeader #35001

pull ismaelsadeeq wants to merge 1 commits into bitcoin:master from ismaelsadeeq:04-2026-remove-redundant-block-man changing 1 files +3 −3
  1. ismaelsadeeq commented at 1:42 am on April 4, 2026: member

    The BlockManager reference in ContextualCheckBlockHeader is unused.

    Fix this by removing it from the function parameter list. Remove BlockManager reference argument in ContextualCheckBlockHeader invocations.

  2. validation: Remove stale BlockManager param in ContextualCheckBlockHeader 851152e42a
  3. DrahtBot added the label Validation on Apr 4, 2026
  4. DrahtBot commented at 1:43 am on April 4, 2026: contributor

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

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK stratospher, maflcko

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

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #34895 (fuzz: Fuzzing harnesses for ActivateBestChainStep and ActivateBestChain by RobinDavid)
    • #30342 (kernel, logging: Pass Logger instances to kernel objects by ryanofsky)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  5. stratospher commented at 7:47 am on April 4, 2026: contributor
    ACK 851152e. not used anymore since checkpoints were removed in 3c5d1a4.
  6. maflcko commented at 8:39 am on April 7, 2026: member
    lgtm ACK 851152e42a4cc5933234a1fc2ab52ba3e389e31c
  7. fanquake merged this on Apr 7, 2026
  8. fanquake closed this on Apr 7, 2026

  9. ismaelsadeeq deleted the branch on Apr 7, 2026

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-04-10 21:13 UTC

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