kernel: make blockTip index const #33321

pull stickies-v wants to merge 1 commits into bitcoin:master from stickies-v:2025-09/kernel-blocktip-const changing 4 files +4 −4
  1. stickies-v commented at 3:01 PM on September 5, 2025: contributor

    Notification interface subscribers need to view, but not mutate, the index.

    This change allows improving the #30595 kernel interface, see e.g. BlockTreeEntry where currently a View is constructed from a non-const pointer, whereas really this should be a const btck_BlockTreeEntry* entry.

  2. kernel: make blockTip index const
    Notification interface subscribers need to view, but not mutate,
    the index.
    75d9b72475
  3. DrahtBot added the label Validation on Sep 5, 2025
  4. DrahtBot commented at 3:01 PM on September 5, 2025: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

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

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK TheCharlatan, l0rinc, yuvicc, achow101

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #30595 (kernel: Introduce initial C header API by TheCharlatan)

    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.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  5. TheCharlatan approved
  6. TheCharlatan commented at 3:16 PM on September 5, 2025: contributor

    ACK 75d9b72475708ee0da13fb23ef65dcced805b6af

  7. fanquake requested review from theuni on Sep 5, 2025
  8. l0rinc approved
  9. l0rinc commented at 4:21 PM on September 5, 2025: contributor

    Code review ACK 75d9b72475708ee0da13fb23ef65dcced805b6af

  10. yuvicc commented at 7:43 AM on September 6, 2025: contributor

    Code review ACK 75d9b72475708ee0da13fb23ef65dcced805b6af

  11. achow101 commented at 8:10 PM on September 11, 2025: member

    ACK 75d9b72475708ee0da13fb23ef65dcced805b6af

  12. achow101 merged this on Sep 11, 2025
  13. achow101 closed this on Sep 11, 2025

  14. stickies-v deleted the branch on Sep 12, 2025

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-05-09 15:12 UTC

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