valgrind: remove outdated suppressions #18879

pull fanquake wants to merge 2 commits into bitcoin:master from fanquake:remove_outdated_supp changing 1 files +0 −13

fanquake commented at 3:33 AM on May 5, 2020: member

https://github.com/bitcoin/bitcoin/commit/708e3c7e85a666d5b8da8638a819c0f3973fcca4: Suppress rest_blockhash_by_height should no longer be needed after #18785.

https://github.com/bitcoin/bitcoin/commit/d7120f7f78cda5ed1ab91f83e9b546de68dbee47 : Removes a duplicate Suppress BCLog::Logger::StartLogging() suppression that was added in #17770.
valgrind: remove rest_blockhash_by_height suppression
This should no-longer be necessary after #18785.
708e3c7e85

valgrind : remove duplicate BCLog::Logger suppression

This was added in #17770, but is identical to the supression above.

d7120f7f78

fanquake added the label Tests on May 5, 2020
MarcoFalke commented at 11:17 AM on May 5, 2020: member

ACK d7120f7f78cda5ed1ab91f83e9b546de68dbee47
practicalswift commented at 11:41 AM on May 5, 2020: contributor

ACK d7120f7f78cda5ed1ab91f83e9b546de68dbee47 -- patch looks correct and valgrind Travis job is happy
MarcoFalke merged this on May 5, 2020
MarcoFalke closed this on May 5, 2020
fanquake deleted the branch on May 5, 2020
DrahtBot locked this on Feb 15, 2022

Contributors

Labels