Command-line parsing changes, and no irc by default #803

pull gavinandresen wants to merge 3 commits into bitcoin:master from gavinandresen:argparsing changing 7 files +265 −42

gavinandresen commented at 7:43 PM on February 6, 2012: contributor
I decided I'd typed "./bitcoind --nolisten" enough times to make command-line parsing work the way I think it should.

Changes are:
- Unit tests for the GetArg() methods
- --foo=... is automatically interpreted as -foo=... (so you can use double-dashes if you like for all args)
- Boolean arguments can be specified as either -foo (true) or -nofoo (false).
- Internally, all boolean args are looked up using the positive form and default value (e.g. GetBoolArg("-dnsseed", true))
- Tweaked the --help text
- Default for -irc to bootstrap via IRC channel was changed from true to false
sipa commented at 9:27 PM on February 6, 2012: member

ACK
Unit tests for the GetArg() methods 3ae0735553
New GetArg features: allow --, and booleans can be -foo or -nofoo 3ad9f8a70f
Look for flushwallet/listen/irc/dnsseed/upnp instead of noflushwallet/etc. And switch default for irc to 0. 9d952d17bb
gavinandresen merged this on Feb 7, 2012
gavinandresen closed this on Feb 7, 2012
SanJoseCosta cross-referenced this on Dec 4, 2018 from issue Bitcoind crashes with Too Many Files error by SanJoseCosta
lateminer referenced this in commit 065f94118d on Feb 28, 2019
bitcoin locked this on Sep 8, 2021

Contributors