Differential D887

Replace more rand() % NUM by randranges
ClosedPublic
Actions

Authored by deadalnix on Jan 4 2018, 14:41.

Details

Reviewers

schancel

Group Reviewers

Restricted Project

Commits

rSTAGINGf73b598fd20d: Replace more rand() % NUM by randranges
rABCf73b598fd20d: Replace more rand() % NUM by randranges

Summary

As per title.

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

testrandconvert

Lint

Lint Passed

Severity	Location	Code	Message
Auto-Fix	src/test/crypto_tests.cpp:1	CFMT	Code style violation

Unit

No Test Coverage

Build Status

Buildable 1507
Build 1507: arc lint + arc unit

Event Timeline

deadalnix created this revision.Jan 4 2018, 14:41

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 4 2018, 14:41

deadalnix added a child revision: D889: Use randbits instead of ad-hoc emulation in prevector tests.Jan 4 2018, 15:02

schancel accepted this revision.Jan 5 2018, 04:28

This revision is now accepted and ready to land.Jan 5 2018, 04:28

Closed by commit rABCf73b598fd20d: Replace more rand() % NUM by randranges (authored by Pieter Wuille <pieter.wuille@gmail.com>, committed by deadalnix). · Explain WhyJan 5 2018, 12:38

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

test/

6 lines

2 lines

2 lines

2 lines

16 lines

4 lines

8 lines

Diff 2337

View Options

src/test/coins_tests.cpp

View Options

src/test/crypto_tests.cpp

View Options

src/test/merkle_tests.cpp

View Options

src/test/pmt_tests.cpp

View Options

src/test/prevector_tests.cpp

View Options

src/test/sighash_tests.cpp

View Options

src/test/skiplist_tests.cpp

Replace more rand() % NUM by randrangesClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 2337

src/test/coins_tests.cpp

src/test/crypto_tests.cpp

src/test/merkle_tests.cpp

src/test/pmt_tests.cpp

src/test/prevector_tests.cpp

src/test/sighash_tests.cpp

src/test/skiplist_tests.cpp

Replace more rand() % NUM by randranges
ClosedPublic
Actions

Revision Contents
Changeset List