Page MenuHomePhabricator

Merge #14985: test: Remove thread_local from test_bitcoin
ClosedPublic

Authored by markblundeberg on Jul 10 2019, 04:04.

Details

Summary

PR14985 backport https://github.com/bitcoin/bitcoin/pull/14985/files
fa61202cae test: Add comment to g_insecure_rand_ctx (MarcoFalke)
fa0d3c4407 test: Undo thread_local g_insecure_rand_ctx (MarcoFalke)

Pull request description:

`thread_local` seems to be highly controversial according to the discussion in #14953, so remove it again from the tests.

Also remove boost::thread_group in the test that uses it, since I am touching it anyway.

Backport note: corresponding parent in ABC was D3485; in Core it was
only five days before that was reverted with this PR.

Test Plan

make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR14985
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6789
Build 11625: Bitcoin ABC Buildbot (legacy)
Build 11624: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jul 10 2019, 14:32