Differential D541

Remove duplicate destination decoding
ClosedPublic
Actions

Authored by dagurval on Sep 13 2017, 17:04.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Commits

rSTAGING3648e1bde29f: Remove duplicate destination decoding
rABC3648e1bde29f: Remove duplicate destination decoding

Summary

Port of https://github.com/bitcoin/bitcoin/pull/11259

Depends on D540

Test Plan

None

Diff Detail

Repository

rABC Bitcoin ABC

Branch

duplicate-dest-decoding

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 863
Build 863: arc lint + arc unit

Event Timeline

dagurval created this revision.Sep 13 2017, 17:04

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 13 2017, 17:04

Harbormaster completed remote builds in B863: Diff 1371.Sep 13 2017, 17:04

deadalnix added a subscriber: deadalnix.Sep 14 2017, 17:20

deadalnix added inline comments.

src/qt/transactiondesc.cpp
94	This change doesn't look related to the rest of the diff.

deadalnix accepted this revision.Sep 25 2017, 11:32

This revision is now accepted and ready to land.Sep 25 2017, 11:32

rebase

Closed by commit rABC3648e1bde29f: Remove duplicate destination decoding (authored by João Barbosa <joao.paulo.barbosa@gmail.com>, committed by dagurval). · Explain WhySep 27 2017, 06:18

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

base58.cpp

18 lines

qt/

transactiondesc.cpp

4 lines

Diff 1371

View Options

src/base58.cpp