It's not tested and presumably not used at all. Nobody maintains it.
Details
Details
- Reviewers
jasonbcox - Group Reviewers
Restricted Project - Commits
- rSTAGINGd4dbad530f96: Remove blockprioritypercentage config parameter.
rABCd4dbad530f96: Remove blockprioritypercentage config parameter.
ninja all check-all
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- minertxpri
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 8652 Build 15290: Default Diff Build & Tests Build 15289: arc lint + arc unit
Event Timeline
Comment Actions
Over a year ago, it was mentioned that keeping this code around until we could fix up the block priority stuff might be useful. It's clear that the cost of maintaining this code is not worth keeping it around, so I'm happy to see this go. Maybe one day we can revisit the concept.
Reviewer note: This is similar in many ways to PR9602 which removes -blockprioritysize which was the original name of the argument before it was changed to -blockprioritypercentage in D485.
Comment Actions
Ah I was going to do this but thanks :-) . I think there are still some priority leftovers after D4745 and this, but not much.