doc: warn that CheckBlock() underestimates sigops #31624

pull Sjors wants to merge 1 commits into bitcoin:master from Sjors:2025/01/doc-sigops changing 1 files +1 −0
  1. Sjors commented at 11:08 am on January 9, 2025: member

    Counting sigops in the witness requires context that CheckBlock() does not have, so it only counts sigops for non-segwit transactions.

    It’s useful to document, but it should not be a problem.

  2. doc: warn that CheckBlock() underestimates sigops
    Counting sigops in the witness requires context that CheckBlock()  does not have,
    so it only counts sigops for non-segwit transactions.
    
    It's useful to document, but it should not be a problem.
    0ac19a98f3
  3. DrahtBot commented at 11:08 am on January 9, 2025: 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/31624.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  4. DrahtBot added the label Docs on Jan 9, 2025


Sjors DrahtBot

Labels
Docs


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: 2025-01-21 03:12 UTC

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