.
. #33832
pull 777genius wants to merge 1 commits into bitcoin:master from 777genius:refactor/add-missing-include-util-result changing 1 files +1 −0-
777genius commented at 5:04 PM on November 9, 2025: none
-
a96cace30f
refactor: Add missing include in util/result.h
Add missing #include <type_traits> for std::conditional_t and std::is_same_v used in the Result template class. Following the same principle as #33825, headers should directly include standard library headers for types they use in their public interface, rather than relying on transitive includes.
- DrahtBot added the label Refactoring on Nov 9, 2025
-
DrahtBot commented at 5:04 PM on November 9, 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/33832.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process. A summary of reviews will appear here.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
- achow101 closed this on Nov 9, 2025
- bitcoin locked this on Nov 9, 2025
- achow101 renamed this:
refactor: Add missing include in util/result.h
.
on Nov 9, 2025 - fanquake removed the label Refactoring on Nov 10, 2025
