tests: Fix cuckoocache_tests TSAN failure introduced in 14935. #14969
pull practicalswift wants to merge 1 commits into bitcoin:master from practicalswift:cuckoocache_tests changing 2 files +6 −3-
practicalswift commented at 9:51 am on December 15, 2018: contributor
-
in src/test/cuckoocache_tests.cpp:224 in c07910b184 outdated
220@@ -221,7 +221,7 @@ static void test_cache_erase_parallel(size_t megabytes) 221 size_t start = ntodo*x; 222 size_t end = ntodo*(x+1); 223 for (uint32_t i = start; i < end; ++i) 224- BOOST_CHECK(set.contains(hashes[i], true)); 225+ set.contains(hashes[i], true);
sipa commented at 9:53 am on December 15, 2018:This is a statement with no effectfanquake added the label Tests on Dec 15, 2018ken2812221 commented at 12:52 pm on December 15, 2018: contributorCould use assert insteadfanquake renamed this:
Fix cuckoocache_tests TSAN failure introduced in #14935. Fixes #14967.
tests: Fix cuckoocache_tests TSAN failure introduced in 14935.
on Dec 15, 2018Fix cuckoocache_tests TSAN failure introduced in #14935. Fixes #14967. d98a29ec40practicalswift commented at 3:20 pm on December 15, 2018: contributor@sipa @ken2812221 Updated. Please re-review :-)laanwj commented at 6:48 pm on December 15, 2018: memberutACK (but squash please)practicalswift force-pushed on Dec 15, 2018practicalswift commented at 7:40 pm on December 15, 2018: contributor@laanwj Squashed! :-)Empact commented at 7:50 pm on December 15, 2018: memberutACK d98a29eken2812221 commented at 6:40 am on December 16, 2018: contributorutACK d98a29ec408590e54f405a7f8d232cd9dc5b14dalaanwj merged this on Dec 16, 2018laanwj closed this on Dec 16, 2018
laanwj referenced this in commit 3424171685 on Dec 16, 2018deadalnix referenced this in commit a06ba8d57a on Jun 22, 2020practicalswift deleted the branch on Apr 10, 2021dzutto referenced this in commit f339f3de57 on Sep 24, 2021dzutto referenced this in commit 9d408b364a on Sep 24, 2021dzutto referenced this in commit eb0a3d833d on Sep 24, 2021dzutto referenced this in commit 298a64983f on Sep 27, 2021dzutto referenced this in commit 6e91a22d80 on Sep 30, 2021UdjinM6 referenced this in commit 4c72a1a975 on Oct 1, 2021kittywhiskers referenced this in commit 1983b3602d on Oct 12, 2021DrahtBot locked this on Aug 18, 2022
practicalswift sipa ken2812221 laanwj EmpactLabels
Tests
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 15:12 UTC
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 15:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me