fanquake
commented at 4:05 PM on September 21, 2022:
member
We already use a mix of <cstdio> and stdio.h unconditionally throughout
the codebase.
Us checking this header also duplicates work already done by autotools.
Currently stdio.h is checked for 3 times during a ./configure run, after
this change, at least it's only twice.
refactor: use <cstdio> over stdio.h
We currently use both. Consolidate on the former.
b95633121b
build: remove stdio.h from header checks
We already use a mix of <cstdio> and stdio.h unconditionally throughout
the codebase.
Us checking this header also duplicates work already done by autotools.
Currently stdio.h is checked for 3 times during a ./configure run, after
this change, at least it's only twice.
55aad5f3a9
fanquake added the label Build system on Sep 21, 2022
fanquake requested review from TheCharlatan on Sep 21, 2022
kristapsk approved
kristapsk
commented at 8:26 PM on September 21, 2022:
contributor
ACK55aad5f3a957296776cfd8d7a4a251361355b8e8
TheCharlatan
commented at 11:16 AM on September 22, 2022:
contributor
ACK55aad5f3a957296776cfd8d7a4a251361355b8e8
fanquake merged this on Sep 22, 2022
fanquake closed this on Sep 22, 2022
fanquake deleted the branch on Sep 22, 2022
sidhujag referenced this in commit fd278e9de1 on Sep 23, 2022
MarcoFalke referenced this in commit ec13810a00 on Sep 25, 2022
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: 2026-04-26 06:13 UTC
This site is hosted by @0xB10C More mirrored repositories can be found on mirror.b10c.me