diff options
author | MarcoFalke <falke.marco@gmail.com> | 2021-10-04 15:05:55 +0200 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2021-10-04 15:06:37 +0200 |
commit | 42fedb4acd3cfa813059fcc3f96b2a41f78d9074 (patch) | |
tree | 9fe2a104daa2bc50d0b7d1d3b2c56edb5a2de779 /ci | |
parent | cdb4dfcbf1c82ef4c4eb7bb982b35219d5dbf827 (diff) | |
parent | fa9d72a7947d2cff541794e21e0040c3c1d43b32 (diff) | |
download | bitcoin-42fedb4acd3cfa813059fcc3f96b2a41f78d9074.tar.xz |
Merge bitcoin/bitcoin#23156: refactor: Remove unused ParsePrechecks and ParseDouble
fa9d72a7947d2cff541794e21e0040c3c1d43b32 Remove unused ParseDouble and ParsePrechecks (MarcoFalke)
fa3cd2853530c86c261ac7266ffe4f1726fe9ce6 refactor: Remove unused ParsePrechecks from ParseIntegral (MarcoFalke)
Pull request description:
All of the `ParsePrechecks` are already done by `ToIntegral`, so remove them from `ParseIntegral`.
Also:
* Remove redundant `{}`. See https://github.com/bitcoin/bitcoin/pull/20457#discussion_r720116866
* Add missing failing c-string test case
* Add missing failing test cases for non-int32_t integral types
ACKs for top commit:
laanwj:
Code review ACK fa9d72a7947d2cff541794e21e0040c3c1d43b32, good find on ParseDouble not being used at all, and testing for behavior of embedded NULL characters is always a good thing.
practicalswift:
cr ACK fa9d72a7947d2cff541794e21e0040c3c1d43b32
Tree-SHA512: 3d654dcaebbf312dd57e54241f9aa6d35b1d1d213c37e4c6b8b9a69bcbe8267a397474a8b86b57740fbdd8e3d03b4cdb6a189a9eb8e05cd38035dab195410aa7
Diffstat (limited to 'ci')
0 files changed, 0 insertions, 0 deletions