Merge #10514: Bugfix: missing == 0 after randrange
ClosedPublic
Actions

Authored by nakihito on Jun 6 2019, 22:18.

Details

Reviewers

jasonbcox
deadalnix
Fabien

Group Reviewers

Restricted Owners Package	(Owns No Changed Paths)
Restricted Project

Commits

rSTAGINGcaf1ca72b3dd: Merge #10514: Bugfix: missing == 0 after randrange
rABCcaf1ca72b3dd: Merge #10514: Bugfix: missing == 0 after randrange

Summary

9aa215b Bugfixes: missing == 0 after randrange (Pieter Wuille)

Tree-SHA512: 160657ac09553f23ad7a3966c753a30ba938ce6f7ccfd34a4ef0d05d73d712362f7eef97e44a96e37a181b8347caa9d8e1584cc4485f69674ab2de3d8a247373

Backport of Core PR10514
https://github.com/bitcoin/bitcoin/pull/10514/

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

PR10514

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 6194
Build 10435: Bitcoin ABC Buildbot (legacy)
Build 10434: arc lint + arc unit

Event Timeline

nakihito created this revision.Jun 6 2019, 22:18

Owners added a reviewer: Restricted Owners Package.Jun 6 2019, 22:18

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 6 2019, 22:18

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B6194: Diff 9216.Jun 6 2019, 22:29

deadalnix accepted this revision.Jun 6 2019, 22:37

This revision is now accepted and ready to land.Jun 6 2019, 22:37

Closed by commit rABCcaf1ca72b3dd: Merge #10514: Bugfix: missing == 0 after randrange (authored by Wladimir J. van der Laan <laanwj@gmail.com>, committed by nakihito). · Explain WhyJun 6 2019, 23:10

This revision was automatically updated to reflect the committed changes.

Nico Guiton <nico@bitframe.org> added a commit: rABCcaf1ca72b3dd: Merge #10514: Bugfix: missing == 0 after randrange.

Nico Guiton <nico@bitframe.org> added a commit: rSTAGINGcaf1ca72b3dd: Merge #10514: Bugfix: missing == 0 after randrange.Sep 19 2019, 00:08

Revision Contents
Changeset List

Path

Size

src/

test/

coins_tests.cpp

8 lines

Diff 9216

View Options

Merge #10514: Bugfix: missing == 0 after randrangeClosedPublicActions