diff options
| author | fanquake <fanquake@gmail.com> | 2023-03-04 08:17:31 +0100 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2023-03-04 08:17:37 +0100 |
| commit | 40c6c85c05812ee8bf824b639307b1ac17a001c4 (patch) | |
| tree | be8476ad96928b21f58ffbfdf5e0604b472b972f /src/netgroup.h | |
| parent | 236cd231d01e68fdabea1cc7f1c72805d72955ff (diff) | |
| parent | 8847ce44e0713350a6d3524f62eaeb10ba548bae (diff) | |
Merge bitcoin/bitcoin#27192: util: add missing include and fix function signature
8847ce44e0713350a6d3524f62eaeb10ba548bae util: add missing include and fix function signature (Cory Fields)
Pull request description:
ping hebasto
Discovered while testing pre-compiled header support with CMake: https://github.com/theuni/bitcoin/commits/cmake-pch-poc. Compilation of that branch fails without this fix and succeeds with it.
Similar to the fix in #27144.
The problem of having a default argument in the definition was masked by the missing include. Using PCH forces that include, so we end up with the compiler error we should've been getting all along.
ACKs for top commit:
fanquake:
ACK 8847ce44e0713350a6d3524f62eaeb10ba548bae
Tree-SHA512: 5eb9a6691ee37cbc5033a48aedcbf5c93af3b234614ae14c3fcc858f1ee505f630ad68f8bbb69ffa280080c8d0f91451362cb3819290b741ce906b2b3224a622
Diffstat (limited to 'src/netgroup.h')
0 files changed, 0 insertions, 0 deletions