Page MenuHomePhabricator

Track negated options in the option parser
ClosedPublic

Authored by deadalnix on Sep 28 2018, 13:17.

Details

Summary
  • Add additional tests for GetBoolArg()

This is meant to be an intermediate commit to prove that the next does not
introduce any changes in the semantics of boolean option parsing.

  • Track negated arguments in the argument paser.

This commit adds tracking for negated arguments. This change will be used in a
future commit that allows disabling the debug.log file using -nodebuglogfile.

This is a backport of Core's PR12713

Depends on D1832 and D1830

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
negatedoptionparser
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3432
Build 4940: Bitcoin ABC Buildbot (legacy)
Build 4939: arc lint + arc unit