diff options
| author | Wladimir J. van der Laan <laanwj@protonmail.com> | 2021-03-16 11:57:47 +0100 |
|---|---|---|
| committer | Wladimir J. van der Laan <laanwj@protonmail.com> | 2021-03-16 12:06:26 +0100 |
| commit | 3b408d21e0938b08ef8dfc79b45508421638988c (patch) | |
| tree | 094cc03011762f91a33918b6c29f90d2c84e73b8 /src/qt/sendcoinsdialog.cpp | |
| parent | af6ee1754511876198a199005eb0273b3c856e60 (diff) | |
| parent | 76782e560b87dc87296144aeb4ee85eb3763b8e9 (diff) | |
| download | bitcoin-3b408d21e0938b08ef8dfc79b45508421638988c.tar.xz | |
Merge #21438: test: add ParseUInt8() test coverage
76782e560b87dc87296144aeb4ee85eb3763b8e9 refactor: reuse test string with embedded null char in util_tests (Jon Atack)
24c6546946f28951aaef7631451a8490888a92b2 test: add ParseUInt8() unit and fuzz test coverage (Jon Atack)
Pull request description:
We have unit test and fuzzer coverage for
- `ParseInt64()`
- `ParseInt32()`
- `ParseUInt64()`
- `ParseUInt32()`
but not `ParseUInt8()`, so this pull adds it.
I was tempted to add a commit that applies clang formatting to the file, or one that updates the C-style casts to named casts, but resisted the temptation unless reviewers request it.
ACKs for top commit:
laanwj:
Code review ACK 76782e560b87dc87296144aeb4ee85eb3763b8e9
MarcoFalke:
cr ACK 76782e560b87dc87296144aeb4ee85eb3763b8e9
Tree-SHA512: 1d7948b3385632094a3b0f0e38f87dccddabf74002e68aa055a51408866b057828ffa15c4b22aa9adde458155fbb5e443b66a9dbf3d7713358fc98a14d64bdcf
Diffstat (limited to 'src/qt/sendcoinsdialog.cpp')
0 files changed, 0 insertions, 0 deletions