Page MenuHomePhabricator

Merge #14816: Add CScriptNum decode python implementation in functional suite
ClosedPublic

Authored by jasonbcox on Jun 24 2020, 19:03.

Details

Summary

2012d4df2 Add CScriptNum decode python implementation in functional suite (Gregory Sanders)

Pull request description:

I needed this for reasons and thought it'd be good to upsteam it.

Tree-SHA512: 6ea89fa2a5f5a7759ba722f2b4ed5cd6423ebfff4e83ac8b8b5c935e6aa479684e626c5f41fa020816d2a9079a99af5564e30808594d5c13e3b51ec9b474926d

Backport of Core PR14816

Test Plan

test_runner.py mining_basic

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

jasonbcox created this revision.

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

majcosta added a subscriber: majcosta.
majcosta added inline comments.
test/functional/mining_basic.py
108–115 ↗(On Diff #21710)

nit: it'd be nice if the linter picked one layout and stuck to it for readability xD

This revision is now accepted and ready to land.Jun 24 2020, 19:40