Page MenuHomePhabricator

Merge #14985: test: Remove thread_local from test_bitcoin

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



PR14985 backport
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

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.