Page MenuHomePhabricator

[refactor] Add a base DescriptorImpl with most common logic
ClosedPublic

Authored by deadalnix on May 21 2020, 13:54.

Details

Summary
Test Plan
ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

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

Fabien requested changes to this revision.May 21 2020, 14:12
Fabien added a subscriber: Fabien.
Fabien added inline comments.
src/script/descriptor.cpp
13 ↗(On Diff #20450)

Macro wildnewlineappeared:

253 ↗(On Diff #20450)

No WPKH

256 ↗(On Diff #20450)

Nor WSH

373 ↗(On Diff #20450)

You lost the braces

548 ↗(On Diff #20450)

Restore

559 ↗(On Diff #20450)

Dito

606 ↗(On Diff #20450)

Dito

620 ↗(On Diff #20450)

Braces

622 ↗(On Diff #20450)

Restore C++ constructor

759 ↗(On Diff #20450)

Braces

768 ↗(On Diff #20450)

Dito

806 ↗(On Diff #20450)

Revert to static and auto

This revision now requires changes to proceed.May 21 2020, 14:12
This revision is now accepted and ready to land.May 21 2020, 14:28