(See commit msg)
This should also fix CI issues, such as #35885 (comment)
refactor: Remove unused #include in common/system #35886
pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2608-iwyu changing 2 files +1 −2-
maflcko commented at 8:24 PM on August 4, 2026: member
-
fa7304f3a5
refactor: Remove unused #include in common/system
Now that clamping to std::numeric_limits<size_t>::max() is removed, the limits include is no longer needed. Similarly after changing size_t -> u64, the cstddef include needs to be replaced by cstdint.
- DrahtBot added the label Refactoring on Aug 4, 2026
-
DrahtBot commented at 8:24 PM on August 4, 2026: 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/35886.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline and AI policy for information on the review process.
Type Reviewers ACK hebasto If your review is incorrectly listed, please copy-paste <code><!--meta-tag:bot-skip--></code> into the comment that the bot should ignore.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
- hebasto approved
-
hebasto commented at 8:38 PM on August 4, 2026: member
ACK fa7304f3a5b876eec7116e70bed2599eb0870126.
There were a silent conflict between bitcoin/bitcoin#34995 and bitcoin/bitcoin#35205.
- hebasto merged this on Aug 4, 2026
- hebasto closed this on Aug 4, 2026
- maflcko deleted the branch on Aug 5, 2026
Labels