doc: Clarify usage of checked witness commitment flag #29539

pull fjahr wants to merge 1 commits into bitcoin:master from fjahr:2024-03-checkwitcomment changing 1 files +7 −0
  1. fjahr commented at 9:41 pm on March 2, 2024: contributor

    Follow-up to #29412

    Adds a clarifying comment about the usage of m_checked_witness_commitment based on the conversation in this thread: #29412 (review)

  2. doc: Clarify usage of checked witness flag
    Co-authored-by: maflcko
    c2d6d9fa0d
  3. DrahtBot commented at 9:41 pm on March 2, 2024: contributor

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

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    Reviews

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

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #29540 (validation: Use witness maleation flag for non-segwit blocks by fjahr)

    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.

  4. DrahtBot added the label Docs on Mar 2, 2024
  5. fjahr commented at 9:45 pm on March 2, 2024: contributor
    Actually I think this may not be needed anymore if #29524 is merged, I am just taking a look at this now.
  6. fjahr commented at 11:15 pm on March 10, 2024: contributor
    Closing for now since #29524 made the documented issue obsolete
  7. fjahr closed this on Mar 10, 2024


fjahr 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: 2024-11-23 09:12 UTC

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