Update libmultiprocess subtree to fix clang-tidy errors #32641

pull ryanofsky wants to merge 2 commits into bitcoin:master from ryanofsky:pr/subtree-2 changing 8 files +108 −82
  1. ryanofsky commented at 6:28 pm on May 29, 2025: contributor

    Includes:

    These changes are needed to fix CI errors in #31802.

    The changes can be verified by running test/lint/git-subtree-check.sh src/ipc/libmultiprocess as described in developer notes and lint instructions

  2. Squashed 'src/ipc/libmultiprocess/' changes from 35944ffd23fa..27c7e8e5a581
    27c7e8e5a581 Merge chaincodelabs/libmultiprocess#172: refactor: fix warnings from clang-tidy-20 and bitcoin-tidy
    2fe87d016be4 Merge chaincodelabs/libmultiprocess#173: doc: Fix error string typo
    57a65b854664 clang-tidy: Suppress bitcoin-nontrivial-threadlocal error
    0d8012f656fe Merge chaincodelabs/libmultiprocess#165: clang-tidy: fix warnings introduced in version 19
    3a96cdc18f2d clang-tidy: Fix bugprone-move-forwarding-reference error
    c1e8c1a02864 clang-tidy: Fix bugprone-move-forwarding-reference errors
    aa19285303ff use ranges transform
    a78137ca73b8 make member function const
    ca3226ec8ab7 replace custom tuple unpacking code with `std::apply`
    949fe85fc91f replace SFINAE trick with `if constexpr`
    44ee4b40b89a doc: Fix error string typo
    
    git-subtree-dir: src/ipc/libmultiprocess
    git-subtree-split: 27c7e8e5a581b3c41330e758951251ef11807b11
    154af1eea1
  3. Merge commit '154af1eea1170f5626aa1c5f19cc77d1434bcc9d' into HEAD 9f6565488f
  4. DrahtBot commented at 6:28 pm on May 29, 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/32641.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK TheCharlatan, Sjors

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

  5. TheCharlatan approved
  6. TheCharlatan commented at 7:37 pm on May 29, 2025: contributor
    ACK 9f6565488fc169898abc5f7ee620ce7a9a4e5e50
  7. Sjors commented at 5:20 am on May 30, 2025: member

    ACK 9f6565488fc169898abc5f7ee620ce7a9a4e5e50

    Tested with git-subtree-check.sh -r

    I reviewed https://github.com/bitcoin-core/libmultiprocess/pull/172 myself, but for the other pull requests I only glanced over the final diff here.

    For future subtree updates we may also want to do https://github.com/bitcoin-core/libmultiprocess/issues/178. That would give some assurance that the correct merge policy is followed there.

  8. fanquake merged this on May 30, 2025
  9. fanquake closed this on May 30, 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: 2025-06-15 06:13 UTC

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