diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2022-02-01 09:42:25 +0100 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2022-02-01 09:42:34 +0100 |
| commit | fcac16fff87b9a949b925831167c2ca5e22eac2c (patch) | |
| tree | 91f3d329ff4a1441d4dd7869d6372b6633265862 /ci/test/wrap-qemu.sh | |
| parent | 8f137e69caeb2a2ffe1aa930bd6fbc49cee4087c (diff) | |
| parent | faa630aa15bbda0f3b0cf3b6f31cf8fdaeb66975 (diff) | |
Merge bitcoin/bitcoin#24190: test: Fix sanitizer suppresions in streams_tests
faa630aa15bbda0f3b0cf3b6f31cf8fdaeb66975 test: Fix sanitizer suppresions in streams_tests (MarcoFalke)
Pull request description:
Two changes (that also make sense on their own) to remove the file-wide sanitizer suppression:
* `FindByte` no longer takes a `char`, but an `uint8_t`, after commit 196b4599201dbce3e0317e9b98753fa6a244b82d.
* The `key` vector of unsigned chars can be removed and inlined as initializer-list. This avoids a bunch of verbose code like `clear()` and `push_back` of `char`s.
ACKs for top commit:
PastaPastaPasta:
utACK faa630aa15bbda0f3b0cf3b6f31cf8fdaeb66975, I have reviewed the changes and agree it makes sense to merge
Tree-SHA512: 747b9d4676fad6d07f3955668639c93333625e69199ff4c499f01167de3875990d93db85e775a7f5b1b684575dceaec8aa000b4db15525fc47b699bac1c85e3d
Diffstat (limited to 'ci/test/wrap-qemu.sh')
0 files changed, 0 insertions, 0 deletions