diff options
author | fanquake <fanquake@gmail.com> | 2023-08-18 11:21:52 +0100 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2023-08-18 11:38:38 +0100 |
commit | 7bf078f2b7d4a0339d053144b4fb35fe020dac25 (patch) | |
tree | 415c62503f7fbd2674bf8be834181e0ad6413ecb /ci | |
parent | 5eb669024f3121f7e8ddd82351da2d07c61013e5 (diff) | |
parent | fa60fa3b0cba4a30726af8e0e9d1e84e14849eda (diff) | |
download | bitcoin-7bf078f2b7d4a0339d053144b4fb35fe020dac25.tar.xz |
Merge bitcoin/bitcoin#28237: refactor: Enforce C-str fmt strings in WalletLogPrintf()
fa60fa3b0cba4a30726af8e0e9d1e84e14849eda bitcoin-tidy: Apply bitcoin-unterminated-logprintf to spkm as well (MarcoFalke)
faa11434fe38aa82892802adb6d879d112ae1675 refactor: Enable all clang-tidy plugin bitcoin tests (MarcoFalke)
fa6dc57760e0a04dbb2e365ca7ad9fd8171ebfdb refactor: Enforce C-str fmt strings in WalletLogPrintf() (MarcoFalke)
fa244f3321de7884f530bb38493a8d0a0cec86ab doc: Fix bitcoin-unterminated-logprintf tidy comments (MarcoFalke)
Pull request description:
All fmt functions only accept a raw C-string as argument.
There should never be a need to pass a format string that is not a compile-time string literal, so disallow it in `WalletLogPrintf()` to avoid accidentally introducing it.
Apart from consistency, this also fixes the clang-tidy plugin bug https://github.com/bitcoin/bitcoin/pull/26296#discussion_r1286821141.
ACKs for top commit:
theuni:
ACK fa60fa3b0cba4a30726af8e0e9d1e84e14849eda
Tree-SHA512: fa6f4984c50f9b34e850bdfee7236706af586e512d866cc869cf0cdfaf9aa707029c210ca72d91f85e75fcbd8efe0d77084701de8c3d2004abfd7e46b6fa9072
Diffstat (limited to 'ci')
0 files changed, 0 insertions, 0 deletions