Don't create change at the dust limit, even if it means paying more than expected
ClosedPublic
Actions

Authored by deadalnix on Jun 11 2019, 23:24.

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGINGf4321052c578: Don't create change at the dust limit, even if it means paying more than…
rABCf4321052c578: Don't create change at the dust limit, even if it means paying more than…

Summary

This is a backport of Core PR9343

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

nodust

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 6275
Build 10597: Bitcoin ABC Buildbot (legacy)
Build 10596: arc lint + arc unit

Event Timeline

deadalnix created this revision.Jun 11 2019, 23:24

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 11 2019, 23:24

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B6275: Diff 9334.Jun 11 2019, 23:31

jasonbcox accepted this revision.Jun 12 2019, 00:34

This revision is now accepted and ready to land.Jun 12 2019, 00:34

deadalnix added a child revision: D3301: Have SelectCoinsMinConf and SelectCoins use BnB or Knapsack and use it.Jun 12 2019, 12:48

Closed by commit rABCf4321052c578: Don't create change at the dust limit, even if it means paying more than… (authored by Alex Morcos <morcos@chaincode.com>, committed by deadalnix). · Explain WhyJun 12 2019, 13:49

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABCf4321052c578: Don't create change at the dust limit, even if it means paying more than….

deadalnix added a commit: rSTAGINGf4321052c578: Don't create change at the dust limit, even if it means paying more than….Sep 19 2019, 00:13

Revision Contents
Changeset List

Path

Size

src/

qt/

coincontroldialog.cpp

10 lines

wallet/

wallet.cpp

27 lines

Diff 9334

View Options

src/qt/coincontroldialog.cpp