Page MenuHomePhabricator

Merge #10431: Prevent shadowing the global dustRelayFee
ClosedPublic

Authored by nakihito on May 14 2019, 01:46.

Details

Reviewers
deadalnix
Fabien
jasonbcox
markblundeberg
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

43c5877 Prevent shadowing the global dustRelayFee. (Pavel Janík)

Tree-SHA512: 9765931a7753c484990003396afd0bb65a53f42d1cad9502017720618ce90b3c5ae68591db01e3524adecdbe6925a5eeeebf04012ba644ef3b65073af207ae5d

Backport of Core PR10431
https://github.com/bitcoin/bitcoin/pull/10431/

Depends on D3039

Test Plan
make check
test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR10431
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5841
Build 9744: Bitcoin ABC Buildbot (legacy)
Build 9743: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.May 14 2019, 01:46
nakihito edited the test plan for this revision. (Show Details)EditedMay 14 2019, 01:47

The warnings core had don't actually occur in our code base because of some other changes made before (we don't need to use the global dustRelayFee). However, renaming the arguments should help to prevent some confusion and will hopefully keep merge conflicts in future backports to a minimum.

I get the warnings, which compiler do you use ?

This revision is now accepted and ready to land.May 14 2019, 09:56
In D3040#71594, @Fabien wrote:

I get the warnings, which compiler do you use ?

Ah, I'm getting them again on the D3039 branch. I may have just missed them last time or wasn't doing a clean build.