Page MenuHomePhabricator

[tests] Use FastRandomContext instead of boost::random::*
ClosedPublic

Authored by Fabien on Mon, Jun 10, 12:08.

Details

Summary
Use FastRandomContext instead of
boost::random::{mt19937,uniform_int_distribution}.

Also include the fix from PR14093 fixing type mismatch (int truncated to
bool).

Backport of core PR10547 and PR14093
https://github.com/bitcoin/bitcoin/pull/10547/files
https://github.com/bitcoin/bitcoin/pull/14093/files

Test Plan
./src/test/test_bitcoin -t scheduler_tests

Diff Detail

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

Event Timeline

Fabien created this revision.Mon, Jun 10, 12:08
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Jun 10, 12:08
Fabien planned changes to this revision.Mon, Jun 10, 12:28
Fabien edited the summary of this revision. (Show Details)Mon, Jun 10, 12:33
Fabien updated this revision to Diff 9286.Mon, Jun 10, 12:34

Squash the fix

jasonbcox accepted this revision.Mon, Jun 10, 16:45
This revision is now accepted and ready to land.Mon, Jun 10, 16:45