diff options
author | MarcoFalke <*~=`'#}+{/-|&$^_@721217.xyz> | 2023-02-06 10:32:39 +0100 |
---|---|---|
committer | MarcoFalke <*~=`'#}+{/-|&$^_@721217.xyz> | 2023-02-06 10:32:55 +0100 |
commit | aff75463e2c75a7a969cdaa535c7be05fd146420 (patch) | |
tree | 869bae708f709b4ad2517477f3705e9a8a829bf7 /src/wallet/feebumper.cpp | |
parent | d8f982603722590f6be7578591390bcc8a459c18 (diff) | |
parent | b8032293e67a3b61ecad531412be5330f7cb39e3 (diff) |
Merge bitcoin/bitcoin#27036: test: Remove last uses of snprintf and simplify
b8032293e67a3b61ecad531412be5330f7cb39e3 Remove use of snprintf and simplify (John Moffett)
Pull request description:
These are the only remaining uses of `snprintf` in our project, and they can cause unexpected issues -- for example, see https://github.com/bitcoin/bitcoin/issues/27014. Change them to use our `ToString` (which uses a locale-independent version of `std::to_string`) to convert an `int` to `std::string`. Also remove resulting unused parts of `StringContentsSerializer`.
Closes https://github.com/bitcoin/bitcoin/issues/27014
ACKs for top commit:
Sjors:
tACK b8032293e67a3b61ecad531412be5330f7cb39e3, fixes #27014.
Tree-SHA512: c903977e654711929decafe8887d0de13b38a340d7082875acc5d41950d834dcfde074e9cabecaf5f9a760f62c34322297b4b156af29761650ef5803b1a54b59
Diffstat (limited to 'src/wallet/feebumper.cpp')
0 files changed, 0 insertions, 0 deletions