Page MenuHomePhabricator

Backport PR11309: Minor cleanups for AcceptToMemoryPool

Authored by markblundeberg on Dec 22 2019, 03:28.



This is based on #11309 however
the removal of fLimitFree and the fix for reorgs was already done in D4745.
So, this is basically just renaming fOverrideMempoolLimit -> bypass_limits,
removing defaultness of two args, and making sure that literal arguments are
documented with /* */. (test_accept from D3079 stays as default arg, like it
is in Core today)

Test Plan

ninja check

Diff Detail

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

Event Timeline

markblundeberg added inline comments.
34 ↗(On Diff #15066)

oops, missed /* pfMissingInputs */ here

markblundeberg edited the summary of this revision. (Show Details)

args defaultness, also missed one place

This revision is now accepted and ready to land.Dec 23 2019, 00:05