Page MenuHomePhabricator

Backport base58 tests from PR11167
ClosedPublic

Authored by deadalnix on Jul 1 2019, 21:57.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
pr11167
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6650
Build 11347: Bitcoin ABC Buildbot (legacy)
Build 11346: arc lint + arc unit

Event Timeline

Fabien requested changes to this revision.Jul 2 2019, 06:53
Fabien added a subscriber: Fabien.
Fabien added inline comments.
src/test/base58_tests.cpp
16 ↗(On Diff #9886)

Revert

20 ↗(On Diff #9886)

Dito

206 ↗(On Diff #9886)

As you are looping over the various chains, I don't understand the added in mainnet ?

Edit: https://github.com/bitcoin/bitcoin/pull/11167#discussion_r140595269

209 ↗(On Diff #9886)

Dito

This revision now requires changes to proceed.Jul 2 2019, 06:53
This revision is now accepted and ready to land.Jul 3 2019, 14:17