doc: fix header guard placement in net_permissions.h #35802

pull Rajkaran-122 wants to merge 1 commits into bitcoin:master from Rajkaran-122:doc-net-permissions-header-guard changing 1 files +3 −2

Rajkaran-122 commented at 8:45 AM on July 25, 2026: none

This PR fixes an unconventional header guard placement in src/net_permissions.h. Previously, the #include directives were placed before the #ifndef BITCOIN_NET_PERMISSIONS_H header guard.

This change moves the #include directives inside the header guard, aligning with standard C++ practices and the rest of the Bitcoin Core codebase.
doc: fix header guard placement in net_permissions.h 187c40bbcd
DrahtBot added the label Docs on Jul 25, 2026
DrahtBot commented at 8:45 AM on July 25, 2026: 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/35802.



Reviews

See the guideline and AI policy for information on the review process. A summary of reviews will appear here.
Rajkaran-122 force-pushed on Jul 25, 2026
Rajkaran-122 force-pushed on Jul 25, 2026
fanquake commented at 9:54 AM on July 25, 2026: member

Thanks, however this will be fixed during the next IWYU changes.
fanquake closed this on Jul 25, 2026

Contributors

Labels