HomePhabricator

Merge #14985: test: Remove thread_local from test_bitcoin

Description

Merge #14985: test: Remove thread_local from test_bitcoin

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

Reviewers: deadalnix, Fabien, #bitcoin_abc

Reviewed By: Fabien, #bitcoin_abc

Differential Revision: https://reviews.bitcoinabc.org/D3628

Details

Provenance
MarcoFalke <falke.marco@gmail.com>Authored on Dec 18 2018, 18:40
Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com>Committed on Jul 10 2019, 15:01
deadalnixPushed on Jul 10 2019, 17:31
Reviewer
Restricted Project
Differential Revision
D3628: Merge #14985: test: Remove thread_local from test_bitcoin
Parents
rSTAGING6509f482538a: Rename wallet_accounts.py test
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/10208