HomePhabricator

Introduce wrappers around CBitcoinAddress
831a13bf63c3Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Introduce wrappers around CBitcoinAddress

Summary:
This patch removes the need for the intermediary Base58 type
CBitcoinAddress, by providing {Encode,Decode,IsValid}Destination
function that directly operate on the conversion between strings
and CTxDestination.

This is a port of https://github.com/bitcoin/bitcoin/pull/11117

Depends on D539

Test Plan: make check, rpc tests, manual QT testing

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Subscribers: deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D540

Details

Provenance
Pieter Wuille <pieter.wuille@gmail.com>Authored on Aug 23 2017, 01:02
dagurvalCommitted on Sep 23 2017, 21:13
schancelPushed on Jan 5 2018, 21:58
schancelPushed on Jan 5 2018, 21:39
schancelPushed on Jan 5 2018, 21:17
Reviewer
Restricted Project
Differential Revision
D540: Introduce wrappers around CBitcoinAddress
Parents
rSTAGINGe2ecbe002807: qt: Use IsMine to validate custom change address
Branches
Unknown
Tags
Unknown

Event Timeline