Page MenuHomePhabricator

Remove unused parameter `validFeeEstimates` from `CTxMempool::addUnchecked`
ClosedPublic

Authored by schancel on Apr 8 2019, 01:21.

Details

Reviewers
deadalnix
Fabien
jasonbcox
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABC0043646b87d1: Remove unused parameter `validFeeEstimates` from `CTxMempool::addUnchecked`
Summary

Remove unused parameter validFeeEstimates from CTxMempool::addUnchecked, and the
associated function for calculating this flag (IsCurrentForFeeEstimation).

The code which made use of this parameter was removed with:

Test Plan
make VERBOSE=1 check

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

schancel created this revision.Apr 8 2019, 01:21
Owners added a reviewer: Restricted Owners Package.Apr 8 2019, 01:21
Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 8 2019, 01:21
schancel updated this revision to Diff 7987.Apr 8 2019, 01:47

Fix a comment

Fabien accepted this revision.Apr 8 2019, 08:34
This revision is now accepted and ready to land.Apr 8 2019, 08:34
jasonbcox accepted this revision.Apr 8 2019, 16:25
This revision was automatically updated to reflect the committed changes.