test: Bundle all integer sanitizer suppressions of dependencies #23973

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2201-testSup changing 1 files +27 −27

MarcoFalke commented at 11:15 AM on January 4, 2022: member

And remove three that are no longer needed. Can be reviewed with --color-moved=dimmed-zebra.

Having separate sections for out-of-tree dependencies and in-tree suppressions makes it easier to categorize while reading.
MarcoFalke added the label Refactoring on Jan 4, 2022
MarcoFalke added the label Tests on Jan 4, 2022

test: Bundle all integer sanitizer suppressions of dependencies

And remove three that are no longer needed.
Can be reviewed with --color-moved=dimmed-zebra

fa77dc9ce2

fanquake commented at 3:02 PM on January 4, 2022: member

ACK fa77dc9ce2ba6ab18a0add0ff520db133213ec2c Dropped:

unsigned-integer-overflow:basic_string.h
unsigned-integer-overflow:coded_stream.h
unsigned-integer-overflow:stl_bvector.h

DrahtBot commented at 3:21 PM on January 4, 2022: member


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #23970 (Remove pointless and confusing shift in RelayAddress by MarcoFalke)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
MarcoFalke merged this on Jan 4, 2022
MarcoFalke closed this on Jan 4, 2022
MarcoFalke deleted the branch on Jan 4, 2022
sidhujag referenced this in commit ce0941a16f on Jan 4, 2022
DrahtBot locked this on Jan 4, 2023

Contributors

Labels